From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id CEFEF5A8C for ; Mon, 30 Mar 2015 17:35:46 +0200 (CEST) Received: from orsmga003.jf.intel.com ([10.7.209.27]) by fmsmga101.fm.intel.com with ESMTP; 30 Mar 2015 08:35:35 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.11,494,1422950400"; d="scan'208";a="548414479" Received: from kmsmsx152.gar.corp.intel.com ([172.21.73.87]) by orsmga003.jf.intel.com with ESMTP; 30 Mar 2015 08:35:32 -0700 Received: from shsmsx152.ccr.corp.intel.com (10.239.6.52) by KMSMSX152.gar.corp.intel.com (172.21.73.87) with Microsoft SMTP Server (TLS) id 14.3.224.2; Mon, 30 Mar 2015 23:35:31 +0800 Received: from shsmsx101.ccr.corp.intel.com ([169.254.1.24]) by SHSMSX152.ccr.corp.intel.com ([169.254.6.71]) with mapi id 14.03.0224.002; Mon, 30 Mar 2015 23:35:09 +0800 From: "Xie, Huawei" To: Thomas Monjalon Thread-Topic: [dpdk-dev] [PATCH] claim responsibility for KVM virtio PMD, vhost backend, and XEN virtio solution. Thread-Index: AQHQavkfhMt9shAL6k+KO+jZv4MUqg== Date: Mon, 30 Mar 2015 15:35:09 +0000 Message-ID: References: <1425915850-14934-1-git-send-email-huawei.xie@intel.com> <3159342.39RF9GQ1Cv@xps13> <4775667.Icro2s1yKj@xps13> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] [PATCH] claim responsibility for KVM virtio PMD, vhost backend, and XEN virtio solution. X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 30 Mar 2015 15:35:47 -0000 On 3/30/2015 11:33 PM, Thomas Monjalon wrote:=0A= > 2015-03-30 15:23, Xie, Huawei:=0A= >> On 3/30/2015 11:08 PM, Thomas Monjalon wrote:=0A= >>> 2015-03-30 14:52, Xie, Huawei:=0A= >>>> On 3/30/2015 8:25 PM, Thomas Monjalon wrote:=0A= >>>>> 2015-03-30 08:31, Xie, Huawei:=0A= >>>>>> On 3/9/2015 11:45 PM, Huawei Xie wrote:=0A= >>>>>>> ---=0A= >>>>>>> MAINTAINERS | 7 +++++++=0A= >>>>>>> 1 file changed, 7 insertions(+)=0A= >>>>>>>=0A= >>>>>>> diff --git a/MAINTAINERS b/MAINTAINERS=0A= >>>>>>> index 07fdf5e..b4327d3 100644=0A= >>>>>>> --- a/MAINTAINERS=0A= >>>>>>> +++ b/MAINTAINERS=0A= >>>>>>> @@ -131,6 +131,12 @@ F: app/test-pmd/mempool_*=0A= >>>>>>> F: examples/vhost_xen/=0A= >>>>>>> F: doc/guides/prog_guide/intel_dpdk_xen_based_packet_switch_sol.rs= t=0A= >>>>>>> =0A= >>>>>>> +Linux Xen VIRTIO=0A= >>>>>>> +M: Huawei Xie =0A= >>>>>>> +F: lib/librte_pmd_xenvirt/=0A= >>>>>>> +F: examples/vhost_xen/=0A= >>>>>>> +F: doc/guides/prog_guide/intel_dpdk_xen_based_packet_switch_sol.rs= t=0A= >>>>>>> +=0A= >>>>>>> FreeBSD EAL (with overlaps)=0A= >>>>>>> M: Bruce Richardson =0A= >>>>>>> F: lib/librte_eal/bsdapp/Makefile=0A= >>>>>>> @@ -226,6 +232,7 @@ F: lib/librte_pmd_mlx4/=0A= >>>>>>> F: doc/guides/prog_guide/mlx4_poll_mode_drv.rst=0A= >>>>>>> =0A= >>>>>>> RedHat virtio=0A= >>>>>>> +M: Huawei Xie =0A= >>>>>>> M: Changchun Ouyang =0A= >>>>>>> F: lib/librte_pmd_virtio/=0A= >>>>>>> F: doc/guides/prog_guide/poll_mode_drv_emulated_virtio_nic.rst=0A= >>>>>> Thomas:=0A= >>>>>> Comments with this patch? Need i add "MAINTAINERS:" into the subject= ?=0A= >>>>> This patch is wrong because you are duplicating an existing section.= =0A= >>>>> I think you want to split Xen support maintenance. If you are sure it= 's a=0A= >>>>> good idea, please send a correct patch.=0A= >>>> Yes, i am mainly the author of XEN virtio PMD and host part and would= =0A= >>>> like to review that part.=0A= >>>> Should i split those two sections and move XEN virtio PMD and host par= t=0A= >>>> to driver section?=0A= >>> Yes you cannot duplicate lib/librte_pmd_xenvirt/.=0A= >>> If you want to split "Linux Xen" section, just do it.=0A= >>> I already explained that to Jijiang. I don't even understand why we hav= e to=0A= >>> discuss it again and again. Please just do the right split.=0A= >>>=0A= >>>>> About vhost, maybe you want to split virtio and vhost?=0A= >>>>>=0A= >>>> About vhost, i will maintain those two parts. Is there issue with the= =0A= >>>> format?=0A= >>> In my understanding Changchun was maintaining virtio, so I thought you = were=0A= >>> interested in vhost only.=0A= >>>=0A= >> Is there issue with two co-maintainers?=0A= >> I was the co-author(with Rashmin) of original static virtio-pmd library,= =0A= >> and would review the patch with changchun to keep it healthy.=0A= > No it's OK. I now clearly understand that this part of the patch is right= .=0A= > You want to co-maintain virtio and vhost. OK=0A= > I had a doubt because you failed with Xen split.=0A= >=0A= >=0A= That explains. Would submit a new patch.=0A=