Re: [Intel-wired-lan] [PATCH V3 RFC 2/2] ixgbe: ixgbe_atr() compute l4_proto only if non-paged data has network/transport headers
On Mon, 2016-10-17 at 18:47 -0400, Sowmini Varadhan wrote: > On (10/17/16 15:37), Jeff Kirsher wrote: > > > Reviewed-by: Alexander Duyck> > > > Sowmini, can you re-submit this to intel-wired-lan but without the RFC > in > > the title? > > V4 resubmitted.. I think I just inadvertently forgot to add Alex as the > reviewed-by.. could you please fix that (or I can resubmit v5 if needed). No need to resubmit, I can make sure Alex's reviewed-by gets added. signature.asc Description: This is a digitally signed message part
Re: [Intel-wired-lan] [PATCH V3 RFC 2/2] ixgbe: ixgbe_atr() compute l4_proto only if non-paged data has network/transport headers
On (10/17/16 15:37), Jeff Kirsher wrote: > > Reviewed-by: Alexander Duyck> > Sowmini, can you re-submit this to intel-wired-lan but without the RFC in > the title? V4 resubmitted.. I think I just inadvertently forgot to add Alex as the reviewed-by.. could you please fix that (or I can resubmit v5 if needed). --Sowmini
Re: [Intel-wired-lan] [PATCH V3 RFC 2/2] ixgbe: ixgbe_atr() compute l4_proto only if non-paged data has network/transport headers
On Mon, 2016-10-17 at 15:29 -0700, Alexander Duyck wrote: > On Mon, Oct 17, 2016 at 2:12 PM, Sowmini Varadhan >wrote: > > > > For some Tx paths (e.g., tpacket_snd()), ixgbe_atr may be > > passed down an sk_buff that has the network and transport > > header in the paged data, so it needs to make sure these > > headers are available in the headlen bytes to calculate the > > l4_proto. > > > > This patch expect that network and transport headers are > > already available in the non-paged header dat. The assumption > > is that the caller has set this up if l4_proto based Tx > > steering is desired. > > > > Signed-off-by: Sowmini Varadhan > > This all looks correct to me. I would recommend having Jeff pull it > in to be submitted to the net queue. > > Reviewed-by: Alexander Duyck Sowmini, can you re-submit this to intel-wired-lan but without the RFC in the title? signature.asc Description: This is a digitally signed message part
Re: [Intel-wired-lan] [PATCH V3 RFC 2/2] ixgbe: ixgbe_atr() compute l4_proto only if non-paged data has network/transport headers
On Mon, Oct 17, 2016 at 2:12 PM, Sowmini Varadhanwrote: > For some Tx paths (e.g., tpacket_snd()), ixgbe_atr may be > passed down an sk_buff that has the network and transport > header in the paged data, so it needs to make sure these > headers are available in the headlen bytes to calculate the > l4_proto. > > This patch expect that network and transport headers are > already available in the non-paged header dat. The assumption > is that the caller has set this up if l4_proto based Tx > steering is desired. > > Signed-off-by: Sowmini Varadhan This all looks correct to me. I would recommend having Jeff pull it in to be submitted to the net queue. Reviewed-by: Alexander Duyck