Moti, Hello and sorry for be reply late until now, definitely as gaetan said 
that there might be some change after the version, anyway I will create a new 
version to benefit you all to review and further test.  

Best regards,
Jeff Guo

-----Original Message-----
From: Gaëtan Rivet [mailto:gaetan.ri...@6wind.com] 
Sent: Thursday, December 14, 2017 6:21 PM
To: Mordechay Haimovsky <mo...@mellanox.com>
Cc: Guo, Jia <jia....@intel.com>; dev@dpdk.org
Subject: Re: [dpdk-dev] [PATCH v6 0/2] add uevent monitor for hot plug

Hello Moti,

On Thu, Dec 14, 2017 at 09:48:23AM +0000, Mordechay Haimovsky wrote:
> Hello,
>  I would like to apply this patch in order to review it.
> 

In absence of answer from Jeff,

Those two paths were modified during the 17.08 release: both pci and vdev buses 
have been moved to drivers/bus.

> Trying to apply it on 17.11 (and latest) fails due to missing 
> lib/librte_eal/common/eal_common_vdev.c             
> Trying to apply it on 17.08.1 fails on missing 
> drivers/bus/pci/bsd/pci.c file
> 

Only the pci bus move was integrated by Jeff to this version of the udev 
monitor. The vdev bus move however came later and should be rebased upon.

> So, on what DPDK version should I apply it ? 
> Or maybe there is a bunch of other  patches I have to apply in order to use 
> this patch ?
> 

You should apply it on 17.11 IMO.
Either you take upon yourself to make it work with the new tree, or wait for 
Jeff to send a new version.

--
Gaëtan Rivet
6WIND

Reply via email to