diff mbox series

[Xen-devel] MAINTAINERS: Move xen/lib/x86 under x86 maintainership

Message ID 20190404140410.1156-1-julien.grall@arm.com
State New
Headers show
Series [Xen-devel] MAINTAINERS: Move xen/lib/x86 under x86 maintainership | expand

Commit Message

Julien Grall April 4, 2019, 2:04 p.m. UTC
At the moment, xen/lib/x86 is covered by the "REST". However, this is
x86-only, so this can fall under the x86 maintainership.

Signed-off-by: Julien Grall <julien.grall@arm.com>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

Comments

Andrew Cooper April 4, 2019, 2:10 p.m. UTC | #1
On 04/04/2019 15:04, Julien Grall wrote:
> At the moment, xen/lib/x86 is covered by the "REST". However, this is
> x86-only, so this can fall under the x86 maintainership.
>
> Signed-off-by: Julien Grall <julien.grall@arm.com>

Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>

Sorry - this is almost certainly my fault
Jan Beulich April 4, 2019, 2:19 p.m. UTC | #2
>>> On 04.04.19 at 16:04, <julien.grall@arm.com> wrote:
> At the moment, xen/lib/x86 is covered by the "REST". However, this is
> x86-only, so this can fall under the x86 maintainership.
> 
> Signed-off-by: Julien Grall <julien.grall@arm.com>

Acked-by: Jan Beulich <jbeulich@suse.com>

Thanks for noticing. Would you mind adding xen/include/xen/lib/x86/
at this occasion as well?

Jan
Julien Grall April 4, 2019, 2:20 p.m. UTC | #3
Hi,

On 04/04/2019 15:19, Jan Beulich wrote:
>>>> On 04.04.19 at 16:04, <julien.grall@arm.com> wrote:
>> At the moment, xen/lib/x86 is covered by the "REST". However, this is
>> x86-only, so this can fall under the x86 maintainership.
>>
>> Signed-off-by: Julien Grall <julien.grall@arm.com>
> 
> Acked-by: Jan Beulich <jbeulich@suse.com>
> 
> Thanks for noticing. Would you mind adding xen/include/xen/lib/x86/
> at this occasion as well?

I will do it. Are you happy if I do it on commit?

@Andrew: May I keep your reviewed-by with this addition?

Cheers,

> 
> Jan
> 
>
Jan Beulich April 4, 2019, 2:27 p.m. UTC | #4
>>> On 04.04.19 at 16:20, <julien.grall@arm.com> wrote:
> On 04/04/2019 15:19, Jan Beulich wrote:
>>>>> On 04.04.19 at 16:04, <julien.grall@arm.com> wrote:
>>> At the moment, xen/lib/x86 is covered by the "REST". However, this is
>>> x86-only, so this can fall under the x86 maintainership.
>>>
>>> Signed-off-by: Julien Grall <julien.grall@arm.com>
>> 
>> Acked-by: Jan Beulich <jbeulich@suse.com>
>> 
>> Thanks for noticing. Would you mind adding xen/include/xen/lib/x86/
>> at this occasion as well?
> 
> I will do it. Are you happy if I do it on commit?

Sure. I'm sorry for not having said so right away.

Jan
Andrew Cooper April 4, 2019, 2:33 p.m. UTC | #5
On 04/04/2019 15:20, Julien Grall wrote:
> Hi,
>
> On 04/04/2019 15:19, Jan Beulich wrote:
>>>>> On 04.04.19 at 16:04, <julien.grall@arm.com> wrote:
>>> At the moment, xen/lib/x86 is covered by the "REST". However, this is
>>> x86-only, so this can fall under the x86 maintainership.
>>>
>>> Signed-off-by: Julien Grall <julien.grall@arm.com>
>>
>> Acked-by: Jan Beulich <jbeulich@suse.com>
>>
>> Thanks for noticing. Would you mind adding xen/include/xen/lib/x86/
>> at this occasion as well?
>
> I will do it. Are you happy if I do it on commit?
>
> @Andrew: May I keep your reviewed-by with this addition?

Yes

~Andrew
Julien Grall April 5, 2019, 2 p.m. UTC | #6
On 04/04/2019 15:33, Andrew Cooper wrote:
> On 04/04/2019 15:20, Julien Grall wrote:
>> Hi,
>>
>> On 04/04/2019 15:19, Jan Beulich wrote:
>>>>>> On 04.04.19 at 16:04, <julien.grall@arm.com> wrote:
>>>> At the moment, xen/lib/x86 is covered by the "REST". However, this is
>>>> x86-only, so this can fall under the x86 maintainership.
>>>>
>>>> Signed-off-by: Julien Grall <julien.grall@arm.com>
>>>
>>> Acked-by: Jan Beulich <jbeulich@suse.com>
>>>
>>> Thanks for noticing. Would you mind adding xen/include/xen/lib/x86/
>>> at this occasion as well?
>>
>> I will do it. Are you happy if I do it on commit?
>>
>> @Andrew: May I keep your reviewed-by with this addition?
> 
> Yes

Thank you to you and Jan! It is now committed.

Cheers,
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index ba7527c423..a9929a91de 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -455,6 +455,7 @@  L:	xen-devel@lists.xenproject.org
 F:	xen/arch/x86/
 F:	xen/include/asm-x86/
 F:	xen/include/public/arch-x86/
+F:	xen/lib/x86
 F:	tools/firmware/hvmloader/
 F:	tools/firmware/rombios/
 F:	tools/firmware/vgabios/