The cio_ignore handling for zFCP devices was not complete. Modify
linuxrc.s390 to write out the FCP_* lines from the CMS conf file to
/etc/zfcp.conf. In storage/zfcp.py, modify the startup() method to run
/sbin/zfcp_cio_free, which reads /etc/zfcp.conf and removes them from
the device blacklist. Modify the onlineDevice() method to not free the
device, but rather set up the other properties.
The use of zfcp_cio_free should handle blocking on the device free
operation until it becomes available.
---
loader/linuxrc.s390 | 4 +-
scripts/upd-instroot | 1 +
storage/zfcp.py | 108 ++++++++++++++------------------------------------
3 files changed, 33 insertions(+), 80 deletions(-)
diff --git a/loader/linuxrc.s390 b/loader/linuxrc.s390
index 83f5c94..64b38c9 100644
--- a/loader/linuxrc.s390
+++ b/loader/linuxrc.s390
@@ -2950,9 +2950,9 @@ fi
syntax_check_fcp
# currently we ignore failed syntax checks since FCP parms are non-interactive
for i in ${!FCP_*}; do
- echo "${!i}" >> /tmp/fcpconfig
+ echo "${!i}" >> /etc/zfcp.conf
done
-# cio_ignore handling for FCP should happen when the content of /tmp/fcpconfig
+# cio_ignore handling for FCP should happen when the content of /etc/zfcp.conf
# will actually be processed which is in anaconda's zfcp.py ZFCP::readConfig()
# TODO/FIXME: also need to pass IPv6 decision to loader/anaconda
diff --git a/scripts/upd-instroot b/scripts/upd-instroot
index 88ff473..fad0071 100755
--- a/scripts/upd-instroot
+++ b/scripts/upd-instroot
@@ -735,6 +735,7 @@ sbin/qetharp
sbin/qetharp-2.6
sbin/qethconf
sbin/sysctl
+sbin/zfcp_cio_free
usr/bin/dirname
usr/bin/expr
usr/bin/getopt
diff --git a/storage/zfcp.py b/storage/zfcp.py
index 7692cad..48e326b 100644
--- a/storage/zfcp.py
+++ b/storage/zfcp.py
@@ -27,9 +27,9 @@ from udev import udev_settle
import gettext
_ = lambda x: gettext.ldgettext("anaconda", x)
+import iutil
import logging
log = logging.getLogger("anaconda")
-import warnings
def loggedWriteLineToFile(fn, value):
f = open(fn, "w")
@@ -39,6 +39,7 @@ def loggedWriteLineToFile(fn, value):
zfcpsysfs = "/sys/bus/ccw/drivers/zfcp"
scsidevsysfs = "/sys/bus/scsi/devices"
+zfcpconf = "/etc/zfcp.conf"
class ZFCPDevice:
def __init__(self, devnum, wwpn, fcplun):
@@ -115,41 +116,12 @@ class ZFCPDevice:
checkValidWWPN = checkValidFCPLun = checkValid64BitHex
def onlineDevice(self):
- online = "%s/%s/online" %(zfcpsysfs, self.devnum)
portadd = "%s/%s/port_add" %(zfcpsysfs, self.devnum)
portdir = "%s/%s/%s" %(zfcpsysfs, self.devnum, self.wwpn)
unitadd = "%s/unit_add" %(portdir)
unitdir = "%s/%s" %(portdir, self.fcplun)
failed = "%s/failed" %(unitdir)
- try:
- if not os.path.exists(online):
- loggedWriteLineToFile("/proc/cio_ignore",
- "free %s" %(self.devnum,))
- udev_settle()
- except IOError as e:
- raise ValueError, _("Could not free zFCP device %(devnum)s from "
- "device ignore list (%(e)s).") \
- % {'devnum': self.devnum, 'e': e}
-
- if not os.path.exists(online):
- raise ValueError, _(
- "zFCP device %s not found, not even in device ignore list."
- %(self.devnum,))
-
- try:
- f = open(online, "r")
- devonline = f.readline().strip()
- f.close()
- if devonline != "1":
- loggedWriteLineToFile(online, "1")
- else:
- log.info("zFCP device %s already online." %(self.devnum,))
- except IOError as e:
- raise ValueError, _("Could not set zFCP device %(devnum)s "
- "online (%(e)s).") \
- % {'devnum': self.devnum, 'e': e}
-
if not os.path.exists(portdir):
if os.path.exists(portadd):
# older zfcp sysfs interface
@@ -326,59 +298,44 @@ class ZFCP:
""" ZFCP utility class.
This class will automatically online to ZFCP drives configured in
- /tmp/fcpconfig when the startup() method gets called. It can also be
+ /etc/zfcp.conf when the startup() method gets called. It can also be
used to manually configure ZFCP devices through the addFCP() method.
- As this class needs to make sure that /tmp/fcpconfig configured
+ As this class needs to make sure that /etc/zfcp.conf configured
drives are only onlined once and as it keeps a global list of all ZFCP
devices it is implemented as a Singleton.
"""
def __init__(self):
self.fcpdevs = []
- self.hasReadConfig = False
self.down = True
- # So that users can write zfcp() to get the singleton instance
- def __call__(self):
- return self
+ if os.path.isfile(zfcpconf):
+ log.info("Enabling zFCP devices listed in %s" % (zfcpconf,))
+ iutil.execWithRedirect("zfcp_cio_free", ["-V"],
+ stdout="/dev/tty5", stderr="/dev/tty5")
- def readConfig(self):
- try:
- f = open("/tmp/fcpconfig", "r")
- except IOError:
- log.info("no /tmp/fcpconfig; not configuring zfcp")
- return
+ # read in all zFCP devices
+ f = open(zfcpconf, "r")
+ lines = map(lambda x: x.strip(), f.readlines())
+ f.close()
- lines = f.readlines()
- f.close()
- for line in lines:
- # each line is a string separated list of values to describe a dev
- # there are two valid formats for the line:
- # devnum scsiid wwpn scsilun fcplun (scsiid + scsilun ignored)
- # devnum wwpn fcplun
- line = string.strip(line).lower()
- if line.startswith("#"):
- continue
- fcpconf = string.split(line)
- if len(fcpconf) == 3:
- devnum = fcpconf[0]
- wwpn = fcpconf[1]
- fcplun = fcpconf[2]
- elif len(fcpconf) == 5:
- warnings.warn("SCSI ID and SCSI LUN values for ZFCP devices are ignored and deprecated.", DeprecationWarning)
- devnum = fcpconf[0]
- wwpn = fcpconf[2]
- fcplun = fcpconf[4]
- else:
- log.warn("Invalid line found in /tmp/fcpconfig!")
- continue
+ for line in lines:
+ if line.startswith("#") or line == '':
+ continue
- try:
- self.addFCP(devnum, wwpn, fcplun)
- except ValueError, e:
- log.warn(str(e))
- continue
+ fields = line.split()
+
+ if len(fields) == 3:
+ self.addFCP(fields[0], fields[1], fields[2])
+ elif len(fields) == 5:
+ # support old syntax of:
+ # devno scsiid wwpn scsilun fcplun
+ self.addFCP(fields[0], fields[2], fields[4])
+
+ # So that users can write zfcp() to get the singleton instance
+ def __call__(self):
+ return self
def addFCP(self, devnum, wwpn, fcplun):
d = ZFCPDevice(devnum, wwpn, fcplun)
@@ -401,12 +358,7 @@ class ZFCP:
if not self.down:
return
self.down = False
- if not self.hasReadConfig:
- self.readConfig()
- self.hasReadConfig = True
- # readConfig calls addFCP which calls onlineDevice already
- return
-
+
if len(self.fcpdevs) == 0:
return
for d in self.fcpdevs:
@@ -426,11 +378,11 @@ class ZFCP:
def write(self, instPath):
if len(self.fcpdevs) == 0:
return
- f = open(instPath + "/etc/zfcp.conf", "w")
+ f = open(os.path.join(instPath, zfcpconf), "w")
for d in self.fcpdevs:
f.write("%s\n" %(d,))
f.close()
-
+
f = open(instPath + "/etc/modprobe.conf", "a")
f.write("alias scsi_hostadapter zfcp\n")
f.close()
--
1.6.6.1
_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
