From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linux.org.uk header.i=@linux.org.uk header.b="uXCBppqR" Received: from zeniv.linux.org.uk (zeniv.linux.org.uk [IPv6:2a03:a000:7:0:5054:ff:fe1c:15ff]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 38E4FD4E; Thu, 23 Nov 2023 23:55:12 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=linux.org.uk; s=zeniv-20220401; h=Sender:In-Reply-To: Content-Transfer-Encoding:Content-Type:MIME-Version:References:Message-ID: Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description; bh=SpnOTVFE3iMqz3KRySppqp/Q/IzJfp9bgY48+5Sf17o=; b=uXCBppqR0PWBWuVqHPbwN/k+sW GgH3oXcHjTtzGlFmeN1H5YGPIX7jqKrtnIUR6LRXMNixFVX9ONXc0jrwZtSGQq9hk6YPzrf37QhWt WamxFoelqO9TT3zqWrySVbI4CGDHEdAVMHn1b/0z6ZuZS6uYdhIxbQxLFoFNvYz4MtlEkWJjmBTHO Wd8oWzGxOVnMlVIwddQcjI6dV9eebpep4/URkwjyhnNI/pOWpLIiUdVGlAso22NyhdJlu5NaXn8LH 9ICgdwB9Gk1mccLNWo/oBN7pMQQqVeX840xDLhqhsYiBjM85w8FamxLKswTyhYcR+UXjkFhN+1UGW iTihpb4Q==; Received: from viro by zeniv.linux.org.uk with local (Exim 4.96 #2 (Red Hat Linux)) id 1r6R1p-002S76-2h; Fri, 24 Nov 2023 07:55:10 +0000 Date: Fri, 24 Nov 2023 07:55:09 +0000 From: Al Viro To: Amir Goldstein Cc: linux-fsdevel@vger.kernel.org, Linus Torvalds , Christian Brauner , linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 03/21] dentry: switch the lists of children to hlist Message-ID: <20231124075509.GU38156@ZenIV> References: <20231124060200.GR38156@ZenIV> <20231124060422.576198-1-viro@zeniv.linux.org.uk> <20231124060422.576198-3-viro@zeniv.linux.org.uk> Precedence: bulk X-Mailing-List: linux-fsdevel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Sender: Al Viro On Fri, Nov 24, 2023 at 09:44:27AM +0200, Amir Goldstein wrote: > On Fri, Nov 24, 2023 at 8:05 AM Al Viro wrote: > > > > Saves a pointer per struct dentry and actually makes the things less > > clumsy. Cleaned the d_walk() and dcache_readdir() a bit by use > > of hlist_for_... iterators. > > > > A couple of new helpers - d_first_child() and d_next_sibling(), > > to make the expressions less awful. > > > > X-fuck-kABI: gladly > > ??? It breaks kABI. Hard. For a good reason. And if you need an elaboration of the reasons why kABI is, er, not universally liked - let's take it to private email. Some rants are really too unprintable for maillists...