On Fri, Mar 31, 2017 at 09:29:20AM +0800, Qu Wenruo wrote:
>
>
> At 03/31/2017 12:49 AM, Liu Bo wrote:
> > On Thu, Mar 30, 2017 at 02:32:47PM +0800, Qu Wenruo wrote:
> > > Unlike mirror based profiles, RAID5/6 recovery needs to read out the
> > > whole full stripe.
> > >
> > > And if we don't do proper protect, it can easily cause race condition.
> > >
> > > Introduce 2 new functions: lock_full_stripe() and unlock_full_stripe()
> > > for RAID5/6.
> > > Which stores a rb_tree of mutex for full stripes, so scrub callers can
> > > use them to lock a full stripe to avoid race.
> > >
> > > Signed-off-by: Qu Wenruo <quwenruo@xxxxxxxxxxxxxx>
> > > Reviewed-by: Liu Bo <bo.li.liu@xxxxxxxxxx>
> > > ---
> > > fs/btrfs/ctree.h | 17 ++++
> > > fs/btrfs/extent-tree.c | 11 +++
> > > fs/btrfs/scrub.c | 217 +++++++++++++++++++++++++++++++++++++++++++++++++
> > > 3 files changed, 245 insertions(+)
> > >
> > > diff --git a/fs/btrfs/ctree.h b/fs/btrfs/ctree.h
> > > index 29b7fc28c607..9fe56da21fed 100644
> > > --- a/fs/btrfs/ctree.h
> > > +++ b/fs/btrfs/ctree.h
[...]
> > > +/*
> > > + * Helper to get full stripe logical from a normal bytenr.
> > > + *
> > > + * Caller must ensure @cache is a RAID56 block group.
> > > + */
> > > +static u64 get_full_stripe_logical(struct btrfs_block_group_cache *cache,
> > > + u64 bytenr)
> > > +{
> > > + u64 ret;
> > > +
> > > + /*
> > > + * round_down() can only handle power of 2, while RAID56 full
> > > + * stripe len can be 64KiB * n, so need manual round down.
> > > + */
> > > + ret = (bytenr - cache->key.objectid) / cache->full_stripe_len *
> > > + cache->full_stripe_len + cache->key.objectid;
> >
> > Can you please use div_u64 instead? '/' would cause building errors.
>
> No problem, but I'm still curious about under which arch/compiler it would
> cause build error?
>
Sorry, it should be div64_u64 since cache->full_stripe_len is (unsigend long).
Building errors might not be true, it's from my memory.
But in runtime, it could end up with 'divide error'.
Thanks,
-liubo
> Thanks,
> Qu
> >
> > Reviewed-by: Liu Bo <bo.li.liu@xxxxxxxxxx>
> >
> > Thanks,
> >
> > -liubo
> >
> >
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
