Thread (10 messages) 10 messages, 3 authors, 2019-01-04

RE: [PATCH] net: tsn: add an netlink interface between kernel and application layer

From: Vinicius Costa Gomes <vinicius.gomes@intel.com>
Date: 2019-01-02 19:01:43
Also in: lkml

Hi Po Liu,

PO LIU [off-list ref] writes:
Hi Vinicius,

Thank you very much for your feedback.

I know the CBS is used to be most important part of AVB. And qdiscs is good tool to configure qos. 

But as you know, the TSN family is a cluster of protocols and much extending the AVB. The protocols have different  functionalities and they may have more than hundred  parameters. For example NXP ls1028a support Qbv/Qci/Qbu/Qav and also the 8021CB (not included in this patch yet).

Some protocols target to configure the traffic class(like Qav CBS).
Some to config the port(like Qbv). But some for the whole ethernet
controller(like Qci, the control entries for the whole controller,
which input ports and which output ports).
Reading your email, now I understand your point a little better. You are
interested in multi-port devices. I admit that I am not too familiar
with how multi-port devices are exposed in Linux, I was only focused on
the end-station use cases, until now.
So I do think all the TSN configuration should not mix in the ethernet
driver itself. I mean the driver should separate a xxx_tsn.c(for I210,
may igb_tsn.c) to maintain the tsn operations.
As far as using qdiscs or the interface of generic netlink. I think
both could configuring the TSN protocols interface layer. Just what I
provided the patch net/tsn/genl_tsn.c. But I do believe it is better
using a standalone TSN middle layer to maintain the TSN capability
ports. Because the TSN ports include not only the end station and also
the switch. LS1028 is such a kind of device.
I think this is the "interesting" part of the discussion. From my point
of view the question now is:

"We already have an acceptable way to expoose TSN features for end
stations. What can we do for multi-port devices?"

What are the options here? From a quick look, it seems that extending
switchdev is a possible solution. What else?

Thinking a little more, if all the ports have netdevices associated with
them, then it could be that exposing those features via qdiscs could be
considered still. Perhaps taking a look at how tc-flower offloading is
done can give some ideas.

And about the process, usually when a new interface is proposed, the
patches are directed to net-next and have the RFC tag, so the readers
(and their tools) know what to expect.
And your advises are precious for us. Let's make out an easy and
flexible interface for TSN.

Br,
Po Liu
Cheers,
--
Vinicius
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help