src-test2/sys/dev/mpr, branch release/11.0.0 FreeBSD source tree https://cgit-dev.freebsd.org/src-test2/atom?h=release%2F11.0.0 2016-07-19T16:46:27Z MFC r302673 2016-07-19T16:46:27Z Stephen McConnell slm@FreeBSD.org 2016-07-19T16:46:27Z urn:sha1:fe49fb6bb11fb19d3e09b200e02e1ba6c2a72050 Use real values to calculate Max I/O size instead of guessing. Reviewed by: ken, scottl Approved by: re(gjb), ken, scottl, ambrisko (mentors) Differential Revision: https://reviews.freebsd.org/D7043 Remove NULL checks after M_WAITOK allocations from mpr(4). 2016-05-10T15:04:24Z Edward Tomasz Napierala trasz@FreeBSD.org 2016-05-10T15:04:24Z urn:sha1:13a89428278aa7bcd19286e7c653705c0dc34c09 Reviewed by: asomers@ MFC after: 1 month Sponsored by: The FreeBSD Foundation Differential Revision: https://reviews.freebsd.org/D6297 Bump version of mpr driver to 13.00.00.00-fbsd 2016-05-09T16:38:51Z Stephen McConnell slm@FreeBSD.org 2016-05-09T16:38:51Z urn:sha1:e769dfac4945d24ec7cc7accaf9d006edcb80439 Approved by: ken, scottl, ambrisko MFC after: 1 week Disks can go missing until a reboot is done in some cases. 2016-05-09T16:36:40Z Stephen McConnell slm@FreeBSD.org 2016-05-09T16:36:40Z urn:sha1:58581c1363b5aa5b03cdb4161c0be81edd4300ac This is due to the DevHandle not being released, which causes the Firmware to not allow that disk to be re-added. Reviewed by: ken, scottl, ambrisko, asomers Approved by: ken, scottl, ambrisko MFC after: 1 week Differential Revision: https://reviews.freebsd.org/D6102 done_ccb pointer can be used if it is NULL. 2016-05-09T16:34:06Z Stephen McConnell slm@FreeBSD.org 2016-05-09T16:34:06Z urn:sha1:5d21655f671994ceca29e27a8c78e1eb1d671599 To prevent this, move check for done_ccb == NULL to before done_ccb is used in mprsas_stop_unit_done(). Reviewed by: ken, scottl, ambrisko, asomers Approved by: ken, scottl, ambrisko MFC after: 1 week Differential Revision: https://reviews.freebsd.org/D6099 Use callout_reset_sbt() instead of callout_reset() if FreeBSD ver is >= 1000029 2016-05-09T16:31:43Z Stephen McConnell slm@FreeBSD.org 2016-05-09T16:31:43Z urn:sha1:407073a044f3b28a337b339f59ae67643c7bafbf Reviewed by: ken, scottl, ambrisko, asomers Approved by: ken, scottl, ambrisko MFC after: 1 week Differential Revision: https://reviews.freebsd.org/D6101 No need to set the MPRSAS_SHUTDOWN flag because it's never used. 2016-05-09T16:29:25Z Stephen McConnell slm@FreeBSD.org 2016-05-09T16:29:25Z urn:sha1:dd9f4a95b2a1eb9b5ee16ad30cfc48811b2fb69c Approved by: ken, scottl, ambrisko MFC after: 1 week Fix possible use of invalid pointer. 2016-05-09T16:26:34Z Stephen McConnell slm@FreeBSD.org 2016-05-09T16:26:34Z urn:sha1:88619392216a5446c44e6633629936f4ff1895fa It was possible to use an invalid pointer to get the target ID value. To fix this, initialize a local Target ID variable to an invalid value and change that variable to a valid value only if the pointer to the Target ID is not NULL. Reviewed by: ken, scottl, ambrisko, asomers Approved by: ken, scottl, ambrisko MFC after: 1 week Differential Revision: https://reviews.freebsd.org/D6100 Change logging level for a debug string to use MPR_LOG instead of MPR_INFO. 2016-05-09T16:24:16Z Stephen McConnell slm@FreeBSD.org 2016-05-09T16:24:16Z urn:sha1:b41c6ff97a5e2b217f80598adcbb81de2ca52ecd Approved by: ken, scottl, ambrisko MFC after: 1 week No log bit in IOCStatus and endian-safe changes. 2016-05-09T16:21:14Z Stephen McConnell slm@FreeBSD.org 2016-05-09T16:21:14Z urn:sha1:d3f6eabf131ddfb53caf67b90d53b2538cb4ab79 Use MPI2_IOCSTATUS_MASK when checking IOCStatus to mask off the log bit, and make a few more things endian-safe. Reviewed by: ken, scottl, ambrisko, asomers Approved by: ken, scottl, ambrisko MFC after: 1 week Differential Revision: https://reviews.freebsd.org/D6097