From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id CA32CA00C4 for ; Wed, 2 Nov 2022 12:41:48 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id C1D7E40A80; Wed, 2 Nov 2022 12:41:48 +0100 (CET) Received: from relay8-d.mail.gandi.net (relay8-d.mail.gandi.net [217.70.183.201]) by mails.dpdk.org (Postfix) with ESMTP id 1C71840041; Wed, 2 Nov 2022 12:41:46 +0100 (CET) Received: (Authenticated sender: i.maximets@ovn.org) by mail.gandi.net (Postfix) with ESMTPSA id 6C9AD1BF20A; Wed, 2 Nov 2022 11:41:35 +0000 (UTC) Message-ID: <6618b308-842f-ca4d-3252-5496f0e6c571@ovn.org> Date: Wed, 2 Nov 2022 12:41:34 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.4.0 Cc: i.maximets@ovn.org, dev@dpdk.org, Ajit Khaparde , Somnath Kotur , Rahul Lakkireddy , Hemant Agrawal , Sachin Saxena , Simei Su , Wenjun Wu , John Daley , Hyong Youb Kim , Ziyang Xuan , Xiaoyun Wang , Guoyang Zhou , Dongdong Liu , Yisen Zhuang , Yuying Zhang , Beilei Xing , Jingjing Wu , Qiming Yang , Qi Zhang , Junfeng Guo , Rosen Xu , Matan Azrad , Viacheslav Ovsiienko , Liron Himi , Jiawen Wu , Jian Wang , Dekel Peled , Ori Kam , stable@dpdk.org, andrew.rybchenko@oktetlabs.ru, ferruh.yigit@amd.com, bruce.richardson@intel.com, david.marchand@redhat.com Content-Language: en-US To: Thomas Monjalon References: <20221013104849.2677995-1-i.maximets@ovn.org> <3640720.MHq7AAxBmi@thomas> From: Ilya Maximets Subject: Re: [PATCH v2] doc: fix support table for ETH and VLAN flow items In-Reply-To: <3640720.MHq7AAxBmi@thomas> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org On 10/26/22 17:34, Thomas Monjalon wrote: > 13/10/2022 12:48, Ilya Maximets: >> 'has_vlan' attribute is only supported by sfc, mlx5 and cnxk. >> Other drivers doesn't support it. Most of them (like i40e) just >> ignore it silently. Some drivers (like mlx4) never had a full >> support of the eth item even before introduction of 'has_vlan' >> (mlx4 allows to match on the destination MAC only). >> >> Same for the 'has_more_vlan' flag of the vlan item. >> >> 'has_vlan' is part of 'rte_flow_item_eth', so changing 'eth' >> field to 'partial support' in documentation for all such drivers. >> 'has_more_vlan' is part of 'rte_flow_item_vlan', so changing >> 'vlan' to 'partial support' as well. >> >> This doesn't solve the issue, but at least marks the problematic >> drivers. >> >> Some details are available in: >> https://bugs.dpdk.org/show_bug.cgi?id=958 >> >> Fixes: 09315fc83861 ("ethdev: add VLAN attributes to ethernet and VLAN items") >> Cc: stable@dpdk.org >> >> Signed-off-by: Ilya Maximets > > Applied, thanks. Thanks! > > How can we encourage fixing the problematic drivers? That's a tough question and I don't have a good answer. One thought that I have is to make all drivers report a mask for each ITEM. That mask should represent all the bits in the ITEM theoretically supported by the driver. If some driver doesn't report a mask for an ITEM, it is treated as all-zero mask. While installing the flow or on flow validation, the generic layer will check the rte_flow against masks reported by the driver. If there are no extra bits in the flow, it is passed to the driver for further validation and installation. If there are extra bits, flow is rejected at the high level. Further validation is still required because only the driver is able to check dependencies between fields. Once such a checking is merged, it will break offloading for all the drivers. Maintainers will need to create appropriate masks to unblock offloading. memset() on the whole ITEM or similar operations must not be allowed for these masks in the driver code. Thorough review of these patches is necessary. This will not help with encouraging to implement the features per se, but will force maintainers to re-check their drivers. And should eliminate issues where certain fields are silently ignored, because until the driver is updated with a new mask generic validation will reject offloading of flows that contain matches on new fields. At least applications will not fear to use certain features knowing that they will be correctly rejected on validation. Best regards, Ilya Maximets.