Re: [Patch 10/18] fs/logfs/inode.c

[JÃrn Engel]

On Sun, 10 June 2007 19:24:28 +0200, Arnd Bergmann wrote:
> 
> I think this is missing code that sets the initial i_uid/i_gid,
> but there may be more missing. Changing the uid value works,
> but creating files as non-root user doesn't.

Fixed in patch 557, see
http://logfs.org/logfs/patches

The initial storm of review comments has calmed down.  I get the
impression that people either lose interest or run out of simple things
to point out.  Maybe I should wait a bit before resending?

JÃrn

-- 
The competent programmer is fully aware of the strictly limited size of
his own skull; therefore he approaches the programming task in full
humility, and among other things he avoids clever tricks like the plague.
-- Edsger W. Dijkstra
-
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/