Date:   Sun, 19 Feb 2023 14:58:20 +0200
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     Ferenc Fejes <fejes@inf.elte.hu>
Cc:     netdev@vger.kernel.org, "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        Jamal Hadi Salim <jhs@mojatatu.com>,
        Cong Wang <xiyou.wangcong@gmail.com>,
        Jiri Pirko <jiri@resnulli.us>,
        Vinicius Costa Gomes <vinicius.gomes@intel.com>,
        Kurt Kanzenbach <kurt@linutronix.de>,
        Gerhard Engleder <gerhard@engleder-embedded.com>,
        Amritha Nambiar <amritha.nambiar@intel.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        UNGLinuxDriver@microchip.com, Andrew Lunn <andrew@lunn.ch>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Xiaoliang Yang <xiaoliang.yang_1@nxp.com>,
        linux-kernel@vger.kernel.org,
        Pranavi Somisetty <pranavi.somisetty@amd.com>,
        Harini Katakam <harini.katakam@amd.com>
Subject: Re: [PATCH net-next 00/12] Add tc-mqprio and tc-taprio support for
 preemptible traffic classes
Message-ID: <20230219125820.mw3uchmqr4bvohle@skbuf>
References: <20230216232126.3402975-1-vladimir.oltean@nxp.com>
 <20230218152021.puhz7m26uu2lzved@skbuf>
 <dd782435586a73ada32c099150c274c79e1c3003.camel@inf.elte.hu>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <dd782435586a73ada32c099150c274c79e1c3003.camel@inf.elte.hu>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?ddwJDTHtZNQhGwVZXMQtIcW0zPIfmq3uNBJmx3Y6I4tCbJot7SY897/9opKI?=
 =?us-ascii?Q?Yjw58Fk7UhRqOfJj+gTKA9gvrCpBg39/8ifwAt72QAqBp3SkiL+/Sp/izgCg?=
 =?us-ascii?Q?B060ITYnyofb7EAGTe27hrquAJr/hRcAlxQbteiO92vVZDR+nO5ir2Mpmn5p?=
 =?us-ascii?Q?SARq295EMRTAL/vxV3HuacRH2ORwUsplBFU6EB4naasnf0SudtPHM9wz+hMs?=
 =?us-ascii?Q?4pRlk1lzAYIzMHeugbFegdVk0gQVvHEPT65Wy3Q02IatsPrOuIIXbXBz1AYu?=
 =?us-ascii?Q?cYmbhiEYlmli6MUzk0k82JdEPtdZHNqnKqpGWytX0Z7/2eHF2/TuvVldyH2C?=
 =?us-ascii?Q?FkMQWsXNlqqT5PyLbVaeST66kRzplALcpPPZ+HaMAduXQTsRhneX7H7sNVlJ?=
 =?us-ascii?Q?EQY5QVxDsyXt4GA/uNkWwuOvJP+tszP7BFqAOMyVRXlysxUKDU4ehQtVG71b?=
 =?us-ascii?Q?n6kVpJ2sazXm0T8PHe5/CHMqPuzlKsvBCmT8KDJUCjzRZLvD0/YKAg7i9um3?=
 =?us-ascii?Q?hwwthuSaAabjCgbAuD9C0wvRvwDHOwIICQGwutu5L7GgXP4lBieQOixx18Cd?=
 =?us-ascii?Q?hebrxySpmZ9keaTnXZOx5qUMuXHa8Huy3UFo+fKBfw/aYnm1JA+7fTzZdBfy?=
 =?us-ascii?Q?FHKuIe9gu2lV/OltIElrkyroFfW1A3Jz3RwA3cYZ2a+LTzfK+mCuLRl/8BzR?=
 =?us-ascii?Q?ssTF19Umse4cnYSEHAHibX00Z1BMjY60ubrYiBo2KwKMniOCpqIlxbux0kiS?=
 =?us-ascii?Q?3uSgpLj2qWst3dPH2MkaiN3l51mK+gYKdIjiTWogb7ELiA7HuGEYS3IlRRa2?=
 =?us-ascii?Q?l/Yv5185ecONwH6accvLiCETSbBptBGA5jmf6HCZJQwOt4LB5lS76yttNZwd?=
 =?us-ascii?Q?E/JdgrW/4h9yXyzV+CF38+QTeSp9y+6KmFPUmwvHIOCpEbxZmnbzYYxhxmgt?=
 =?us-ascii?Q?cFeZmo30EodKmiaUJVyRbgvbP1UTudEO+kPp8f66okTWXNjJH+i1O5GB1v6o?=
 =?us-ascii?Q?GBV04wY29J0vRMTP6Y02SOITbdOlk2jSAGWRxNe9uPwTeb9zlVzxFoPtEWg/?=
 =?us-ascii?Q?blOW4i/iYiYi4crGuoXGdwbqHEvOyi7Y5uu613CPqmHwiTDMXCYbQO/r5n5p?=
 =?us-ascii?Q?K4jnqjJB7T03PUNa3pClnV1S9sa55Bp9Knla7P5LQlBQtXFeleGyckng907v?=
 =?us-ascii?Q?15N+jDIqluy/HsNJQ2fByLinYTNIPbjQ8IG9JcmE0oomeF+xFPLdjEHWg5Z8?=
 =?us-ascii?Q?kzZdbIZsda5G/3gbf3VPwb+8Aj2Q5n04sldwF4ub81nsyErufD1kAdMhpnVy?=
 =?us-ascii?Q?wJbSQ/8pl7A28Cf/JUh0zKchuVZTytFZwyEGJLFSo32m/eCKWHbkuvb2gXjG?=
 =?us-ascii?Q?ff9Jukk3HxNXZIBTmI2S3iPxPFgNGG8fFtwrV1mXljV/CAlYHEDs3Uya8/vI?=
 =?us-ascii?Q?ObtyGQQvc3NcH38a9O5eXVk3veu4qzgMafGnZgfHk3YG+OG8tGiqchNIfLHj?=
 =?us-ascii?Q?oNDYXm5k2vtoJ3IW6VZIdVKf56CyD6EMGa3FtALZUvmn77cePAmHdU1DV/GI?=
 =?us-ascii?Q?sH/IsGwKDbP5vO9D1Wp/MfOlc8bWiUtq89M+rZRjQU2L3UT+pJIlav3Zgnqf?=
 =?us-ascii?Q?LQ=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 1df90fb5-eacd-469e-533e-08db1278fc64
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Feb 2023 12:58:24.6303
 (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: RMTcHRgJwX9Fw4+zMRuXhs+ohuonmwk07VOhz79tAsg4HJ53EHUmbXMqMmDhopTkUjvvRPI0hq1Uf++zFjBZFg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM9PR04MB8588
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Ferenc,

On Sun, Feb 19, 2023 at 10:47:31AM +0100, Ferenc Fejes wrote:
> Do you have the iproute2 part? Sorry if I missed it, but it would be
> nice to see how is that UAPI exposed for the config tools. Is there any
> new parameter for mqprio/taprio?

I haven't posted the iproute2 part (yet). For those familiar with my
recent development, FP is a per-traffic-class netlink attribute just
like queueMaxSDU from tc-taprio. That was exposed in iproute2 as an
array of values, one per tc.

What I have in my tree would allow something like this:

tc qdisc replace dev $swp1 root stab overhead 20 taprio \
	num_tc 8 \
	map 0 1 2 3 4 5 6 7 \
	queues 1@0 1@1 1@2 1@3 1@4 1@5 1@6 1@7 \
	base-time 0 \
	sched-entry S 0x7e 900000 \
	sched-entry S 0x82 100000 \
	max-sdu 0 0 0 0 0 0 0 200 \
	fp P E E E E E E E \   # this is new (one entry per tc)
	flags 0x2

tc qdisc replace dev $swp1 root mqprio \
	num_tc 8 \
	map 0 1 2 3 4 5 6 7 \
	queues 1@0 1@1 1@2 1@3 1@4 1@5 1@6 1@7 \
	fp P E E E E E E E \   # this is new (one entry per tc)
	hw 1

of course the exact syntax is a potential matter of debate on its own,
and does not really matter for the purpose of defining the kernel UAPI,
which is why I wanted to keep discussions separate.

For hardware which understands preemptible queues rather than traffic
classes, how many queues are preemptible, and what are their offsets,
will be deduced by translating the "queues" argument.

For hardware which understands preemptible priorities rather than
traffic classes, which priorities are preemptible will be deduced by
translating the "map" argument.

The traffic class is the kernel entity which has the preemptible
priority in my proposed UAPI because this is what my analysis of the
standard has deduced that the preemptible quality is fundamentally
attached to.

Considering that the UAPI for FP is a topic that has been discussed to
death at least since August without any really new input since then, I'm
going to submit v2 later today, and the iproute2 patch set afterwards
(still need to write man page entries for that).