On Mon, Aug 20, 2012 at 8:14 PM, Jianpeng Ma <majianpeng@xxxxxxxxx> wrote: > On 2012-08-21 01:51 Dan Williams <djbw@xxxxxx> Wrote: >>On Mon, Aug 20, 2012 at 1:17 AM, Jianpeng Ma <majianpeng@xxxxxxxxx> wrote: >>> I think step4 maybe return ioerror.But it's success. >> >>You would need to check that stripe0 got recycled between step2 and >>step3. If not then the uptodate stripe is in the cache and the read >>can be serviced from the cached data. >> >>...which seems to be the case because of: >> >>> [19277.585936] locked=0 uptodate=4 to_read=0 to_write=0 failed=1 failed_num=3,-1 >> > > Thanks! If stripe had a cache function.I think the chunk_aligned_read should use it for small read situation. But the stripe cache is not meant to be a read cache as it would be duplicating what the page cache offers. Most aligned reads are likely to miss the stripe cache so adding a lookup for every read would be mostly wasted overhead. -- Dan -- 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
