Hi, Just sent a new version with the commit log reworded to answer the "why" question you mentioned. Thx for your reviews.
Kindest regards, Raslan Darawsheh > -----Original Message----- > From: Thomas Monjalon <[email protected]> > Sent: Wednesday, October 10, 2018 5:08 PM > To: Wiles, Keith <[email protected]>; Raslan Darawsheh > <[email protected]> > Cc: [email protected]; Shahaf Shuler <[email protected]>; Ori Kam > <[email protected]>; Yigit, Ferruh <[email protected]> > Subject: Re: [PATCH v5 1/3] net/tap: add queue and port ids in Rx/Tx queues > structures > > 10/10/2018 14:50, Wiles, Keith: > > > > > On Oct 10, 2018, at 2:03 AM, Raslan Darawsheh <[email protected]> > wrote: > > > > > > Signed-off-by: Raslan Darawsheh <[email protected]> > > > > This title for the patch is the what we did not why we did it, should that > > be > changed? To me it does not convey the reason or we would need to add a > more complete comment body text to explain why we wanted the change. It > is a bit of nit picking. > > I think the question "why" must be answered in the body. > >
