Date:   Mon, 10 Feb 2020 15:17:13 -0500
From:   Murali Karicheri <m-karicheri2@...com>
To:     Vladimir Oltean <olteanv@...il.com>
CC:     Vinicius Costa Gomes <vinicius.gomes@...el.com>,
        Po Liu <po.liu@....com>,
        "davem@...emloft.net" <davem@...emloft.net>,
        "hauke.mehrtens@...el.com" <hauke.mehrtens@...el.com>,
        "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>,
        "allison@...utok.net" <allison@...utok.net>,
        "tglx@...utronix.de" <tglx@...utronix.de>,
        "hkallweit1@...il.com" <hkallweit1@...il.com>,
        "saeedm@...lanox.com" <saeedm@...lanox.com>,
        "andrew@...n.ch" <andrew@...n.ch>,
        "f.fainelli@...il.com" <f.fainelli@...il.com>,
        "alexandru.ardelean@...log.com" <alexandru.ardelean@...log.com>,
        "jiri@...lanox.com" <jiri@...lanox.com>,
        "ayal@...lanox.com" <ayal@...lanox.com>,
        "pablo@...filter.org" <pablo@...filter.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
        "simon.horman@...ronome.com" <simon.horman@...ronome.com>,
        Claudiu Manoil <claudiu.manoil@....com>,
        Vladimir Oltean <vladimir.oltean@....com>,
        Alexandru Marginean <alexandru.marginean@....com>,
        Xiaoliang Yang <xiaoliang.yang_1@....com>,
        Roy Zang <roy.zang@....com>, Mingkai Hu <mingkai.hu@....com>,
        Jerry Huang <jerry.huang@....com>, Leo Li <leoyang.li@....com>
Subject: Re: [v1,net-next, 1/2] ethtool: add setting frame preemption of
 traffic classes

Hi Vladimir,

On 01/23/2020 08:30 AM, Vladimir Oltean wrote:
> Hi Murali,
> 
> On Wed, 22 Jan 2020 at 20:04, Murali Karicheri <m-karicheri2@...com> wrote:
>>
>> I have question about the below parameters in The Gate Parameter Table
>> that are not currently supported by tc command. Looks like they need to
>> be shown to user for management.
>>
>>       - ConfigChange - Looks like this needs to be controlled by
>>         user. After sending admin command, user send this trigger to start
>>         copying admin schedule to operation schedule. Is this getting
>>         added to tc command?
> 
> "The ConfigChange parameter signals the start of a
> configuration change for the gate
> when it is set to TRUE. This should only be done
> when the various administrative parameters
> are all set to appropriate values."
> 
> As far as my understanding goes, all tc-taprio commands currently
> behave as though this boolean is implicitly set to TRUE after the
> structures have been set up. I'm not sure there is any value in doing
> otherwise.
> 
That is my understanding as well. However I found this in the 802.1Q
and want to see if someone has insight into this parameter. So perhaps
we can ignore this for now and re-visit when there is a real need for
the same.

>>       - ConfigChangeTime - The time at which the administrative variables
>>         that determine the cycle are to be copied across to the
>>         corresponding operational variables, expressed as a PTP timescale
> 
> This is the base-time of the admin schedule, no?

I think there is cycle time extension possible and in that case,
ConfigChangeTime may be different from Admin  base-time. So this gives
the actual time when the cycle configuration is actually applied.

> 
> "The PTPtime at which the next config change is scheduled to occur.
> The value is a representation of a PTPtime value,
> consisting of a 48-bit integer
> number of seconds and a 32-bit integer number of nanoseconds."
> 
>>       - TickGranularity - the management parameters specified in Gate
>>         Parameter Table allow a management station to discover the
>>         characteristics of an implementation’s cycle timer clock
>>         (TickGranularity) and to set the parameters for the gating cycle
>>         accordingly.
> 
> Not sure who is going to use this and for what purpose, but ok.

Same here. Not sure how this will be used by application.

> 
>>       - ConfigPending - A Boolean variable, set TRUE to indicate that
>>         there is a new cycle configuration awaiting installation.
> 
> I had tried to export something like this (driver calls back into
> sch_taprio.c when hw has applied the config, this would result in
> ConfigPending = FALSE), but ultimately didn't finish the idea, and it
> caused some problems too, due to incorrect RCU usage.
> 

Ok. Hope you plan to send a patch for this in the future.

>>       - ConfigChangeError - Error in configuration (AdminBaseTime <
>>         CurrentTime)
> 
> This can be exported similarly.

Ok.

> 
>>       - SupportedListMax - Maximum supported Admin/Open shed list.
>>
>> Is there a plan to export these from driver through tc show or such
>> command? The reason being, there would be applications developed to
>> manage configuration/schedule of TSN nodes that would requires these
>> information from the node. So would need a support either in tc or
>> some other means to retrieve them from hardware or driver. That is my
>> understanding...
>>
> 
> Not sure what answer you expect to receive for "is there any plan".

To avoid duplicating the work if someone is already working on this.

> You can go ahead and propose something, as long as it is reasonably
> useful to have.
Ok. Will keep in mind.
> 
>> Regards,
>>
>> Murali
>>
>> --
>> Murali Karicheri
>> Texas Instruments
> 
> Thanks,
> -Vladimir
> 

-- 
Murali Karicheri
Texas Instruments

Powered by blists - more mailing lists