Re: [Xen-devel] [PATCH v5 4/4] x86/mem_sharing: compile mem_sharing subsystem only when kconfig is enabled

[Tamas K Lengyel <tamas@xxxxxxxxxxxxx>]

On Mon, Jun 3, 2019 at 10:40 AM Julien Grall <julien.grall@xxxxxxx> wrote:
>
> Hi,
>
> On 03/06/2019 17:38, Tamas K Lengyel wrote:
> > On Mon, Jun 3, 2019 at 2:26 AM Jan Beulich <JBeulich@xxxxxxxx> wrote:
> >>
> >>>>> On 16.05.19 at 23:37, <tamas@xxxxxxxxxxxxx> wrote:
> >>> Disable it by default as it is only an experimental subsystem.
> >>>
> >>> Signed-off-by: Tamas K Lengyel <tamas@xxxxxxxxxxxxx>
> >>> Cc: Jan Beulich <jbeulich@xxxxxxxx>
> >>> Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> >>> Cc: Wei Liu <wei.liu2@xxxxxxxxxx>
> >>> Cc: Roger Pau Monne <roger.pau@xxxxxxxxxx>
> >>> Cc: George Dunlap <George.Dunlap@xxxxxxxxxxxxx>
> >>> Cc: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
> >>> Cc: Julien Grall <julien.grall@xxxxxxx>
> >>> Cc: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
> >>> Cc: Stefano Stabellini <sstabellini@xxxxxxxxxx>
> >>> Cc: Tim Deegan <tim@xxxxxxx>
> >>> Cc: George Dunlap <george.dunlap@xxxxxxxxxxxxx>
> >>> ----
> >>> v4: add ASSERT_UNREACHABLE to inlined functions where applicable & other
> >>> fixups
> >>> ---
> >>>   xen/arch/x86/Kconfig              |  6 +++++-
> >>>   xen/arch/x86/domain.c             |  2 ++
> >>>   xen/arch/x86/domctl.c             |  2 ++
> >>>   xen/arch/x86/mm/Makefile          |  2 +-
> >>>   xen/arch/x86/x86_64/compat/mm.c   |  2 ++
> >>>   xen/arch/x86/x86_64/mm.c          |  2 ++
> >>>   xen/common/Kconfig                |  3 ---
> >>>   xen/common/domain.c               |  2 +-
> >>>   xen/common/grant_table.c          |  2 +-
> >>>   xen/common/memory.c               |  2 +-
> >>>   xen/common/vm_event.c             |  6 +++---
> >>>   xen/include/asm-x86/mem_sharing.h | 28 ++++++++++++++++++++++++++++
> >>>   xen/include/asm-x86/mm.h          |  3 +++
> >>>   xen/include/xen/sched.h           |  2 +-
> >>>   xen/include/xsm/dummy.h           |  2 +-
> >>>   xen/include/xsm/xsm.h             |  4 ++--
> >>>   xen/xsm/dummy.c                   |  2 +-
> >>>   xen/xsm/flask/hooks.c             |  4 ++--
> >>>   18 files changed, 58 insertions(+), 18 deletions(-)
> >>
> >> Daniel, it looks like you weren't Cc-ed here, but your ack is needed.
> >
> > Indeed, I've also seem to have missed CC-ing Razvan (fixed now).
>
> Nowadays, add_maintainers.pl should do the right for you without having to
> modifying by hand each patch:
>
> 42sh> scripts/add_maintainers.pl -d .

Hi Julien,
ah good call! I've been using scripts/get_maintainers.pl which
evidently is more error-prone in case the patch goes through revisions
and I forget to re-check if more people need CC-ing now.

Thanks,
Tamas

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel