Re: [PATCH 7/9] vfs: reorganize do_lookup

From: Miklos Szeredi
Date: Tue Mar 13 2012 - 07:04:02 EST


Christoph Hellwig <hch@xxxxxxxxxxxxx> writes:

>> + /*
>> + * Don't bother with __d_lookup: callers are for creat as
>> + * well as unlink, so a lot of the time it would cost
>> + * a double lookup.
>> + */
>> + dentry = d_lookup(base, name);
>
> This comment isn't true anymore now, although the actual code should
> still be fine given that we already did the __d_lookup before for
> regular lookups.

Okay.

Thanks,
Miklos
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/