Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     Xiaoliang Yang <xiaoliang.yang_1@nxp.com>
CC:     Vinicius Costa Gomes <vinicius.gomes@intel.com>,
        "davem@davemloft.net" <davem@davemloft.net>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "Arvid.Brodin@xdin.com" <Arvid.Brodin@xdin.com>,
        "m-karicheri2@ti.com" <m-karicheri2@ti.com>,
        "michael.chan@broadcom.com" <michael.chan@broadcom.com>,
        "vishal@chelsio.com" <vishal@chelsio.com>,
        "saeedm@mellanox.com" <saeedm@mellanox.com>,
        "jiri@mellanox.com" <jiri@mellanox.com>,
        "idosch@mellanox.com" <idosch@mellanox.com>,
        "alexandre.belloni@bootlin.com" <alexandre.belloni@bootlin.com>,
        "ivan.khoronzhuk@linaro.org" <ivan.khoronzhuk@linaro.org>,
        "andre.guedes@linux.intel.com" <andre.guedes@linux.intel.com>,
        "allan.nielsen@microchip.com" <allan.nielsen@microchip.com>,
        "joergen.andreasen@microchip.com" <joergen.andreasen@microchip.com>,
        "jhs@mojatatu.com" <jhs@mojatatu.com>
Subject: Re: [EXT] Re: [RFC, net-next] net: qos: introduce a frer action to
 implement 802.1CB
Thread-Topic: [EXT] Re: [RFC, net-next] net: qos: introduce a frer action to
 implement 802.1CB
Thread-Index: AQHXtFznyGH2JvuT50mNvd/a5fV2Jau6CVIAgADGvQCAA4bJAA==
Date:   Fri, 1 Oct 2021 16:17:11 +0000
Message-ID: <20211001161710.2sdz6o6lh3yg7k6p@skbuf>
References: <20210928114451.24956-1-xiaoliang.yang_1@nxp.com>
 <87czos9vnj.fsf@linux.intel.com>
 <DB8PR04MB5785F3128FEB1FB1B2F9AC0DF0A99@DB8PR04MB5785.eurprd04.prod.outlook.com>
In-Reply-To: <DB8PR04MB5785F3128FEB1FB1B2F9AC0DF0A99@DB8PR04MB5785.eurprd04.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?YL8cp9Km921ukK/3Zwb6SwXHyrMO5u7xg83wyU7YLrQLWjNs8DwXU2mRcSBc?=
 =?us-ascii?Q?S9WEOQ+QLZqbuDIzL5FDgOngd7s2v5bXOjVdQYCGJTTZNlXyDFXXDmZZRUl8?=
 =?us-ascii?Q?cNWjRkSNaoM0WZfjnFM14ATApQVv7WyigwKB5sue1ZN1+TFWTHlBiw0vskcm?=
 =?us-ascii?Q?HbDgfZYOWzhcbCm0D4PgzeXiTdrpDlkYdIernll8HHOZnDl0p5M55qWu2/GV?=
 =?us-ascii?Q?DwBkl+JGZmh0ZNHymYynu+SOvLxM/niUhRvKscf+gbLOislCwIn8rflhCbFH?=
 =?us-ascii?Q?kIqitU5FJ4x2ojsdj4PwqlINq5Br1zcDzm1WXAItJnvHJgwKW/XKTf4PUDDp?=
 =?us-ascii?Q?ephB4w/gRkzZg6MYbZ/Xhajhz6k2aFggYo2VfSgg9n3FfGIa5ZYTG1EkyrQ0?=
 =?us-ascii?Q?p86PBu6XlY4KRsIsMNSwhMkO6+wrDorKdRa6kYMLvfBiGIzt/j0rRa/JyCbz?=
 =?us-ascii?Q?4cfrIYre12OmJAmzZZLQPc+vjq63bl2nY7oO9qQEUUwRLokT5E5gj+611mpb?=
 =?us-ascii?Q?NoaClv2B1BeWo6PxOBlOmC8ysJd5OUYhwmlhMdQlKt4PJZmCDCFRzNxwZ+EH?=
 =?us-ascii?Q?pNaoHzSgtPEYhDvRnquBHozJ4N954xYf5NYqcGUMbMNFkfQBPXEh3pDc7P5m?=
 =?us-ascii?Q?fDvEQZKElVdpZSruKXiNpCoX/bvAe4Wvvu/Y1i6S9McX2sGVloLe+kax1XTK?=
 =?us-ascii?Q?IkKNTFZwrFAKQk5Kv8sGlV5hJspzRlUejIDeOgqYlIWf9HlfFXJ/9b0fzzZR?=
 =?us-ascii?Q?wEybmBd7CByfNQkMiM8HBmLg7HoDciycAweb9KOeabynBweh7PfPxw6RuSch?=
 =?us-ascii?Q?WmzAu2GPz+NYXOv5rzai7PqKgVgu6ADCE87kAPQoxDUIP8VAgK3R0Lufg2ak?=
 =?us-ascii?Q?F5ZhW8gdmAVf7d2eiZqWI7ZW62OR4U9HFZzIDp1rweL6POtgusVcDQ/BJmKG?=
 =?us-ascii?Q?i8hnOCalcu5hi/Ttbl4OlCjeoEJCPYY4dgRvPRBfM7H1/PHtgTEutbWp4akK?=
 =?us-ascii?Q?cL4XlkWIpNQ1wBItuYyKEFfNRLV7IQCsIEQ3JrE8OMIlg07P+drYXlQPrxx5?=
 =?us-ascii?Q?7Pvpu6P5iJgtnmQTp6KqVsn3NUYNnS5X965uooI8X6fFEIfN+jIWv90qmwGI?=
 =?us-ascii?Q?ITfPdniF9y+c3h7qMp637mmk8hTAsLPDVn5tZKHrzfNrG9NEsskYQ+RblWNr?=
 =?us-ascii?Q?4+hY+HKTySZtFzcNx0xppklXVdHjz5d/AAAtrG8QYEVhRuoWnZ5Jp5yXMDyz?=
 =?us-ascii?Q?mlJRG3v/3TBuXHavpoXCcVadmWVCSKdF/fioQsuUKijqwL3QAHUHzMTKGiVV?=
 =?us-ascii?Q?A0IW6EvjV2W5zjREaUctAMsA?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <8BA9500280EDB04E8D5A3434AC3896A1@eurprd04.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f497b6f5-8f3b-4988-fbae-08d984f6ec75
