Re: [PATCH 20/22] xfs: add support for lazy direct read bounce buffering

Christoph Hellwig <[email protected]>
Newsgroups org.kernel.vger.linux-xfs,org.kernel.vger.linux-block,org.kernel.vger.linux-fsdevel
Message-ID <[email protected]>
On Fri, Jul 24, 2026 at 08:24:43AM +0200, Christoph Hellwig wrote:
> > > +	/* .../xfs/<dev>/csum/ */
> > > +	error = xfs_sysfs_init(&mp->m_csum_kobj, &xfs_csum_ktype,
> > > +			       &mp->m_kobj, "csum");
> > > +	if (error)
> > > +		goto out_remove_error_dir;
> > 
> > /me wonders if this is a debugging knob and therefore should go in
> > debugfs?  Or is there a solid usecase for normal sysadmins to be able to
> > control this?
> 
> I consider it an insurance again mispredicting user workloads.  debugfs
> also is really painful in general, and especially for trivial files
> likes this where it requires tons of boilerplate for no reason.

Also: I'm not particular happy about the placement of this file
in sysfs.  Do we want a misc group, or rw?  The current one feels
a bit too specific, which means we'd keep on growing way too many
groups in the future.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.