From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <yskoh@mellanox.com>
Received: from EUR01-VE1-obe.outbound.protection.outlook.com
 (mail-ve1eur01on0081.outbound.protection.outlook.com [104.47.1.81])
 by dpdk.org (Postfix) with ESMTP id 0F5A21B296;
 Mon,  6 Aug 2018 22:02:03 +0200 (CEST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Mellanox.com;
 s=selector1;
 h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;
 bh=8GJIUoEk+J18OL+9n5rBKYrXs29WFK/xfucx9263z1w=;
 b=M+wdPQCVCOvbrrRfPpCPamw3onNKwe3vhWISkQ+XLTal0VTiMDjusQnIU2JZ9U0iYbusHT8793RHIHyIPDhCUGKVcDcKxIajUjkkDgCvKnN4oYRBN0gXgYeRTjuQGZJLm3Ablt2W52bDh8ftXUc7rCq2CWF/BZBhmZze1R4dr1I=
Authentication-Results: spf=none (sender IP is )
 smtp.mailfrom=yskoh@mellanox.com; 
Received: from yongseok-MBP.local (209.116.155.178) by
 AM5PR0501MB2034.eurprd05.prod.outlook.com (2603:10a6:203:1a::20) with
 Microsoft SMTP Server (version=TLS1_2,
 cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1017.15; Mon, 6 Aug
 2018 20:02:00 +0000
Date: Mon, 6 Aug 2018 13:01:45 -0700
From: Yongseok Koh <yskoh@mellanox.com>
To: Matan Azrad <matan@mellanox.com>
Cc: Shahaf Shuler <shahafs@mellanox.com>, "dev@dpdk.org" <dev@dpdk.org>,
 "stable@dpdk.org" <stable@dpdk.org>
Message-ID: <20180806200144.GB7392@yongseok-MBP.local>
References: <20180802205400.10316-1-yskoh@mellanox.com>
 <AM0PR0502MB4019E730E9E0F5DA7E0333AAD2210@AM0PR0502MB4019.eurprd05.prod.outlook.com>
 <AM0PR0502MB40194D24FDC639CC5A5E641AD2210@AM0PR0502MB4019.eurprd05.prod.outlook.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <AM0PR0502MB40194D24FDC639CC5A5E641AD2210@AM0PR0502MB4019.eurprd05.prod.outlook.com>
User-Agent: Mutt/1.9.3 (2018-01-21)
X-Originating-IP: [209.116.155.178]
X-ClientProxiedBy: MWHPR1201CA0014.namprd12.prod.outlook.com
 (2603:10b6:301:4a::24) To AM5PR0501MB2034.eurprd05.prod.outlook.com
 (2603:10a6:203:1a::20)
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id: d5c625a8-a254-4adc-1d7d-08d5fbd77957
X-MS-Office365-Filtering-HT: Tenant
X-Microsoft-Antispam: BCL:0; PCL:0;
 RULEID:(7020095)(4652040)(8989117)(5600074)(711020)(4618075)(2017052603328)(7153060)(7193020);
 SRVR:AM5PR0501MB2034; 
X-Microsoft-Exchange-Diagnostics: 1; AM5PR0501MB2034;
 3:pGYMHvT+Ru4FVqcZ9E3JOzCpm+cAEJ2vmk7intDE2s97OpL1lhEyCLPmqtjrUlKvEn4ClRRqNCUvjN7H/D19LLy5KRJRPMfsZ9k9YjoWzpnMAmLwopyg+HjhdDfasMUfT7JbbyWUTLTMr+0qDTtlqVZ7YHfVPLe6epGooyxMHL78e81fJInm+dHyPsMDed5tuWiyHWt9LsCVNeSlFSs0uvrS3cVch9J0tlvHw/j1egvBy/7pYfgqA9aFCzSG+a/d;
 25:9xOGLrysC01IujPizbQ9XvQWhqLjoiGyJYaJ/RtvYBa52uJwakGzwlZnoj2XbTRXzR771qKNiOewbbpZcVd8SktbFkAJ/q47EB8Usmq/5E+GcxiPIqewkFgXy0aD6UH4FM3OXzC4pYImL1nvzH9zFWeQp+0kUJfJgJrtLjOpxS3d2l88HQ2yU20t339942BCFn8rgWz8S1uZaTDHfoe9vagELBinjAPSXsz6FZpmm4yKm8kRmv5Gcxp1y6aH8CtnffHm7TL5ysMXLjMmCVtJtc/nMX4O6/1WD8Gi52LDGdc719EslbmhVrMcw/VCdDr4mecNERJZi6KTDRah3b+gnA==;
 31:FHrtdGLMiUnOMNd9lHGLXfBGkXDaxeCiXk0OkGurA3Zqt3SV/1PqM7O1OMsqQnJ6iHShLI9sx+DkjV+KNYJ1L/BYLt0b91uEyzjLqSq1JCpRChqmo2G96GYFDW4Ntj8s3NnxQaU1Y8cLIqDp3eiE6gW8+RYf1UPvJd5SmgFxHdcds3R10orWi6US8y//I3p3gYrwSEeYEp50ew7eqPLrw0DoO/35UbeZgL+/FaXAEGA=
X-MS-TrafficTypeDiagnostic: AM5PR0501MB2034:
X-Microsoft-Exchange-Diagnostics: 1; AM5PR0501MB2034;
 20:RoknlGyD/VgqP1e2AOKF/Mv4Uc3UxG+GC9Fpl/k2j2DaZTuf9BTfY/izrqqMmXCqQ4RZSpUl181+vjTUpO8o7+o45qlXbp1F+5ig/PpGXXiJzxlWC8xm/vVprJ0+YeDl6PJWL/+i8cwZiXyXLlZvLaDb19uco1ZC6NzyGM5SiSNVdG3SDA3CQI0ZwDJ1vbItiWjgH8Qpq7ofGnxaKzwFSD3mAaoxc51MFEnon9Zs+2DgjbRc0sLTOn230q6go6nc7s1gaMOsQM+u41lzfCAeGuOoMli0qBxjGI1dxKbCYjmEzBhaKncCMK4d6UhIZbYoCnPbOXPq4OapcmBWROd4AJ0eM2z9MVu5UUfdSTdWYKhq2tUbaQ4MY6q5Cb/N/EvJL43D6GYE3/ZctTazKIDlw+cFdGvZo7V/1YfRP3XIQfAhJy2Gt5S2JrYvAyWKX3fQq7gKRBoW3RE2sKzTEL1BAij/+NI80D1Ye9V/Vaw3iFM0qLodzzdhtk9EXhnPKuvC;
 4:AWe6mY43+y2/vGSYHKGSZQQvrmuW39ZKXzFPRZdTmyDRAmLp2pjG9PFrWN5r8E92LMNB6/oHRN7Wq8JbuhvJLVUjU2ZSpO8t0RNnNQ0OCHmqO3hXUZ2GvpVuo0WPAKMJ29BRGUy7sN/c9+YlK59L/r6y5IRZM31927iIiZM8fD/1nArLQ7bu00EMAmn7cCsiTvj8oW3sq7TPt8/ClQD8FXsH6Oh+7Yn7es7POSFr+ZdkYdRmCLrYXEHx9U2iRKnn8mtJjKD6DbZLOp0pM+D3CQ==
X-Microsoft-Antispam-PRVS: <AM5PR0501MB20342E2BD7A22314689D9A94C3200@AM5PR0501MB2034.eurprd05.prod.outlook.com>
X-Exchange-Antispam-Report-Test: UriScan:;
X-MS-Exchange-SenderADCheck: 1
X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0;
 RULEID:(8211001083)(6040522)(2401047)(8121501046)(5005006)(93006095)(93001095)(3231311)(944501410)(52105095)(3002001)(10201501046)(6055026)(149027)(150027)(6041310)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123560045)(20161123564045)(20161123562045)(20161123558120)(6072148)(201708071742011)(7699016);
 SRVR:AM5PR0501MB2034; BCL:0; PCL:0; RULEID:; SRVR:AM5PR0501MB2034; 
X-Forefront-PRVS: 07562C22DA
X-Forefront-Antispam-Report: SFV:NSPM;
 SFS:(10009020)(39860400002)(136003)(396003)(376002)(366004)(346002)(189003)(199004)(316002)(58126008)(229853002)(16586007)(33656002)(6246003)(6666003)(6116002)(68736007)(6636002)(3846002)(450100002)(97736004)(5660300001)(25786009)(106356001)(478600001)(6306002)(9686003)(2906002)(966005)(55016002)(53936002)(6862004)(50466002)(1076002)(98436002)(23726003)(8936002)(66066001)(47776003)(86362001)(7696005)(52116002)(76176011)(4326008)(956004)(11346002)(476003)(486006)(16526019)(14444005)(446003)(81166006)(81156014)(54906003)(33896004)(6506007)(386003)(305945005)(7736002)(26005)(105586002)(18370500001);
 DIR:OUT; SFP:1101; SCL:1; SRVR:AM5PR0501MB2034; H:yongseok-MBP.local; FPR:;
 SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1; 
Received-SPF: None (protection.outlook.com: mellanox.com does not designate
 permitted sender hosts)
X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; AM5PR0501MB2034;
 23:kYfRWy5TSxX9uAztWpbPD5QIeLHjFkzd9xZFc3L?=
 =?us-ascii?Q?jHHwRE7QWxRR8bGNE7daU8bQrMoIMR/VSeRxwvSOEDPHcuH2xVxx1LIlyKRd?=
 =?us-ascii?Q?7DeCb2tWF+rz3ntYlTNjL3IgZWK9MfOI7CVHNoxWZwRs/TivUaZi2OdpZPOg?=
 =?us-ascii?Q?7/septNvqFxdk5Llr2d1f+t+sAwQHNTN/mSC0abF/rjWxVE+8ImX5jI8K6R3?=
 =?us-ascii?Q?sv9AAs0I7Rdi72gIq0op1Xce2omVEoAiko2v1lwBIZBK+67Y4MAmyExDiPrn?=
 =?us-ascii?Q?G3Snov13kAC+14+31m5HK/CnTRCUKYIKQJSonsf0PCdBZrHB570qkJ9YWBEm?=
 =?us-ascii?Q?qC/nG3PQFkqZUpz62gRbJiu8BgOwDJ/wVlN/FySZ8RLFBJ0wShdY+gnTWR1X?=
 =?us-ascii?Q?g8YisTpHzWIYlFquEUlPtInTg5/0S0es9HdlO0mzn28+Y8hz7N39/NOq217Q?=
 =?us-ascii?Q?Zl0e293lHawkP56JDDJf8pBfymq9aIdmMsYXDNvNBLEPeYylg7h5KZT2+FjD?=
 =?us-ascii?Q?QRq0MWqq0sSsj1BERcnHSOYjQTxFAYNnUXMZy3PlQWYOgVezGx6nXu1aa0GB?=
 =?us-ascii?Q?QVXwQBVhjtzS+zmIrRXfZRXqB8nxvKn8t73NtPaEIyh4Q67eP3yM8gjpXQ6G?=
 =?us-ascii?Q?dArepc9wk2Z5waeUXCvp8p3nYXMwQmhv91RnxG3bSR1Qsw3pYBLGjax37iPZ?=
 =?us-ascii?Q?3cjtm5wZyuskq3S9xoMGmXQoJ0MY1CI2ks0exvQCI7yyDPlpkgZATALnuzqO?=
 =?us-ascii?Q?1WcGyWy5H0QP4qfiM3f9O0fh/pIyhUj5xsmA8+XBFVHKwA+LhaXliR/eJXMR?=
 =?us-ascii?Q?5yLf0mB0WzCIn6auh7/+0QFlY38YJNTNwzA1iNqp353sW4traGEpnyfZc3G9?=
 =?us-ascii?Q?0QAxKVyfVyRuEOljAJcE3cc6LtnP/x8g8k0zO+yLYRTG2++25R8p3b1yLabt?=
 =?us-ascii?Q?c8hi8vcwH3+LtRtRBlbjF9XJ95/rK2NyXoaolQesi8HLnBVvlaq9FEWpBdo5?=
 =?us-ascii?Q?5vrioybeoaaTGvHwbYXILBy3AQ51lJYog73/KG7ckQaiNm36KCjD7VFVr2Kb?=
 =?us-ascii?Q?IPO8+fEHiWsMOBfiNvtEx7MKMA2yW+tcuwIvz0Q2fVC671i9BupNlVdzaVgj?=
 =?us-ascii?Q?7ta8H3LGoVtDSqbyUhmdjuxSxlwG0zgzXBAVmxp9RSA4tm63JDGAUyE/ai6s?=
 =?us-ascii?Q?aBAvdB7Ej17ydLQ3kNEg3VpEJPES2PSWK6AIqKkZu9GqC1McL/GQBGX4FrZl?=
 =?us-ascii?Q?Qoi/MohWwW5AcgMq1rmUc50fE0fjE8GWca3MC+kE5uAmI5sprRJAnD04ddce?=
 =?us-ascii?Q?wJLiMuIHyKbkLI3nUgbAXLvnzAR9xAhqfaX2aTFVud1bqeLTCSI5OWE6tzg/?=
 =?us-ascii?Q?ozVV1Aw=3D=3D?=
X-Microsoft-Antispam-Message-Info: Ms3RJYuMrw+KTnEeSR06KnrFttIam3M173yxU8SLbGdPQExosIgomObcfbKx/CPDSfz85On5cMeHziW6cH1cCfMmlv4KGT33oy8ryuWgS3dMq3upk3H1PhBQo5Tks8J1zpJlI8ygJzMNpmEZL038+bhaq5s5zsRpEXiQe1OVwI24xzKxiodSxuXG9DvPgdZ5vrDXYIRXBIyx0Ip02LLqHZtuIoAtaLgntXge7Albyjy5LwXbAJo9Jm5MTnKEqOAsZ39/6o8ZkQbcdA23+lZQxbFTGGA+D/jaxBnYi28OqAfxwfNlLDZILg4JxFFGcuw5NL5gLg1XZ46XNNZqkZRB/TBiZa8OJztTRdC76nM6sL0=
X-Microsoft-Exchange-Diagnostics: 1; AM5PR0501MB2034;
 6:Td1AfOymxxXR5UqHvPKf2xNnN4gJ58RPPakZxi+6721cAiuJlucgc2Qb5JZ1LI9xX2F6iK7Do1t7OXkP1DcuT7tZ+3aXB1hdXodJmAPa3vWPPPZ+rOXykZsSC2y7noshwC89Bh7q6gl3hu2IEO/jS4prCqTWci/BsZSV/fxFnlDUFNMMnBGEGoRhvDBSs0nVG7IhJpuLD81wKgCR9RxSzZUeDDol2LGXLhKXDro3m2BXfu8z8psH+vURG2BlND4r3qt7NMcYfTKW5/yylmzJXA445MoULToMrzQKyOE6OJVT6sS7nj1+3DMZkgXN+WPwyn2aP7kOK5R95HdrmzUxs1DHbD/11CS+alLV1sCywi2N1whAOvaC2D9L+EbgEUcJ4PYVoWr8xMx3U44Fa8d4u/mhKFoo5XNhWh8rQXVyC4/0+ZND9KvTyYAt+kcPHafyy92/1pvylv5N6Z3yIHUZJQ==;
 5:HcB0nnccM6IHMTt4OIeFD6CtgEEfH0N2eZUrOuX3fVqEKXTip19KwVDIvPv31I0b2xc8HK1jGNWLjKyZheRdCTP7lC/1wcfD2zQtV9/tg3kxramF7yjn1JvYqx2ftzqMqYzkkU13YE5FHcOaY68mSu4xcLLMnZoUzqoB86TwLnA=;
 7:qmbxG7HC4psmnGl09HhPJZ/kyNj0fPQ6G9ubDlkkYhrceCuNukoa3VlI0xXudz6wQgmlTWwPFjnGiWB/qG1U9tjAw8BVM/n8FHpluJhl3THBQXr/p+T/93L0cklYGizZlWKmK3bQhritmV5MrMGymplCJpICDR9NPwn+F+QJ9MBntj0d/eJJPs7ktP8yjRvUlefQHsyrAHNV/VpYjjD7/ZTux4gHbQY185j0mfWo4FqNDafrc9ViaFUe4FtypH0D
SpamDiagnosticOutput: 1:99
SpamDiagnosticMetadata: NSPM
X-OriginatorOrg: Mellanox.com
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Aug 2018 20:02:00.4317 (UTC)
X-MS-Exchange-CrossTenant-Network-Message-Id: d5c625a8-a254-4adc-1d7d-08d5fbd77957
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: a652971c-7d2e-4d9b-a6a4-d149256f461b
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM5PR0501MB2034
Subject: Re: [dpdk-dev] [PATCH] net/mlx5: prohibit wildcard match for	GRE
 protocol
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Mon, 06 Aug 2018 20:02:03 -0000

On Sun, Aug 05, 2018 at 12:01:01AM -0700, Matan Azrad wrote:
> Hi
> More info below...
> 
> From: Matan Azrad
> > Hi Koh
> > 
> > From: Yongseok Koh
> > > Currently, driver sets full mask (0xffff) for GRE protocol to device.
> > > Until it is changed, PMD can't support wildcard match.
> > >
> > > Fixes: f4b901a46aec ("net/mlx5: add flow GRE item")
> > > Cc: stable@dpdk.org
> > >
> > > Signed-off-by: Yongseok Koh <yskoh@mellanox.com>
> > > ---
> > >  drivers/net/mlx5/mlx5_flow.c | 8 ++++++--
> > >  1 file changed, 6 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/drivers/net/mlx5/mlx5_flow.c
> > > b/drivers/net/mlx5/mlx5_flow.c index
> > > b7500ec9d6..83ac6cbb85 100644
> > > --- a/drivers/net/mlx5/mlx5_flow.c
> > > +++ b/drivers/net/mlx5/mlx5_flow.c
> > > @@ -1690,8 +1690,12 @@ mlx5_flow_item_gre(const struct rte_flow_item
> > > *item,
> > >  					  RTE_FLOW_ERROR_TYPE_ITEM,
> > >  					  item,
> > >  					  "L3 Layer is missing");
> > > -	if (!mask)
> > > -		mask = &rte_flow_item_gre_mask;
> > > +	if (!mask || mask->protocol != 0xffff)
> > 
> > Why do you check !mask again, I think it's redundant.. no?
> > 
> > I think we can support also mask->protocol = 0:
> > In this case we just need to not specify a spec.

Right. My original intention is to force users to use 'is' or 'spec' with full
mask being specified not depending on the default mask. But error message isn't
enough. And users know what the default mask is.

> Just to be more accurate:
> 
> The case of !mask led to mask->protocol = 0xffff before this patch, so it's ok and should be supported.
> In addition, we can support also mask->protocol = 0 (means match all the gre protocol values) by remaining empty spec\mask to the verbs.
> It can be something like this;
> 	if (!mask)
> 		mask = &rte_flow_item_gre_mask;
> 	if (mask->protocol == 0)
> 		spec = NULL;

This shouldn't work. If spec is null, ibv_flow_spec_gre will have zeroed val and
mask. And this will lead to protocol being zero with full mask in the driver.
Also, we have one lacking validation check here - driver doesn't support
c_ks_res0_ver yet.

> 	else if (mask->protocol != 0xffff)
> 		ERROR!
> 
> I checked the next flow:
> eth / ipv4 / gre
> 
> It matched all the gre traffic(MLNX_OFED_LINUX-4.4-2.0.4.0).

I figured out the reason for your false positive. This issue has already been
fixed in the latest MOFED release and the patch is pending for upstream [1].

We should void this temp patch but document it appropriately instead as it can
work well with the MOFED release but not with upstream kernel. Let's take it
offline.


[1] https://patchwork.kernel.org/patch/10498671/


Thanks,
Yongseok