Re: [RFC PATCH] blk-cgroup: prevent rcu_sched detected stalls warnings in blkg_destroy_all()

2020-11-25 Thread Tejun Heo
Hello, On Wed, Nov 25, 2020 at 08:49:19PM +0800, yukuai (C) wrote: > > You can't continue iteration after dropping both locks. You'd have to jump > > out of loop and start list_for_each_entry_safe() again. > > Thanks for your review, it's right. On the other hand > blkcg_activate_policy() and blk

Re: [RFC PATCH] blk-cgroup: prevent rcu_sched detected stalls warnings in blkg_destroy_all()

2020-11-25 Thread yukuai (C)
On 2020/11/25 20:32, Tejun Heo wrote: Hello, Thanks for the fix. A couple comments below. On Sat, Nov 21, 2020 at 04:34:20PM +0800, Yu Kuai wrote: +#define BLKG_DESTROY_BATH 4096 I think you meant BLKG_DESTROY_BATCH. static void blkg_destroy_all(struct request_queue *q) { struc

Re: [RFC PATCH] blk-cgroup: prevent rcu_sched detected stalls warnings in blkg_destroy_all()

2020-11-25 Thread Tejun Heo
Hello, Thanks for the fix. A couple comments below. On Sat, Nov 21, 2020 at 04:34:20PM +0800, Yu Kuai wrote: > +#define BLKG_DESTROY_BATH 4096 I think you meant BLKG_DESTROY_BATCH. > static void blkg_destroy_all(struct request_queue *q) > { > struct blkcg_gq *blkg, *n; > + int count

[RFC PATCH] blk-cgroup: prevent rcu_sched detected stalls warnings in blkg_destroy_all()

2020-11-21 Thread Yu Kuai
test procedures: a. create 2 cgroups, and echo "8:0 1" to blkio.throttle.write_bps_device b. echo 1 > /sys/blocd/sda/device/delete test result: CPU: 6 PID: 472 Comm: bash Not tainted 5.10.0-rc4-next-20201120-dirty #62 Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS ?-2019