Date: Fri, 05 Apr 2013 14:22:36 -0700 From: Sean Bruno <seanwbruno@gmail.com> To: Mark Johnston <markj@freebsd.org> Cc: freebsd-scsi@freebsd.org Subject: Re: adding BBU relearn support to mfiutil Message-ID: <1365196956.17311.13.camel@localhost> In-Reply-To: <20130304033836.GA33631@oddish> References: <20130304033836.GA33631@oddish>
next in thread | previous in thread | raw e-mail | index | archive | help
--=-S/UqI/W50Kx8QFyI4S2v Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable On Sun, 2013-03-03 at 22:38 -0500, Mark Johnston wrote: > Hi Everyone, >=20 > I recently needed to add a couple of features to mfiutil related to BBU > relearning. I've pasted a patch below which >=20 > 1. adds extra fields to the output of "mfiutil show battery" showing BBU > properties. This is essentially the output of >=20 > # MegaCli -AdpBbuInfo -GetBbuProperties -aLL >=20 > and consists of info about battery learning: the learn period, the > time at which the controller will start the next relearn, and the BBU > mode (which indicates whether the battery supports transparent > relearning). >=20 > 2. adds a couple of subcommands under "mfiutil bbu" which lets users set > the BBU properties which can be set by MegaCli. >=20 > 3. adds a command "mfiutil start learn" which immediately kicks off a > battery relearn. >=20 > These changes grew out of concern about the fact that the controller > write cache is set to write-through mode during a relearn period (which > usually lasts for several hours). This ended up causing some mysterious > and intermittent performance issues, so I needed a way of getting more > info about what was going on (using MegaCli isn't really an option for > several reasons). Some BBUs support transparent relearning, which > basically means that the controller write cache doesn't get turned off > during a relearn. However, LSI's default config doesn't enable it, and > now mfiutil can be used to do that (through "mfiutil bbu bbu-mode"). >=20 > I was hoping someone would be able to review the patch. If anyone's able > and willing to test it, I'd very much appreciate feedback from that. >=20 > Thanks! > -Mark Just to document for the record. Finally got around to testing this today with Mark providing updates. Looks good overall with a couple of nits that he is handling at the moment (man page and variable name collision). Sean --=-S/UqI/W50Kx8QFyI4S2v Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.13 (FreeBSD) iQEcBAABAgAGBQJRX0CcAAoJEBkJRdwI6BaHXqYH/37KVK+hCK/SzyK2BiGYK+AU iIab4boLwgzCWZr7fYtQwHfyngVwrwubC6grhAsyw6eDdodmKFosTZT1ZPEsXtyW JCUasWrpppxvo+HyaIF708ZlKHVX5V7O+yG2gx5IBM61DkXnh69D9hcn+hiqFU3I vHcvzpZi24gjxL2fH2fXro9RO3Ng0ctQlqW3dvirFtsV04por0XglvDO35hH47m6 wRWL4zNqiO2mbGg2u6ou3RtbwRZdBKJSFn4s9Fi766LX0+q/amwFAMt2hreXGr8M AoSPhvjtPVKulHwvMYamrmePjB8WbrN4YEu+F4YhnQe9abRj7ZELj8OwY9W4oi8= =Bfrb -----END PGP SIGNATURE----- --=-S/UqI/W50Kx8QFyI4S2v--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1365196956.17311.13.camel>