On 07/24/15 at 11:25am, Dave Young wrote:
On 07/24/15 at 10:31am, Minfei Huang wrote:
> On 07/24/15 at 10:19am, Dave Young wrote:
> > On 07/23/15 at 06:29pm, Minfei Huang wrote:
> > > Due to the different format between ipv4 and ipv6 protocol, quote the
> > > ipv6 address with bracket "[]" to make dracut notify.
> > >
> > > Signed-off-by: Minfei Huang <mhuang(a)redhat.com>
> > > Acked-by: Dave Young <dyoung(a)redhat.com>
> > > ---
> > > dracut-module-setup.sh | 4 +++-
> > > 1 file changed, 3 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/dracut-module-setup.sh b/dracut-module-setup.sh
> > > index 04a2e15..2a0d93f 100755
> > > --- a/dracut-module-setup.sh
> > > +++ b/dracut-module-setup.sh
> > > @@ -542,10 +542,12 @@ kdump_setup_iscsi_device() {
> > > kdump_setup_netdev $netdev $srcaddr
> > >
> > > # prepare netroot= command line
> > > - # FIXME: IPV6 addresses require explicit [] around $tgt_ipaddr
> > > # FIXME: Do we need to parse and set other parameters like protocol,
port
> > > # iscsi_iface_name, netdev_name, LUN etc.
> > >
> > > + if is_ipv6_address $tgt_ipaddr; then
> > > + tgt_ipaddr="[$tgt_ipaddr]"
> > > + fi
> > >
netroot_str="netroot=iscsi:${userpwd_str}${userpwd_in_str}@$tgt_ipaddr::::$tgt_name"
> > >
> > > [[ -f $netroot_conf ]] || touch $netroot_conf
> >
> > Relooking this patch, have you really tested iscsi/ipv6?
> >
> > Below part need a change for ipv6 as well:
> >
> > netdev=$(/sbin/ip route get to ${tgt_ipaddr} | \
> > sed 's|.*dev \(.*\).*|\1|g')
> > srcaddr=$(echo $netdev | awk '{ print $3; exit }')
> > netdev=$(echo $netdev | awk '{ print $1; exit }')
> >
> > kdump_setup_netdev $netdev $srcaddr
>
> Due to the limitation of the hardware, we do not test the iscsi to
> support ipv6, just to satisfy the format to dracut.
>
> I think I told you before about it.
>
The code is for software iscsi so you only need test soft iscsi...
Ok. I will test the software iscsi to confirm it.
Thanks
Minfei