Re: [dpdk-dev] [PATCH v17 0/5] User-space Ethtool

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

2015-07-16 21:55, Wang, Liang-min:
> Thomas,
> 	Do you want me to create a separate patch just include the example/l2fwd-ethtool?

Yes

> 	Do you also mean besides the identified Makefiles, you see more rework needs to be done,
> 	or I just need to fix Makefile issue? If just Makefile issue, I could try to make another attempt tomorrow?

After checking the build there is probably more review to do.
Let's take more time to have something clean and maybe more complete in 2.2.
This patchset is your first contribution to DPDK and is already a nice achievement.
The new API must now be implemented in more drivers to be effective.


> > -----Original Message-----
> > From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com]
> > Sent: Thursday, July 16, 2015 5:48 PM
> > To: Wang, Liang-min
> > Cc: dev@dpdk.org
> > Subject: Re: [dpdk-dev] [PATCH v17 0/5] User-space Ethtool
> > 
> > 2015-07-16 09:25, Liang-Min Larry Wang:
> > > This implementation is designed to provide a familar interface for
> > applications that rely on kernel-space driver to support ethtool_op and
> > net_device_op for device management. The initial implementation focuses
> > on ops that can be implemented through existing netdev APIs. More ops will
> > be supported in latter release.
> > 
> > Applied without example which needs more work, thanks