From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.9 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id E9590C34026 for ; Tue, 18 Feb 2020 18:28:44 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id C115E21D56 for ; Tue, 18 Feb 2020 18:28:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1582050524; bh=jJkCDGG5ZFNXNyHV2P6h6E/KRBQPh8St4HaG6x44ZrY=; h=References:In-Reply-To:From:Date:Subject:To:Cc:List-ID:From; b=mQv7Jox2MOV554PGINS4L9nXvLSSWnwdsortPjYHQqwGCz/0MwgKWt1O7vnCZTsqp m5Ue5H/L4XYmQfE3nZ8dwXscft8gIkcq7FhinLCW453EwNBvxw4DOdHGy+wHYUPVDc +3pDPA0pv3032HB6Qw2zfVMM1QxA+ShtMzren+X8= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726439AbgBRS2o (ORCPT ); Tue, 18 Feb 2020 13:28:44 -0500 Received: from mail-lj1-f196.google.com ([209.85.208.196]:33906 "EHLO mail-lj1-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726225AbgBRS2o (ORCPT ); Tue, 18 Feb 2020 13:28:44 -0500 Received: by mail-lj1-f196.google.com with SMTP id x7so24131056ljc.1 for ; Tue, 18 Feb 2020 10:28:42 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux-foundation.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=+QhJ1rq4OwV2iGSlB588wtXJvujKNK9tlxfzQJwyHS8=; b=ejxkrareXzY6YTUjQpMNQmMbBAVxAeHZpnsjqRJOfVCFhXRJpBgATWPZbLfOiAxW/a th7vIR385GnvQUqIU39rpkayyQ2QV5OZ329vYyqfpQ0PO3TGv7Tsc77viMB/prpd/EN7 mxfaTk+8rftAr7S1xcTb0hs67cWZ0/zooHpA0= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=+QhJ1rq4OwV2iGSlB588wtXJvujKNK9tlxfzQJwyHS8=; b=Uf8ytzZlg8xx37cD7nHvXpmESyok7whfU9t24BoSloc7ECjpj+Gh2LAsIls7luHd2J hrKovWPX/2viIj6OIf+qLo4fRG5k3Hh8Y1T+G7O6Yn2FaO+wwtRETMBL2sxYSDiZ1JnC VThFRM8zQxEyOlzAiuDjJAijfuU33lRw1PdnNkbMmR05GhMl4BKQTwlohUwB7ll2Kguq GEWWeDC5RSIOCxqqCbWRXAdxwthxalfdCFPraP3dF3FcL9N/c22JNkP73vbdLGZJHEgD yJlShlKQMrdImlVhBfYFvrnCCTrxrxS4aFyHMXD5M1HCAE71EL1jQXtMo0i1LFZc1xB6 HUaw== X-Gm-Message-State: APjAAAWwTs2ywBTAOV/FVyodpyTf2WvjyrV09cv7IWoGopF/4Nbl49EV 46lo9M/T7WMrsA4BLMm9DHGv6iCY4bA= X-Google-Smtp-Source: APXvYqwroLmxhdfIrA3qS4jH3UMK2N3B1h1UWrA/jcMHNqdjNossfzk/NNqMgNGaorzyFf/T+7J3YA== X-Received: by 2002:a2e:9143:: with SMTP id q3mr12951898ljg.199.1582050521630; Tue, 18 Feb 2020 10:28:41 -0800 (PST) Received: from mail-lf1-f42.google.com (mail-lf1-f42.google.com. [209.85.167.42]) by smtp.gmail.com with ESMTPSA id p26sm2719663lfh.64.2020.02.18.10.28.39 for (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 18 Feb 2020 10:28:40 -0800 (PST) Received: by mail-lf1-f42.google.com with SMTP id v201so15203253lfa.11 for ; Tue, 18 Feb 2020 10:28:39 -0800 (PST) X-Received: by 2002:a19:f514:: with SMTP id j20mr11229869lfb.31.1582050519448; Tue, 18 Feb 2020 10:28:39 -0800 (PST) MIME-Version: 1.0 References: <20200214154854.6746-1-sashal@kernel.org> <20200214154854.6746-542-sashal@kernel.org> <20200218182041.GB24185@bombadil.infradead.org> In-Reply-To: <20200218182041.GB24185@bombadil.infradead.org> From: Linus Torvalds Date: Tue, 18 Feb 2020 10:28:23 -0800 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH AUTOSEL 5.5 542/542] pipe: use exclusive waits when reading or writing To: Matthew Wilcox Cc: Andrei Vagin , LKML , Sasha Levin , stable , Josh Triplett , linux-fsdevel Content-Type: multipart/mixed; boundary="0000000000003aa18b059eddd731" Sender: stable-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org --0000000000003aa18b059eddd731 Content-Type: text/plain; charset="UTF-8" On Tue, Feb 18, 2020 at 10:20 AM Matthew Wilcox wrote: > > You don't want to move wake_up_partner() up and call it from pipe_release()? I was actually thinking of going the other way - two of three users of wake_up_partner() are redundantly waking up the wrong side, and the third user is pointlessly written too. So I was _thinking_ of a patch like the appended (which is on top of the previous patch), but ended up not doing it. Until you brought it up. But I won't bother committing this, since it shouldn't really matter. Linus --0000000000003aa18b059eddd731 Content-Type: text/x-patch; charset="US-ASCII"; name="patch.diff" Content-Disposition: attachment; filename="patch.diff" Content-Transfer-Encoding: base64 Content-ID: X-Attachment-Id: f_k6s80fwq0 IGZzL3BpcGUuYyB8IDE4ICsrKysrKy0tLS0tLS0tLS0tLQogMSBmaWxlIGNoYW5nZWQsIDYgaW5z ZXJ0aW9ucygrKSwgMTIgZGVsZXRpb25zKC0pCgpkaWZmIC0tZ2l0IGEvZnMvcGlwZS5jIGIvZnMv cGlwZS5jCmluZGV4IDIxNDQ1MDc0NDdjNS4uNzliYTYxNDMwZjljIDEwMDY0NAotLS0gYS9mcy9w aXBlLmMKKysrIGIvZnMvcGlwZS5jCkBAIC0xMDI1LDEyICsxMDI1LDYgQEAgc3RhdGljIGludCB3 YWl0X2Zvcl9wYXJ0bmVyKHN0cnVjdCBwaXBlX2lub2RlX2luZm8gKnBpcGUsIHVuc2lnbmVkIGlu dCAqY250KQogCXJldHVybiBjdXIgPT0gKmNudCA/IC1FUkVTVEFSVFNZUyA6IDA7CiB9CiAKLXN0 YXRpYyB2b2lkIHdha2VfdXBfcGFydG5lcihzdHJ1Y3QgcGlwZV9pbm9kZV9pbmZvICpwaXBlKQot ewotCXdha2VfdXBfaW50ZXJydXB0aWJsZV9hbGwoJnBpcGUtPnJkX3dhaXQpOwotCXdha2VfdXBf aW50ZXJydXB0aWJsZV9hbGwoJnBpcGUtPndyX3dhaXQpOwotfQotCiBzdGF0aWMgaW50IGZpZm9f b3BlbihzdHJ1Y3QgaW5vZGUgKmlub2RlLCBzdHJ1Y3QgZmlsZSAqZmlscCkKIHsKIAlzdHJ1Y3Qg cGlwZV9pbm9kZV9pbmZvICpwaXBlOwpAQCAtMTA3OCw3ICsxMDcyLDcgQEAgc3RhdGljIGludCBm aWZvX29wZW4oc3RydWN0IGlub2RlICppbm9kZSwgc3RydWN0IGZpbGUgKmZpbHApCiAJICovCiAJ CXBpcGUtPnJfY291bnRlcisrOwogCQlpZiAocGlwZS0+cmVhZGVycysrID09IDApCi0JCQl3YWtl X3VwX3BhcnRuZXIocGlwZSk7CisJCQl3YWtlX3VwX2ludGVycnVwdGlibGVfYWxsKCZwaXBlLT53 cl93YWl0KTsKIAogCQlpZiAoIWlzX3BpcGUgJiYgIXBpcGUtPndyaXRlcnMpIHsKIAkJCWlmICgo ZmlscC0+Zl9mbGFncyAmIE9fTk9OQkxPQ0spKSB7CkBAIC0xMTA0LDcgKzEwOTgsNyBAQCBzdGF0 aWMgaW50IGZpZm9fb3BlbihzdHJ1Y3QgaW5vZGUgKmlub2RlLCBzdHJ1Y3QgZmlsZSAqZmlscCkK IAogCQlwaXBlLT53X2NvdW50ZXIrKzsKIAkJaWYgKCFwaXBlLT53cml0ZXJzKyspCi0JCQl3YWtl X3VwX3BhcnRuZXIocGlwZSk7CisJCQl3YWtlX3VwX2ludGVycnVwdGlibGVfYWxsKCZwaXBlLT5y ZF93YWl0KTsKIAogCQlpZiAoIWlzX3BpcGUgJiYgIXBpcGUtPnJlYWRlcnMpIHsKIAkJCWlmICh3 YWl0X2Zvcl9wYXJ0bmVyKHBpcGUsICZwaXBlLT5yX2NvdW50ZXIpKQpAQCAtMTEyMCwxMiArMTEx NCwxMiBAQCBzdGF0aWMgaW50IGZpZm9fb3BlbihzdHJ1Y3QgaW5vZGUgKmlub2RlLCBzdHJ1Y3Qg ZmlsZSAqZmlscCkKIAkgKiAgdGhlIHByb2Nlc3MgY2FuIGF0IGxlYXN0IHRhbGsgdG8gaXRzZWxm LgogCSAqLwogCi0JCXBpcGUtPnJlYWRlcnMrKzsKLQkJcGlwZS0+d3JpdGVycysrOworCQlpZiAo cGlwZS0+cmVhZGVycysrID09IDApCisJCQl3YWtlX3VwX2ludGVycnVwdGlibGVfYWxsKCZwaXBl LT53cl93YWl0KTsKKwkJaWYgKHBpcGUtPndyaXRlcnMrKyA9PSAwKQorCQkJd2FrZV91cF9pbnRl cnJ1cHRpYmxlX2FsbCgmcGlwZS0+cmRfd2FpdCk7CiAJCXBpcGUtPnJfY291bnRlcisrOwogCQlw aXBlLT53X2NvdW50ZXIrKzsKLQkJaWYgKHBpcGUtPnJlYWRlcnMgPT0gMSB8fCBwaXBlLT53cml0 ZXJzID09IDEpCi0JCQl3YWtlX3VwX3BhcnRuZXIocGlwZSk7CiAJCWJyZWFrOwogCiAJZGVmYXVs dDoK --0000000000003aa18b059eddd731--