On Monday June 19, raziebe@xxxxxxxxx wrote: > Neil hello > > if i am not mistaken here: > > in first instance of : if(bi) ... > ... > > you return without setting to NULL > Yes, you are right. Thanks. And fixing that bug removes the crash. However.... I've been doing a few tests and it is hard to measure much improvement, which is strange. I can maybe see a 1% improvement but that could just be noise. I do some more and see if I can find out what is happening. Interestingly, with a simple dd if=/dev/md1 of=/dev/null bs=1024k test, 2.6.16 is substantially faster (10%) than 2.6.17-rc6-mm2 before that patches are added. There is something weird there. Have you done any testing? NeilBrown - To unsubscribe from this list: send the line "unsubscribe linux-raid" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html