Received: by 2002:a25:824b:0:0:0:0:0 with SMTP id d11csp829858ybn; Tue, 24 Sep 2019 10:08:16 -0700 (PDT) X-Google-Smtp-Source: APXvYqxJf9NyVcAYj8h/CK3Q2jWCQxD4BUe9XA74a4U0Qw/t/dm8iKz4MC65UGNqTmVXFq5MyGaH X-Received: by 2002:a1c:7914:: with SMTP id l20mr1204413wme.155.1569344896606; Tue, 24 Sep 2019 10:08:16 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1569344896; cv=none; d=google.com; s=arc-20160816; b=ku8xf/NvORFZsjyrterIgQrFECAG91m8sgzvXP8berwYnGyddAxGAJ94+C74Yyltqv RLDVSIoBpcRpSc/rSq0dHf1au0cgg4TXemPy0bqEYQ2EWL3fTaQi7xmm5f5BmmwwfnVJ hBvXRr9bMmsBDeK1rdQTQg4Ie5tPr2wo1ttwPDlrG1zhqwlH9ty2HF5AV5gogE/Qzrmn LK8z0LmESCzgftZtOY1QU/vGXiwRwLQXHSmNjXcAJZFSfxvDqbbnS2BBuoX+FF8sK5a2 G/2XIpqJ3xptl51Z5mKDCX6NWnfiFOAI6TErxGcPQBg1NxvFRsNtXmpT/BVlEALBjYNT 8q3A== 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:message-id:subject:cc :to:from:date; bh=NkEOoYFdo9d/uoxWYvOIZ9BDP4nFyFqOUCtve3sJ6J8=; b=aClAMd3NQeE2Pzps29CjLaBieY11cTas47/Ek5x5L6RbyxcUPPSoQJ3+ixLW2RNQ51 SLHOo4yhv5x/LoKmHpNTrvmoilXthudVeQhyTgVQUt1RXm+sarZ0nwFHl3h1z1zubO3w PHoDTUzYoYqkZ7ZLokFEInRk+ZZDdNHuFvbEDHOKvpY400lEV8gJh76mmetJA/hpt8QR GaSOc43Mi2iVwV/bQAk6XxscMJG2/A5nOqXfardGHFqIEYsz25jbvnEIRSut7aaWhGaF xUwZWuJI1Q7VhkhOrZm7j9SvrW99Hxiv1dAsVTBgX3pBKXBJ5QIJVlXxXksHc0aOlbp+ aU7g== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-ext4-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-ext4-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id l54si1445250edb.206.2019.09.24.10.07.51; Tue, 24 Sep 2019 10:08:16 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-ext4-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-ext4-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-ext4-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2439616AbfIWNTF (ORCPT + 99 others); Mon, 23 Sep 2019 09:19:05 -0400 Received: from outgoing-auth-1.mit.edu ([18.9.28.11]:51834 "EHLO outgoing.mit.edu" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S2439591AbfIWNTF (ORCPT ); Mon, 23 Sep 2019 09:19:05 -0400 Received: from callcc.thunk.org (guestnat-104-133-0-98.corp.google.com [104.133.0.98] (may be forged)) (authenticated bits=0) (User authenticated as tytso@ATHENA.MIT.EDU) by outgoing.mit.edu (8.14.7/8.12.4) with ESMTP id x8NDIEHw002725 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Mon, 23 Sep 2019 09:18:15 -0400 Received: by callcc.thunk.org (Postfix, from userid 15806) id 7F45B420811; Mon, 23 Sep 2019 09:18:14 -0400 (EDT) Date: Mon, 23 Sep 2019 09:18:14 -0400 From: "Theodore Y. Ts'o" To: Jan Kara Cc: Goldwyn Rodrigues , linux-fsdevel@vger.kernel.org, linux-ext4@vger.kernel.org, linux-btrfs@vger.kernel.org, hch@infradead.org, andres@anarazel.de, david@fromorbit.com, riteshh@linux.ibm.com, linux-f2fs-devel@lists.sourceforge.net, Goldwyn Rodrigues Subject: Re: [PATCH 2/3] ext4: fix inode rwsem regression Message-ID: <20190923131814.GB6005@mit.edu> References: <20190911093926.pfkkx25mffzeuo32@alap3.anarazel.de> <20190911164517.16130-1-rgoldwyn@suse.de> <20190911164517.16130-3-rgoldwyn@suse.de> <20190923101042.GA25332@quack2.suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190923101042.GA25332@quack2.suse.cz> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-ext4-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-ext4@vger.kernel.org On Mon, Sep 23, 2019 at 12:10:42PM +0200, Jan Kara wrote: > On Wed 11-09-19 11:45:16, Goldwyn Rodrigues wrote: > > From: Goldwyn Rodrigues > > > > This is similar to 942491c9e6d6 ("xfs: fix AIM7 regression") > > Apparently our current rwsem code doesn't like doing the trylock, then > > lock for real scheme. So change our read/write methods to just do the > > trylock for the RWF_NOWAIT case. > > > > Fixes: 728fbc0e10b7 ("ext4: nowait aio support") > > Signed-off-by: Goldwyn Rodrigues > > Thanks for fixing this! The patch looks good to me. You can add: > > Reviewed-by: Jan Kara > > BTW, I've also added Ted as ext4 maintainer to CC. Thanks, I've been following along, and once the merge window is over I'll start going through the patch backlog. Cheers, - Ted