[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [XEN PATCH v6 06/31] x86/mm: avoid building multiple .o from a single .c file


  • To: Jan Beulich <jbeulich@xxxxxxxx>
  • From: Anthony PERARD <anthony.perard@xxxxxxxxxx>
  • Date: Mon, 12 Jul 2021 13:49:15 +0100
  • Authentication-results: esa2.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none
  • Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Ian Jackson <iwj@xxxxxxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, "Tim Deegan" <tim@xxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Mon, 12 Jul 2021 12:49:27 +0000
  • Ironport-hdrordr: A9a23:uMzOu6BXjgiU9krlHemu55DYdb4zR+YMi2TC1yhKKCC9Vvbo8P xG/c5rsSMc5wx8ZJhNo7+90ey7MBXhHP1OkOws1NWZLWrbUQKTRekIh+bfKn/bak/DH4ZmpN 5dmsNFaOEYY2IVsfrH
  • Ironport-sdr: oKsT2lLZY3nNM1eeJ8Q/FszhjtyMX765Bhr8p2+87MorSaP/cl+d3HvydRXYm7TfF7cvqUZw3u 9reLguJfwWNOy3N6bCBdt4aM7Kp1KYgNYoCZ56DHfeU4nGFLI2ywOxsn3KQu708YoCtkZHdpRn q8QfDKHB8CGcBHHNbWH6d6X+k56oQ7GeVMay3yatontqxSAtlg8fZu1ZTCafu4ohKNGOGaAdXZ XZd4JrMzAooTC+GTJ/Gnkd2ep/FUxO3XgLjpP/DKkbPKbhvYMc5ogHrcCSALcmQg8L8jxQ2qBX bzk=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Wed, Jul 07, 2021 at 04:45:11PM +0200, Jan Beulich wrote:
> On 01.07.2021 16:09, Anthony PERARD wrote:
> >  xen/Makefile                            | 11 -----------
> >  xen/Rules.mk                            |  2 +-
> >  xen/arch/x86/mm/Makefile                |  9 ---------
> >  xen/arch/x86/mm/guest_walk.c            |  3 ---
> >  xen/arch/x86/mm/guest_walk_2.c          |  2 ++
> >  xen/arch/x86/mm/guest_walk_3.c          |  2 ++
> >  xen/arch/x86/mm/guest_walk_4.c          |  2 ++
> >  xen/arch/x86/mm/hap/Makefile            |  9 ---------
> >  xen/arch/x86/mm/hap/guest_walk.c        |  3 ---
> >  xen/arch/x86/mm/hap/guest_walk_2level.c |  2 ++
> >  xen/arch/x86/mm/hap/guest_walk_3level.c |  2 ++
> >  xen/arch/x86/mm/hap/guest_walk_4level.c |  2 ++
> 
> Is there a particular reason you've kept the "level" in these three
> file names? Preferably with them shortened (and ideally dashes used
> everywhere in the new file names instead of underscores)

I had no reason to rename them. (Also, renaming them makes the patch
bigger ;-) as we would rename the objects files.)

But I can rename as part of the patch. Should I rename all of them (mm
hap and shadow) to "guest-walk-$level.c" ? Or just "guest-$level.c" like
in shadow/ ? On the other hand, it would probably be helpful to have the same
basename for both the source and the .c that includes the source (e.g.
"guest_walk.c" and "guest_walk_2.c" have "guest_walk" in common). So if
we were to replace underscores by dashes, we should probably rename
"guest_walk.c" to "guest-walk.c" as well.

So I'll remove the "level" from the filenames in hap/ to start with in
an update to this patch, it it worth it to do more that that?

> Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>

Thanks,

-- 
Anthony PERARD



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.