Hi Thaomas, > From: Bruce Richardson [mailto:bruce.richardson at intel.com] > Sent: Tuesday, March 22, 2016 7:30 AM > > On Thu, Mar 10, 2016 at 05:47:56PM -0800, Rasesh Mody wrote: > > Signed-off-by: Harish Patil <harish.patil at qlogic.com> > > Signed-off-by: Rasesh Mody <rasesh.mody at qlogic.com> > > --- > > drivers/net/bnx2x/bnx2x.c | 207 +++++++++++++++++------------------ > --- > > drivers/net/bnx2x/bnx2x.h | 4 +- > > drivers/net/bnx2x/bnx2x_ethdev.c | 53 ++++++++++ > > drivers/net/bnx2x/bnx2x_rxtx.c | 47 +++++---- > > 4 files changed, 173 insertions(+), 138 deletions(-) > > > This patch needs a commit message with a description of changes and how > they help with performance. > > If there are multiple changes being made to improve the performance, or > add in extra TX features, they would be better split into separate patches, > with one unique change per patch. If any of the changes are bug-fixes as > opposed to performance improvements (since the word "fixes" appears in > the title here), they should each be in a separate patch, with a message > describing the fix and including a "fixes" line in the message as described > here: > http://dpdk.org/doc/guides/contributing/patches.html#commit-messages- > body > > Thanks, > /Bruce
The feedback is addressed and v2 patches have been submitted. Pls. apply to rel_16_04. Thanks! Rasesh
