Received-SPF: pass (google.com: domain of linux-kernel+bounces-95812-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Date: Thu, 7 Mar 2024 16:24:27 +0100
From: Maciej Fijalkowski <maciej.fijalkowski@intel.com>
To: suresh ks <suresh2514@gmail.com>
CC: "Loktionov, Aleksandr" <aleksandr.loktionov@intel.com>, "Kitszel,
 Przemyslaw" <przemyslaw.kitszel@intel.com>, "Brandeburg, Jesse"
	<jesse.brandeburg@intel.com>, "Nguyen, Anthony L"
	<anthony.l.nguyen@intel.com>, "davem@davemloft.net" <davem@davemloft.net>,
	"edumazet@google.com" <edumazet@google.com>, "kuba@kernel.org"
	<kuba@kernel.org>, "pabeni@redhat.com" <pabeni@redhat.com>,
	"intel-wired-lan@lists.osuosl.org" <intel-wired-lan@lists.osuosl.org>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH iwl v2] i40e: print correct hw max rss count in kernel
 ring buffer
Message-ID: <ZencK4bR0R1igXb5@boxer>
References: <20240120072806.8554-1-suresh2514@gmail.com>
 <e524c57e-fe43-4582-bb05-c50f3e529848@intel.com>
 <SJ0PR11MB5866EAA3CBAF4E28842305D5E57D2@SJ0PR11MB5866.namprd11.prod.outlook.com>
 <Zc46QEBEpCOL75qN@boxer>
 <CABAyFk54urKYzroBt5ii=h8hMHJ=rgPpGwivK1DjNS+pWDFq5Q@mail.gmail.com>
