Re: [PATCH v3 9/9] xfsrestore: check for compatible xfsrestore

[Christoph Hellwig <hch@xxxxxxxxxxxxx>]

On Tue, Nov 16, 2010 at 09:05:11AM -0600, wkendall@xxxxxxx wrote:
> When resuming a restore or doing a cumulative restore, xfsrestore
> reads state information left around by the previous invocation.
> This patch adds logic to determine whether or not restore is
> able to make sense of the sense information.
> 
> The xfsrestore man page has also been updated to make the user
> aware of the requirement to use a compatible restore and
> system when resuming restores.

Shouldn't you use the opportunity to switch everything in the on-disk
layout to explicitly sized types?  Seeing things like the bool types
in the persistant structures really scares me.