Re: [dpdk-dev] [PATCH] claim responsibility for KVM virtio PMD, vhost backend, and XEN virtio solution.

[Thomas Monjalon <thomas.monjalon@6wind.com>]

2015-03-30 14:52, Xie, Huawei:
> On 3/30/2015 8:25 PM, Thomas Monjalon wrote:
> > 2015-03-30 08:31, Xie, Huawei:
> >> On 3/9/2015 11:45 PM, Huawei Xie wrote:
> >>> ---
> >>>  MAINTAINERS | 7 +++++++
> >>>  1 file changed, 7 insertions(+)
> >>>
> >>> diff --git a/MAINTAINERS b/MAINTAINERS
> >>> index 07fdf5e..b4327d3 100644
> >>> --- a/MAINTAINERS
> >>> +++ b/MAINTAINERS
> >>> @@ -131,6 +131,12 @@ F: app/test-pmd/mempool_*
> >>>  F: examples/vhost_xen/
> >>>  F: doc/guides/prog_guide/intel_dpdk_xen_based_packet_switch_sol.rst
> >>>  
> >>> +Linux Xen VIRTIO
> >>> +M: Huawei Xie <huawei.xie@intel.com>
> >>> +F: lib/librte_pmd_xenvirt/
> >>> +F: examples/vhost_xen/
> >>> +F: doc/guides/prog_guide/intel_dpdk_xen_based_packet_switch_sol.rst
> >>> +
> >>>  FreeBSD EAL (with overlaps)
> >>>  M: Bruce Richardson <bruce.richardson@intel.com>
> >>>  F: lib/librte_eal/bsdapp/Makefile
> >>> @@ -226,6 +232,7 @@ F: lib/librte_pmd_mlx4/
> >>>  F: doc/guides/prog_guide/mlx4_poll_mode_drv.rst
> >>>  
> >>>  RedHat virtio
> >>> +M: Huawei Xie <huawei.xie@intel.com>
> >>>  M: Changchun Ouyang <changchun.ouyang@intel.com>
> >>>  F: lib/librte_pmd_virtio/
> >>>  F: doc/guides/prog_guide/poll_mode_drv_emulated_virtio_nic.rst
> >> Thomas:
> >> Comments with this patch? Need i add "MAINTAINERS:" into the subject?
> > 
> > This patch is wrong because you are duplicating an existing section.
> > I think you want to split Xen support maintenance. If you are sure it's a
> > good idea, please send a correct patch.
> 
> Yes, i am mainly the author of XEN virtio PMD and host part and would
> like to review that part.
> Should i split those two sections and move XEN virtio PMD and host part
> to driver section?

Yes you cannot duplicate lib/librte_pmd_xenvirt/.
If you want to split "Linux Xen" section, just do it.
I already explained that to Jijiang. I don't even understand why we have to
discuss it again and again. Please just do the right split.

> > About vhost, maybe you want to split virtio and vhost?
> >
> About vhost, i will maintain those two parts. Is there issue with the
> format?

In my understanding Changchun was maintaining virtio, so I thought you were
interested in vhost only.