From mboxrd@z Thu Jan 1 00:00:00 1970 From: Cornelia Huck Subject: Re: [PATCH v7 13/13] MAINTAINERS: Add vfio-ccw maintainer Date: Tue, 9 May 2017 09:33:39 +0200 Message-ID: <20170509093339.67cdb945.cornelia.huck@de.ibm.com> References: <20170505020352.8984-1-bjsdjshi@linux.vnet.ibm.com> <20170505020352.8984-14-bjsdjshi@linux.vnet.ibm.com> <20170505142014.28c849b8.cornelia.huck@de.ibm.com> <20170508052927.GD15974@bjsdjshi@linux.vnet.ibm.com> <20170508110927.5713c101.cornelia.huck@de.ibm.com> <20170509014143.GG15974@bjsdjshi@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20170509014143.GG15974@bjsdjshi@linux.vnet.ibm.com> Sender: kvm-owner@vger.kernel.org List-Archive: List-Post: To: Dong Jia Shi Cc: kvm@vger.kernel.org, linux-s390@vger.kernel.org, qemu-devel@nongnu.org, borntraeger@de.ibm.com, agraf@suse.com, alex.williamson@redhat.com, eric.auger@redhat.com List-ID: On Tue, 9 May 2017 09:41:43 +0800 Dong Jia Shi wrote: > * Cornelia Huck [2017-05-08 11:09:27 +0200]: > > > On Mon, 8 May 2017 13:29:27 +0800 > > Dong Jia Shi wrote: > > > > > * Cornelia Huck [2017-05-05 14:20:14 +0200]: > > > > > > > On Fri, 5 May 2017 04:03:52 +0200 > > > > Dong Jia Shi wrote: > > > > > > > > > Add Cornelia Huck as the vfio-ccw maintainer. > > > > > > > > > > Signed-off-by: Dong Jia Shi > > > > > --- > > > > > MAINTAINERS | 5 +++++ > > > > > 1 file changed, 5 insertions(+) > > > > > > > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > > > > index cae3b09..c1f9917 100644 > > > > > --- a/MAINTAINERS > > > > > +++ b/MAINTAINERS > > > > > @@ -999,6 +999,11 @@ S: Supported > > > > > F: hw/vfio/* > > > > > F: include/hw/vfio/ > > > > > > > > > > +vfio-ccw > > > > > +M: Cornelia Huck > > > > > +S: Supported > > > > > +F: hw/vfio/ccw.c > > > > > > > > The other code related to vfio-ccw is entangled with the other virtual > > > > css code, so this is probably really the only file that needs to be > > > > listed separately. > > > "hw/s390x/" and "include/hw/s390x/" are listed under the "S390 Machines" > > > part. > > > > > > Maybe you would like to list the following files separately here: > > > hw/s390x/s390-ccw.c > > > hw/s390x/s390-ccw.h > > > > Yes, that makes sense. > > > > But seeing the path name here: Should s390-ccw.h rather go into > > include/hw/s390x/ (even if hw/s390x/ already contains several .h > > files...)? > Ok. You will find it in include/hw/s390x/ in next version. > > Would you like me making a sepate patch to move all of the other .h > files to the include folder? Let's just leave them where they are for now.