From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754636AbcBGUjZ (ORCPT ); Sun, 7 Feb 2016 15:39:25 -0500 Received: from tiger.mobileactivedefense.com ([217.174.251.109]:48522 "EHLO tiger.mobileactivedefense.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753314AbcBGUjX (ORCPT ); Sun, 7 Feb 2016 15:39:23 -0500 From: Rainer Weikusat To: Eric Dumazet Cc: Rainer Weikusat , davem@davemloft.net, hannes@stressinduktion.org, edumazet@google.com, dhowells@redhat.com, ying.xue@windriver.com, "netdev\@vger.kernel.org" , LKML , "stable\@vger.kernel.org" , Joseph Salisbury Subject: Re: [V4.4-rc6 Regression] af_unix: Revert 'lock_interruptible' in stream receive code In-Reply-To: <8737t4jr99.fsf@doppelsaurus.mobileactivedefense.com> (Rainer Weikusat's message of "Sun, 07 Feb 2016 18:43:14 +0000") References: <56B4EF04.2060407@canonical.com> <87bn7ude8g.fsf@doppelsaurus.mobileactivedefense.com> <1454713398.7627.325.camel@edumazet-glaptop2.roam.corp.google.com> <8737t4jr99.fsf@doppelsaurus.mobileactivedefense.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.4 (gnu/linux) Date: Sun, 07 Feb 2016 20:39:02 +0000 Message-ID: <8760y0l0gp.fsf@doppelsaurus.mobileactivedefense.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.4.3 (tiger.mobileactivedefense.com [217.174.251.109]); Sun, 07 Feb 2016 20:39:12 +0000 (GMT) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Rainer Weikusat writes: [...] > the real problem is that the function disagrees with itself on how to > use the err variable: The start uses that to record an error which > might need to be reported, the return statement uses it to indicate > that an error has occurred. This should have been "as indication that an errors has occured" (if no data was copied).