Re: [PATCH v1 3/3] MAINTAINERS: xen, kvm: track pvclock-abi.h changes

From	Juergen Gross <jgross@suse.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v1 3/3] MAINTAINERS: xen, kvm: track pvclock-abi.h changes
Date	2017-01-27 16:00 +0100
Message-ID	<t4gam-5qf-9@gated-at.bofh.it> (permalink)
References	<t3zI6-4Hf-3@gated-at.bofh.it> <t3zI6-4Hf-19@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On 25/01/17 18:33, Joao Martins wrote:
> This file defines an ABI shared between guest and hypervisor(s)
> (KVM, Xen) and as such there should be an correspondent entry in
> MAINTAINERS file. Notice that there's already a text notice at the
> top of the header file, hence this commit simply enforces it more
> explicitly and have both peers noticed when such changes happen.
> 
> Signed-off-by: Joao Martins <joao.m.martins@oracle.com>
> ---
> This was suggested by folks at xen-devel as we missed some of the
> ABI additions (e.g. flags field in pvti, TSC stable bit) - so this
> patch is to help preventing that from happening. Alternatively I
> could instead add a "PVCLOCK ABI" section in this file with the
> two mailing lists.

I don't mind either way. In case of an own section: make it more generic
like "PARAVIRT ABIS" as there might be more than PVCLOCK (if not now
maybe in future).

For both variants:

Acked-by: Juergen Gross <jgross@suse.com>


Juergen

> ---
>  MAINTAINERS | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 26edd83..c4315d1 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -7041,6 +7041,7 @@ F:	Documentation/virtual/kvm/
>  F:	arch/*/kvm/
>  F:	arch/x86/kernel/kvm.c
>  F:	arch/x86/kernel/kvmclock.c
> +F:	arch/x86/include/asm/pvclock-abi.h
>  F:	arch/*/include/asm/kvm*
>  F:	include/linux/kvm*
>  F:	include/uapi/linux/kvm*
> @@ -13483,6 +13484,7 @@ M:	Juergen Gross <jgross@suse.com>
>  L:	xen-devel@lists.xenproject.org (moderated for non-subscribers)
>  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git
>  S:	Supported
> +F:	arch/x86/include/asm/pvclock-abi.h
>  F:	arch/x86/xen/
>  F:	drivers/*/xen-*front.c
>  F:	drivers/xen/
>

Thread

[PATCH v1 0/3] x86/xen: pvclock vdso support Joao Martins <joao.m.martins@oracle.com> - 2017-01-25 18:40 +0100
  [PATCH v1 2/3] x86/xen/time: setup vcpu 0 time info page Joao Martins <joao.m.martins@oracle.com> - 2017-01-25 18:40 +0100
    Re: [PATCH v1 2/3] x86/xen/time: setup vcpu 0 time info page Boris Ostrovsky <boris.ostrovsky@oracle.com> - 2017-01-25 20:30 +0100
      Re: [PATCH v1 2/3] x86/xen/time: setup vcpu 0 time info page Joao Martins <joao.m.martins@oracle.com> - 2017-01-26 14:30 +0100
    Re: [PATCH v1 2/3] x86/xen/time: setup vcpu 0 time info page Andy Lutomirski <luto@amacapital.net> - 2017-01-26 18:20 +0100
  [PATCH v1 3/3] MAINTAINERS: xen, kvm: track pvclock-abi.h changes Joao Martins <joao.m.martins@oracle.com> - 2017-01-25 18:40 +0100
    Re: [PATCH v1 3/3] MAINTAINERS: xen, kvm: track pvclock-abi.h changes Andy Lutomirski <luto@amacapital.net> - 2017-01-26 18:30 +0100
      Re: [PATCH v1 3/3] MAINTAINERS: xen, kvm: track pvclock-abi.h changes Joao Martins <joao.m.martins@oracle.com> - 2017-01-26 21:10 +0100
    Re: [PATCH v1 3/3] MAINTAINERS: xen, kvm: track pvclock-abi.h changes Juergen Gross <jgross@suse.com> - 2017-01-27 16:00 +0100
  [PATCH v1 1/3] x86/pvclock: add setter for pvclock_pvti_cpu0_va Joao Martins <joao.m.martins@oracle.com> - 2017-01-25 18:40 +0100
    Re: [PATCH v1 1/3] x86/pvclock: add setter for pvclock_pvti_cpu0_va Andy Lutomirski <luto@amacapital.net> - 2017-01-26 18:30 +0100
      Re: [PATCH v1 1/3] x86/pvclock: add setter for pvclock_pvti_cpu0_va Joao Martins <joao.m.martins@oracle.com> - 2017-01-26 21:00 +0100

csiph-web