mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Jacob Keller <jacob.e.keller@intel.com>
To: Simon Horman <horms@kernel.org>, Jakub Kicinski <kuba@kernel.org>
Cc: Jiri Pirko <jiri@resnulli.us>,
	"David S. Miller" <davem@davemloft.net>,
	Eric Dumazet <edumazet@google.com>,
	Jakub Kicinski <kuba@kernel.org>,
	"Paolo Abeni" <pabeni@redhat.com>,
	Jonathan Corbet <corbet@lwn.net>,
	Tony Nguyen <anthony.l.nguyen@intel.com>,
	Przemek Kitszel <przemyslaw.kitszel@intel.com>,
	Andrew Lunn <andrew+netdev@lunn.ch>,
	Alexander Lobakin <aleksander.lobakin@intel.com>,
	<netdev@vger.kernel.org>, <linux-doc@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>, Dan Nowlin <dan.nowlin@intel.com>,
	Qi Zhang <qi.z.zhang@intel.com>, Jie Wang <jie1x.wang@intel.com>,
	Junfeng Guo <junfeng.guo@intel.com>,
	"Jedrzej Jagielski" <jedrzej.jagielski@intel.com>,
	Aleksandr Loktionov <aleksandr.loktionov@intel.com>,
	Rafal Romanowski <rafal.romanowski@intel.com>
Subject: Re: [PATCH net-next 06/14] ice: Extend PTYPE bitmap coverage for GTP encapsulated flows
Date: Thu, 16 Oct 2025 10:20:25 -0700	[thread overview]
Message-ID: <64d3e25a-c9d6-4a43-84dd-cffe60ac9848@intel.com> (raw)
In-Reply-To: <aPDjUeXzS1lA2owf@horms.kernel.org>


[-- Attachment #1.1: Type: text/plain, Size: 1872 bytes --]



On 10/16/2025 5:21 AM, Simon Horman wrote:
> On Wed, Oct 15, 2025 at 12:32:02PM -0700, Jacob Keller wrote:
>> From: Przemek Kitszel <przemyslaw.kitszel@intel.com>
>>
>> Consolidate updates to the Protocol Type (PTYPE) bitmap definitions
>> across multiple flow types in the Intel ICE driver to support GTP
>> (GPRS Tunneling Protocol) encapsulated traffic.
>>
>> Enable improved Receive Side Scaling (RSS) configuration for both user
>> and control plane GTP flows.
>>
>> Cover a wide range of protocol and encapsulation scenarios, including:
>>  - MAC OFOS and IL
>>  - IPv4 and IPv6 (OFOS, IL, ALL, no-L4)
>>  - TCP, SCTP, ICMP
>>  - GRE OF
>>  - GTPC (control plane)
>>
>> Expand the PTYPE bitmap entries to improve classification and
>> distribution of GTP traffic across multiple queues, enhancing
>> performance and scalability in mobile network environments.
>>
>> --
> 
> Hi Jacob,
> 
> Perhaps surprisingly, git truncates the commit message at
> the ('--') line above. So, importantly, the tags below are absent.
> 

Its somewhat surprising, since I thought you had to use '---' for that.
Regardless, this shouldn't be in the commit message at all.
> Also, the two lines below seem out of place.
> 
>>  ice_flow.c |   54 +++++++++++++++++++++++++++---------------------------
>>  1 file changed, 26 insertions(+), 26 deletions(-)
>>

Yep these shouldn't have been here at all. I checked, and for some
reason it was included in the original message id of the patch. b4
happily picked it up when using b4 shazam.

See:
https://lore.kernel.org/intel-wired-lan/20250915133928.3308335-5-aleksandr.loktionov@intel.com/

I am not sure if this is the fault of b4, though it has different
behavior than other git tooling here.

I fixed this on my end, and can resubmit after the 24hr period if needed.

Thanks,
Jake

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 236 bytes --]

  reply	other threads:[~2025-10-16 17:20 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-15 19:31 [PATCH net-next 00/14] Intel Wired LAN Driver Updates 2025-10-15 (ice, iavf, ixgbe, i40e, e1000e) Jacob Keller
2025-10-15 19:31 ` [PATCH net-next 01/14] devlink: Add new "max_mac_per_vf" generic device param Jacob Keller
2025-10-15 19:31 ` [PATCH net-next 02/14] i40e: support generic devlink param "max_mac_per_vf" Jacob Keller
2025-10-15 19:31 ` [PATCH net-next 03/14] ice: add flow parsing for GTP and new protocol field support Jacob Keller
2025-10-15 19:32 ` [PATCH net-next 04/14] ice: add virtchnl and VF context support for GTP RSS Jacob Keller
2025-10-15 19:32 ` [PATCH net-next 05/14] ice: improve TCAM priority handling for RSS profiles Jacob Keller
2025-10-15 19:32 ` [PATCH net-next 06/14] ice: Extend PTYPE bitmap coverage for GTP encapsulated flows Jacob Keller
2025-10-16 12:21   ` Simon Horman
2025-10-16 17:20     ` Jacob Keller [this message]
2025-10-16 19:03       ` Simon Horman
2025-10-16 21:37         ` Jacob Keller
2025-10-16 23:56           ` Jakub Kicinski
2025-10-17  0:31             ` Jacob Keller
2025-10-17  6:10               ` Jacob Keller
2025-10-15 19:32 ` [PATCH net-next 07/14] iavf: add RSS support for GTP protocol via ethtool Jacob Keller
2025-10-15 19:32 ` [PATCH net-next 08/14] net: docs: add missing features that can have stats Jacob Keller
2025-10-15 19:32 ` [PATCH net-next 09/14] ice: implement ethtool standard stats Jacob Keller
2025-10-15 19:32 ` [PATCH net-next 10/14] ice: add tracking of good transmit timestamps Jacob Keller
2025-10-15 19:32 ` [PATCH net-next 11/14] ice: implement transmit hardware timestamp statistics Jacob Keller
2025-10-15 19:32 ` [PATCH net-next 12/14] ice: refactor to use helpers Jacob Keller
2025-10-15 19:32 ` [PATCH net-next 13/14] ixgbe: preserve RSS indirection table across admin down/up Jacob Keller
2025-10-15 19:32 ` [PATCH net-next 14/14] e1000e: Introduce private flag to disable K1 Jacob Keller

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=64d3e25a-c9d6-4a43-84dd-cffe60ac9848@intel.com \
    --to=jacob.e.keller@intel.com \
    --cc=aleksander.lobakin@intel.com \
    --cc=aleksandr.loktionov@intel.com \
    --cc=andrew+netdev@lunn.ch \
    --cc=anthony.l.nguyen@intel.com \
    --cc=corbet@lwn.net \
    --cc=dan.nowlin@intel.com \
    --cc=davem@davemloft.net \
    --cc=edumazet@google.com \
    --cc=horms@kernel.org \
    --cc=jedrzej.jagielski@intel.com \
    --cc=jie1x.wang@intel.com \
    --cc=jiri@resnulli.us \
    --cc=junfeng.guo@intel.com \
    --cc=kuba@kernel.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.com \
    --cc=przemyslaw.kitszel@intel.com \
    --cc=qi.z.zhang@intel.com \
    --cc=rafal.romanowski@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

Powered by JetHome