From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ilias Apalodimas Subject: Re: [PATCH] net: tsn: add an netlink interface between kernel and application layer Date: Thu, 3 Jan 2019 11:16:05 +0200 Message-ID: <20190103091605.GA4597@apalos> References: <1545968772-7237-1-git-send-email-Po.Liu@nxp.com> <1545968945-7290-1-git-send-email-Po.Liu@nxp.com> <87r2e14fgr.fsf@intel.com> <87k1jm51ey.fsf@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: PO LIU , "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "davem@davemloft.net" , "haustad@cisco.com" , "nicolas.ferre@microchip.com" , "gregkh@linuxfoundation.org" , Mingkai Hu , Roy Zang To: Vinicius Costa Gomes Return-path: Content-Disposition: inline In-Reply-To: <87k1jm51ey.fsf@intel.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org Hi Po, > > 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. Have you considered a switchdev-based driver for multi-port devices? What you ask of TSN configuration is currently doable with switch switchdev for VLANs and other similar networking functionality. Instead of rewriting this from scratch, we not extend the currect TC and switchdev functionality for that ? Regards /Ilias