| 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 |
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
>>> 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
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 > >
>>> 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
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
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 --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/
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(+)