2009-10-29 13:34:14

by Rakesh Ranjan

[permalink] [raw]
Subject: [RFC-PATCH] dhcp provisioning support in cxgb3i

Hi Mike,

Herein attached patches for having dhcp provisioning support in cxgb3i.
I have added one new iscsi netlink message ISCSI_UEVENT_REQ_IPCONF.
Please have a look and share suggestions.

Regards
Rakesh Ranjan


Attachments:
0001-added-one-new-netlink-message-ISCSI_UEVENT_REQ_IPCON.patch (2.80 kB)
0002-Implemented-dhcp-client-support-in-cxgb3i.patch (16.12 kB)
Download all attachments

2009-10-29 14:06:54

by Ben Hutchings

[permalink] [raw]
Subject: Re: [RFC-PATCH] dhcp provisioning support in cxgb3i

On Thu, 2009-10-29 at 18:46 +0530, Rakesh Ranjan wrote:
> Hi Mike,
>
> Herein attached patches for having dhcp provisioning support in
> cxgb3i.
> I have added one new iscsi netlink message ISCSI_UEVENT_REQ_IPCONF.
> Please have a look and share suggestions.
[...]

Why does cxgb3i need its very own DHCP client? This seems like
something that's generically useful to firmware-based iSCSI adapters.

(It would be better still if this could be left to user-space, but
although a user-space program could send out requests on the net device
using the iSCSI device's MAC address, I don't see how it would get
replies.)

Ben.

--
Ben Hutchings, Senior Software Engineer, Solarflare Communications
Not speaking for my employer; that's the marketing department's job.
They asked us to note that Solarflare product names are trademarked.

2009-10-29 21:10:09

by Mike Christie

[permalink] [raw]
Subject: Re: [RFC-PATCH] dhcp provisioning support in cxgb3i

Rakesh Ranjan wrote:
> Hi Mike,
>
> Herein attached patches for having dhcp provisioning support in cxgb3i.
> I have added one new iscsi netlink message ISCSI_UEVENT_REQ_IPCONF.

Is the idea to have iscsid/uip send down this msg?

Was it not possible to hook in more like how bnx2i does dhcp?

2009-10-30 05:24:17

by Rakesh Ranjan

[permalink] [raw]
Subject: Re: [RFC-PATCH] dhcp provisioning support in cxgb3i

Mike Christie wrote:
> Rakesh Ranjan wrote:
>> Hi Mike,
>>
>> Herein attached patches for having dhcp provisioning support in
>> cxgb3i. I have added one new iscsi netlink message
>> ISCSI_UEVENT_REQ_IPCONF.
>
> Is the idea to have iscsid/uip send down this msg?
>
> Was it not possible to hook in more like how bnx2i does dhcp?

yep, idea is to have iscsid send down ISCSI_UEVENT_REQ_IPCONF msg. bnx2i
approach in our case for at least T3 is not feasible.

Regards
Rakesh Ranjan

2009-11-02 23:48:32

by Mike Christie

[permalink] [raw]
Subject: Re: [RFC-PATCH] dhcp provisioning support in cxgb3i

Rakesh Ranjan wrote:
> Mike Christie wrote:
>> Rakesh Ranjan wrote:
>>> Hi Mike,
>>>
>>> Herein attached patches for having dhcp provisioning support in
>>> cxgb3i. I have added one new iscsi netlink message
>>> ISCSI_UEVENT_REQ_IPCONF.
>>
>> Is the idea to have iscsid/uip send down this msg?
>>
>> Was it not possible to hook in more like how bnx2i does dhcp?
>
> yep, idea is to have iscsid send down ISCSI_UEVENT_REQ_IPCONF msg. bnx2i
> approach in our case for at least T3 is not feasible.
>

I think adding DHCP in the kernel is getting a little crazy :) If we go
down this path, I agree with the other person that stated it should at
least be generic.

2009-11-04 18:24:21

by Rakesh Ranjan

[permalink] [raw]
Subject: Re: [RFC-PATCH] dhcp provisioning support in cxgb3i

Mike Christie wrote:
> Rakesh Ranjan wrote:
>> Mike Christie wrote:
>>> Rakesh Ranjan wrote:
>>>> Hi Mike,
>>>>
>>>> Herein attached patches for having dhcp provisioning support in
>>>> cxgb3i. I have added one new iscsi netlink message
>>>> ISCSI_UEVENT_REQ_IPCONF.
>>>
>>> Is the idea to have iscsid/uip send down this msg?
>>>
>>> Was it not possible to hook in more like how bnx2i does dhcp?
>>
>> yep, idea is to have iscsid send down ISCSI_UEVENT_REQ_IPCONF msg.
>> bnx2i approach in our case for at least T3 is not feasible.
>>
>
> I think adding DHCP in the kernel is getting a little crazy :) If we go
> down this path, I agree with the other person that stated it should at
> least be generic.

Hi Mike,

What do you mean be generic ? Do you want to have a generic interface
for DHCP handler that could be part of libiscsi and could be used by
other LLD's.

Regards
Rakesh Ranjan

2009-11-04 19:09:27

by Stephen Hemminger

[permalink] [raw]
Subject: Re: [RFC-PATCH] dhcp provisioning support in cxgb3i

On Wed, 04 Nov 2009 23:54:07 +0530
Rakesh Ranjan <[email protected]> wrote:

> Mike Christie wrote:
> > Rakesh Ranjan wrote:
> >> Mike Christie wrote:
> >>> Rakesh Ranjan wrote:
> >>>> Hi Mike,
> >>>>
> >>>> Herein attached patches for having dhcp provisioning support in
> >>>> cxgb3i. I have added one new iscsi netlink message
> >>>> ISCSI_UEVENT_REQ_IPCONF.
> >>>
> >>> Is the idea to have iscsid/uip send down this msg?
> >>>
> >>> Was it not possible to hook in more like how bnx2i does dhcp?
> >>
> >> yep, idea is to have iscsid send down ISCSI_UEVENT_REQ_IPCONF msg.
> >> bnx2i approach in our case for at least T3 is not feasible.
> >>
> >
> > I think adding DHCP in the kernel is getting a little crazy :) If we go
> > down this path, I agree with the other person that stated it should at
> > least be generic.
>
> Hi Mike,
>
> What do you mean be generic ? Do you want to have a generic interface
> for DHCP handler that could be part of libiscsi and could be used by
> other LLD's.

Is there anyway to make existing code (in net/ipv4/ipconfig.c) more generic
and useful for both?