Content-Type: text/plain; charset="utf-8"
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <CABAyFk54urKYzroBt5ii=h8hMHJ=rgPpGwivK1DjNS+pWDFq5Q@mail.gmail.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?dys4VlV1cytRNys1YkFKa3A3bEZTNE4rQi9RZlZkdmJieWo0TVdpbWZCNzRv?=
 =?utf-8?B?RElmTE9ITmUrelVQV2p3T2k4bDkrZnBNZXFqTlJRTzlHbkZncWcxQlJlOFU0?=
 =?utf-8?B?eUJWNjB1ME45WHlmdlJIVUNuU2ZqSGU1YWc1U2N1Vmxza0RGWlBpSUlPWjdz?=
 =?utf-8?B?NnkvcUc1QUVlWGthRm5oY3VpV0VmVzlqN09yMk8yYjRhRHRGMk5hcC9KYWo0?=
 =?utf-8?B?UnpJWG1aMDJHRmdwN1M2TjgxOTF4NC9TNkNPUGhRdW85VHRmN0xFeE96M0xT?=
 =?utf-8?B?b1JYS3FieDlkeWF2dmU5Uko3Z3JZdk1TYWpkekN1NG1Gck13MzhCN0ZldXpE?=
 =?utf-8?B?TXNDU1laOWVnRUpYdWV2a1BaNENIOE40ekk2Q1Q2azN4WUlER2R0d3hScVd5?=
 =?utf-8?B?bGVHTVdBNjk5d0dJNkxZdDQ5aWJ3c1ZtdUNxbWY1YnErOGNOb1J5VXdEcE1y?=
 =?utf-8?B?N1B1OTdLMmduUXNQMks5ZU5XSHJHeElJTWErOUZoWFBJS3BLNk9qNVpUbWo5?=
 =?utf-8?B?TEd0WUk2bVVqcTgrRUphMnd3ZDJqVjVOMU9FVmMyWnJaWVdoalNQQ2c2OWhJ?=
 =?utf-8?B?UGVSWHUrQ1JZcUFJMGdkWFhYNkIrUFlnU01HRmFKUStVTjlZcGJsRE0zQ3lu?=
 =?utf-8?B?UmJ0dmFoN1dBQmZGc0xGWXVzakt0bmRMRlJCRE5iQURHNVg5K2UybGY4TlFC?=
 =?utf-8?B?dCs0b1pud2QrY2pnWkVVY244RFZzMUtWY2IzaCt3ajYreTBqbXBNa3VtL3pt?=
 =?utf-8?B?OExmTFBhd3BRK2R0RWVtUDY5TC9mdmhaVDBSYTBDZnVhNjBzS3FFSnQvVXd3?=
 =?utf-8?B?eGRaOWU0MlZpNzBUWjc5L1VLZ2ZjTTFld21NUjhFS3NqTDQ5RXRnZ21oVWFZ?=
 =?utf-8?B?WmEzMm11eXVBRFpsSThaSkl4dTByZzFzU2JGZ1h0MDRHSFFqV05JTWlrSmto?=
 =?utf-8?B?NVM3akJJT2plYzNrVmEyYitQeGFzYllQMkNVK3NNMkRKNzhhK1cxZjJuWllu?=
 =?utf-8?B?T1ByWWVXT0UwSlZhVFJocllNanBQTHc3M25lYm4zaHphSWttdDlMeVdsaitP?=
 =?utf-8?B?c2lIRWJ1T29iSGJwOGsram5GakVqT3ZmbWtPTnJFUi8ySklvNnhJMFovU1RG?=
 =?utf-8?B?UDdGK0RyUjVpSEdvZHYvZFVIdzJ6b2hQZHpMSmlIOVlBTTJpYkY3cThpN2Uv?=
 =?utf-8?B?Q2s2VXVWZmFzVkRRNFU4eGM4eTFnemxiRDE0dXQ4cVlranpvUEpJNGZkSlc1?=
 =?utf-8?B?UFNHRU9WdjBsL2dZTm5DWmx4Tno3THVNdWhRUWNxRE1iRm9kSitIeDdJSHAy?=
 =?utf-8?B?ZjlabC96UitoS2JOdDBvYy9ibmd4UUo5NloxeDF2R0t2SmxXUi9MK1p3SXpM?=
 =?utf-8?B?cy85UjlyWnBGUkxoMWdjUXdJUkRpZElUQVQ0TkU1Vyt0ZWtPZHV1bWRhanE5?=
 =?utf-8?B?Y05mUFRYM0c4cjNvYlJISmtzWXFLdWw5Mmg3NjA3dzBDd1lOa2RjeGZtZjRh?=
 =?utf-8?B?SklTSjM5RVRKRjJNRHAwOElnbko1UzlJRkV6dUVVT1Nwbm8vUFV0ZVhBa3F1?=
 =?utf-8?B?ZFVzUnFXcWRJbzJBamlRZDBJczJaUzZGUHVpVi9wbDVrTzA1bXRpdnlRWVcy?=
 =?utf-8?B?aHpueUg0RDVLTytWamNqZjh6RlpwM3RmeGg1dHRwY1VWU3ltRExFS25IOTFi?=
 =?utf-8?B?ODFwY09SUzF0b0p2L2x3b3BQVkU2eEdReXdsRCtYdklzNjZuTTJPeDFmL01R?=
 =?utf-8?B?cHkvU3UzNkcweXhFa2J3b1oyajhyUVA3a1VZbEo0WTRSQytVbHhvM2xJOW5L?=
 =?utf-8?B?UTRjR2ozZXpTMmxHc01pUFQ0ZlQ4akozRVZhZlJ0akJ2M0ZjRG1IZm13T3lT?=
 =?utf-8?B?U0NIVlNKejFabmh5RmEwVlZpS3Y0Qm4rdGlJQmU3b2dub21MSzVqakNnSGNj?=
 =?utf-8?B?WWlEWDVCdWlvRmVmNW9rdkhlbEhvd2xEdGdPSzdDT1ZiVFh1anMrWlhHZkNp?=
 =?utf-8?B?eVZjcWFSODJ5OFliUXZhWTZTNU1nUkduQWE3cGxCSjhTVkxxQ0tlUjNqWnFy?=
 =?utf-8?B?YlBvS3ZzNTdNclNuZTRSNkpCK0l6V0NFWU8xajVPMnVIQ0pRL0Ryc3BaRTJl?=
 =?utf-8?B?M1l2N2FadWdZUm8vQzNKUTZpdGhVaThkOUpMQzlvaDJvK0huNEpzMEZxblFC?=
 =?utf-8?B?Z0E9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: a766fd9e-e0fe-4dd8-dc75-08dc3ebab385
X-MS-Exchange-CrossTenant-AuthSource: DM4PR11MB6117.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Mar 2024 15:24:37.7954
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: xal3yI2o52VKz2PH6j/QvdvI1peBplMcQk2unuDDcwb0IIg/bBaLE8ycnZuXnuwT6A3wyWmijVeUAt0PBGSSa9Mp4fy6bMuI2bl3dTsghvQ=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR11MB4535
X-OriginatorOrg: intel.com

On Thu, Mar 07, 2024 at 08:09:47PM +0530, suresh ks wrote:
> Hi Maciej,
> 
> Thanks for the links and taking time to review.

No problem, but please don't top post on linux mailing lists nor don't
respond in HTML. These are 101 rules to participate in upstream
discussions.

> 
> I was following the ethtool codes and suggested vsi->num_queue_pairs.
> Maybe vsi->alloc_queue_pairs is the right option as you suggested. I have
> not read much of i40e codes. Just that one of our customers reported this
> issue and I was reviewing the codes.
> 
> I also felt rss_size_max comes from the early i40e days. So I was not sure
> what I would add in the 'Fixes:' tag.

