Messages in this thread Patch in this message | | | Date | Wed, 25 May 2011 10:46:28 +0200 | From | Jens Axboe <> | Subject | Re: Mysterious CFQ crash and RCU |
| |
On 2011-05-25 10:28, Paul Bolle wrote: > On Tue, 2011-05-24 at 11:41 +0200, Jens Axboe wrote: >> Paul, can we see a dmesg from your running system? Perhaps there's some >> dependency on a particular driver or device that makes this easier to >> reproduce. > > Sure. What follows is (1K+ lines output of) the dmesg of a session in > which I managed to trigger this issue (three times) with my favorite > testing tool (ie, "mandb -c"). Feel free to prod me for the details of > my system that are not apparent from the flood of information in this > log.
Great, thanks Paul. Are you easily able to test a patch out? Having a test case on your machine that is as easily reproducible as it seems to be, would be a great place to test ideas out.
I don't think we are dealing with bad RCU usage in CFQ. My gut tells me that this is related to the merging of cooperating queues. It fits roughly with the time frame of when this issue started occuring, and some of that reference logic looks fragile/racy.
So if you _can_ test a patch easily, please try this one. It'll disable that logic.
diff --git a/block/cfq-iosched.c b/block/cfq-iosched.c index ab7a9e6..f37e3df 100644 --- a/block/cfq-iosched.c +++ b/block/cfq-iosched.c @@ -1827,6 +1827,7 @@ static struct cfq_queue *cfqq_close(struct cfq_data *cfqd, static struct cfq_queue *cfq_close_cooperator(struct cfq_data *cfqd, struct cfq_queue *cur_cfqq) { +#if 0 struct cfq_queue *cfqq; if (cfq_class_idle(cur_cfqq)) @@ -1870,6 +1871,9 @@ static struct cfq_queue *cfq_close_cooperator(struct cfq_data *cfqd, return NULL; return cfqq; +#else + return cfqq; +#endif } /* -- Jens Axboe
| |