[prev in list] [next in list] [prev in thread] [next in thread] 

List:       dm-devel
Subject:    Re: [dm-devel] [PATCH 2/2] dm: Avoid use-after-free of a mapped device
From:       "Jun'ichi Nomura" <j-nomura () ce ! jp ! nec ! com>
Date:       2013-02-26 0:30:57
Message-ID: 512C0241.7050500 () ce ! jp ! nec ! com
[Download RAW message or body]

On 02/26/13 00:09, Bart Van Assche wrote:
> Without your patch my test failed after two or three iterations. With your patch my \
> test is still running after 53 iterations. So if you want you can add Tested-by: \
> Bart Van Assche <bvanassche@acm.org>.

Great. Thanks for testing.
I'll submit a patch with your Reported-by and Tested-by.

> Your e-mail and the above patch are also interesting because these explain why \
> reverting to the v3.7 of drivers/md made my test succeed. 
> Note: even if this patch gets accepted I think it's still useful to modify \
> blk_run_queue() such that it converts recursion into iteration.

Yes. That's a separate discussion.
Though I'm not sure if it's ok in general to implicitly convert
sync run-queue to async one.

-- 
Jun'ichi Nomura, NEC Corporation

--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel


[prev in list] [next in list] [prev in thread] [next in thread]