submitting-patches.html has it explained ;)

> 
> Sorry for getting late here. Was on a vacation.
> 
> 
> thanks....
> *Suresh KS*
> suresh2514@gmail.com
> 91-7709100053
> 
> 
> 
> On Thu, Feb 15, 2024 at 9:52 PM Maciej Fijalkowski <
> maciej.fijalkowski@intel.com> wrote:
> 
> > On Tue, Jan 30, 2024 at 08:33:43AM +0000, Loktionov, Aleksandr wrote:
> > > > -----Original Message-----
> > > > From: Kitszel, Przemyslaw <przemyslaw.kitszel@intel.com>
> > > > Sent: Tuesday, January 30, 2024 9:26 AM
> > > > To: Loktionov, Aleksandr <aleksandr.loktionov@intel.com>
> > > > Subject: Fwd: [PATCH iwl v2] i40e: print correct hw max rss count
> >
> > Subject should be iwl-net and you should have Fixes: tag.
> > I also would like you to go through
> > https://docs.kernel.org/process/submitting-patches.html
> >
> > or any other document that would prepare you for your first submission.
> >
> > > > in kernel ring buffer
> > > >
> > > > FWD to Alex
> > > >
> > > >
> > > > -------- Forwarded Message --------
> > > > Subject: [PATCH iwl v2] i40e: print correct hw max rss count in
> > > > kernel ring buffer
> > > > Date: Sat, 20 Jan 2024 12:58:06 +0530
> > > > From: Suresh Kumar <suresh2514@gmail.com>
> > > > To: jesse.brandeburg@intel.com, anthony.l.nguyen@intel.com,
> > > > davem@davemloft.net, edumazet@google.com, kuba@kernel.org,
> > > > pabeni@redhat.com, intel-wired-lan@lists.osuosl.org,
> > > > netdev@vger.kernel.org, linux-kernel@vger.kernel.org
> > > > CC: Suresh Kumar <suresh2514@gmail.com>
> > > >
> > > > pf->rss_size_max is hardcoded and always prints max rss count as
> > > > 64.
> > > >
> > > > Eg:
> > > >    kernel: i40e 0000:af:00.1: User requested queue count/HW max RSS
> > > > count:  104/64
> > > >
> > > > whereas  ethtool reports the correct value from "vsi-
> > > > >num_queue_pairs"
> > > >
> > > > Channel parameters for eno33:
> > > > Pre-set maximums:
> > > > RX:     n/a
> > > > TX:     n/a
> > > > Other:      1
> > > > Combined:   104
> > > > Current hardware settings:
> > > > RX:     n/a
> > > > TX:     n/a
> > > > Other:      1
> > > > Combined:   104  <-------
> > > >
> > > > and is misleading.
> > > >
> > > > Change it to vsi->num_queue_pairs
> > >
> > > Please reject this patch, it breaks driver logging.
> > > The massage clearly states that it dumps max rss queues number that f/w
> > supports.
> >
> > ...which would imply that you would be able to work with anything more
> > than 64 queues. From a quick glance rss_size_max comes from early i40e
> > days and if that would be the limit indeed then driver allowing 104 queues
> > would be a disaster, no?
> >
> > >
> > > Thank you
> > >
> > > > Signed-off-by: Suresh Kumar <suresh2514@gmail.com>
> > > > ---
> > > >   drivers/net/ethernet/intel/i40e/i40e_main.c | 2 +-
> > > >   1 file changed, 1 insertion(+), 1 deletion(-)
> > > >
> > > > diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c
> > > > b/drivers/net/ethernet/intel/i40e/i40e_main.c
> > > > index d5519af34657..f5c1ec190f7e 100644
> > > > --- a/drivers/net/ethernet/intel/i40e/i40e_main.c
> > > > +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c
> > > > @@ -12524,7 +12524,7 @@ int i40e_reconfig_rss_queues(struct i40e_pf
> > > > *pf, int queue_count)
> > > >             i40e_pf_config_rss(pf);
> > > >     }
> > > >     dev_info(&pf->pdev->dev, "User requested queue count/HW max
> > > > RSS
> > > > count:  %d/%d\n",
> > > > -            vsi->req_queue_pairs, pf->rss_size_max);
> > > > +            vsi->req_queue_pairs, vsi->num_queue_pairs);
> >
> > IMHO this should be vsi->alloc_queue_pairs instead.
> >
> > > >     return pf->alloc_rss_size;
> > > >   }
> > > >   -- 2.43.0
> > > >
> > >
> >