Re: [PATCH 2/2] dasd: remove dynamic ioctl registration

From: Ric Wheeler
Date: Fri Jan 06 2006 - 09:30:21 EST




Christoph Hellwig wrote:

If you could hold off just a couple of weeks, I hope that we can get through our EMC process junk and get the GPL'ed patch out to fix the symmetrix support part of this rolled in as well,



Why? We never do things to support legally questionable binary modules.
And on the practical side, does emc even ship modules for -mm release?
What's the point of not putting it into -mm?


No need for an EMC module, I think that we can issue a simple patch to dasd.c that removes the (silly) binary module that was there.

I would prefer that the clean up not break one of the few (and relatively common) devices supported by the dasd.c driver. If for no other reason, it would seem to make it more likely to be able test the existing patch properly.

ric

-
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/