X-MS-Exchange-CrossTenant-originalarrivaltime: 01 Oct 2021 16:17:11.1773
 (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: sRAATmoNkENWtFNpL6BAAXyeAocWGH6aKl2Nu0eOk8errl54QEnwsToN3mNS8KtknriJluXANrRnAadzrcKI7Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VE1PR04MB7471
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Vinicius,

On Wed, Sep 29, 2021 at 10:25:58AM +0000, Xiaoliang Yang wrote:
> Hi Vinicius,
>
> On Sep 29, 2021 at 6:35:59 +0000, Vinicius Costa Gomes wrote:
> > > This patch introduce a frer action to implement frame replication and
> > > elimination for reliability, which is defined in IEEE P802.1CB.
> > >
> >
> > An action seems, to me, a bit too limiting/fine grained for a frame rep=
lication
> > and elimination feature.
> >
> > At least I want to hear the reasons that the current hsr/prp support ca=
nnot be
> > extended to support one more tag format/protocol.
> >
> > And the current name for the spec is IEEE 802.1CB-2017.
> >
> 802.1CB can be set on bridge ports, and need to use bridge forward
> Function as a relay system. It only works on identified streams,
> unrecognized flows still need to pass through the bridged network
> normally.
>
> But current hsr/prp seems only support two ports, and cannot use the
> ports in bridge. It's hard to implement FRER functions on current HSR
> driver.
>
> You can see chapter "D.2 Example 2: Various stack positions" in IEEE 802.=
1CB-2017,
> Protocol stack for relay system is like follows:
>
>              Stream Transfer Function
>                 |             |
>                 |        Sequence generation
>                 |            Sequence encode/decode
>   Stream identification        Active Stream identification
>                 |              |
>                 |        Internal LAN---- Relay system forwarding
>                 |                        |        |
>                 MAC                    MAC        MAC
>
> Use port actions to easily implement FRER tag add/delete, split, and
> recover functions.
>
> Current HSR/PRP driver can be used for port HSR/PRP set, and tc-frer
> Action to be used for stream RTAG/HSR/PRP set and recover.

Did Xiaoliang answer your question satisfactorily? :)=