Re: [f2fs-dev] [PATCH v4 1/6] fs: enhance and rename shutdown() callback to remove_bdev()

2025-07-10 Thread Kent Overstreet
On Thu, Jul 10, 2025 at 03:10:04PM +0200, Jan Kara wrote: > On Wed 09-07-25 13:49:12, Kent Overstreet wrote: > > On Wed, Jul 09, 2025 at 07:23:07PM +0200, Jan Kara wrote: > > > > It also avoids the problem of ->mark_dead events being generated > > > > from a con

Re: [f2fs-dev] [PATCH v4 1/6] fs: enhance and rename shutdown() callback to remove_bdev()

2025-07-09 Thread Kent Overstreet
On Wed, Jul 09, 2025 at 07:23:07PM +0200, Jan Kara wrote: > On Wed 09-07-25 08:59:42, Dave Chinner wrote: > > This means that device removal processing can be performed > > without global filesystem/VFS locks needing to be held. Hence issues > > like re-entrancy deadlocks when there are concurrent/

Re: [f2fs-dev] [PATCH v4 1/6] fs: enhance and rename shutdown() callback to remove_bdev()

2025-07-08 Thread Kent Overstreet
On Wed, Jul 09, 2025 at 10:25:08AM +0930, Qu Wenruo wrote: > 在 2025/7/9 10:05, Kent Overstreet 写道: > > Consider that the thing that has a block device open might not even be a > > filesystem, or at least a VFS filesystem. > > > > It could be a stacking block device dr

Re: [f2fs-dev] [PATCH v4 1/6] fs: enhance and rename shutdown() callback to remove_bdev()

2025-07-08 Thread Kent Overstreet
On Wed, Jul 09, 2025 at 08:37:05AM +0930, Qu Wenruo wrote: > 在 2025/7/9 08:29, Dave Chinner 写道: > > On Tue, Jul 08, 2025 at 09:55:14AM +0200, Christian Brauner wrote: > > > On Mon, Jul 07, 2025 at 05:45:32PM -0700, Darrick J. Wong wrote: > > > > On Tue, Jul 08, 2025 at 08:52:47AM +0930, Qu Wenruo w

Re: [f2fs-dev] [PATCH 00/10] convert the majority of file systems to mmap_prepare

2025-06-16 Thread Kent Overstreet
Mailserver is rejecting with "too many recipients" - ai On Mon, Jun 16, 2025 at 08:33:19PM +0100, Lorenzo Stoakes wrote: > REVIEWER'S NOTES > > > I am basing this on the mm-new branch in Andrew's tree, so let me know if I > should rebase anything here. Given the mm bits touch

Re: [f2fs-dev] [PATCH v2 1/2] workqueue: new struct io_work

2024-07-02 Thread Kent Overstreet
On Tue, Jul 02, 2024 at 05:27:19PM GMT, yi sun wrote: > Yes, adding the io priority attribute to work will bring huge benefits in the > following scenarios: > The system has huge IO pressure (these IOs may all be low-priority IOs), > and a work (we hope to complete quickly) is also doing IO. If thi

Re: [f2fs-dev] [PATCH v2 1/2] workqueue: new struct io_work

2024-07-01 Thread Kent Overstreet
On Mon, Jul 01, 2024 at 07:32:23AM GMT, Tejun Heo wrote: > Hello, > > On Mon, Jul 01, 2024 at 03:51:37PM +0800, Yi Sun wrote: > > +/* > > + * If a work may do disk IO, it is recommended to use struct io_work > > + * instead of struct work_struct. > > + */ > > +struct io_work { > > + struct work_

Re: [f2fs-dev] [PATCH 1/2] workqueue: add io priority to work_struct

2024-06-21 Thread Kent Overstreet
On Fri, Jun 21, 2024 at 02:26:16PM +0800, Yi Sun wrote: > Many works will go to submit_bio(), and in many cases the io priority of > kworker cannot meet the real-time requirements of this work. > > So add the basic attribute ioprio to work_struct, and kworker can adjust > its io priority according

Re: [f2fs-dev] [PATCH v3 13/13] bcachefs: fiemap: emit new COMPRESSED state

2024-04-05 Thread Kent Overstreet
On Fri, Apr 05, 2024 at 01:17:45PM -0600, Andreas Dilger wrote: > On Apr 3, 2024, at 1:22 AM, Sweet Tea Dorminy > wrote: > > > > Signed-off-by: Sweet Tea Dorminy > > --- > > fs/bcachefs/fs.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/fs/bcachefs/fs.c b/fs/bca

Re: [f2fs-dev] [PATCH v3 02/13] fiemap: update fiemap_fill_next_extent() signature

2024-04-05 Thread Kent Overstreet
On Fri, Apr 05, 2024 at 01:05:01PM -0600, Andreas Dilger wrote: > On Apr 3, 2024, at 1:22 AM, Sweet Tea Dorminy > wrote: > > > > Update the signature of fiemap_fill_next_extent() to allow passing a > > physical length. Update all callers to pass a 0 physical length -- since > > none set the EXTE

Re: [f2fs-dev] [PATCH v3 11/13] bcachefs: fiemap: return correct extent physical length

2024-04-03 Thread Kent Overstreet
On Wed, Apr 03, 2024 at 01:00:11PM -0400, Brian Foster wrote: > On Wed, Apr 03, 2024 at 03:22:52AM -0400, Sweet Tea Dorminy wrote: > > Signed-off-by: Sweet Tea Dorminy > > --- > > fs/bcachefs/fs.c | 18 -- > > 1 file changed, 12 insertions(+), 6 deletions(-) > > > > diff --git a/

Re: [f2fs-dev] [PATCH v3 00/13] fiemap extension for more physical information

2024-04-03 Thread Kent Overstreet
On Wed, Apr 03, 2024 at 03:22:41AM -0400, Sweet Tea Dorminy wrote: > For many years, various btrfs users have written programs to discover > the actual disk space used by files, using root-only interfaces. > However, this information is a great fit for fiemap: it is inherently > tied to extent info

Re: [f2fs-dev] [FYI][PATCH] tracing/treewide: Remove second parameter of __assign_str()

2024-02-23 Thread Kent Overstreet
On Fri, Feb 23, 2024 at 01:46:53PM -0500, Steven Rostedt wrote: > On Fri, 23 Feb 2024 10:30:45 -0800 > Jeff Johnson wrote: > > > On 2/23/2024 9:56 AM, Steven Rostedt wrote: > > > From: "Steven Rostedt (Google)" > > > > > > [ > > >This is a treewide change. I will likely re-create this patch

Re: [f2fs-dev] [PATCH 1/2] fscrypto: don't use on-stack buffer for filename encryption

2016-11-05 Thread Kent Overstreet
On Thu, Nov 03, 2016 at 03:03:01PM -0700, Eric Biggers wrote: > With the new (in 4.9) option to use a virtually-mapped stack > (CONFIG_VMAP_STACK), stack buffers cannot be used as input/output for > the scatterlist crypto API because they may not be directly mappable to > struct page. For short fi