xfs
[Top] [All Lists]

Re: [PATCH] xfs_mdrestore: initialize sb prior to xfs_sb_from_disk()

To: Dave Chinner <david@xxxxxxxxxxxxx>
Subject: Re: [PATCH] xfs_mdrestore: initialize sb prior to xfs_sb_from_disk()
From: Eric Sandeen <sandeen@xxxxxxxxxx>
Date: Thu, 05 Jun 2014 19:00:02 -0500
Cc: xfs-oss <xfs@xxxxxxxxxxx>
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20140605235633.GC4453@dastard>
References: <5390F89C.2050305@xxxxxxxxxx> <20140605235633.GC4453@dastard>
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:24.0) Gecko/20100101 Thunderbird/24.5.0
On 6/5/14, 6:56 PM, Dave Chinner wrote:
> On Thu, Jun 05, 2014 at 06:09:16PM -0500, Eric Sandeen wrote:

...

>> But ... should we maybe just do this once and for all in
>> xfs_sb_from_disk?  I'm not sure leaving it up to every
>> caller is a good idea, unless somebody ahs a reason to
>> pre-populate some fields - I can't imagine why that would
>> be, though...
> 
> We don't ever read in the CRC field into the in-memory structures
> because it has no meaning in memory. Simiarly, we don't ever write
> the CRC field from the in-core structure because we always
> re-calculate it in the IO path if CRCs are configured. That is
> consistent behaviour across the entire code-base.

<snip stuff>

> Perhaps we should move the memset() to within xfs_sb_from_disk()
> to make this explicit?

Yes, that's what I meant by "this" in "do this once and for all" - 
sorry, that wasn't clear.  memset(0) in xfs_sb_from_disk().

Yeah, the more I think about it, the more I think that's probably
the obviously correct thing to do.

-Eric

> Cheers,
> 
> Dave.
> 

<Prev in Thread] Current Thread [Next in Thread>