Mike Christie wrote: > On 01/07/2010 08:16 AM, Alex Zeffertt wrote: >> Mike Christie wrote: >>> >>> Thanks for doing this. Sorry for the late reply. >>> >>> >>> Just one comment on the patch. Could you move the code in the 'n' case >>> >>> + case 'n': >>> + /* >>> + * Bring up NICs required by targets in iBFT >>> + * using IP addresses and routing info from iBFT. >>> + */ >>> >>> ...... >>> >>> >>> to some helper function, so it is not so crowed and a little easier to >>> read? >>> >> >> No problem. Please find a new patch attached. >> > > Merged in commit 39f4761231e2aa20b468cd5258785d6f56472772. It should > show up in the git tree in a little bit and in the next major release. > Thanks! > Finally found some time to comment on this one. In principle this is a good idea; it certainly saves quite some trouble for eg initrd as we don't have to worry about interface naming etc.
However: Having iscsiadm setting up the network interfaces we do have the problem that we're doing it alongside the 'normal' system network configuration. IE the system hasn't any idea that we've already configured the interface nor how it should handle it. This is especially a problem during shutdown, where the networking scripts are prone to shutdown any unknown interface. And having the networking scripts calling out to iscsiadm is not earning us much favours I would assume. A possible route here would be to modify the networking scripts to check the ibft variables; however, this requires quite some bit of sysfs tree walking. And given the somewhat fluctuating nature of sysfs it will earn us even less favours. It would be cool if we had an 'ibft' link in the network interface, though ... Hmm. Maybe I can trick Kay Sievers into doing one ... But apart from that is a pretty cool thing. It especially fixes the problem we have right now that changes in the iBFT BIOS are not picked up once the machines are installed. Cheers, Hannes -- Dr. Hannes Reinecke zSeries & Storage h...@suse.de +49 911 74053 688 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: Markus Rex, HRB 16746 (AG Nürnberg)
-- You received this message because you are subscribed to the Google Groups "open-iscsi" group. To post to this group, send email to open-is...@googlegroups.com. To unsubscribe from this group, send email to open-iscsi+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/open-iscsi?hl=en.
