From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olivier MATZ Subject: Re: [PATCH v2 02/13] ixgbe: fix remaining pkt_flags variable size to 64 bits Date: Mon, 17 Nov 2014 18:03:37 +0100 Message-ID: <546A2A69.5080004@6wind.com> References: <1415635166-1364-1-git-send-email-olivier.matz@6wind.com> <1415984609-2484-1-git-send-email-olivier.matz@6wind.com> <1415984609-2484-3-git-send-email-olivier.matz@6wind.com> <7C4248CAE043B144B1CD242D275626532FDFFE4E@IRSMSX104.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: "jigsaw-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org" To: "Walukiewicz, Miroslaw" , "dev-VfR2kkLFssw@public.gmane.org" , Thomas Monjalon Return-path: In-Reply-To: <7C4248CAE043B144B1CD242D275626532FDFFE4E-kPTMFJFq+rGvNW/NfzhIbrfspsVTdybXVpNB7YpNyf8@public.gmane.org> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" Hi Miroslaw, On 11/17/2014 05:47 PM, Walukiewicz, Miroslaw wrote: > > >> -----Original Message----- >> From: Olivier Matz [mailto:olivier.matz-pdR9zngts4EAvxtiuMwx3w@public.gmane.org] >> Sent: Friday, November 14, 2014 6:03 PM >> To: dev-VfR2kkLFssw@public.gmane.org >> Cc: olivier.matz-pdR9zngts4EAvxtiuMwx3w@public.gmane.org; Walukiewicz, Miroslaw; Liu, Jijiang; Liu, Yong; >> jigsaw-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org; Richardson, Bruce >> Subject: [PATCH v2 02/13] ixgbe: fix remaining pkt_flags variable size to 64 >> bits >> >> Since commit 4332beee9 "mbuf: expand ol_flags field to 64-bits", the >> packet flags are now 64 bits wide. Some occurences were forgotten in >> the ixgbe driver. > > I think it should be present in separate patch. I do no not see any relation to TSO You are right, there is no relation with TSO. The reason why I initially added it in the same patchset is because I discovered this bug while implementing TSO and I wanted to avoid too much noise on the list. I can take out some patches from the series, but maybe it's too late and it would confuse patchwork. Thomas, what do you think? Regards, Olivier