* [PATCH] MAINTAINERS: add arch specific public headers to arch file groups
@ 2017-09-01 10:45 Jan Beulich
2017-09-01 10:56 ` Andrew Cooper
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Jan Beulich @ 2017-09-01 10:45 UTC (permalink / raw)
To: xen-devel
Cc: Stefano Stabellini, Wei Liu, George Dunlap, Andrew Cooper,
Ian Jackson, Tim Deegan
I've recently got sufficiently annoyed by people not applying enough
common sense to get_maintainer.pl output, Cc-ing all REST maintainers
on ARM-only public interface changes.
Sort ARM's xen/ groups of path specifications at the same time.
Signed-off-by: Jan Beulich <jbeulich@suse.com>
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -166,14 +166,15 @@ S: Supported
L: xen-devel@lists.xen.org
F: docs/misc/arm/
F: xen/arch/arm/
-F: xen/include/asm-arm/
-F: xen/drivers/char/cadence-uart.c
F: xen/drivers/char/arm-uart.c
+F: xen/drivers/char/cadence-uart.c
F: xen/drivers/char/exynos4210-uart.c
F: xen/drivers/char/omap-uart.c
F: xen/drivers/char/pl011.c
F: xen/drivers/char/scif-uart.c
F: xen/drivers/passthrough/arm/
+F: xen/include/asm-arm/
+F: xen/include/public/arch-arm/
CPU POOLS
M: Juergen Gross <jgross@suse.com>
@@ -434,6 +435,7 @@ S: Supported
L: xen-devel@lists.xen.org
F: xen/arch/x86/
F: xen/include/asm-x86/
+F: xen/include/public/arch-x86/
F: tools/firmware/hvmloader/
F: tools/firmware/rombios/
F: tools/firmware/vgabios/
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] MAINTAINERS: add arch specific public headers to arch file groups
2017-09-01 10:45 [PATCH] MAINTAINERS: add arch specific public headers to arch file groups Jan Beulich
@ 2017-09-01 10:56 ` Andrew Cooper
2017-09-04 8:33 ` Wei Liu
2017-09-18 10:10 ` Julien Grall
2 siblings, 0 replies; 6+ messages in thread
From: Andrew Cooper @ 2017-09-01 10:56 UTC (permalink / raw)
To: Jan Beulich, xen-devel
Cc: Stefano Stabellini, Wei Liu, George Dunlap, Tim Deegan, Ian Jackson
On 01/09/17 11:45, Jan Beulich wrote:
> I've recently got sufficiently annoyed by people not applying enough
> common sense to get_maintainer.pl output, Cc-ing all REST maintainers
> on ARM-only public interface changes.
>
> Sort ARM's xen/ groups of path specifications at the same time.
>
> Signed-off-by: Jan Beulich <jbeulich@suse.com>
Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>
>
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -166,14 +166,15 @@ S: Supported
> L: xen-devel@lists.xen.org
> F: docs/misc/arm/
> F: xen/arch/arm/
> -F: xen/include/asm-arm/
> -F: xen/drivers/char/cadence-uart.c
> F: xen/drivers/char/arm-uart.c
> +F: xen/drivers/char/cadence-uart.c
> F: xen/drivers/char/exynos4210-uart.c
> F: xen/drivers/char/omap-uart.c
> F: xen/drivers/char/pl011.c
> F: xen/drivers/char/scif-uart.c
> F: xen/drivers/passthrough/arm/
> +F: xen/include/asm-arm/
> +F: xen/include/public/arch-arm/
>
> CPU POOLS
> M: Juergen Gross <jgross@suse.com>
> @@ -434,6 +435,7 @@ S: Supported
> L: xen-devel@lists.xen.org
> F: xen/arch/x86/
> F: xen/include/asm-x86/
> +F: xen/include/public/arch-x86/
> F: tools/firmware/hvmloader/
> F: tools/firmware/rombios/
> F: tools/firmware/vgabios/
>
>
>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] MAINTAINERS: add arch specific public headers to arch file groups
2017-09-01 10:45 [PATCH] MAINTAINERS: add arch specific public headers to arch file groups Jan Beulich
2017-09-01 10:56 ` Andrew Cooper
@ 2017-09-04 8:33 ` Wei Liu
2017-09-04 9:06 ` Jan Beulich
2017-09-18 10:10 ` Julien Grall
2 siblings, 1 reply; 6+ messages in thread
From: Wei Liu @ 2017-09-04 8:33 UTC (permalink / raw)
To: Jan Beulich
Cc: Stefano Stabellini, Wei Liu, George Dunlap, Andrew Cooper,
Ian Jackson, Tim Deegan, xen-devel
On Fri, Sep 01, 2017 at 04:45:12AM -0600, Jan Beulich wrote:
> I've recently got sufficiently annoyed by people not applying enough
> common sense to get_maintainer.pl output, Cc-ing all REST maintainers
> on ARM-only public interface changes.
>
> Sort ARM's xen/ groups of path specifications at the same time.
>
> Signed-off-by: Jan Beulich <jbeulich@suse.com>
I tried to apply this patch from a downloaded maildir but it got
rejected.
I manually committed the changes, copied the commit message over and
pushed it, but it turns out I forgot to change the authorship to you. I
hope you don't mind.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] MAINTAINERS: add arch specific public headers to arch file groups
2017-09-04 8:33 ` Wei Liu
@ 2017-09-04 9:06 ` Jan Beulich
0 siblings, 0 replies; 6+ messages in thread
From: Jan Beulich @ 2017-09-04 9:06 UTC (permalink / raw)
To: Wei Liu
Cc: Stefano Stabellini, George Dunlap, Andrew Cooper, Ian Jackson,
Tim Deegan, xen-devel
>>> On 04.09.17 at 10:33, <wei.liu2@citrix.com> wrote:
> On Fri, Sep 01, 2017 at 04:45:12AM -0600, Jan Beulich wrote:
>> I've recently got sufficiently annoyed by people not applying enough
>> common sense to get_maintainer.pl output, Cc-ing all REST maintainers
>> on ARM-only public interface changes.
>>
>> Sort ARM's xen/ groups of path specifications at the same time.
>>
>> Signed-off-by: Jan Beulich <jbeulich@suse.com>
>
> I tried to apply this patch from a downloaded maildir but it got
> rejected.
>
> I manually committed the changes, copied the commit message over and
> pushed it, but it turns out I forgot to change the authorship to you. I
> hope you don't mind.
No problem at all - the main thing is that the change is in place.
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] MAINTAINERS: add arch specific public headers to arch file groups
2017-09-01 10:45 [PATCH] MAINTAINERS: add arch specific public headers to arch file groups Jan Beulich
2017-09-01 10:56 ` Andrew Cooper
2017-09-04 8:33 ` Wei Liu
@ 2017-09-18 10:10 ` Julien Grall
2017-09-18 10:37 ` Jan Beulich
2 siblings, 1 reply; 6+ messages in thread
From: Julien Grall @ 2017-09-18 10:10 UTC (permalink / raw)
To: Jan Beulich, xen-devel
Cc: Stefano Stabellini, Wei Liu, George Dunlap, Andrew Cooper,
Ian Jackson, Tim Deegan, Bhupinder Thakur
Hi Jan,
On 01/09/17 11:45, Jan Beulich wrote:
> I've recently got sufficiently annoyed by people not applying enough
> common sense to get_maintainer.pl output, Cc-ing all REST maintainers
> on ARM-only public interface changes.
It seems that you didn't move arch-arm.h, arch-x86_{32,64}.h under resp.
ARM and x86 maintainership. Was it intentional?
Cheers,
>
> Sort ARM's xen/ groups of path specifications at the same time.
>
> Signed-off-by: Jan Beulich <jbeulich@suse.com>
>
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -166,14 +166,15 @@ S: Supported
> L: xen-devel@lists.xen.org
> F: docs/misc/arm/
> F: xen/arch/arm/
> -F: xen/include/asm-arm/
> -F: xen/drivers/char/cadence-uart.c
> F: xen/drivers/char/arm-uart.c
> +F: xen/drivers/char/cadence-uart.c
> F: xen/drivers/char/exynos4210-uart.c
> F: xen/drivers/char/omap-uart.c
> F: xen/drivers/char/pl011.c
> F: xen/drivers/char/scif-uart.c
> F: xen/drivers/passthrough/arm/
> +F: xen/include/asm-arm/
> +F: xen/include/public/arch-arm/
>
> CPU POOLS
> M: Juergen Gross <jgross@suse.com>
> @@ -434,6 +435,7 @@ S: Supported
> L: xen-devel@lists.xen.org
> F: xen/arch/x86/
> F: xen/include/asm-x86/
> +F: xen/include/public/arch-x86/
> F: tools/firmware/hvmloader/
> F: tools/firmware/rombios/
> F: tools/firmware/vgabios/
>
>
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> https://lists.xen.org/xen-devel
>
--
Julien Grall
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] MAINTAINERS: add arch specific public headers to arch file groups
2017-09-18 10:10 ` Julien Grall
@ 2017-09-18 10:37 ` Jan Beulich
0 siblings, 0 replies; 6+ messages in thread
From: Jan Beulich @ 2017-09-18 10:37 UTC (permalink / raw)
To: Julien Grall
Cc: Stefano Stabellini, Wei Liu, George Dunlap, Andrew Cooper,
Ian Jackson, Tim Deegan, Bhupinder Thakur, xen-devel
>>> On 18.09.17 at 12:10, <julien.grall@arm.com> wrote:
> Hi Jan,
>
> On 01/09/17 11:45, Jan Beulich wrote:
>> I've recently got sufficiently annoyed by people not applying enough
>> common sense to get_maintainer.pl output, Cc-ing all REST maintainers
>> on ARM-only public interface changes.
>
> It seems that you didn't move arch-arm.h, arch-x86_{32,64}.h under resp.
> ARM and x86 maintainership. Was it intentional?
No, but the two x86 ones you mention are merely compatibility
stubs, which aren't really expected to ever change, so who is
their formal maintainer doesn't matter. For arch-arm.h I agree
it should be put under ARM maintainership.
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-09-18 10:37 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-09-01 10:45 [PATCH] MAINTAINERS: add arch specific public headers to arch file groups Jan Beulich
2017-09-01 10:56 ` Andrew Cooper
2017-09-04 8:33 ` Wei Liu
2017-09-04 9:06 ` Jan Beulich
2017-09-18 10:10 ` Julien Grall
2017-09-18 10:37 ` Jan Beulich
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.