Re: [PATCH] nbd: nbd sysfs framework

From: Greg KH
Date: Wed Aug 03 2011 - 19:34:55 EST


On Wed, Aug 03, 2011 at 07:15:51PM -0400, Paul Clements wrote:
> Description: The patch creates a new sysfs entry framework for nbd.

Why? What does this buy us except an increase in code size for no added
benifit? You seem to be stripping out the driver layer here and using
"raw" kobjects, why? That's generally never a good idea (hint, I think
you just caused a race and userspace bug with your change as well
because of this...)

confused,

greg k-h
--
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/