Re: [PATCH] i2o_block Fix, possible CFQ elevator problem?

[Warren Togami]

Jens Axboe wrote:
> > > Not necessarily, it's most likely a CFQ bug. Otherwise it would have
> > > surfaced before :-)
> >
> > I forgot to mention in the previous reports:
> >
> > Prior to three of your original suggested cleanups of i2o_block, four 
> > simultaneous bonnie++'s on four independent arrays would almost 
> > immediately cause the crash while running elevator=cfq.  After those 
> > three cleanups four simultaneous bonnie++ would survive for a while 
> > without crashing... until you run "sync" in another terminal.  We 
> > however did not test it enough times to determine if without "sync" it 
> > can survive the test run.  Do you want this tested without "sync"?
>
>
> Repeat the tests that made it crash. The last patch I sent should work
> for you, at least until the real issue is found.

Tested your patch, it indeed does seem to keep the system stable.  If I 
am understanding it right, the patch disables merging in the case where 
it would have caused a BUG condition?  (Less efficiency.)

In any case, for now we are doing our i2o development using the other 
schedulers like deadline.  Let us know if you have updated cfq patches 
to try, and we will.

Warren Togami
wtogami@xxxxxxxxxx
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/