Received-SPF: pass (google.com: domain of linux-kernel+bounces-199266-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Date: Mon, 3 Jun 2024 17:05:59 +0300
From: Vladimir Oltean <vladimir.oltean@nxp.com>
To: MD Danish Anwar <danishanwar@ti.com>
Cc: Jan Kiszka <jan.kiszka@siemens.com>,
	Dan Carpenter <dan.carpenter@linaro.org>,
	Andrew Lunn <andrew@lunn.ch>, Simon Horman <horms@kernel.org>,
	Diogo Ivo <diogo.ivo@siemens.com>,
	Wolfram Sang <wsa+renesas@sang-engineering.com>,
	Randy Dunlap <rdunlap@infradead.org>,
	Niklas Schnelle <schnelle@linux.ibm.com>,
	Vignesh Raghavendra <vigneshr@ti.com>,
	Richard Cochran <richardcochran@gmail.com>,
	Roger Quadros <rogerq@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
	Jakub Kicinski <kuba@kernel.org>,
	Eric Dumazet <edumazet@google.com>,
	"David S. Miller" <davem@davemloft.net>,
	linux-arm-kernel@lists.infradead.org, netdev@vger.kernel.org,
	linux-kernel@vger.kernel.org, srk@ti.com,
	Jacob Keller <jacob.e.keller@intel.com>,
	Roger Quadros <rogerq@ti.com>
Subject: Re: [PATCH net-next v9 2/2] net: ti: icssg_prueth: add TAPRIO
 offload support
Message-ID: <20240603140559.krc6ap5qbltutsvj@skbuf>
References: <20240531044512.981587-3-danishanwar@ti.com>
 <20240531135157.aaxgslyur5br6zkb@skbuf>
 <20240531044512.981587-1-danishanwar@ti.com>
 <20240531044512.981587-1-danishanwar@ti.com>
 <20240531044512.981587-3-danishanwar@ti.com>
 <20240531044512.981587-3-danishanwar@ti.com>
 <20240531135157.aaxgslyur5br6zkb@skbuf>
 <9bcc04a9-645a-4571-a679-ffe67300877a@ti.com>
 <9bcc04a9-645a-4571-a679-ffe67300877a@ti.com>
 <20240603135100.t57lr4u3j6h6zszd@skbuf>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20240603135100.t57lr4u3j6h6zszd@skbuf>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?2hECoWV2KGY03SKSO1FyDvA7CN5/LZssEc+4qTH1Z7ojtikkM8Fucu+RrvCG?=
 =?us-ascii?Q?qfDrx31TVLI7BFri1CTzh+DVLQt2Abi1+f8N9mcqPpq065WULOIhHt5rVA+Y?=
 =?us-ascii?Q?FuGPBZP0rOl6cDSzGAi8MFavI4H7IvYjhHHYYRcLb2DYBesJn6C/1ty3kVB7?=
 =?us-ascii?Q?+samfaiZw2f4KCQqs11AECDmOEQqErGrcdnxywP0r4RavJHXB/AnbYWbI8S0?=
 =?us-ascii?Q?ru+5sEsoJ4THFdPys5+ZRtiK4Rgagc/qmegEPfemQPYJOx4w/G//vPFLfZjF?=
 =?us-ascii?Q?RIM771q6FZwpTbSf0CbuZd0aEbEH2lT++ovvlgy74DNHThe88zZBw09Mf+Uj?=
 =?us-ascii?Q?EuoYwVoJP8rQLyV/yXWPk8tfKSK+1p6RRnY7cFjkhgshTXCtRLrTdP+sFffG?=
 =?us-ascii?Q?P2iFqFrOVHC2yJ8UEtbYx4ICg0v2mwTbREXT9VYxCsksgbyUAO1fP6ggI6nN?=
 =?us-ascii?Q?0s85X6aJpvlXamyRpvqXm48PgYmtqRAqyd5ZCcbdFi7pTVWnQDPMf3CXeO+g?=
 =?us-ascii?Q?zCqHPkuCa7xEzOa1jD5Le16jEdbNd9clmgqlxepwVHDWz5x++OfGkGIFJ67q?=
 =?us-ascii?Q?xYLIUeOe0cEi1KUh/JDncxLw1vOi3k0bcu8f7fVTUunBlTchBcKK+2Jw6DgL?=
 =?us-ascii?Q?41ZXs3hRm4nT4TD6TTsJ1KorvymAtzH7cPpkVhVkXj28+i4ONvVUIQlDAqzU?=
 =?us-ascii?Q?0DJdrkAPNh6R2dJlZ1N8ISlcULOl0buq5kDCyJVqff0KLYIQlhZtAXg4yDEB?=
 =?us-ascii?Q?4t+EAJg6eKmGaMQ5vwIKpJ2qJeUlqIMc1BA31yaLmHnnAqLALXyeBwWIqNBq?=
 =?us-ascii?Q?15O+0S3Ed2ucCgoi0BcBdvHoN17cXX8SfP3OeEAiWqfVL678MQNnDKfgjPTq?=
 =?us-ascii?Q?Ki+8uv1IoQYh+mvgyVy0RiKk/mbwyAsmWrngyDRTH3ZSekUSsA7aSzG5DGiU?=
 =?us-ascii?Q?yk60fhXjUt6Ypg3H5LbFunAoqq6UdCGicZYdLy8Ic6Gy0Ub0m9/qo+aiRGhh?=
 =?us-ascii?Q?wybuKyjWz1Jndxp7qPg7IgZ2wbKuoZ45ZhhpRPWweSZtogXOUmvsPPaTajri?=
 =?us-ascii?Q?x+cL8tN975U0B3Ep2JTtY52JW7fanVoGuRf63LslHUTboprfq0FAYjWrCx2s?=
 =?us-ascii?Q?/Jux4/n30IAeY3fKRsgwRR3kiJT2f+A3Q5yRNgDEbzHKv/nQwms+yj/n0Ehv?=
 =?us-ascii?Q?/WnvGgF2KKzRZ+8pHoSBXQek1z5tsQGseUCIjpblD+GTmlJ/KXedmlwXTWCe?=
 =?us-ascii?Q?GvLkhSf/wceVZbWep2dZvRtwetKyKZxAVn7kKqKCvcfLs0RddhRhFp+/GOIB?=
 =?us-ascii?Q?owsVQ576rKNLPmc2UiB78eYOlfLgmuU+03Fwh+e883xiE2h1Bbu7HkNOYN6Q?=
 =?us-ascii?Q?33aXlD4W2hFlaQjNfFSBDqkSGvisOO3MQZRfRzKodAoxoRSYEKCvdSE98q3p?=
 =?us-ascii?Q?fYwWV7MqCmnnhDD0qSro0HicGeok0+drpm6cRyMHImDEysFSOZdqTEEG3xe1?=
 =?us-ascii?Q?IcxzFb/GnO3EyKNBU02qV4KxwkDMNuaQlSmNQXF4/kPt4w3ojKNzpW0xt4dK?=
 =?us-ascii?Q?AlcdoCQtMNCw52NCW/duNM4ikKq4yigfvc/TA6CTsHVrNAe48wZ9uyMX1/+U?=
 =?us-ascii?Q?hQ=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f1a3ba35-89e8-4b3e-4332-08dc83d64d97
X-MS-Exchange-CrossTenant-AuthSource: AM6PR04MB4551.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Jun 2024 14:06:02.9671
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: bGsbtDcl+vJdwdXF/lsSSgPtSIFeMC8fpnDge0U38Tu1nzjuAR7wGtXKmzRu1oCMMFffQ7hMzsEAbhipQF9vdA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI2PR04MB10571

On Mon, Jun 03, 2024 at 04:51:00PM +0300, Vladimir Oltean wrote:
> > >> +static void tas_reset(struct prueth_emac *emac)
> > >> +{
> > >> +	struct tas_config *tas = &emac->qos.tas.config;
> > >> +	int i;
> > >> +
> > >> +	for (i = 0; i < TAS_MAX_NUM_QUEUES; i++)
> > >> +		tas->max_sdu_table.max_sdu[i] = 2048;
> > > 
> > > Macro + short comment for the magic number, please.
> > > 
> > 
> > Sure I will add it. Each elements in this array is a 2 byte value
> > showing the maximum length of frame to be allowed through each gate.
> 
> Is the queueMaxSDU[] array active even with the TAS being in the reset
> state? Does this configuration have any impact upon the device MTU?
> I don't know why 2048 was chosen.

Another comment here is: in the tc-taprio UAPI, a max-sdu value of 0
is special and means "no maxSDU limit for this TX queue". You are
programming the values from taprio straight away to hardware, so,
assuming there's no bug there, it means that the hardware also
understands 0 to mean "no maxSDU limit".

If so, then during tas_reset(), after which the TAS should be disabled,
why aren't you also using 0 as a default value, but 2048?