[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <DB3PR05MB0764D7198D5FC6D534A90B89AC6E0@DB3PR05MB0764.eurprd05.prod.outlook.com>
Date: Tue, 3 Jan 2017 16:13:46 +0000
From: Yotam Gigi <yotamg@...lanox.com>
To: David Miller <davem@...emloft.net>
CC: "jhs@...atatu.com" <jhs@...atatu.com>,
Elad Raz <eladr@...lanox.com>,
"Jiri Pirko" <jiri@...lanox.com>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>
Subject: RE: [PATCH net-next] net/sched: cls_matchall: Fix error path
>-----Original Message-----
>From: David Miller [mailto:davem@...emloft.net]
>Sent: Tuesday, January 03, 2017 6:08 PM
>To: Yotam Gigi <yotamg@...lanox.com>
>Cc: jhs@...atatu.com; Elad Raz <eladr@...lanox.com>; Jiri Pirko
><jiri@...lanox.com>; netdev@...r.kernel.org
>Subject: Re: [PATCH net-next] net/sched: cls_matchall: Fix error path
>
>From: Yotam Gigi <yotamg@...lanox.com>
>Date: Tue, 3 Jan 2017 17:47:02 +0200
>
>> Fix several error paths in matchall:
>> - Release reference to actions in case the hardware fails offloading
>> (relevant to skip_sw only)
>> - Fix error path in case tcf_exts initialization fails
>>
>> Fixes: bf3994d2ed31 ("net/sched: introduce Match-all classifier")
>> Signed-off-by: Yotam Gigi <yotamg@...lanox.com>
>
>Nothing is checking the tcf_exts_init() return value for errors either,
>and I think you should fix this alongside these release problems at the
>same time.
Ok. Will send v2 soon.
Thanks!
>
>Thanks.
Powered by blists - more mailing lists