Re: [PATCH v3 3/5] btrfs: scrub: Don't append on-disk pages for raid56 scrub

2017-03-29 Thread Liu Bo
On Wed, Mar 29, 2017 at 09:33:20AM +0800, Qu Wenruo wrote: > In the following situation, scrub will calculate wrong parity to > overwrite correct one: > > RAID5 full stripe: > > Before > | Dev 1 | Dev 2 | Dev 3 | > | Data stripe 1 | Data stripe 2 | Parity Stripe | >

[PATCH v3 3/5] btrfs: scrub: Don't append on-disk pages for raid56 scrub

2017-03-28 Thread Qu Wenruo
In the following situation, scrub will calculate wrong parity to overwrite correct one: RAID5 full stripe: Before | Dev 1 | Dev 2 | Dev 3 | | Data stripe 1 | Data stripe 2 | Parity Stripe | --- 0 | 0x (Bad) |