Messages in this thread | | | Date | Sun, 19 Feb 2006 01:25:17 +0000 | From | Alasdair G Kergon <> | Subject | Re: [dm-devel] Re: [PATCH] User-configurable HDIO_GETGEO for dm volumes |
| |
On Fri, Feb 17, 2006 at 04:59:58PM -0800, Darrick J. Wong wrote: > Actually, I was not aware that a device could exist without a table. > However, I suppose that geometry is a property of a device, not its > underlying configuration, so the forced_geometry is better off in struct > mapped_device.
There are 0, 1 or 2 tables associated with a device. dm restricts which of them you can address through the ioctl interface according to the state of the device. Things are simpler if you can use this ioctl regardless of the state of the tables, so a device-level one is more appropriate.
> Here's the third revision, with the geometry pushed into mapped_device > as well as fixes for the problems that you pointed out wrt string > passing, lack of warning messages, etc.
Much better: I'll do a complete review now, fix up remaining minor things and get it submitted.
> Also, the v2 patch should have had the appropriate entries in > include/linux/compat_ioctl.h. Maybe it fell off? In any case, it is > present in this v3.
*32 isn't but the other *32 ones are?
[In the userspace patch where you add the ioctl to the list you need to use the version number which first included it i.e. 4, 6, 0 and you also have to add it to the libdm-compat file for v1.]
Alasdair -- agk@redhat.com - 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/
| |