Re: [PATCH] UDF: check for allocated memory for inode data

[Christoph Hellwig]

On Fri, May 11, 2007 at 01:01:27PM +0400, Cyrill Gorcunov wrote:
> [Christoph Hellwig - Fri, May 11, 2007 at 08:29:39AM +0100]
> 
> ...
> 
> | And please get rid of the UDF_I_* macro for everything you touch, just
> | put a
> | 
> | 	struct udf_inode_info *uip = UDF_I(inode);
> | 
> | at the beginning of the function and use the fields directly.
> | 
> 
> Actually to properly remove UDF_I* and UDF_SB_* macroses in the
> whole UDF subsystem - is _lot_ of work. I'm going to make it but
> not now (too busy).

Doing it completely is a lot of work, yes.  I was more thinking of
converting a piece of code once you do major changes.  But if you
want to convert all the code as a separate patch I'm more than happy
aswell.
-
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/