> -----Original Message----- > From: Xen-devel [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of Jan > Beulich > Sent: 31 March 2016 08:52 > To: Paul Durrant > Cc: xen-de...@lists.xenproject.org > Subject: Re: [Xen-devel] [PATCH] MAINTAINERS: Add myself as x86 I/O > emulation and viridian maintainer > > >>> On 31.03.16 at 09:28, <paul.durr...@citrix.com> wrote: > > I have made many modifications to this code over the past few years > > so I'm probably the one most familiar with it. > > > > Signed-off-by: Paul Durrant <paul.durr...@citrix.com> > > --- > > MAINTAINERS | 16 ++++++++++++++++ > > 1 file changed, 16 insertions(+) > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > index e765311..43bbb8f 100644 > > --- a/MAINTAINERS > > +++ b/MAINTAINERS > > @@ -384,6 +384,16 @@ F: xen/include/asm-x86/ > > F: tools/firmware/hvmloader/ > > F: tools/tests/x86_emulator/ > > > > +X86 I/O EMULATION > > +M: Paul Durrant <paul.durr...@citrix.com> > > +S: Supported > > +F: xen/arch/x86/hvm/emulate.c > > +F: xen/arch/x86/hvm/hvm.c > > This file doesn't really belong here (as containing a lot of other > stuff).
I'm concerned about missing any changes to the ioreq server code that's in there. Would it perhaps be better to split that code out at this point? > Instead I think you also want ... > > > +F: xen/arch/x86/hvm/intercept.c > > F: xen/arch/x86/hvm/io.c Yes, I missed that. > > > +F: xen/include/asm-x86/hvm/hvm.h > > And this again doesn't belong here, while > > F: xen/include/asm-x86/hvm/emulate.h > > seems missing. > Yes, that should be there too. > If you agree, I can do these adjustments while committing. > I agree to the additions and dropping hvm.h, but I'd like to keep hvm.c pending potentially splitting out the ioreq server code. Paul > Jan > > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xen.org > http://lists.xen.org/xen-devel _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
