Debug logging improvements
authorBryn M. Reeves <bmr@redhat.com>
Wed, 23 Jan 2013 11:11:58 +0000 (11:11 +0000)
committerBryn M. Reeves <bmr@redhat.com>
Wed, 23 Jan 2013 11:11:58 +0000 (11:11 +0000)
* Do not hard-code the function name in _debug() calls; use the
bash $FUNCNAME variable instead.

* Fix whitespace in a couple of device debugging messages.

* Ensure all dmsetup calls have proper message prefixes.

blkdevalias

index 7a2896f..5cbfea9 100755 (executable)
@@ -148,7 +148,7 @@ ba_add_wwid_mapping() {
     _TYPE="$3"
     _PART="$4"
     _DEV="$5"
-    _debug "ba_add_wwid_mapping: WWID=\"$_WWID\" ALIAS=\"$_ALIAS\"" \
+    _debug "$FUNCNAME: WWID=\"$_WWID\" ALIAS=\"$_ALIAS\"" \
         "TYPE=\"$_TYPE\" PART=\"$_PART\""
     if [ "$_WWID" == "" ]; then
         _warn "no wwid given"
@@ -162,12 +162,12 @@ ba_add_wwid_mapping() {
         _warn "no partition number given"
     fi
     if [ "${WWIDMAP[$_WWID]}" != "" ]; then
-        _debug "ba_add_wwid_mapping: WWIDMAP[\$WWID]=${WWIDMAP[$_WWID]}"
+        _debug "$FUNCNAME: WWIDMAP[\$WWID]=${WWIDMAP[$_WWID]}"
         _warn "$DEV already mapped to alias ${WWIDMAP[$_WWID]}; replacing it."
         ba_del_wwid_mapping "$_WWID"
     fi
     if [ "${NAMEMAP[$_ALIAS]}" != "" ]; then
-        _debug "ba_add_wwid_mapping: NAMEMAP[\$ALIAS]=${NAMEMAP[$_ALIAS]}"
+        _debug "$FUNCNAME: NAMEMAP[\$ALIAS]=${NAMEMAP[$_ALIAS]}"
         _warn "$DEV already mapped to disk ${NAMEMAP[$_ALIAS]}; replacing it."
         ba_del_wwid_mapping ${NAMEMAP[$_ALIAS]}
     fi
@@ -203,15 +203,15 @@ ba_get_partnum () {
     if [[ $_DEV == dm-* ]]; then
         _DEV=$DM_NAME
     fi
-    _debug "ba_get_partnum: $_DEV"
-    _debug calling "$BA_DMSETUP info $_DEV"
+    _debug "$FUNCNAME: $_DEV"
+    _debug "$FUNCNAME: calling $BA_DMSETUP info $_DEV"
     $BA_DMSETUP info $_DEV > /dev/null 2>&1
     if [ "$?" == "0" ]; then
         _NUM=$(echo $_DEV | sed -e 's/\/dev.*\///' -e 's/.*p\([0-9]*\).*/\1/')
-        _debug "ba_get_partnum: dm device $_DEV NUM=$_NUM"
+        _debug "$FUNCNAME: dm device $_DEV NUM=$_NUM"
     else
         _NUM=$(echo $_DEV | sed 's/\(\/dev\/\)*sd[a-z]*//')
-        _debug "ba_get_partnum: non-dm device $_DEV NUM=$_NUM"
+        _debug "$FUNCNAME: non-dm device $_DEV NUM=$_NUM"
     fi
     if [ "$_NUM" == "" ]; then
         _NUM="0"
@@ -223,14 +223,14 @@ ba_get_partnum () {
 ba_scsi_id () {
     local _DEV
     _DEV="$1"
-    _debug "ba_scsi_id: getting ID for device $_DEV"
+    _debug "$FUNCNAME: getting ID for device $_DEV"
     $BA_DMSETUP info $_DEV > /dev/null  2>&1
     if [ $? -eq 0 ]; then
         _DEV=$(echo $_DEV | sed 's/p[0-9].*//')
     else
         _DEV=$(echo $_DEV | sed 's/[0-9].*//')
     fi
-     _debug "ba_scsi_id: calling \"$BA_SCSI_ID $BA_SCSI_ID_FLAGS $_DEV\""
+     _debug "$FUNCNAME: calling \"$BA_SCSI_ID $BA_SCSI_ID_FLAGS $_DEV\""
     $BA_SUDO $BA_SCSI_ID $BA_SCSI_ID_FLAGS $_DEV
 }
 
@@ -241,7 +241,7 @@ ba_chown_configs () {
     _GROUP="$2"
     $BA_CHOWN -R "$_USER:$_GROUP" "$BA_CONF_DIR"
     ls -l "$BA_CONF_DIR" | while read _OUT; do
-        _debug "ba_chown_configs: $_OUT"
+        _debug "$FUNCNAME: $_OUT"
     done
     return 0
 }
@@ -251,7 +251,7 @@ ba_refresh_alias () {
     local _ALIAS _DEVPATH
     _ALIAS="$1"
     _DEVPATH="/dev/$BA_DEV_PATH/$_ALIAS"    
-    _debug "ba_refresh_alias: ALIAS=$_ALIAS DEVPATH=$_DEVPATH"
+    _debug "$FUNCNAME: ALIAS=$_ALIAS DEVPATH=$_DEVPATH"
     ba_refresh_device $_DEVPATH
 }
 
@@ -259,7 +259,7 @@ ba_refresh_alias () {
 ba_refresh_device () {
     local _DEVPATH _DEVNAME _LINKDEST _PARTNUM
     _DEVPATH="$1"
-    _debug "ba_refresh_device: refreshing device at path $_DEVPATH"
+    _debug "$FUNCNAME: refreshing device at path $_DEVPATH"
     _LINKDEST=$($BA_READLINK $_DEVPATH)
     if [ "$?" != 0 ]; then
         _DEVNAME=$($BA_BASENAME $_DEVPATH | $BA_SED 's/p\+[1-9]*$//')
@@ -267,14 +267,14 @@ ba_refresh_device () {
         if [ "$_PARTNUM" != "0" ]; then
             _DEVNAME="$(echo ${_DEVNAME} | sed 's/[0-9]*$//')/$_DEVNAME"
         fi
-        _debug "ba_refresh_device: non-symlinked dev at " \
+        _debug "$FUNCNAME: non-symlinked dev at" \
             "$_DEVPATH has name $_DEVNAME"
     else
         _DEVNAME=$($BA_BASENAME $_LINKDEST)
-        _debug "ba_refresh_device: symlinked dev at" \
+        _debug "$FUNCNAME: symlinked dev at" \
             "$_DEVPATH has name $_DEVNAME"
     fi
-    _debug "ba_refresh_device: requesting change uevent at" \
+    _debug "$FUNCNAME: requesting change uevent at" \
             "sysfs path /sys/block/$_DEVNAME"
     $BA_SUDO $BA_SH -c "echo change > /sys/block/$_DEVNAME/uevent"
 }
@@ -363,14 +363,14 @@ ba_createdisk () {
     if [ "$_DEV" == "" ] || [ "$_ALIAS" == "" ]; then
         ba_usage "createdisk <alias> <device>"
     fi
-    _debug "ba_createdisk: DEV=\"$_DEV\" ALIAS=\"$_ALIAS\""
+    _debug "$FUNCNAME: DEV=\"$_DEV\" ALIAS=\"$_ALIAS\""
 
     _WWID=$(ba_scsi_id $_DEV)
     if [ "$_WWID" == "" ]; then
         _error "could not get SCSI ID for $_DEV"
         exit 1
     fi
-    _debug "ba_createdisk: WWID=\"$_WWID\""
+    _debug "$FUNCNAME: WWID=\"$_WWID\""
 
     if $BA_DMSETUP info $_DEV > /dev/null  2>&1; then
         _TYPE="mpath"
@@ -379,7 +379,7 @@ ba_createdisk () {
     fi
 
     _PART=$(ba_get_partnum $_DEV) 
-    _debug "ba_createdisk: PART=\"$_PART\""
+    _debug "$FUNCNAME: PART=\"$_PART\""
 
     ba_add_wwid_mapping $_WWID $_ALIAS $_TYPE $_PART $_DEV
     ba_store_wwid_map
@@ -434,14 +434,14 @@ ba_querydisk () {
         exit 2
     fi
     _WWID=${NAMEMAP[$_ALIAS]}
-    _debug "ba_querydisk: ALIAS=\"$_ALIAS\"" \
+    _debug "$FUNCNAME: ALIAS=\"$_ALIAS\"" \
         "WWID=\"$_WWID\" LISTDEV=\"$_LISTDEV\""
     if [ "$_LISTDEV" == "1" ]; then
         DEV="/dev/$BA_DEV_PATH/$($BA_READLINK "/dev/$BA_DEV_PATH/$_ALIAS")"
         MAJ="$[0x$(stat --format "%t" $DEV)]"
         MIN="$[0x$(stat --format "%T" $DEV)]"
         _ONDEV="on device [$MAJ,$MIN]"
-        _debug "ba_querydisk: DEV=\"$DEV\" MAJ=\"$MAJ\" MIN=\"$MIN\""
+        _debug "$FUNCNAME: DEV=\"$DEV\" MAJ=\"$MAJ\" MIN=\"$MIN\""
     fi
     if [ "$_WWID" != "" ]; then
         echo "$_ALIAS is a valid blkdevalias disk $_ONDEV"
@@ -457,12 +457,12 @@ ba_scandisks () {
     else
         _REFRESH="no"
     fi
-    _debug "ba_scandisks: device refresh requested: $_REFRESH"
+    _debug "$FUNCNAME: device refresh requested: $_REFRESH"
     for _DEV in /dev/$BA_DEV_PATH/*; do
         _TARGET=$($BA_READLINK $_DEV)
         # perform refresh on pre-canonicalised device path
         if [ "$_REFRESH" == "yes" ]; then
-            _debug "ba_scandisks: refreshing device at $_TARGET"
+            _debug "$FUNCNAME: refreshing device at $_TARGET"
             ba_refresh_device $_TARGET
         fi
         if [[ $_TARGET == *dm-* ]]; then
@@ -481,7 +481,7 @@ ba_scandisks () {
 ba_map () {
     local _WWID _TYPE _PART
     _WWID="$1"
-    _debug "ba_map: $_WWID -> ${WWIDMAP[$_WWID]}"
+    _debug "$FUNCNAME: $_WWID -> ${WWIDMAP[$_WWID]}"
     if [ "$_WWID" == "" ] || [ "${WWIDMAP[$_WWID]}" == "" ]; then
         ba_usage "map <wwid>"
         exit 2
@@ -492,7 +492,7 @@ ba_map () {
     else
         _TYPE=mpath
     fi
-    _debug "ba_map: WWID=$_WWID PART=$_PART TYPE=$_TYPE"
+    _debug "$FUNCNAME: WWID=$_WWID PART=$_PART TYPE=$_TYPE"
     if [ "$_PART" != ${PARTMAP[$_WWID]} ] \
         || [ "$_TYPE" != ${TYPEMAP[$_WWID]} ] ; then
         exit 1