[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <37d33d830812221103xc922c8dla400503ac6de4760@mail.gmail.com>
Date: Tue, 23 Dec 2008 00:33:02 +0530
From: "Sandeep K Sinha" <sandeepksinha@...il.com>
To: "Sandeep K Sinha" <sandeepksinha@...il.com>,
linux-kernel@...r.kernel.org, dm-devel@...hat.com
Subject: Re: Device mapper support for more than one target ?
On Tue, Dec 23, 2008 at 12:24 AM, Sandeep K Sinha
<sandeepksinha@...il.com> wrote:
> Hey,
>
> On Tue, Dec 23, 2008 at 12:20 AM, Alasdair G Kergon <agk@...hat.com> wrote:
>> On Mon, Dec 22, 2008 at 10:52:08PM +0530, Sandeep K Sinha wrote:
>>> After looking at the complete implementation of device mappers, I
>>> figured out that we can have more than one target for a mapped device
>>> then why do we have a check for the number of target to be equal to
>>> one , in dm_blk_ioctl ( ) in drivers/md/dm.c
>>>
>>> http://lxr.linux.no/linux+v2.6.27.10/drivers/md/dm.c#L337
>>
>> Because whether or not it makes sense to send any specific ioctl to more
>> than one target in parallel depends on what that ioctl does. When we
>> added that code we said that we could add hard-coding for specific
>> ioctls if the need arose, but so far it hasn't.
>>
Also, I would like to raise a couple of other points too here.
The number of targets that will be generated will be completely
transparent to the user and there is no way to figure that out from
the userland.
All that the user know is the LV thats created, and it will try to
issue ioctl's on that. Also, the error that is returned is not that
friendly, i mean it just return with an error value of -1, without
even providing any error messages.
I wish to add atleast a proper error message to the same, if we are
not providing ioctl's at the mapped_device level.
> No, the philosophy should be that we send the ioctl's to the mapped
> device and not to the targets underlying that mapped-device.
>
> And doing so, I should be able to access the complete map that belongs
> to that mapped device. If I implement an ioctl of my own and try to
> serve it, the problem would be that it would never allow me to serve
> it if has more than one target.
>
>> Alasdair
>> --
>> agk@...hat.com
>>
>
>
>
> --
> Regards,
> Sandeep.
>
>
>
>
>
>
> "To learn is to change. Education is a process that changes the learner."
>
--
Regards,
Sandeep.
"To learn is to change. Education is a process that changes the learner."
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists