[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <20200102.163302.1487790740822950185.davem@davemloft.net>
Date: Thu, 02 Jan 2020 16:33:02 -0800 (PST)
From: David Miller <davem@...emloft.net>
To: po.liu@....com
Cc: linux-kernel@...r.kernel.org, netdev@...r.kernel.org,
vinicius.gomes@...el.com, claudiu.manoil@....com,
vladimir.oltean@....com, alexandru.marginean@....com,
xiaoliang.yang_1@....com, roy.zang@....com, mingkai.hu@....com,
jerry.huang@....com, leoyang.li@....com, ivan.khoronzhuk@...aro.org
Subject: Re: [v3,net-next] enetc: add support time specific departure base
on the qos etf
From: Po Liu <po.liu@....com>
Date: Thu, 2 Jan 2020 04:59:24 +0000
> ENETC implement time specific departure capability, which enables
> the user to specify when a frame can be transmitted. When this
> capability is enabled, the device will delay the transmission of
> the frame so that it can be transmitted at the precisely specified time.
> The delay departure time up to 0.5 seconds in the future. If the
> departure time in the transmit BD has not yet been reached, based
> on the current time, the packet will not be transmitted.
>
> This driver was loaded by Qos driver ETF. User could load it by tc
> commands. Here are the example commands:
>
> tc qdisc add dev eth0 root handle 1: mqprio \
> num_tc 8 map 0 1 2 3 4 5 6 7 hw 1
> tc qdisc replace dev eth0 parent 1:8 etf \
> clockid CLOCK_TAI delta 30000 offload
>
> These example try to set queue mapping first and then set queue 7
> with 30us ahead dequeue time.
>
> Then user send test frame should set SO_TXTIME feature for socket.
>
> There are also some limitations for this feature in hardware:
> - Transmit checksum offloads and time specific departure operation
> are mutually exclusive.
> - Time Aware Shaper feature (Qbv) offload and time specific departure
> operation are mutually exclusive.
>
> Signed-off-by: Po Liu <Po.Liu@....com>
> Reviewed-by: Vladimir Oltean <vladimir.oltean@....com>
> ---
> Changes v2-v3:
> - Avoid tx checking sum offload when setting TXTIME offload. This is
> not support in hardware.
This looks a lot better, applied, thank you.
Powered by blists - more mailing lists