Re: [dpdk-dev] [PATCH V17 3/4] eal/linux: uevent parse and process

2018-04-01 Thread Guo, Jia
On 3/30/2018 12:59 AM, Stephen Hemminger wrote: On Fri, 30 Mar 2018 00:00:04 +0800 Jeff Guo wrote: + ret = ioctl(netlink_fd, FIONBIO, &nonblock); + if (ret != 0) { + RTE_LOG(ERR, EAL, "ioctl(FIONBIO) failed.\n"); + goto err; + } + retu Sin

Re: [dpdk-dev] [PATCH V17 3/4] eal/linux: uevent parse and process

2018-04-01 Thread Guo, Jia
On 3/30/2018 1:00 AM, Stephen Hemminger wrote: On Fri, 30 Mar 2018 00:00:04 +0800 Jeff Guo wrote: +dev_uev_monitor_create(int netlink_fd) +{ + struct sockaddr_nl addr; + int ret; + int size = 64 * 1024; + int nonblock = 1; + + memset(&addr, 0, sizeof(addr)); +

Re: [dpdk-dev] [PATCH V17 3/4] eal/linux: uevent parse and process

2018-03-29 Thread Stephen Hemminger
On Fri, 30 Mar 2018 00:00:04 +0800 Jeff Guo wrote: > +dev_uev_monitor_create(int netlink_fd) > +{ > + struct sockaddr_nl addr; > + int ret; > + int size = 64 * 1024; > + int nonblock = 1; > + > + memset(&addr, 0, sizeof(addr)); > + addr.nl_family = AF_NETLINK; > + addr

Re: [dpdk-dev] [PATCH V17 3/4] eal/linux: uevent parse and process

2018-03-29 Thread Stephen Hemminger
On Fri, 30 Mar 2018 00:00:04 +0800 Jeff Guo wrote: > + ret = ioctl(netlink_fd, FIONBIO, &nonblock); > + if (ret != 0) { > + RTE_LOG(ERR, EAL, "ioctl(FIONBIO) failed.\n"); > + goto err; > + } > + retu Since you use NOWAIT option, this is unnecessary.

[dpdk-dev] [PATCH V17 3/4] eal/linux: uevent parse and process

2018-03-29 Thread Jeff Guo
In order to handle the uevent which have been detected from the kernel side, add uevent parse and process function to translate the uevent into device event, which user has subscribe to monitor. Signed-off-by: Jeff Guo --- v17->v16: add new file into meson.build, modify coding sytle and add print