From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from verein.lst.de (verein.lst.de [213.95.11.211]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 911BE10F7; Thu, 7 Dec 2023 09:46:21 -0800 (PST) Received: by verein.lst.de (Postfix, from userid 2407) id 5A17B227A87; Thu, 7 Dec 2023 18:46:18 +0100 (CET) Date: Thu, 7 Dec 2023 18:46:17 +0100 From: Christoph Hellwig To: Bart Van Assche Cc: "Martin K . Petersen" , linux-scsi@vger.kernel.org, linux-block@vger.kernel.org, linux-fsdevel@vger.kernel.org, Jens Axboe , Christoph Hellwig , Daejun Park , Kanchan Joshi , Jeff Layton , Chuck Lever , Dave Chinner , Chaitanya Kulkarni Subject: Re: [PATCH v5 04/17] fs: Restore F_[GS]ET_FILE_RW_HINT support Message-ID: <20231207174617.GD31184@lst.de> References: <20231130013322.175290-1-bvanassche@acm.org> <20231130013322.175290-5-bvanassche@acm.org> Precedence: bulk X-Mailing-List: linux-scsi@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20231130013322.175290-5-bvanassche@acm.org> User-Agent: Mutt/1.5.17 (2007-11-01) On Wed, Nov 29, 2023 at 05:33:09PM -0800, Bart Van Assche wrote: > Revert commit 7b12e49669c9 ("fs: remove fs.f_write_hint") to enable testing > write hint support with fio and direct I/O. To enable testing seems like a pretty bad argument for bloating struct file. I'd much prefer to not restore it, but if you want to do so please write a convincing commit log.