Messages in this thread |  | | Date | Fri, 12 Oct 2001 08:56:05 -0600 | From | Richard Gooch <> | Subject | Re: [PATCH] EFI GUID Partition Tables |
| |
Martin Wilck writes: > Richard, > > > You've put the devfs_unregister_slave() inside an #ifdef. Yuk! It > > shouldn't be conditional. > > I did that because I didn't want to pollute your code. The function > was only needed for the UUID patch. > > > And I'm not really sure that I like this > > function in the first place, but that's not something I want to get > > into right now. > > I did not see a possibility to cleanly remove a slave that was > registered before. Did I oversee something? Do you thing that > functionality is superfluous?
The whole slave mechanism is a hack, created because of the badly structured genhd layer. It's a convenience for lazy programmers (like me, when I was hacking the genhd layer:-). I'd prefer to see it used very sparingly.
Regards,
Richard.... Permanent: rgooch@atnf.csiro.au Current: rgooch@ras.ucalgary.ca - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |