Re: [PATCH] xfsprogs: initialize filetype for lost+found creation

Eric Sandeen
2013-09-18T21:04:48Z

On 9/18/13 3:55 PM, Mark Tinguely wrote:
> On 09/17/13 12:07, Eric Sandeen wrote:
>> If we create lost+found make sure it's got the proper filetype.
>>
>> Signed-off-by: Eric Sandeen<sandeen@redhat.com>
>> ---
>>
>> (sorry, untested, not sure how to force creation when I can't use
>> xfs_db to craft a filesystem that needs lost+found)
>
> The {lib}xfs_dir_lookup() doesn't use the field - just like extended attributes use xfs_names but it won't use the field entry, but it does not hurt to have it initialized to something.

Thanks for the review - just FWIW, I was thinking that it needs
to be initialized so that when created, it is created with the proper
type, rather than random stack noise.

It seems, though, that we're still missing bits in libxfs to actually
write the types to disk; I'll look into that.

Thanks,
-Eric

> Reviewed-by: Mark Tinguely <tinguely@sgi.com>
>
> _______________________________________________
> xfs mailing list
> xfs@oss.sgi.com
> http://oss.sgi.com/mailman/listinfo/xfs