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 Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1ED67C3DA6E for ; Thu, 28 Dec 2023 16:58:42 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 4F7506B00E8; Thu, 28 Dec 2023 11:58:42 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 4A72F6B00E9; Thu, 28 Dec 2023 11:58:42 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 3BD216B00EA; Thu, 28 Dec 2023 11:58:42 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id 2ACCA6B00E8 for ; Thu, 28 Dec 2023 11:58:42 -0500 (EST) Received: from smtpin01.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id ECBE0A01F5 for ; Thu, 28 Dec 2023 16:58:41 +0000 (UTC) X-FDA: 81616836042.01.E330144 Received: from sin.source.kernel.org (sin.source.kernel.org [145.40.73.55]) by imf30.hostedemail.com (Postfix) with ESMTP id C753380002 for ; Thu, 28 Dec 2023 16:58:39 +0000 (UTC) Authentication-Results: imf30.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=lWpAGoY8; dmarc=pass (policy=none) header.from=kernel.org; spf=pass (imf30.hostedemail.com: domain of nathan@kernel.org designates 145.40.73.55 as permitted sender) smtp.mailfrom=nathan@kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1703782720; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=xLUJphbdSDlSH/ELyDWRlAkxZ9MemPxjaCkG2TgdsBU=; b=aMjvT0qBae0m0BrrHCBuZ2pq2j7G10Y6gGugr9myzZWNxOwJDp4QY7Hyk7EhHN0DH2tblf 1VNBNTpZ2TTVgbaOypl8PMyvHxNyRYua8BsQZnroI15qlq5j3+QoEg3cDero9kGyMcu0/F 6/vgQkoSbKKJmmnPHWqGjZdP75eR658= ARC-Authentication-Results: i=1; imf30.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=lWpAGoY8; dmarc=pass (policy=none) header.from=kernel.org; spf=pass (imf30.hostedemail.com: domain of nathan@kernel.org designates 145.40.73.55 as permitted sender) smtp.mailfrom=nathan@kernel.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1703782720; a=rsa-sha256; cv=none; b=zK1D8Zr6zQxXEdiGXjmX2usTA3ALZjiDrtJTa5xbsRpsgXYPnTaSrs0hoL6FxtvQf8PwiJ +GyAQLxmtLGDMIwH/5RobBXPFTtkGWgf48025QESGLlk1VFn3liCSnDiaiBLwN9JC388DR faANus8ANy2dP3E1Xr6o7cq0Rb/6fNc= Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sin.source.kernel.org (Postfix) with ESMTP id A6BEDCE1741; Thu, 28 Dec 2023 16:58:35 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 9499CC433C9; Thu, 28 Dec 2023 16:58:33 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1703782714; bh=LtB9LsZRFoK2mRick0kUYbdhu9aPS6cGfKbd+Bt8FtI=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=lWpAGoY88Go6sFhC5oHMizbI6XpsDbe3P9/irSiWfwoLQGwfrrdzSswLuoybnRUFT Y+XqOp9YWoQsYRrr6bvMkd8mxSERkT48kgdNRqDjCUUttR5Q3H3U0h6pPQweVEORBc I8WSnOqHV1rhNFbW5BbgKKPNaTH+Ac5AbrB8xN5rrfXe+tCgqQs5Rp1ezx8NBaLOU2 N34MiXPPv82Oc2WqM4yvpi5XSNSYcpVl0ehK6ySaDLDFWybSeNgF8bZuALCNNz0wJQ 6D8Um8C0blfpXdkHQK5vaSPd/HhcXAfTjTliP0Y6lp9eN5VXRU/sim+RtVXRhf+tQ6 UVcz4gfMLuMIw== Date: Thu, 28 Dec 2023 09:58:31 -0700 From: Nathan Chancellor To: Christian Brauner Cc: David Howells , Jeff Layton , Steve French , Matthew Wilcox , Marc Dionne , Paulo Alcantara , Shyam Prasad N , Tom Talpey , Dominique Martinet , Eric Van Hensbergen , Ilya Dryomov , Christian Brauner , linux-cachefs@redhat.com, linux-afs@lists.infradead.org, linux-cifs@vger.kernel.org, linux-nfs@vger.kernel.org, ceph-devel@vger.kernel.org, v9fs@lists.linux.dev, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, llvm@lists.linux.dev, Kees Cook Subject: Re: [PATCH v5 15/40] netfs: Add support for DIO buffering Message-ID: <20231228165831.GA348702@dev-arch.thelio-3990X> References: <20231221132400.1601991-1-dhowells@redhat.com> <20231221132400.1601991-16-dhowells@redhat.com> <20231226165442.GA1202197@dev-arch.thelio-3990X> <20231228-wohlbefinden-museen-c5efad4e0d84@brauner> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20231228-wohlbefinden-museen-c5efad4e0d84@brauner> X-Rspamd-Queue-Id: C753380002 X-Rspam-User: X-Rspamd-Server: rspam04 X-Stat-Signature: eyu4rgd7n5p3buomo4rapec9fpwdbeiq X-HE-Tag: 1703782719-644169 X-HE-Meta: U2FsdGVkX1/X6IR7fypJqI69NxE8K0dLSbESs1NaK7Yhe2SrFM3d1GJTiTLucQ4LeLPK1fVwD3nyLz8n3l7OWh6eXAmG1CvASm6wO7bSyvR5nWJQTMudmR70ms6Cy3OhYtNthdwWcQyd4b0X/uBOHU3AfMUyUcJsvaOiSmfvNYrWz380sXph85R644QaNdGsbkkD3mbPO7JxloLr9Qra1SX3PP2vG51vDRnmJXp2dN5O33vK5SUQsc8feqL+l+3fxrtizCnPn6m3zAhhIZMxtmvLs0bPi+x87P0jxU9spqACer/+iWavzeofHwmxCuonCrWD7LBK8Jp3iHLZ+44706q23s42pQYV/oLWtpZRSylwAFJW2F4ubeJ0naInOPOp+J7pQDsmJpxlS27wrMmE0ul5GiuaU7Fs86pLZ80c8Kwkzg7e+zBhNMp+GoZqacRYAQj0c0xIyGZKXrlmG0ZuJ5kMLhnoAcqH7JTTt3Vc8+UHHyRq0dzmQbDjU9zq3zVvdzUlLD9tP2CNdYp/XDynMQt1OS4FuevqtOwsHRDGeLyeP88MFi/Pr02mKAQdKoJVWJlvn5t0gPd5kfLgKOtv8O7jY5LOcmbT/SGAMhwmqZ5XVJMu87oyQJN5+B+chYLHnGiMAej+lQ64o+4e0mWWafVWUzDa0HhPBlZmsVpXSeaEk7zSSwOEgTiHLQ+FhF7QzsuQ7Erxc+iaDH7vd5ASyS+v6yP73miQuIplpsXfXOn98u1ikrBxaUmoqsU4ahFs5Hkb2iscB18Kk3ba3SUTYjmNqHbjrV5hA+K/ZeBHStTtVbQI8WBYxdsFHcHYzmEfTJYDge+mz82eP6dV7Ro3cdnjkN/n+AViVBTsTOI9CM3E6seWAFtxpnx6wDkNs9zPf9MRY6UPbro= X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On Thu, Dec 28, 2023 at 11:47:45AM +0100, Christian Brauner wrote: > > This will break the build with versions of clang that have support for > > counted_by (as it has been reverted in main but reapplication to main is > > being actively worked on) because while annotating pointers with this > > attribute is a goal of the counted_by attribute, it is not ready yet. > > Please consider removing this and adding a TODO to annotate it when > > support is available. > > It's really unpleasant that we keep getting new attributes that we > seemingly are encouraged to use and get sent patches for it. And then we > learn a little later that that stuff isn't ready yet. It's annoying. I I will assume the "get sent patches for it" is referring to the patches that Kees has been sending out to add this attribute to flexible array members. In his defense, that part of the attribute is very nearly ready (it is only the pointer annotations that are not ready, as in not worked on at all as far as I am aware). In fact, it was merged in clang's main branch for some time and the only reason that it was backed out was because adoption in the kernel had pointed out bugs in the original implementation that were harder to fix than initially thought; in other words, only because we started adding this attribute to the kernel were we able to realize that the initial implementation in clang needed to be improved, otherwise this feature may have shipped completely broken in clang 18.1.0 because it had not been stress tested yet. Now we can get it right. However, I do not necessarily disagree that it is annoying for maintainers who are not following this saga but are just receiving patches to add these annotatations because adds additional things to check for. Perhaps there should be some guidance added to the __counted_by definition or Documentation around how it is expected to be used so that there is clear advice for both developers and maintainers? > know it isn't your fault but it would be wise to be a little more > careful. IOW, unless both clang and gcc do support that thing > appropriately don't send patches to various subsystems for this. I will assume this was not necessarily directed at me because I have not sent any patches for __counted_by. > In any case, this is now fixed. I pulled an updated version from David. Thanks a lot. Cheers, Nathan