Received: by 2002:a05:6a10:a0d1:0:0:0:0 with SMTP id j17csp110047pxa; Fri, 21 Aug 2020 02:33:36 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwMO1HatvT3nYAAVbxOOJUTrxsgpYqZpCBAyqoQfmzzpx+6vbXKoRayHrvSn+C0qRw2a1w8 X-Received: by 2002:a17:906:1c84:: with SMTP id g4mr2163744ejh.50.1598002416751; Fri, 21 Aug 2020 02:33:36 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1598002416; cv=none; d=google.com; s=arc-20160816; b=E7/G+HNLq2t852dtfKIpFmjTJDVMjnGzllR0jrBvNqy0LgGKpu0MYKFBs7edgMMeaN 6vETnWifQDKNTE4jLimMeVFt44xymvM/wJE11QxdtmBqLN/eKgIjh5cWFMOuYTdDQRs4 yEpbc9WnN95GHW7afw9AfY2j43zkjIzRm1Br/dgRy3BaDBfpsw0ahVDemBZUXMuj5ERq 8OMjXRQ+FHmDSz5z2nLf6NBns7zLnaFKMnNipUc3O6ulTaC6zVA5isphmguVlC51PCEM RAe1YWj9mdbtkbxJ6LxLQQ4BrQWc26BgFnV3VLf5MWmOnrCHh7wGC+7dMzxgs/8ZGTIn QWwg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:mail-followup-to :reply-to:message-id:subject:cc:to:from:date; bh=V2YyeRWUiI55RngZlLnam8dhGURsOvH3XWUJPWlY37Q=; b=1Jnu1k4sLqGg1Cddkk8tyUIPIbIrFDzIQo1hLNDNp7yOBlqnrh5K4gqG4mdUdfPf3H zGdlazAaFVEyCLpXz6EbiHq4kALVlnmOTWxEtEyLn6nFEcvFJEw8V4qpdkjbedm0II60 i6VfenfkoS31c5ur/bL4ixznbb+hG5YNe3/bxvHs86/YjV1uv7NtYrGRzcjBssqC1v2y lSb0A5cG3Tmw0MQ1IvNrpd6FEEeu8nhfZgOiZGDZl1eGoHiWwrJm6aP2C1thyv8LZcjH 01MUcUIs/fjWiaR9RLnWbbZZMSr8JwDV+wucQftxuodQ7nKP9mjpt8QrqHigYoGqFxFa sIHQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id c15si768811ejm.289.2020.08.21.02.33.13; Fri, 21 Aug 2020 02:33:36 -0700 (PDT) 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; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728478AbgHUJ3w (ORCPT + 99 others); Fri, 21 Aug 2020 05:29:52 -0400 Received: from mx2.suse.de ([195.135.220.15]:53232 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728095AbgHUJ3w (ORCPT ); Fri, 21 Aug 2020 05:29:52 -0400 X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay2.suse.de (unknown [195.135.221.27]) by mx2.suse.de (Postfix) with ESMTP id C8EC9B189; Fri, 21 Aug 2020 09:30:18 +0000 (UTC) Received: by ds.suse.cz (Postfix, from userid 10065) id D6D5DDA730; Fri, 21 Aug 2020 11:28:44 +0200 (CEST) Date: Fri, 21 Aug 2020 11:28:44 +0200 From: David Sterba To: Tong Zhang Cc: jikos@kernel.org, dsterba@suse.com, gregkh@linuxfoundation.org, jirislaby@kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] tty: ipwireless: fix error handling Message-ID: <20200821092844.GB2026@suse.cz> Reply-To: dsterba@suse.cz Mail-Followup-To: dsterba@suse.cz, Tong Zhang , jikos@kernel.org, dsterba@suse.com, gregkh@linuxfoundation.org, jirislaby@kernel.org, linux-kernel@vger.kernel.org References: <20200818160401.4736-1-ztong0001@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200818160401.4736-1-ztong0001@gmail.com> User-Agent: Mutt/1.5.23.1-rc1 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Aug 18, 2020 at 12:03:58PM -0400, Tong Zhang wrote: > ipwireless_send_packet() can only return 0 on success and -ENOMEM on > error, the caller should check non zero for error condition Thanks. Acked-by: David Sterba --- > v2: - According to Jiri's comment, I made the checking consistent with > the rest of the kernel. I also rebased the code using f684668a24ec. > Thank you Jiri! --- This paragraph should not be in the changelog. The patches to ipwireless go via Greg's tree, please send an updated v3 so he can just apply that without further edits. Thanks.