Carlo Nonato 4db2f87e15 block, bfq: fix overwrite of bfq_group pointer in bfq_find_set_group() ... [ Upstream commit 14afc59361 ] The bfq_find_set_group() function takes as input a blkcg (which represents a cgroup) and retrieves the corresponding bfq_group, then it updates the bfq internal group hierarchy (see comments inside the function for why this is needed) and finally it returns the bfq_group. In the hierarchy update cycle, the pointer holding the correct bfq_group that has to be returned is mistakenly used to traverse the hierarchy bottom to top, meaning that in each iteration it gets overwritten with the parent of the current group. Since the update cycle stops at root's children (depth = 2), the overwrite becomes a problem only if the blkcg describes a cgroup at a hierarchy level deeper than that (depth > 2). In this case the root's child that happens to be also an ancestor of the correct bfq_group is returned. The main consequence is that processes contained in a cgroup at depth greater than 2 are wrongly placed in the group described above by BFQ. This commits fixes this problem by using a different bfq_group pointer in the update cycle in order to avoid the overwrite of the variable holding the original group reference. Reported-by: Kwon Je Oh <kwonje.oh2@gmail.com> Signed-off-by: Carlo Nonato <carlo.nonato95@gmail.com> Signed-off-by: Paolo Valente <paolo.valente@linaro.org> Signed-off-by: Jens Axboe <axboe@kernel.dk> Signed-off-by: Sasha Levin <sashal@kernel.org>		2020-03-25 08:25:49 +01:00
..
partitions	…
Kconfig	…
Kconfig.iosched	…
Makefile	…
badblocks.c	…
bfq-cgroup.c	block, bfq: fix overwrite of bfq_group pointer in bfq_find_set_group()	2020-03-25 08:25:49 +01:00
bfq-iosched.c	block, bfq: remove ifdefs from around gets/puts of bfq groups	2020-03-12 13:00:32 +01:00
bfq-iosched.h	block, bfq: remove ifdefs from around gets/puts of bfq groups	2020-03-12 13:00:32 +01:00
bfq-wf2q.c	block, bfq: get a ref to a group when adding it to a service tree	2020-03-12 13:00:32 +01:00
bio-integrity.c	block: fix memleak of bio integrity data	2020-01-26 10:01:09 +01:00
bio.c	block: fix memleak of bio integrity data	2020-01-26 10:01:09 +01:00
blk-cgroup.c	…
blk-core.c	block: end bio with BLK_STS_AGAIN in case of non-mq devs and REQ_NOWAIT	2020-01-12 12:21:37 +01:00
blk-exec.c	…
blk-flush.c	blk-mq: insert passthrough request into hctx->dispatch directly	2020-03-21 08:11:52 +01:00
blk-integrity.c	…
blk-ioc.c	…
blk-iocost.c	blk-iocost: fix incorrect vtime comparison in iocg_is_idle()	2020-03-18 07:17:52 +01:00
blk-iolatency.c	…
blk-lib.c	…
blk-map.c	block: fix memleak when __blk_rq_map_user_iov() is failed	2020-01-12 12:21:43 +01:00
blk-merge.c	…
blk-mq-cpumap.c	…
blk-mq-debugfs-zoned.c	…
blk-mq-debugfs.c	…
blk-mq-debugfs.h	…
blk-mq-pci.c	…
blk-mq-rdma.c	…
blk-mq-sched.c	blk-mq: insert flush request to the front of dispatch queue	2020-03-21 08:11:57 +01:00
blk-mq-sched.h	…
blk-mq-sysfs.c	…
blk-mq-tag.c	…
blk-mq-tag.h	…
blk-mq-virtio.c	…
blk-mq.c	blk-mq: insert passthrough request into hctx->dispatch directly	2020-03-21 08:11:52 +01:00
blk-mq.h	blk-mq: insert passthrough request into hctx->dispatch directly	2020-03-21 08:11:52 +01:00
blk-pm.c	…
blk-pm.h	…
blk-rq-qos.c	…
blk-rq-qos.h	…
blk-settings.c	block: fix an integer overflow in logical block size	2020-01-23 08:22:32 +01:00
blk-softirq.c	…
blk-stat.c	…
blk-stat.h	…
blk-sysfs.c	…
blk-throttle.c	…
blk-timeout.c	…
blk-wbt.c	…
blk-wbt.h	…
blk-zoned.c	…
blk.h	block: fix memleak of bio integrity data	2020-01-26 10:01:09 +01:00
bounce.c	…
bsg-lib.c	block: Fix the type of 'sts' in bsg_queue_rq()	2020-01-23 08:22:44 +01:00
bsg.c	…
cmdline-parser.c	…
compat_ioctl.c	…
elevator.c	…
genhd.c	…
ioctl.c	…
ioprio.c	…
kyber-iosched.c	…
mq-deadline.c	…
opal_proto.h	…
partition-generic.c	…
scsi_ioctl.c	…
sed-opal.c	…
t10-pi.c	…