On Thu 07-02-08 08:06:37, Rasmus Rohde wrote: > Ok - I have checked get_parent and it works as expected. > I used the "Neil Brown"-test mentioned elsewhere in this thread and > added a few printk's to make sure we actually got the code covered. > > > There's still a few trivial warnings from scripts/checkpatch.pl that > > should be fixed up: > Fixed that. Sorry for not running checkpatch.pl before submitting. > > Before posting the last and hopefully final patch I'd like to know what > Jan says about open coding the lookup for .. > It will mean a lot of code duplication and I think it makes good sense > for udf_find_entry to be able to handle .. Yes, I think opencoding it would really lead to larger code duplication than I'd like so please keep the change in udf_find_entry(). Thanks. Honza -- Jan Kara <jack@xxxxxxx> SUSE Labs, CR - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html