diff for duplicates of <1496764970.2692.7.camel@sandisk.com> diff --git a/a/1.txt b/N1/1.txt index abce1af..f373272 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,18 +1,16 @@ On Tue, 2017-06-06 at 23:22 +0800, Ming Lei wrote: > If queue is stopped, we shouldn't dispatch request into driver and -> hardware, unfortunately the check is removed in bd166ef183c2(blk-mq-sched= -: +> hardware, unfortunately the check is removed in bd166ef183c2(blk-mq-sched: > add framework for MQ capable IO schedulers). ->=20 +> > This patch fixes the issue by moving the check back into > __blk_mq_try_issue_directly(). ->=20 +> > This patch fixes request use-after-free[1][2] during canceling requets > of NVMe in nvme_dev_disable(), which can be triggered easily during > NVMe reset & remove test. -Although I'm not convinced that this patch fixes [2], the patch itself look= -s +Although I'm not convinced that this patch fixes [2], the patch itself looks fine to me. Hence: -Reviewed-by: Bart Van Assche <Bart.VanAssche@sandisk.com>= +Reviewed-by: Bart Van Assche <Bart.VanAssche@sandisk.com> diff --git a/a/content_digest b/N1/content_digest index f0dc4f5..23aead3 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -12,21 +12,19 @@ "b\0" "On Tue, 2017-06-06 at 23:22 +0800, Ming Lei wrote:\n" "> If queue is stopped, we shouldn't dispatch request into driver and\n" - "> hardware, unfortunately the check is removed in bd166ef183c2(blk-mq-sched=\n" - ":\n" + "> hardware, unfortunately the check is removed in bd166ef183c2(blk-mq-sched:\n" "> add framework for MQ capable IO schedulers).\n" - ">=20\n" + "> \n" "> This patch fixes the issue by moving the check back into\n" "> __blk_mq_try_issue_directly().\n" - ">=20\n" + "> \n" "> This patch fixes request use-after-free[1][2] during canceling requets\n" "> of NVMe in nvme_dev_disable(), which can be triggered easily during\n" "> NVMe reset & remove test.\n" "\n" - "Although I'm not convinced that this patch fixes [2], the patch itself look=\n" - "s\n" + "Although I'm not convinced that this patch fixes [2], the patch itself looks\n" "fine to me. Hence:\n" "\n" - Reviewed-by: Bart Van Assche <Bart.VanAssche@sandisk.com>= + Reviewed-by: Bart Van Assche <Bart.VanAssche@sandisk.com> -86d578d749d5c0b1c6aa03af1c40427ea2350e807e133de49c40074aee740063 +697877bb868e9f4edfaf985681cd9debf0d4e176e660dc1b99b3168a87bced4b
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.