Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 18 Apr 2012 18:58:19 +0000 (UTC)
From:      Doug Ambrisko <ambrisko@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-9@freebsd.org
Subject:   svn commit: r234433 - stable/9/usr.sbin/mfiutil
Message-ID:  <201204181858.q3IIwJoD046156@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: ambrisko
Date: Wed Apr 18 18:58:18 2012
New Revision: 234433
URL: http://svn.freebsd.org/changeset/base/234433

Log:
  MFC r233713
  Remove the magic mfi_array is 288 bytes and just use the
  sizeof the array since it is not 288 bytes.
  
  Change reporting of a "SYSTEM" disk to "JBOD" to match
  LSI MegaCli and firmware reporting.
  
  This means that mfiutil command to "create jbod" is now a
  little confusing since a RAID per drive is not really what
  LSI defines JBOD to be.  This should be fixed in the future
  and support added to really create LSI JBOD and enable that
  feature on cards that support it.

Modified:
  stable/9/usr.sbin/mfiutil/mfi_config.c
  stable/9/usr.sbin/mfiutil/mfi_drive.c
Directory Properties:
  stable/9/   (props changed)
  stable/9/usr.sbin/mfiutil/   (props changed)

Modified: stable/9/usr.sbin/mfiutil/mfi_config.c
==============================================================================
--- stable/9/usr.sbin/mfiutil/mfi_config.c	Wed Apr 18 18:43:01 2012	(r234432)
+++ stable/9/usr.sbin/mfiutil/mfi_config.c	Wed Apr 18 18:58:18 2012	(r234433)
@@ -211,9 +211,8 @@ clear_config(int ac, char **av)
 }
 MFI_COMMAND(top, clear, clear_config);
 
-#define	MFI_ARRAY_SIZE		288
-#define	MAX_DRIVES_PER_ARRAY						\
-	((MFI_ARRAY_SIZE - sizeof(struct mfi_array)) / 8)
+#define MAX_DRIVES_PER_ARRAY MFI_MAX_ROW_SIZE
+#define MFI_ARRAY_SIZE sizeof(struct mfi_array)
 
 #define	RT_RAID0	0
 #define	RT_RAID1	1
@@ -305,7 +304,7 @@ parse_array(int fd, int raid_type, char 
 
 	/* Validate the number of drives for this array. */
 	if (count >= MAX_DRIVES_PER_ARRAY) {
-		warnx("Too many drives for a single array: max is %zu",
+		warnx("Too many drives for a single array: max is %d",
 		    MAX_DRIVES_PER_ARRAY);
 		return (EINVAL);
 	}

Modified: stable/9/usr.sbin/mfiutil/mfi_drive.c
==============================================================================
--- stable/9/usr.sbin/mfiutil/mfi_drive.c	Wed Apr 18 18:43:01 2012	(r234432)
+++ stable/9/usr.sbin/mfiutil/mfi_drive.c	Wed Apr 18 18:58:18 2012	(r234433)
@@ -149,7 +149,7 @@ mfi_pdstate(enum mfi_pd_state state)
 	case MFI_PD_STATE_COPYBACK:
 		return ("COPYBACK");
 	case MFI_PD_STATE_SYSTEM:
-		return ("SYSTEM");
+		return ("JBOD");
 	default:
 		sprintf(buf, "PSTATE 0x%04x", state);
 		return (buf);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201204181858.q3IIwJoD046156>