From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jakub Kicinski Subject: Re: [PATCH bpf] xsk: fix return value of xdp_umem_assign_dev() Date: Thu, 23 Aug 2018 20:29:38 +0200 Message-ID: <20180823202938.17b036f7@cakuba.netronome.com> References: <20180820005425.2604-1-bhole_prashant_q7@lab.ntt.co.jp> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: Alexei Starovoitov , Daniel Borkmann , =?UTF-8?B?QmrDtnJuIFTDtnBlbA==?= , Magnus Karlsson , "David S . Miller" , netdev@vger.kernel.org To: Prashant Bhole Return-path: Received: from mail-qk0-f195.google.com ([209.85.220.195]:32786 "EHLO mail-qk0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726780AbeHWWAl (ORCPT ); Thu, 23 Aug 2018 18:00:41 -0400 Received: by mail-qk0-f195.google.com with SMTP id z78-v6so4281187qka.0 for ; Thu, 23 Aug 2018 11:29:45 -0700 (PDT) In-Reply-To: <20180820005425.2604-1-bhole_prashant_q7@lab.ntt.co.jp> Sender: netdev-owner@vger.kernel.org List-ID: On Mon, 20 Aug 2018 09:54:25 +0900, Prashant Bhole wrote: > s/ENOTSUPP/EOPNOTSUPP/ in function umem_assign_dev(). > This function's return value is directly returned by xsk_bind(). > EOPNOTSUPP is bind()'s possible return value. > > Fixes: f734607e819b ("xsk: refactor xdp_umem_assign_dev()") > Signed-off-by: Prashant Bhole FWIW the refactoring commit just cleaned up the code, is it worth submitting a patch to stable to correct 4.18 as well?