On 06/04/2014 01:50 PM, Harald Hoyer wrote:
On 04.06.2014 09:52, Hannes Reinecke wrote:
Hi all,
I'm trying to setup a machine with root on iSCSI via iBFT
(on SR-IOV, yay!).
But anyhow, I'll be setting 'ip=ibft' on the commandline as per documentation
and the iBFT settings are applied and everything is nice and dandy.
It's only that dracut complains during boot:
Jun 03 16:28:11 localhost dracut-cmdline[80]: ibft
Jun 03 16:28:11 localhost dracut-cmdline[80]: Warning: Please supply bootdev
argument for multiple ip= lines
Jun 03 16:28:11 localhost dracut-cmdline[80]: Warning: Setting bootdev to 'ibft0'
Looking closer, setting 'ip=ibft' triggers 'ibft_to_cmdline()',
which will add _another_ ip= argument to the dracut commandline.
And consequently parse_ip_opts() complains here.
While it's quite easy to fix (just ignore ip=ibft when checking for duplicate
ip arguments), I do wonder whether this is the correct way.
Thing is, 'ip=ibft' is not really an ip setting, but rather a marker that iBFT
should be evaluated.
As such, wouldn't 'rd.iscsi.firmware=ibft' be a more appropriate setting?
Seeing that the ip argument is added to the commandline anyway, having a
duplicate 'ip=ibft' setting does look a bit odd.
And confuses the logic ...
Cheers,
Hannes
Good catch! The bad thing is, that "ip=ibft" is already documented and used, so
we might want "rd.iscsi.ibft=1" and also ignore "ip=ibft" in the check.
"rd.iscsi.firmware=1" is already taken for also starting iscsistart (which also
starts the network interfaces, AFAIK)
Hmm. You are of course aware that rd.iscsi.firmware doesn't
necessarily requires you to start network interfaces, right?
be2iscsi and qla4xxx have a separate PCI function, operating
independently on the network interface. So no need to do anything there.
"rd.iscsi.ibft=1" would probably fit, if you only want to ifup the network
interfaces.
As said above, rd.iscsi.firmware should just cause 'iscsistart' to
run, and we need another setting to setup network interfaces
specified by iBFT. So for that I really would like to use
rd.iscsi.ibft=1, which will allow us to deprecate the 'ip=ibft'
commandline in the long run.
Cheers,
Hannes
--
Dr. Hannes Reinecke zSeries & Storage
hare@xxxxxxx +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: J. Hawn, J. Guild, F. Imendörffer, HRB 16746 (AG Nürnberg)
--
To unsubscribe from this list: send the line "unsubscribe initramfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
