)]}' { "log": [ { "commit": "de2eb4d5c5c25e8fb75d1e19092f24b83cb7d8d5", "tree": "537fae73fbf34d34b5e83151a467275986ffb6b5", "parents": [ "6c6aacbb7787dccc6fb662bae66e599bbf0f07b5" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Tue Mar 13 17:15:11 2012 -0700" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:44 2012 -0700" }, "message": "isci: End the RNC resumption wait when the RNC is destroyed.\n\nWhile the RNC is suspended for I/O cleanup, the remote device can be\nstopped and the RNC setup for destruction. These changes accomodate that\ncase in the abort path.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "6c6aacbb7787dccc6fb662bae66e599bbf0f07b5", "tree": "ec1c60ef970f55d2bb09901fc66f4efca8e09865", "parents": [ "79cbab89ff31b6c6ab896d4ed5e3b2ae65193a96" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Tue Mar 13 17:03:00 2012 -0700" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:44 2012 -0700" }, "message": "isci: Fixed RNC bug that lost the suspension or resumption during destroy\n\nThis fix corrects the saving of resume parameters when the destruction\nof the RNC has already been directed, and makes sure not to overwrite\nthe RNC destruction callbacks.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "79cbab89ff31b6c6ab896d4ed5e3b2ae65193a96", "tree": "cd88e03d748a9bc6b2df873fc430c3b62282e170", "parents": [ "3ef768c6c0caa83b9fe66f19a18898ed0315ac36" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Tue Mar 13 16:36:35 2012 -0700" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:44 2012 -0700" }, "message": "isci: Fix RNC AWAIT_SUSPENSION-\u003eINVALIDATING transition.\n\nThe RNC state machine would incorrectly transition from\nSCI_RNC_AWAIT_SUSPENSION directly to SCI_RNC_INVALIDATING when a destruct\nrequest was made. This would skip the increment of the suspension count\nand the abort of pending TCs (although the invalidating state would at\nleast cleanup outstanding TCs).\n\nInstead, the RNC will transition to SCI_RNC_SUSPENDED and then start the\ndestruction process.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "3ef768c6c0caa83b9fe66f19a18898ed0315ac36", "tree": "cf8233efe91dfac3232f06a51f58c4e4c5a508b3", "parents": [ "f8381807ebdfffa34c2c5aa38eda33673d1a7adf" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Mon Mar 12 17:29:51 2012 -0700" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:44 2012 -0700" }, "message": "isci: Manage the IREQ_NO_AUTO_FREE_TAG under scic_lock.\n\nSince there is a possibilty of a timeout waiting for the RNC suspension,\nhandle the exit case from the task termination under scic_lock, and leave\nthe tag allocated if the termination timed-out.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "f8381807ebdfffa34c2c5aa38eda33673d1a7adf", "tree": "e91cfb2d072f416fa5c5b72acb267e4945fffe94", "parents": [ "397497dd61948b0d59d1d21812b93c97b0eeb2dd" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Sun Mar 04 12:44:53 2012 +0000" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:44 2012 -0700" }, "message": "isci: Remove obviated host callback list.\n\nSince the callbacks to libsas now occur under scic_lock, there is no\nlonger any reason to save the completed requests in a separate list\nfor completion to libsas.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "397497dd61948b0d59d1d21812b93c97b0eeb2dd", "tree": "f156de22337932cba5d5a4dedbb80bfa0054dd4c", "parents": [ "87805162b6af20d2ad386a49aec13b753cca523a" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Sat Mar 10 05:46:46 2012 +0000" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:43 2012 -0700" }, "message": "isci: Check IDEV_GONE before performing abort path operations.\n\nIn the link fail path, set IDEV_GONE for every device on the domain\nwhen the last link in the port fails.\n\nIn the abort path functions like isci_reset_device, make sure that\nthere has not already been a detected domain failure with the device\nby checking IDEV_GONE, before performing any kind of hard reset, SMP\nphy control, or TMF operation.\n\nThe check for IDEV_GONE makes sure that the device in the abort path\nreally has control of the port with which it is associated. This\nprevents starting hard resets at incorrect times and scheduling\nunnecessary LUN resets for SATA devices.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "87805162b6af20d2ad386a49aec13b753cca523a", "tree": "bfe5b09eaa3943636263d8695c946550b883b236", "parents": [ "1f05388933cb6e57ed9e51768c194ff145002f3b" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:42:09 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:43 2012 -0700" }, "message": "isci: Restore the ATAPI device RNC management code.\n\nThe ATAPI specific and STP general RNC suspension code had been\nincorrectly removed from the remote device code.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "1f05388933cb6e57ed9e51768c194ff145002f3b", "tree": "4e4bae448746ffdb0196b8e16a4f25893eb10855", "parents": [ "c5457a82a404db3c447df22e6425c5c140c4bee1" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:42:08 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:43 2012 -0700" }, "message": "isci: Don\u0027t wait for an RNC suspend if it\u0027s being destroyed.\n\nMake sure that the wait for suspend can handle the RNC destruction case.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "c5457a82a404db3c447df22e6425c5c140c4bee1", "tree": "a98780529a483b511af46b85c021b27f26d06e2f", "parents": [ "8c731888bf1be8d15d587ab1b4da80553302e653" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:42:07 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:42 2012 -0700" }, "message": "isci: Change the phy control and link reset interface for HW reasons.\n\nThere is an apparent HW lockup caused when the PE is disabled while there\nis an outstanding TC in progress. This change puts the link into OOB to\nforce the TC to end before the PE is disabled.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "8c731888bf1be8d15d587ab1b4da80553302e653", "tree": "0f5cba0195d75ae7a4b5c1ecf19bf6b1ced36ea5", "parents": [ "28de92bef0fb0c3953aa73d31a961422ef900e6a" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:42:06 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:42 2012 -0700" }, "message": "isci: Added timeouts to RNC suspensions in the abort path.\n\nThis change adds timeouts to the RNC suspension wait. It makes the\nsuspend and resume timeouts the same.\n\nThe previous resume timeout of 5 ms was too short, and timeouts were\nseen in resumptions of devices in the abort task/LUN reset path - which\nwould receive an RNC resumed message within a tenth of a second later.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "28de92bef0fb0c3953aa73d31a961422ef900e6a", "tree": "c428479eb9257fc08f2cb968d2f1377a312c803e", "parents": [ "1db79b3e784bffe7e00f9462a5c3441746e48632" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:42:06 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:42 2012 -0700" }, "message": "isci: Add protocol indicator for TMF requests.\n\nRequests contructed as task management requests need to have the protocol\nindicator set so the completion decode can observe any RNC suspension\nconditions.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "1db79b3e784bffe7e00f9462a5c3441746e48632", "tree": "fbe817512475d81423cbecb861a95e770aac5a70", "parents": [ "0cce165e2814bc8c08ab229db5e17013971dced7" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:42:05 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:42 2012 -0700" }, "message": "isci: Directly control IREQ_ABORT_PATH_ACTIVE when completing TMFs.\n\nTMF requests, unlike normal I/O requests, need to handle I/O management\nconditions in the completion function because TMFs are not handled in the\ncompletion tasklet.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "0cce165e2814bc8c08ab229db5e17013971dced7", "tree": "48dde48b2f1af5fee99bd4439914abde40b62912", "parents": [ "d76689e46c8b2180c08575adc830cfda890ceb87" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:42:04 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:42 2012 -0700" }, "message": "isci: Wait for RNC resumption before leaving the abort path.\n\nIn the case of TMF execution, or device resets, wait for the RNC to fully\nresume before returning to the caller. This ensures that the remote\ndevice will not fail I/O requests while waiting for the RNC resumption to\ncomplete.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "d76689e46c8b2180c08575adc830cfda890ceb87", "tree": "16788cf746428daf83bd6fd03c039e51af2bc05e", "parents": [ "621120ca56850249554996c94efe75f8200a2cc0" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:42:04 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:41 2012 -0700" }, "message": "isci: Fix RNC suspend call for SCI_RESUMING state.\n\nInstead of immediately transitioning to the SCI_RNC_AWAIT_SUSPENSION\nstate, handle the SCI_RNC_RESUMING suspend transition from the\nSCI_RNC_READY state like the SCI_RNC_INVALIDATING --\u003e SCI_RNC_POSTING\ntransitions do now, by setting the destination state for the entry\ninto the READY state.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "621120ca56850249554996c94efe75f8200a2cc0", "tree": "892326cd79b25e0741ee271c01cc04aadb1af8ef", "parents": [ "033d19d298b4245da2d3d6c795ea97e419f9ac61" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:42:03 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:41 2012 -0700" }, "message": "isci: Manage tag releases differently when aborting tasks.\n\nWhen an individual request is being terminated, the request\u0027s tag\nis managed in the terminate function.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "033d19d298b4245da2d3d6c795ea97e419f9ac61", "tree": "2e66627f294b9e2b52174e719ff987464cedaeec", "parents": [ "0c3ce38f1bc8b6a6d8df0959e3c0dece31f9350c" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:42:03 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:41 2012 -0700" }, "message": "isci: Callbacks to libsas occur under scic_lock and are synchronized.\n\nThis patch changes the callback mechanism to libsas to only occur while\nthe scic_lock is held; the abort path cleanup of I/Os also checks to make\nsure IREQ_ABORT_PATH_ACTIVE is clear before proceding.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "0c3ce38f1bc8b6a6d8df0959e3c0dece31f9350c", "tree": "22d5fb8aaea17b1ea51b060e51f5a709e382c6e0", "parents": [ "31a38ef0a5ad12dbe262ca55d0a905657be55a8d" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:42:02 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:41 2012 -0700" }, "message": "isci: When in the abort path, defeat other resume calls until done.\n\nCompletion of I/Os during the one of the abort path interface calls\nfrom libsas can drive remote device state changes and the resumption\nof the device RNC. This is a problem when the abort path is\nattempting to cleanup outstanding I/O at the same time - the resumption\ncan prevent the termination from occuring correctly.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "31a38ef0a5ad12dbe262ca55d0a905657be55a8d", "tree": "cf05c3c5fb97670711e58edc6708d7f52e02f418", "parents": [ "08c031e4e3294a66a64074e12482abda846dd39c" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:42:01 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:41 2012 -0700" }, "message": "isci: Implement waiting for suspend in the abort path.\n\nIn order to prevent a device from receiving an I/O request while still\nin an RNC suspending or resuming state (and therefore failing that\nI/O back to libsas with a reset required status) wait for the RNC state\nchange before proceding in the abort path.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "08c031e4e3294a66a64074e12482abda846dd39c", "tree": "c7fb3392487ed731a39a75239850be657009aa56", "parents": [ "9608b6408e637abeec101abb6aebd3343f0ebac4" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:42:00 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:40 2012 -0700" }, "message": "isci: Make sure all TCs are terminated and cleaned in LUN reset.\n\nIn the libsas error path, SATA disks require extra handling in\nlibata to recover operation. However, libsas expects to be able\nto immediately recover all outstanding I/O once the error handler\nescalation stops. This patch fixes the condition where the libata\nerror handler is scheduled for operation but libsas has already\ndeleted the outstanding sas_tasks.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "9608b6408e637abeec101abb6aebd3343f0ebac4", "tree": "4c06b2ed04c61c2e76dd7d663a54b7cc1ae0717f", "parents": [ "447bfbcee070a0b43dd6abc743063d7a02fe65ca" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:42:00 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:40 2012 -0700" }, "message": "isci: Manage the LLHANG timer enable/disable per-device.\n\nThe LLHANG timer should be enabled once per device. This patch corrects\nboth the timer enable and the timer disable for the remote device.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "447bfbcee070a0b43dd6abc743063d7a02fe65ca", "tree": "7431e8eaadc2112795fcc7e3ba6a355bc1fafad9", "parents": [ "e3c84dfdb8f4c675b0ba5cf3fa252dc4056b7ddd" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:41:59 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:40 2012 -0700" }, "message": "isci: Save the suspension hint for upcoming suspensions.\n\nIn the case of a suspend call while in SCI_RNC_POSTING or INVALIDATING\nstates, the LLHANG detect needed to be saved so the upcoming suspension\nwould enable it correctly. The unused suspend callback parameters were\nremoved.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "e3c84dfdb8f4c675b0ba5cf3fa252dc4056b7ddd", "tree": "d7c7216ccd4740f41a926cd467d101e7a01339ff", "parents": [ "c94fc1ad25de885e1c59f714f19bc726e7a21caf" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:41:58 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:40 2012 -0700" }, "message": "isci: Fix the terminated I/O to not call sas_task_abort().\n\nThis addresses a regression from the commit \"isci: Redesign\ndevice suspension, abort, cleanup.\" in which the sas_task end\ncondition for terminated I/Os was made to call back on\nsas_task_abort()\".\nThis commit will be rolled into the original.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "c94fc1ad25de885e1c59f714f19bc726e7a21caf", "tree": "f522804dabddeb78304384f897264f731588eab0", "parents": [ "d6b2a0e4a066ea51322e16e66b25028cb0b4ca7e" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:41:58 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:40 2012 -0700" }, "message": "isci: Distinguish between remote device suspension cases\n\nFor NCQ error conditions among others, there is no need to enable\nthe link layer hang detect timer.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "d6b2a0e4a066ea51322e16e66b25028cb0b4ca7e", "tree": "77fbde12b41062458824621916498f335c6eef11", "parents": [ "033751f6643adf616b85ac5eea7ce784bdde1b50" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:41:57 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:39 2012 -0700" }, "message": "isci: Remove isci_device reqs_in_process and dev_node from isci_device.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "033751f6643adf616b85ac5eea7ce784bdde1b50", "tree": "77135e835a84ce987c87fe2623238eab195bfa80", "parents": [ "aa20d9343079b1f0bebd43dec82ecfd4af5e43da" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:41:56 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:39 2012 -0700" }, "message": "isci: Only set IDEV_GONE in the device stop path.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "aa20d9343079b1f0bebd43dec82ecfd4af5e43da", "tree": "2e04327fd7625c34fa2732a25763f4119bcfc7f1", "parents": [ "637325028f3a9e9cf411ede96063997611f976e4" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:41:56 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:39 2012 -0700" }, "message": "isci: All pending TCs are terminated when the RNC is invalidated.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "637325028f3a9e9cf411ede96063997611f976e4", "tree": "56e9dc922a1ed8c22cb114cbfc9ee58bd324af93", "parents": [ "59e35396436c564b5019e1a70073900bc3e19f4f" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:41:55 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:39 2012 -0700" }, "message": "isci: Device access in the error path does not depend on IDEV_GONE.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "59e35396436c564b5019e1a70073900bc3e19f4f", "tree": "0bfa36c98fede19f78f136e92c2fd6bb2932051e", "parents": [ "14aaa9f0a318bd04cbb9d822524b817e95d8b343" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:41:54 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:39 2012 -0700" }, "message": "isci: Add suspension cases for RNC INVALIDATING, POSTING states.\n\nThe RNC can be any of the states in the loop from suspended to\nready when the API \"suspend\" or \"resume\" are called. This change\nadds destination states parameters that control the suspension /\nresumption action of the RNC statemachine for those transition states.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "14aaa9f0a318bd04cbb9d822524b817e95d8b343", "tree": "5f6292ec671d0993c1df51586dea4fb7750d1cd9", "parents": [ "d80ecd5726ce49b5da457d562804b40f0183e8f7" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:41:54 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:38 2012 -0700" }, "message": "isci: Redesign device suspension, abort, cleanup.\n\nThis commit changes the means by which outstanding I/Os are handled\nfor cleanup.\nThe likelihood is that this commit will be broken into smaller pieces,\nhowever that will be a later revision. Among the changes:\n\n- All completion structures have been removed from the tmf and\nabort paths.\n- Now using one completed I/O list, with the I/O completed in host bit being\nused to select error or normal callback paths.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "d80ecd5726ce49b5da457d562804b40f0183e8f7", "tree": "bc3d6218f40d014f69edc3467c22fc533e269aa0", "parents": [ "83884014eaaa68834ced39d1c75f1bc20d618ec0" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:41:53 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:38 2012 -0700" }, "message": "isci: Escalate to I_T_Nexus_Reset when the device is gone.\n\nIf LUN reset sees that the device is gone, it returns TMF_RESP_FUNC_FAILED\nto cause libsas to escalate to an I_T_Nexus_Reset.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "83884014eaaa68834ced39d1c75f1bc20d618ec0", "tree": "1a54f8c3b64b95938d0499412a37c780ecb84c5f", "parents": [ "23ec2aa947e83d0a172220f361166b8224875221" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:41:52 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:37 2012 -0700" }, "message": "isci: Remote device stop also suspends the RNC and terminates I/O.\n\nFixing the remote device state machine to suspend and terminate\nall outstanding I/O before the device stopped state is reached.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "23ec2aa947e83d0a172220f361166b8224875221", "tree": "747e299efb5c534d99bc84b9631dedec02b033aa", "parents": [ "5b6bf225e7fc249c703e19bf2c983d1a59178874" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:41:52 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:37 2012 -0700" }, "message": "isci: Remote device must be suspended for NCQ cleanup.\n\nWhen the remote device enters the NCQ error state, the device must\nbe suspended so that the I/O terminations can take place.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "5b6bf225e7fc249c703e19bf2c983d1a59178874", "tree": "9c9b855474b77323c0d89b33ac224e0c1f016c73", "parents": [ "726980d56908f2e230624394f03743689db3110c" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:41:51 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:37 2012 -0700" }, "message": "isci: Manage device suspensions during TC terminations.\n\nTCs must be terminated only while the RNC is suspended. This commit\nadds remote device suspensions and resumptions in the abort, reset and\ntermination paths.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "726980d56908f2e230624394f03743689db3110c", "tree": "6aca898ea8f57c276088feb2eed5770fba6dfd86", "parents": [ "ac78ed0f78eae5c3c918e132b5e2029cdc4fdedc" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:41:50 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:37 2012 -0700" }, "message": "isci: Terminate outstanding TCs on TX/RX RNC suspensions.\n\nTCs must only be terminated when RNCs are suspended.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "ac78ed0f78eae5c3c918e132b5e2029cdc4fdedc", "tree": "ac1e7cc2dc8fcffa22d8791baf265dd748594a1f", "parents": [ "56d7c013e714c6feab2ab5ac854808e29048b069" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:41:50 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:37 2012 -0700" }, "message": "isci: Handle all suspending TC completions\n\nAdd comprehensive decode for all TC completions that generate RNC\nsuspensions.\n\nNote that this commit also removes unconditional resumptions of ATAPI\ndevices when in the SCI_STP_DEV_ATAPI_ERROR state, and STP devices\nwhen in the SCI_STP_DEV_IDLE state. This is because the SCI_STP_DEV_IDLE\nand SCI_STP_DEV_ATAPI state entry functions manage the RNC resumption.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "56d7c013e714c6feab2ab5ac854808e29048b069", "tree": "da627f1e8d49fe6d2350123b31f418e572f15ad0", "parents": [ "6f48844e3f16b7d8a1f9a1a11bd9a11089a5292f" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:41:49 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:36 2012 -0700" }, "message": "isci: Fixed bug in resumption from RNC Tx/Rx suspend state.\n\nThe resumption from the Tx/Rx suspended state should work the same\nas the Tx suspended state.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "6f48844e3f16b7d8a1f9a1a11bd9a11089a5292f", "tree": "aeb95746d05e7bab2b33a9351443e03152e4dfa4", "parents": [ "fc25f79af321c01a739150ba2c09435cf977a63d" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Mar 08 22:41:48 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 14:33:36 2012 -0700" }, "message": "isci: Manage the link layer hang detect timer for RNC suspensions.\n\nFor STP devices under certain protocol conditions, an RNC will not\nsuspend until the current transfer state is broken with a SYNC/ESC\nsequence from the SCU. The SYNC/ESC driven by expiration of the\nSCU link layer hang detect timer, which has too small a dynamic\nrange to support slow SATA devices, so normally it is disabled.\n\nThis change enables the timer with the minimum period at the point\nwhen the suspension is requested.\n\nNote that there is potential collateral damage to other open\nconnections to slow SATA devices on the same port, since there\nis no alternative but to enable the LLHANG timer on every phy in\nthe port for the current suspension request - there is no way to\ntell on which phy the RNC in question is currently active.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "fc25f79af321c01a739150ba2c09435cf977a63d", "tree": "b251a8a7399c9e991d084857b2d1a7b5c07c24e2", "parents": [ "08e73be56b6b2e5459638481a54b755ed562ada8" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Mon Apr 30 11:57:44 2012 -0700" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 12:27:29 2012 -0700" }, "message": "isci: fix oem parameter validation on single controller skus\n\nOEM parameters [1] are parsed from the platform option-rom / efi\ndriver. By default the driver was validating the parameters for the\ndual-controller case, but in single-controller case only the first set\nof parameters may be valid.\n\nLimit the validation to the number of actual controllers detected\notherwise the driver may fail to parse the valid parameters leading to\ndriver-load or runtime failures.\n\n[1] the platform specific set of phy address, configuration,and analog\n tuning values\n\n[stable v3.0+]\nCc: \u003cstable@vger.kernel.org\u003e\nReported-by: Dave Jiang \u003cdave.jiang@intel.com\u003e\nTested-by: Dave Jiang \u003cdave.jiang@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "08e73be56b6b2e5459638481a54b755ed562ada8", "tree": "f3ebe335019d247222b6b404177669d3d583814e", "parents": [ "6119908f0fe3737bba2f64eff70599c3e41d522e" ], "author": { "name": "Maciej Trela", "email": "maciej.trela@intel.com", "time": "Mon Mar 12 23:29:30 2012 +0000" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 12:27:28 2012 -0700" }, "message": "isci: enable BCN in sci_port_add_phy()\n\nEnsure we enable receiving BCN\u0027s from the\nhardware when adding phy to isci_port.\nOtherwise if we get BCN before the port is\ncreated we won\u0027t see any BCN\n\nSigned-off-by: Maciej Trela \u003cmaciej.trela@intel.com\u003e\nReported-by: Richard Boyd \u003crichard.g.boyd@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "6119908f0fe3737bba2f64eff70599c3e41d522e", "tree": "b78a7d433149ba3def1d427a7f4662d17eb00dd6", "parents": [ "d1dc5e2d21a55538167d7ce82aa147c91c5e6788" ], "author": { "name": "Andrzej Jakowski", "email": "andrzej.jakowski@intel.com", "time": "Thu Mar 08 19:38:50 2012 +0000" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 12:27:28 2012 -0700" }, "message": "isci: Changes in COMSAS timings enabling ISCI to detect buggy disc drives.\n\nThis patch extends timings in COMSAS signaling, so ISCI can detect disc\ndrives having issues to send COMSAS in correct time frame.\n\nSigned-off-by: Andrzej Jakowski \u003candrzej.jakowski@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "d1dc5e2d21a55538167d7ce82aa147c91c5e6788", "tree": "b63ebbe0267506d1f355a987229ea18cd0807670", "parents": [ "2396a2650a5a39634e3ad6b29e1104944e5ab88f" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Sat Feb 25 14:29:49 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 12:27:13 2012 -0700" }, "message": "isci: kill isci_host.shost\n\nWe can retrieve the shost from the sas_ha like the rest of libsas and\ndrop this out of our local data structure.\n\nAcked-by: Jacek Danecki \u003cjacek.danecki@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "2396a2650a5a39634e3ad6b29e1104944e5ab88f", "tree": "a5f1e724f1a3eec60da92aef018c6c7840f95bcc", "parents": [ "50a92d93148ec073efd2456b007e04ecae452086" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu Mar 01 17:06:24 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 12:27:12 2012 -0700" }, "message": "isci: fix interrupt disable\n\nThere is a (dubious?) lost irq workaround in sci_controller_isr() that\neffectively nullifies attempts to disable interrupts. Until the\nworkaround can be re-evaluated add some infrastructure to prevent the\ninterrupt handler from inadvertantly re-enabling interrupts.\n\nThe failure mode was interrupts continuing to run after the driver had\nbeen removed and its iomappings torn down.\n\nReported-by: Jacek Danecki \u003cjacek.danecki@intel.com\u003e\nTested-by: Jacek Danecki \u003cjacek.danecki@intel.com\u003e\n[richard: clear remaining interrupts at the end of reset]\nAcked-by: Richard Boyd \u003crichard.g.boyd@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "50a92d93148ec073efd2456b007e04ecae452086", "tree": "1a81ac87e301bdb7bd809daea52bd7be0b3b21b4", "parents": [ "eb608c3cb3f0a6b99252ea6a69fc0d2bbecf1f4f" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Wed Feb 29 01:07:56 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 12:27:12 2012 -0700" }, "message": "isci: fix \u0027link-up\u0027 events occur after \u0027start-complete\u0027\n\nThe call to wait_for_start() is meant to ensure that all links have been\ngiven a chance to come up before letting the kernel proceed with\nprobing. However, the implementation is not correctly syncing with the\nport configuration agent. In the MPC case the ports are hard-coded, in\nthe APC case we need to wait for the port-configuration to form ports\nfrom the started phys.\n\nTowards that end increase the timeout for the APC agent to form ports,\nand delay start complete until all phys are out of link-training.\n\nCc: \u003cstable@vger.kernel.org\u003e\nCc: Richard Boyd \u003crichard.g.boyd@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "eb608c3cb3f0a6b99252ea6a69fc0d2bbecf1f4f", "tree": "e28da3a8a530798c7c3368d16f88003ee3ee0643", "parents": [ "abec912d71c44bbd642ce12ad98aab76f5a53163" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu Feb 23 01:12:10 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 12:27:12 2012 -0700" }, "message": "isci: fix controller stop\n\n1/ notify waiters when controller stop completes (fixes 10 second stall\n unloading the driver)\n2/ make sure phy stop is after port and device stop\n\nCc: Richard Boyd \u003crichard.g.boyd@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "abec912d71c44bbd642ce12ad98aab76f5a53163", "tree": "d8967b23a5a4ea8302b43d4db5e0cd09d21d34d3", "parents": [ "ae904d15cf344bcb426f63982016f6bacc45825b" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Wed Feb 15 13:58:42 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 12:27:12 2012 -0700" }, "message": "isci: refactor initialization for S3/S4\n\nBased on an original implementation by Ed Nadolski and Artur Wojcik\n\nIn preparation for S3/S4 support refactor initialization so that\ndriver-load and resume-from-suspend can share the common init path of\nisci_host_init(). Organize the initialization into objects that are\nself-contained to the driver (initialized by isci_host_init) versus\nthose that have some upward registration (initialized at allocation time\nasd_sas_phy, asd_sas_port, dma allocations). The largest change is\nmoving the the validation of the oem and module parameters from\nisci_host_init() to isci_host_alloc().\n\nThe S3/S4 approach being taken is that libsas will be tasked with\nremembering the state of the domain and the lldd is free to be\nforgetful. In the case of isci we\u0027ll just re-init using a subset of the\nnormal driver load path.\n\n[clean up some unused / mis-indented function definitions in host.h]\n\nSigned-off-by: Ed Nadolski \u003cedmund.nadolski@intel.com\u003e\nSigned-off-by: Artur Wojcik \u003cartur.wojcik@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "ae904d15cf344bcb426f63982016f6bacc45825b", "tree": "76b31d9458213ed3f57656d7e74ea862c41ef04d", "parents": [ "1844e4789fe5c97a9ff3bb82628111abbe7cc846" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Fri Feb 17 16:30:47 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 12:27:12 2012 -0700" }, "message": "isci: kill isci_port.domain_dev_list\n\nAnother unused field, and isci_port_init is overkill.\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "1844e4789fe5c97a9ff3bb82628111abbe7cc846", "tree": "e0ee7084e73d91ef0c0ec29d078f85549d18679e", "parents": [ "944b787d0a469a376f4d6699eb01138823197513" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Wed Feb 15 13:20:31 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 12:27:12 2012 -0700" }, "message": "isci: kill -\u003estatus, and -\u003estate_lock in isci_host\n\nThey serve no incremental purpose over the existing sas_ha state.\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "944b787d0a469a376f4d6699eb01138823197513", "tree": "4a1a59d80cc22afc85729cea30405b60aef2cfe0", "parents": [ "c79dd80d73017a88a2c2ae46e7d5303cba6a32e0" ], "author": { "name": "Tom Jackson", "email": "thomas.p.jackson@intel.com", "time": "Fri Feb 24 09:38:49 2012 +0000" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 12:27:12 2012 -0700" }, "message": "isci: Don\u0027t filter BROADCAST CHANGE primitives\n\nPer the SAS spec, several types of BROADCAST CHANGE primitives\nmust cause re-discovery of the originating expander.\nOnly the standard BROADCAST CHANGE primitive was being\nsent to the LIBSAS layer. The other BC primitives have been\nadded to the sci_phy_event_handler()\n\nSigned-off-by: Tom Jackson \u003cthomas.p.jackson@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "c79dd80d73017a88a2c2ae46e7d5303cba6a32e0", "tree": "4bf1f09df72921272e38bc0c9610374d4c1873e2", "parents": [ "11cc51835af0e6fbb2da9cb012bdaaa036497b7f" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Wed Feb 01 00:44:14 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 12:27:11 2012 -0700" }, "message": "isci: kill sci_phy_protocol and sci_request_protocol\n\nHoldovers from the initial driver cleanup, replace with enum sas_protocol.\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "11cc51835af0e6fbb2da9cb012bdaaa036497b7f", "tree": "c9afaa1c1afc1e185b85fa6137e60282657e1778", "parents": [ "14e99b4a3f5323bb961754de5024daff79e59b98" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Wed Feb 01 00:23:10 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 12:27:11 2012 -0700" }, "message": "isci: kill -\u003eis_direct_attached\n\ndomain_device -\u003eparent conveys the same information.\n\nOccurrences of -\u003eis_direct_attached appear next to incomplete open-coded\nversions of dev_is_sata(), clean those up as well.\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "14e99b4a3f5323bb961754de5024daff79e59b98", "tree": "379c3f28a33865d135969981862eed44e2aba243", "parents": [ "36be50515fe2aef61533b516fa2576a2c7fe7664" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Fri Feb 10 01:05:43 2012 -0800" }, "committer": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu May 17 12:27:11 2012 -0700" }, "message": "isci: improve \u0027invalid state\u0027 warnings\n\nConvert controller state machine warnings to emit the state number (it\nmissed the number to string conversion, but since these error rarely\nhappen not much motivation to go further).\n\nFix up the rnc warnings to use the state name.\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\n" }, { "commit": "424a6f6ef990b7e9f56f6627bfc6c46b493faeb4", "tree": "0028356ed8003495fbbe1f716f359e3c8ebc35b6", "parents": [ "1ab142d499294b844ecc81e8004db4ce029b0b61", "cd8df932d894f3128c884e3ae1b2b484540513db" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Mar 22 12:55:29 2012 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Mar 22 12:55:29 2012 -0700" }, "message": "Merge tag \u0027scsi-misc\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi-misc-2.6\n\nSCSI updates from James Bottomley:\n \"The update includes the usual assortment of driver updates (lpfc,\n qla2xxx, qla4xxx, bfa, bnx2fc, bnx2i, isci, fcoe, hpsa) plus a huge\n amount of infrastructure work in the SAS library and transport class\n as well as an iSCSI update. There\u0027s also a new SCSI based virtio\n driver.\"\n\n* tag \u0027scsi-misc\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi-misc-2.6: (177 commits)\n [SCSI] qla4xxx: Update driver version to 5.02.00-k15\n [SCSI] qla4xxx: trivial cleanup\n [SCSI] qla4xxx: Fix sparse warning\n [SCSI] qla4xxx: Add support for multiple session per host.\n [SCSI] qla4xxx: Export CHAP index as sysfs attribute\n [SCSI] scsi_transport: Export CHAP index as sysfs attribute\n [SCSI] qla4xxx: Add support to display CHAP list and delete CHAP entry\n [SCSI] iscsi_transport: Add support to display CHAP list and delete CHAP entry\n [SCSI] pm8001: fix endian issue with code optimization.\n [SCSI] pm8001: Fix possible racing condition.\n [SCSI] pm8001: Fix bogus interrupt state flag issue.\n [SCSI] ipr: update PCI ID definitions for new adapters\n [SCSI] qla2xxx: handle default case in qla2x00_request_firmware()\n [SCSI] isci: improvements in driver unloading routine\n [SCSI] isci: improve phy event warnings\n [SCSI] isci: debug, provide state-enum-to-string conversions\n [SCSI] scsi_transport_sas: \u0027enable\u0027 phys on reset\n [SCSI] libsas: don\u0027t recover end devices attached to disabled phys\n [SCSI] libsas: fixup target_port_protocols for expanders that don\u0027t report sata\n [SCSI] libsas: set attached device type and target protocols for local phys\n ...\n" }, { "commit": "9f3938346a5c1fa504647670edb5fea5756cfb00", "tree": "7cf6d24d6b076c8db8571494984924cac03703a2", "parents": [ "69a7aebcf019ab3ff5764525ad6858fbe23bb86d", "317b6e128247f75976b0fc2b9fd8d2c20ef13b3a" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Mar 21 09:40:26 2012 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Mar 21 09:40:26 2012 -0700" }, "message": "Merge branch \u0027kmap_atomic\u0027 of git://github.com/congwang/linux\n\nPull kmap_atomic cleanup from Cong Wang.\n\nIt\u0027s been in -next for a long time, and it gets rid of the (no longer\nused) second argument to k[un]map_atomic().\n\nFix up a few trivial conflicts in various drivers, and do an \"evil\nmerge\" to catch some new uses that have come in since Cong\u0027s tree.\n\n* \u0027kmap_atomic\u0027 of git://github.com/congwang/linux: (59 commits)\n feature-removal-schedule.txt: schedule the deprecated form of kmap_atomic() for removal\n highmem: kill all __kmap_atomic() [swarren@nvidia.com: highmem: Fix ARM build break due to __kmap_atomic rename]\n drbd: remove the second argument of k[un]map_atomic()\n zcache: remove the second argument of k[un]map_atomic()\n gma500: remove the second argument of k[un]map_atomic()\n dm: remove the second argument of k[un]map_atomic()\n tomoyo: remove the second argument of k[un]map_atomic()\n sunrpc: remove the second argument of k[un]map_atomic()\n rds: remove the second argument of k[un]map_atomic()\n net: remove the second argument of k[un]map_atomic()\n mm: remove the second argument of k[un]map_atomic()\n lib: remove the second argument of k[un]map_atomic()\n power: remove the second argument of k[un]map_atomic()\n kdb: remove the second argument of k[un]map_atomic()\n udf: remove the second argument of k[un]map_atomic()\n ubifs: remove the second argument of k[un]map_atomic()\n squashfs: remove the second argument of k[un]map_atomic()\n reiserfs: remove the second argument of k[un]map_atomic()\n ocfs2: remove the second argument of k[un]map_atomic()\n ntfs: remove the second argument of k[un]map_atomic()\n ...\n" }, { "commit": "69a7aebcf019ab3ff5764525ad6858fbe23bb86d", "tree": "7211df5704b743a7667159748c670a9744164482", "parents": [ "d464c92b5234227c1698862a1906827e2e398ae0", "f1f996b66cc3908a8f5ffccc2ff41840e92f3b10" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Mar 20 21:12:50 2012 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Mar 20 21:12:50 2012 -0700" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/jikos/trivial\n\nPull trivial tree from Jiri Kosina:\n \"It\u0027s indeed trivial -- mostly documentation updates and a bunch of\n typo fixes from Masanari.\n\n There are also several linux/version.h include removals from Jesper.\"\n\n* \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/jikos/trivial: (101 commits)\n kcore: fix spelling in read_kcore() comment\n constify struct pci_dev * in obvious cases\n Revert \"char: Fix typo in viotape.c\"\n init: fix wording error in mm_init comment\n usb: gadget: Kconfig: fix typo for \u0027different\u0027\n Revert \"power, max8998: Include linux/module.h just once in drivers/power/max8998_charger.c\"\n writeback: fix fn name in writeback_inodes_sb_nr_if_idle() comment header\n writeback: fix typo in the writeback_control comment\n Documentation: Fix multiple typo in Documentation\n tpm_tis: fix tis_lock with respect to RCU\n Revert \"media: Fix typo in mixer_drv.c and hdmi_drv.c\"\n Doc: Update numastat.txt\n qla4xxx: Add missing spaces to error messages\n compiler.h: Fix typo\n security: struct security_operations kerneldoc fix\n Documentation: broken URL in libata.tmpl\n Documentation: broken URL in filesystems.tmpl\n mtd: simplify return logic in do_map_probe()\n mm: fix comment typo of truncate_inode_pages_range\n power: bq27x00: Fix typos in comment\n ...\n" }, { "commit": "77dfce076cbd76c04e90abff188d058cdbff78dd", "tree": "c2f1ac2dd386c68e6bf8dee8d996d0b6e36f9c73", "parents": [ "4679026d783eb5ac90247bc466d66b817b213abf" ], "author": { "name": "Cong Wang", "email": "amwang@redhat.com", "time": "Fri Nov 25 23:14:23 2011 +0800" }, "committer": { "name": "Cong Wang", "email": "xiyou.wangcong@gmail.com", "time": "Tue Mar 20 21:48:19 2012 +0800" }, "message": "scsi: remove the second argument of k[un]map_atomic()\n\nSigned-off-by: Cong Wang \u003camwang@redhat.com\u003e\n" }, { "commit": "13de2c541993394ec4db4949302312a0ce0af7b1", "tree": "5acbb429ab2f09f4147e22fa2a1c578e364d58dc", "parents": [ "e462116afb1e436d4045167e7fe979405ae33d2b" ], "author": { "name": "Andrzej Jakowski", "email": "andrzej.jakowski@intel.com", "time": "Fri Feb 10 01:18:54 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Wed Feb 29 16:24:41 2012 -0600" }, "message": "[SCSI] isci: improvements in driver unloading routine\n\nThis patch fixes scenario where driver removal should be possible\nonly when driver is in READY state. Also it removes redundant\ninvocation of routine disabling SCU interrupts - this method is\ncalled somewhere else in driver deinitialization path.\n\nSigned-off-by: Andrzej Jakowski \u003candrzej.jakowski@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "e462116afb1e436d4045167e7fe979405ae33d2b", "tree": "0ccb050d8c3ce67419a1da7e92dc9c386ad9fd3c", "parents": [ "d7a0ccdd9bd78b5b74d2963ec7ab67c9d896902a" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Fri Feb 10 01:18:49 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Wed Feb 29 16:24:13 2012 -0600" }, "message": "[SCSI] isci: improve phy event warnings\n\nisci occasionally spews messages like:\n\nisci 0000:03:00.0: sci_phy_event_handler: PHY starting substate machine received unexpected event_code b3940000\n\n...which is not very helpful, since we don\u0027t know which controller,\nwhich phy, the exact state, or a decode of the event.\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "d7a0ccdd9bd78b5b74d2963ec7ab67c9d896902a", "tree": "9fc59d3303e7453006c767b0f4daadbf3d4ed5c1", "parents": [ "16d3db1b298529e6aec4a4cf0f56225d9e0bc641" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Fri Feb 10 01:18:44 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Wed Feb 29 16:23:15 2012 -0600" }, "message": "[SCSI] isci: debug, provide state-enum-to-string conversions\n\nDebugging the driver requires tracing the state transtions and tracing\nstate names is less work than decoding numbers.\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "9508a66f898d46e726a318469312b45e0b1d078b", "tree": "e6b61e6c2a7dc8b40fdc0fe34901ff3db4af47a9", "parents": [ "92625f9bff3853951cc75f5bc084ee67c1317d2f" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Wed Jan 18 20:47:01 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Wed Feb 29 15:35:41 2012 -0600" }, "message": "[SCSI] libsas: async ata scanning\n\nlibsas ata error handling is already async but this does not help the\nscan case. Move initial link recovery out from under host-\u003escan_mutex,\nand delay synchronization with eh until after all port probe/recovery\nwork has been queued.\n\nDevice ordering is maintained with scan order by still calling\nsas_rphy_add() in order of domain discovery.\n\nSince we now scan the domain list when invoking libata-eh we need to be\ncareful to check for fully initialized ata ports.\n\nAcked-by: Jack Wang \u003cjack_wang@usish.com\u003e\nAcked-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "5a998328296cb16cb8b0307a402a0f2feab533c5", "tree": "25c3a355256a895189d993b8b45ce029e5eab204", "parents": [ "6a719391e4a13d5fa2ec7642ef0c780d7f3f5716" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Mon Dec 12 20:32:09 2011 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Wed Feb 29 15:16:52 2012 -0600" }, "message": "[SCSI] isci: remove IDEV_EH hack to disable \"discovery-time\" ata resets\n\nPrior to commit 61aaff49 \"isci: filter broadcast change notifications\nduring SMP phy resets\" we borrowed the MVS_DEV_EH approach from the\nmvsas driver for preventing -\u003elldd_I_T_nexus_reset() events during ata\ndiscovery. This hack was protecting against the old -\u003ephy_reset() in\nata_bus_probe(), but since the conversion to the new error handling this\nhack is preventing resets from reaching ata devices.\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "6a719391e4a13d5fa2ec7642ef0c780d7f3f5716", "tree": "669d2569b29f930d2eeb895b2cc81e22e2878472", "parents": [ "687833a03baae8308900486fa6499ef955fd07a9" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu Dec 08 00:37:25 2011 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Wed Feb 29 15:16:05 2012 -0600" }, "message": "[SCSI] isci: remove bus and reset handlers\n\nRemove -\u003eeh_device_reset_handler() and -\u003eeh_bus_reset_handler() for the\nsame reason they are not implemented for libata hosts, they cannot be\nimplemented reliably with ata-eh. ATA error recovery wants to divert\nall resets to the eh thread and wait for completion, these handlers may\nbe invoked from a non-blocking ioctl.\n\nThe other path they are called from is libsas-eh, and if we escalate\npast I_T_nexus reset we have larger problems i.e. tear down all\nin-flight commands in the domain potentially without notification to the\nlldd if it has chosen not to implement -\u003elldd_clear_nexus_port() /\n-\u003elldd_clear_nexus_ha().\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "687833a03baae8308900486fa6499ef955fd07a9", "tree": "76da700de1674f42978b58b35dede2b577a6f901", "parents": [ "43a5ab151f0268459c4368292c2ddb2266b8f243" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu Nov 17 18:01:38 2011 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Wed Feb 29 15:15:07 2012 -0600" }, "message": "[SCSI] isci: -\u003elldd_ata_check_ready handler\n\nReport to libata whether the link to the given domain_device is up and the\nsignature fis has been received.\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "43a5ab151f0268459c4368292c2ddb2266b8f243", "tree": "e8ee97e0d49bc0c454ca90f16a836fd01830b11f", "parents": [ "9277699121b81891e303ada0a53fa1d04b7ffe72" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu Dec 08 23:20:44 2011 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Wed Feb 29 15:13:40 2012 -0600" }, "message": "[SCSI] isci: stop interpreting -\u003elldd_lu_reset() as an ata soft-reset\n\nDriving resets from libsas-eh is pre-mature as libata will make a\ndecision about performing a softreset. Currently libata determines\nwhether to perform a softreset based on ata_eh_followup_srst_needed(),\nand none of those conditions apply to isci.\n\nRemove the srst implementation and translate -\u003elldd_lu_reset() for ata\ndevices as a request to drive a reset via libata-eh.\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "9277699121b81891e303ada0a53fa1d04b7ffe72", "tree": "ed8fde54c953025be2a59697dd912f581536c782", "parents": [ "fca4ecbdc440337b3c257b38c2f4cc8d0ca0286c" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Wed Nov 30 11:57:34 2011 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Wed Feb 29 15:11:54 2012 -0600" }, "message": "[SCSI] isci: fix interpretation of \"hard\" reset\n\nA hard reset to isci in the direct-attached case is one where the driver\ninternally manages debouncing the link. In the sas-expander-attached\ncase a hard reset is one that clears affiliations. The driver should\nnot be prematurely dropping affiliations at run time, that decision (to\nforce expander hard resets to ata devices) is left to userspace to\nmanage. So, arrange for I_T_nexus resets to be sas-link-resets in the\nexpander-attached case and isci-hard-resets in the direct-attached case.\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "fca4ecbdc440337b3c257b38c2f4cc8d0ca0286c", "tree": "4b1af31866accd76f16056a842c5a25de182e838", "parents": [ "c132f692085ac624d7c8123df781846c8dcb3166" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Tue Jan 03 23:26:15 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Wed Feb 29 15:11:19 2012 -0600" }, "message": "[SCSI] isci: kill isci_port-\u003estatus\n\nIt only tracks whether the port is stopping in order to gate new devices\nbeing discovered while the port is stopping. However, since the check\nand subsequent handling is unlocked there is nothing to stop the port\nfrom going down immediately after the check.\n\nDriver is already prepared to handle devices arriving on stale ports,\nand those will be cleaned up by an eventual -\u003elldd_dev_gone()\nnotification.\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "c132f692085ac624d7c8123df781846c8dcb3166", "tree": "c9009fcb9112812544f54efc6c134021dffb6545", "parents": [ "cb48d672bfeb4147f3e20d5b16fa2eb6377231e2" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Tue Jan 03 23:26:08 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Wed Feb 29 15:05:05 2012 -0600" }, "message": "[SCSI] isci: kill iphy-\u003eisci_port lookups\n\nThis field is a holdover from the OS abstraction conversion. The stable\nphy to port lookups are done via iphy-\u003eownining_port under scic_lock.\nAfter this conversion to use port-\u003elldd_port the only volatile lookup is\nthe initial lookup in isci_port_formed(). After that point any lookup\nvia a successfully notified domain_device is guaranteed to be valid\nuntil the domain_device is destroyed.\n\nDelete -\u003estart_complete as it is only set once and is set as a\nconsequence of the port going link up, by definition of getting a port\nformed event the port is \"ready\".\n\nWhile we are correcting port lookups also move the asd_sas_port table\nout from under the isci_port. This is to preclude any temptation to use\ncontainer_of() to convert an asd_sas_port to an isci_port, the\nassociation is dynamic and under libsas control.\n\nTested-by: Maciej Trela \u003cmaciej.trela@intel.com\u003e\n[dmilburn@redhat.com: fix i686 compile error]\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "f41a0c441c3fe43e79ebeb75584dbb5bfa83e5cd", "tree": "5a53adb90ebf31888184a9bff16ccc1869e2e4b3", "parents": [ "3a9c5560f677690f65038f399f4f598c79b83186" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Wed Dec 21 21:33:17 2011 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Wed Feb 29 13:01:06 2012 -0600" }, "message": "[SCSI] libsas: fix sas_find_local_phy(), take phy references\n\nIn the direct-attached case this routine returns the phy on which this\ndevice was first discovered. Which is broken if we want to support\nwide-targets, as this phy reference can become stale even though the\nport is still active.\n\nIn the expander-attached case this routine tries to lookup the phy by\nscanning the attached sas addresses of the parent expander, and BUG_ONs\nif it can\u0027t find it. However since eh and the libsas workqueue run\nindependently we can still be attempting device recovery via eh after\nlibsas has recorded the device as detached. This is even easier to hit\nnow that eh is blocked while device domain rediscovery takes place, and\nthat libata is fed more timed out commands increasing the chances that\nit will try to recover the ata device.\n\nArrange for dev-\u003ephy to always point to a last known good phy, it may be\nstale after the port is torn down, but it will catch up for wide port\nreconfigurations, and never be NULL.\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "312d3e56119a4bc5c36a96818f87f650c069ddc2", "tree": "d8cf7586656301ff1c4e5a49f9cbddde61e3e561", "parents": [ "b1124cd3ec97406c767b90bf7e93ecd2d2915592" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu Nov 17 17:59:50 2011 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Sun Feb 19 13:50:12 2012 -0600" }, "message": "[SCSI] libsas: remove ata_port.lock management duties from lldds\n\nEach libsas driver (mvsas, pm8001, and isci) has invented a different\nmethod for managing the ap-\u003elock. The lock is held by the ata\n-\u003equeuecommand() path. mvsas drops it prior to acquiring any internal\nlocks which allows it to hold its internal lock across calls to\ntask-\u003etask_done(). This capability is important as it is the only way\nthe driver can flush task-\u003etask_done() instances to guarantee that it no\nlonger has any in-flight references to a domain_device at\n-\u003elldd_dev_gone() time.\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "b1124cd3ec97406c767b90bf7e93ecd2d2915592", "tree": "d0936775aacff4492177c14a73175738dfb51ee0", "parents": [ "f8daa6e6d83f60a721752cb53433bfdc1503b45f" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Mon Dec 19 16:42:34 2011 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Sun Feb 19 13:48:51 2012 -0600" }, "message": "[SCSI] libsas: introduce sas_drain_work()\n\nWhen an lldd invokes -\u003enotify_port_event() it can trigger a chain of libsas\nevents to:\n\n 1/ form the port and find the direct attached device\n\n 2/ if the attached device is an expander perform domain discovery\n\nA call to flush_workqueue() will only flush the initial port formation work.\nCurrently libsas users need to call scsi_flush_work() up to the max depth of\nchain (which will grow from 2 to 3 when ata discovery is moved to its own\ndiscovery event). Instead of open coding multiple calls switch to use\ndrain_workqueue() to flush sas work.\n\ndrain_workqueue() does not handle new work submitted during the drain so\nlibsas needs a bit of infrastructure to hold off unchained work submissions\nwhile a drain is in flight. A lldd -\u003enotify() event is considered \u0027unchained\u0027\nwhile a sas_discover_event() is \u0027chained\u0027. As Tejun notes:\n\n \"For now, I think it would be best to add private wrapper in libsas to\n support deferring unchained work items while draining.\"\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "6f4e75a49fd07d707995865493b9f452302ae36b", "tree": "05f623463250c0e5f06d2a491b39f4edb93236e8", "parents": [ "95ac7fd189b7e81a200b4d00b2bb6669b31acf3a" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu Nov 17 17:59:46 2011 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Sun Feb 19 13:36:36 2012 -0600" }, "message": "[SCSI] libsas: kill sas_slave_destroy\n\nPer commit 3e4ec344 \"libata: kill ATA_FLAG_DISABLED\" needing to set\nATA_DEV_NONE is a holdover from before libsas converted to the\n\"new-style\" ata-eh.\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "3d2d752549150c2706f6bf8d8a2cceb89ef9f42e", "tree": "b50821eef1f21902c5d22263001e202b65c3d680", "parents": [ "a6fe35c052c4fdd1e8e21251f14eea0bd4bbd25b" ], "author": { "name": "Dave Jiang", "email": "dave.jiang@intel.com", "time": "Fri Feb 10 01:18:34 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Sun Feb 19 09:14:24 2012 -0600" }, "message": "[SCSI] isci: T10 DIF support\n\nThis allows the controller to do WRITE_INSERT and READ_STRIP for SAS\ndisks that support protection information. SAS disks must be formatted\nwith protection information to use this feature via sg_format.\n\n sg3_utils-1.32 -- sg_format version 1.19 20110730\n sg_format usage:\n sg_format --format --verbose --pinfo /dev/sda\n\nAcked-by: Martin K. Petersen \u003cmartin.petersen@oracle.com\u003e\nSigned-off-by: Dave Jiang \u003cdave.jiang@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "e5cc6aa4b6ef34c3f054af8c61a4f73c157589c3", "tree": "64efa1d1dbece4ce058e7cd19b205bde9ef5bdc1", "parents": [ "e3d338a536330b5ffc9f28d7c6a4cdf6ba51867a" ], "author": { "name": "Marcin Tomczak", "email": "marcin.tomczak@intel.com", "time": "Fri Jan 27 11:14:50 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Sun Feb 19 08:09:00 2012 -0600" }, "message": "[SCSI] isci: enable clock gating\n\nEnabling clock gating for power savings on entry to controller ready\nstate. Disable SCU clock gating for power savings on exit from the\ncontroller ready state.\n\nThe gating is fully automated by silicon after setting the mode.\n\nSigned-off-by: Marcin Tomczak \u003cmarcin.tomczak@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "6d7938f46f89c9773f9396c1d13b20bbc5c6d95b", "tree": "aac6b7756ec3952caf5a9c1b96822eaacf3cb77c", "parents": [ "a92fa25c63a788758bd52e9123504d133210c8b7" ], "author": { "name": "Dave Jiang", "email": "dave.jiang@intel.com", "time": "Fri Jan 27 11:17:37 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Sat Feb 18 08:37:00 2012 -0600" }, "message": "[SCSI] isci: Fix NULL ptr dereference when no firmware is being loaded\n\nNULL orom ptr passed in for verification which caused page fault.\nWe will set a default version when we don\u0027t have orom struct.\n\nReported-by: Dan Melnic \u003cdan@seamicro.com\u003e\nSigned-off-by: Dave Jiang \u003cdave.jiang@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "a2888ec985f36f8584609fedc916e5b1fd37aaac", "tree": "84fd8cf700a6f131ecf7c7829a0bb3cb81b5de2f", "parents": [ "ce64b8d85de96a1a9538f6dbc960ef592ca05fe8" ], "author": { "name": "Jesper Juhl", "email": "jj@chaosbits.net", "time": "Sun Feb 05 01:37:56 2012 +0100" }, "committer": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Fri Feb 10 09:52:19 2012 +0100" }, "message": "SCSI, ISCSI: Just #include \"host.h\" once in host.c\n\nThere\u0027s no need to include the header twice.\n\nSigned-off-by: Jesper Juhl \u003cjj@chaosbits.net\u003e\nSigned-off-by: Jiri Kosina \u003cjkosina@suse.cz\u003e\n" }, { "commit": "05b080fc933bb068b32fa119db00e8efcc10e3bd", "tree": "8442420c0733da0a9c1d7b85828e4e297d0163c0", "parents": [ "0953dbea1d9a84c8443b5f5bb45229a6c9d7f9f3" ], "author": { "name": "Marcin Tomczak", "email": "marcin.tomczak@intel.com", "time": "Wed Jan 04 01:33:41 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Jan 16 11:48:02 2012 +0400" }, "message": "[SCSI] isci: fix, prevent port from getting stuck in the \u0027configuring\u0027 state\n\nWhen expander connected in x2 or x4 mode and with IO runnning, if\na cable from wideport is plugged out from the phy, IO\u0027s start failing\non all the targets.\n\nObserved that when cable is pulled with IO running, cominit is\nhappening on all the links and IO\u0027s start dropping to 0 and eventually\nthe whole IO fails. Second observation, target is trying to open and\nSCU is responding with \"Open reject no destination\".\n\nA cause of the problem is when the port went from the \"ready\nconfiguring substate\" back to \"ready configuring substate\" as a result\nof phy being pulled off, scic suspended the port task scheduler\nregister. As a result no IO was allowed and in the \"substate\nconfiguring enter\" routine the IO never goes back to 0. As a result\nthe port never comes out of \"ready substate configuring\".\n\nThe patch adds a mechanism of activate and deactivate phy when a port\nlink up, which fixes the problem.\n\nSigned-off-by: Bartek Nowakowski \u003cbartek.nowakowski@intel.com\u003e\nSigned-off-by: Maciej Trela \u003cmaciej.trela@intel.com\u003e\nSigned-off-by: Marcin Tomczak \u003cmarcin.tomczak@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "0953dbea1d9a84c8443b5f5bb45229a6c9d7f9f3", "tree": "bb167391780955add1754646ec7689be97d781d7", "parents": [ "be778341812dc75b1c515fab6ebd39c0daf1e2bc" ], "author": { "name": "Marcin Tomczak", "email": "marcin.tomczak@intel.com", "time": "Wed Jan 04 01:33:36 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Jan 16 11:47:27 2012 +0400" }, "message": "[SCSI] isci: fix start OOB\n\nSplit scu_link_layer_start_oob function to reset and enable and\nadd flush after all steps.\n\nSigned-off-by: Marcin Tomczak \u003cmarcin.tomczak@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "be778341812dc75b1c515fab6ebd39c0daf1e2bc", "tree": "f690ff71352d7b0447c7e97e3d1eee929115819c", "parents": [ "472d4d2cfbc169f3868a5f63ce727a482a2fd487" ], "author": { "name": "Marcin Tomczak", "email": "marcin.tomczak@intel.com", "time": "Wed Jan 04 01:33:31 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Jan 16 11:46:59 2012 +0400" }, "message": "[SCSI] isci: fix io failures while wide port links are coming up\n\nWhen the first phy of a wide port comes up, don\u0027t report the port ready\nyet, always wait for 250 miliseconds then config the port with all phys\nadded to the port. So that we can avoid reporting wide port device too\nearly to kernel, which caused the first IOs (report luns, inquirys)\nfailed due to not all the phys are configured into its port. Changes\nalso made that the phys in a wide port don\u0027t need to go through half\nsecond wait time for consuming power.\n\nSigned-off-by: Marcin Tomczak \u003cmarcin.tomczak@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "472d4d2cfbc169f3868a5f63ce727a482a2fd487", "tree": "b9de4784cd8bcd28917e689f046468d4a7fb40bb", "parents": [ "7e629841b8f8f7bc84d74de994d748b0a3282225" ], "author": { "name": "Marcin Tomczak", "email": "marcin.tomczak@intel.com", "time": "Wed Jan 04 01:33:26 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Jan 16 11:46:18 2012 +0400" }, "message": "[SCSI] isci: allow more time for wide port targets\n\nWhen hot insert the wide port device through the mini-sas port,\nthe first IOs (Report Luns or Inquiry) may fail due to the device\ntrying to open to a SCU phy that is still in suspended state. This\nIO failure causes the wide port device stuck in UPDATING_PORT_WIDTH\nstate.\n\nSigned-off-by: Marcin Tomczak \u003cmarcin.tomczak@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "7e629841b8f8f7bc84d74de994d748b0a3282225", "tree": "00c823b35c8c72f3b151c3eabef95255e59b4bc0", "parents": [ "d4ec1cf61fb081a9dde0c0e0b2d0201f4005f937" ], "author": { "name": "Bartek Nowakowski", "email": "bartek.nowakowski@intel.com", "time": "Wed Jan 04 01:33:20 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Jan 16 11:45:43 2012 +0400" }, "message": "[SCSI] isci: enable wide port targets\n\nArrange for task_contexts prepared for the wide targets to account for\nall the attached phys in the port.\n\nSigned-off-by: Bartek Nowakowski \u003cbartek.nowakowski@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "d4ec1cf61fb081a9dde0c0e0b2d0201f4005f937", "tree": "b93baa50731ef128b13a01b0319f9b0bc7e76112", "parents": [ "d3fd2e2bc29b1f0979352994f9035359003461c3" ], "author": { "name": "Marcin Tomczak", "email": "marcin.tomczak@intel.com", "time": "Wed Jan 04 01:33:15 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Jan 16 11:45:19 2012 +0400" }, "message": "[SCSI] isci: Fix IO fails when pull cable from phy in x4 wideport in MPC mode.\n\nFailure seen pulling a cable from a x4 port configured in manual port\nconfiguration (MPC) mode (MPC mode is set by the the OEM paramaters\nprovided by the platform or isci_firmware.bin). While IO running to\ndevices behind and expander, plugging out the cable from phy is causing\nIO failures and IO drops on disks and never recover.\n\nIt happens because during link up/down the phy were being taken out of\nthe port.\n\nFix: during link down the phy is kept in the same logical port.\n\nSigned-off-by: Marcin Tomczak \u003cmarcin.tomczak@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "d3fd2e2bc29b1f0979352994f9035359003461c3", "tree": "a991571aace4f23eb1c69fda48bea49edaadfa90", "parents": [ "27234ab44f97d85bab062a9d18aaff99addd267d" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Wed Jan 04 01:33:10 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Jan 16 11:44:54 2012 +0400" }, "message": "[SCSI] isci: update version to 1.1\n\nBump the version now that the driver has atapi support and the initial\nround of hotplug fixes. The EXPERIMENTAL tag should have been removed a\nwhile back. While we\u0027re here also kill the \"select SCSI_SAS_HOST_SMP\"\nas the build error was separately fixed by commit d962480e \"[SCSI]\nlibsas: fix try_test_sas_gpio_gp_bit() build error\".\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "27234ab44f97d85bab062a9d18aaff99addd267d", "tree": "a54a7da9aa4bb6ddf2da02163cd9de5975240156", "parents": [ "be168a3b89b43b8bb4ef71066e91439d400e20a6" ], "author": { "name": "Maciej Trela", "email": "maciej.trela@intel.com", "time": "Wed Jan 04 01:33:05 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Jan 16 11:44:15 2012 +0400" }, "message": "[SCSI] isci: remove unused \u0027isci_tmf-\u003edevice\u0027 field\n\nAs the field was never set, isci_print_tmf() using \u0027isci_tmf-\u003edevice\u0027\nsometimes causes a kernel crash if the dev_dbg() statement is enabled.\nRemove the unused field both from isci_tmf struct definition and from\nisci_print_tmf()\n\nSigned-off-by: Maciej Trela \u003cmaciej.trela@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "be168a3b89b43b8bb4ef71066e91439d400e20a6", "tree": "4d0d9e0ffda9aa5d2368c2a4f856bb4f949b609a", "parents": [ "6024d38b86ecc7943e398ef01b0dc2765870d444" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Wed Jan 04 01:33:00 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Jan 16 11:43:59 2012 +0400" }, "message": "[SCSI] isci: link speeds default to gen 2\n\nGen-3 operation is marginal, default to gen-2 for now.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "6024d38b86ecc7943e398ef01b0dc2765870d444", "tree": "438704c82c54d11b790e68ad2415bad7f700d2a2", "parents": [ "9fee607f0b29adabd72265a847b8e421dff10d66" ], "author": { "name": "Marcin Tomczak", "email": "marcin.tomczak@intel.com", "time": "Wed Jan 04 01:32:54 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Jan 16 11:43:39 2012 +0400" }, "message": "[SCSI] isci: performance-fix, shorten default \"no outbound task\" timeout\n\n\"No task timeout timer reduced from 20 to 2 This timer controls how\nlong the SCU hardware will hold open the TX side of the connection\nbefore sending a DONE. The timer allows the hardware to attempt to\noptimize the DONE/CLOSE behavior to allow for new COMMAND IU to be\nposted. In practice closing the connection quicker is better.\"\n\nSigned-off-by: Marcin Tomczak \u003cmarcin.tomczak@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "9fee607f0b29adabd72265a847b8e421dff10d66", "tree": "7442c82b8e363a7aa1e87955e9619cf8e949cf3a", "parents": [ "594e566ae5985e0cc3185ac21509a86e90aad577" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Wed Jan 04 01:32:49 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Jan 16 11:43:04 2012 +0400" }, "message": "[SCSI] isci: oem parameter format v1.3 (cable select)\n\nv1.3 allows the attenuation of the attached cables to be specified to\nthe driver in terms of \u0027short\u0027, \u0027medium\u0027, and \u0027long\u0027 (see probe_roms.h).\nThese settings (per phy) are retrieved from the platform oem-parameters\n(BIOS rom) or via a module parameter override.\n\nReviewed-by: Jiangbi Liu \u003cjiangbi.liu@intel.com\u003e\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "594e566ae5985e0cc3185ac21509a86e90aad577", "tree": "f53db9f48084fea7c553077e60e03e7cd130c21a", "parents": [ "afd13a1f2b05157c7621d87dfe89ea6ea9061bd8" ], "author": { "name": "Dave Jiang", "email": "dave.jiang@intel.com", "time": "Wed Jan 04 01:32:44 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Jan 16 11:42:00 2012 +0400" }, "message": "[SCSI] isci: oem parameter format v1.1 (ssc select)\n\nv1.1 allows finer grained tuning of the SSC (spread-spectrum-clocking)\nsettings for SAS and SATA. See notes in probe_roms.h\n\nSigned-off-by: Dave Jiang \u003cdave.jiang@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "afd13a1f2b05157c7621d87dfe89ea6ea9061bd8", "tree": "1dc7a3c2816de4598ada6871f1381336bf1b4192", "parents": [ "2e5da889d44a3a9629f895de3488306e7f5ddf16" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Wed Jan 04 01:32:39 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Jan 16 11:40:29 2012 +0400" }, "message": "[SCSI] isci: update afe (analog-front-end) recipe for C1\n\nC1 silicon requires updates to the phy tuning recipe and also support\nfor user provided cable selects (per-phy) for short, medium, and long\ncables. Default to \u0027short\u0027 awaiting support for selecting the cable via\noem parameters.\n\nReviewed-by: Jiangbi Liu \u003cjiangbi.liu@intel.com\u003e\nSigned-off-by: Marcin Tomczak \u003cmarcin.tomczak@intel.com\u003e\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "2e5da889d44a3a9629f895de3488306e7f5ddf16", "tree": "e7af1c412c85dcdb28023fca51f6089ff31474b5", "parents": [ "7d99b3abaf8412932269a2a2e561138326d61f8b" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Wed Jan 04 01:32:34 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Jan 16 11:39:52 2012 +0400" }, "message": "[SCSI] isci: cleanup oem parameter and recipe handling\n\nBefore updating the code to support the latest platform updates and\nsilicon revision cleanup some of the long deref chains.\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "7d99b3abaf8412932269a2a2e561138326d61f8b", "tree": "85e6fd2e57655463de5c0e1546ca8b8526d6923a", "parents": [ "798edaadba97612224c250d50f880405d75a3af1" ], "author": { "name": "Ben Hutchings", "email": "ben@decadent.org.uk", "time": "Wed Jan 04 01:32:28 2012 -0800" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Jan 16 11:34:37 2012 +0400" }, "message": "[SCSI] isci, firmware: Remove isci fallback parameter blob and generator\n\nThis parameter blob and generator program have been moved to the\nlinux-firmware.git repository.\n\nSigned-off-by: Ben Hutchings \u003cben@decadent.org.uk\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "7000f7c71e2457391e3249eac1ae53c91f49a8c0", "tree": "bf2567c0d927c691932a701b45bcf17a9a7cfdcc", "parents": [ "52d74634335dfc0984ed955ed3c6ad6488495f96" ], "author": { "name": "Andrzej Jakowski", "email": "andrzej.jakowski@intel.com", "time": "Thu Oct 27 15:05:42 2011 -0700" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Oct 31 13:23:18 2011 +0400" }, "message": "[SCSI] isci: overriding max_concurr_spinup oem parameter by max(oem, user)\n\nFixes bug where max_concurr_spinup oem parameter should be\noverriden by max_concurr_spinup user parameter. Override should\nhappen only when max_concurr_spinup user parameter is specified\nin command line (greater than 0). Also this fix shortens variables\nrepresenting max_conxurr_spinup for oem and user parameters.\n\nSigned-off-by: Andrzej Jakowski \u003candrzej.jakowski@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "52d74634335dfc0984ed955ed3c6ad6488495f96", "tree": "647120b0916abcc42f1e2d050f13175993509776", "parents": [ "8e35a1398c5db981cd1a2d7635de9c15dd648527" ], "author": { "name": "Dan Williams", "email": "dan.j.williams@intel.com", "time": "Thu Oct 27 15:05:37 2011 -0700" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Oct 31 13:23:01 2011 +0400" }, "message": "[SCSI] isci: revert bcn filtering\n\nThe initial bcn filtering implementation was validated on a kernel\nbaseline that predated the switch to new libata error handling. Also,\nprior to that conversion we borrowed the mvsas MVS_DEV_EH approach to\nprevent the unwanted extra ap-\u003eops-\u003ephy_reset(ap) that occurred in the\nata_bus_probe() path.\n\nAfter the conversion to new libata eh resets at discovery are more\nfrequent and get filtered prematurely by IDEV_EH. The result is that\nour bcn filtering has been blocked from running and at discovery and it\nappears to stall discovery completion to the point of triggering hung\ntask timeouts. So, revert the implementation for now. When it returns\nit will go into libsas proper.\n\nThe domain rediscovery that takes place due to -\u003elldd_I_T_nexus_reset()\nevents should now be properly waited for by the ata_port_wait_eh() call\nin ata_port_probe(). So the hard coded delay in the isci\n-\u003elldd_I_T_nexus_reset() and other libsas drivers should help debounce\nthe libsas thread from seeing temporary device removals.\n\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "8e35a1398c5db981cd1a2d7635de9c15dd648527", "tree": "9a7de3cae5c5a7194a1a447dc871062f351b3363", "parents": [ "5412e25c55fc0b08041a451d8bee6f2b291099c2" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Oct 27 15:05:32 2011 -0700" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Oct 31 13:22:41 2011 +0400" }, "message": "[SCSI] isci: Fix hard reset timeout conditions.\n\nA hard reset can timeout before or after the last phy in the\nport goes away. If after, then notify the OS that the last\nphy has failed.\n\nThe recovery for the failed hard reset has been removed.\nThis recovery code was unecessary in that the link would\nrecover from the failure normally by a new link reset sequence\nor hotplug of the remote device.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "5412e25c55fc0b08041a451d8bee6f2b291099c2", "tree": "2dbaa34c32c376f4a683338fceba0a78b5c3d00c", "parents": [ "3b34c169f8197e02529fa3ec703703c2ce418c57" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Oct 27 15:05:27 2011 -0700" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Oct 31 13:20:28 2011 +0400" }, "message": "[SCSI] isci: No need to manage the pending reset bit on pending requests.\n\nThe lldd does not need to look at or manage the pending device\nreset bit in pending sas_tasks.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "3b34c169f8197e02529fa3ec703703c2ce418c57", "tree": "4834c97238ea0b7968de0bd9b7efd8c780175eda", "parents": [ "98145cb722b51ccc3ba27166c9803545accba950" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Oct 27 15:05:22 2011 -0700" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Oct 31 13:19:47 2011 +0400" }, "message": "[SCSI] isci: Remove redundant isci_request.ttype field.\n\nUse the existing IREQ_TMF flag as a request type indicator.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "98145cb722b51ccc3ba27166c9803545accba950", "tree": "714aaae27bd45da93793729f3321c6409aad45ae", "parents": [ "db49c2d037d50dfc67b29a4e013d6250ca97c3aa" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Oct 27 15:05:16 2011 -0700" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Oct 31 13:17:48 2011 +0400" }, "message": "[SCSI] isci: Fix task management for SMP, SATA and on dev remove.\n\nlibsas uses the LLDD abort task interface to handle I/O timeouts\nin the SATA/STP and SMP discovery paths, so this change will terminate\nSTP/SMP requests. Also, if the device is gone, the lldd will prevent\nlibsas from further escalations in the error handler.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "db49c2d037d50dfc67b29a4e013d6250ca97c3aa", "tree": "c95a2d7dcb7976a44ed798b3f5b3fd49450df095", "parents": [ "b343dff1a269bcc0eac123ef541c5476b03d52c1" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Oct 27 15:05:11 2011 -0700" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Oct 31 13:17:04 2011 +0400" }, "message": "[SCSI] isci: No task_done callbacks in error handler paths.\n\nlibsas will cleanup pending sas_tasks after error handler\npath functions are called; do not call task_done callbacks.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "b343dff1a269bcc0eac123ef541c5476b03d52c1", "tree": "d272a97f43f44d19908bb7c5ca997a82be2e0028", "parents": [ "d6891682220c18c01bf6838f30e37342c38fde44" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Oct 27 15:05:06 2011 -0700" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Oct 31 13:16:23 2011 +0400" }, "message": "[SCSI] isci: Handle task request timeouts correctly.\n\nIn the case where \"task\" requests timeout (note that this class of\nrequests can also include SATA/STP soft reset FIS transmissions),\nhandle the case where the task was being managed by some call to\nterminate the task request by completing both the tmf and the aborting\nprocess.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "d6891682220c18c01bf6838f30e37342c38fde44", "tree": "ad0c61215cbc9de6c82e95825923f95ebc2532a1", "parents": [ "c2cb8a5fd7d5d8729a4fc25937c4d6564f9a7aa3" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Oct 27 15:05:01 2011 -0700" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Oct 31 13:16:04 2011 +0400" }, "message": "[SCSI] isci: Fix tag leak in tasks and terminated requests.\n\nMake sure terminated requests and completed task tags are freed.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "c2cb8a5fd7d5d8729a4fc25937c4d6564f9a7aa3", "tree": "b562e8cea25801037a4b22a79630bfbc5bd667a3", "parents": [ "0e2e27990e2dcd415f7974e8460a2f05accdddfb" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Oct 27 15:04:56 2011 -0700" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Oct 31 13:15:17 2011 +0400" }, "message": "[SCSI] isci: Immediately fail I/O to removed devices.\n\nIn the case where an I/O fails to start in isci_request_execute,\nonly allow retries if the device is not already gone.\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" }, { "commit": "0e2e27990e2dcd415f7974e8460a2f05accdddfb", "tree": "351eb1905eb0e2e118dac9a9d27951225ba2c492", "parents": [ "5a918353ec97bbce2af83a950eb38e2781bfe9e7" ], "author": { "name": "Jeff Skirvin", "email": "jeffrey.d.skirvin@intel.com", "time": "Thu Oct 27 15:04:50 2011 -0700" }, "committer": { "name": "James Bottomley", "email": "JBottomley@Parallels.com", "time": "Mon Oct 31 13:14:44 2011 +0400" }, "message": "[SCSI] isci: Lookup device references through requests in completions.\n\nThe LLDD needs to obtain a reference to the device through the request\nitself and not through the domain_device, because the\ndomain_device.lldd_dev is set to NULL early in the lldd_dev_gone call.\nThis relies on the fact that the isci_remote_device object is keeping a\nseperate reference count of outstanding requests. TODO: unify the\nrequest count tracking with the isci_remote_device kref.\n\nThe failure signature of this condition looks like the following\nlog, where the important bits are the call to lldd_dev_gone followed\nby a crash in isci_terminate_request_core:\n\n[ 229.151541] isci 0000:0b:00.0: isci_remote_device_gone: domain_device \u003d ffff8801492d4800, isci_device \u003d ffff880143c657d0, isci_port \u003d ffff880143c63658\n[ 229.166007] isci 0000:0b:00.0: isci_remote_device_stop: isci_device \u003d ffff880143c657d0\n[ 229.175317] isci 0000:0b:00.0: isci_terminate_pending_requests: idev\u003dffff880143c657d0 request\u003dffff88014741f000; task\u003dffff8801470f46c0 old_state\u003d2\n[ 229.189702] isci 0000:0b:00.0: isci_terminate_request_core: device \u003d ffff880143c657d0; request \u003d ffff88014741f000\n[ 229.201339] isci 0000:0b:00.0: isci_terminate_request_core: before completion wait (ffff88014741f000/ffff880149715ad0)\n[ 229.213414] isci 0000:0b:00.0: sci_controller_process_completions: completion queue entry:0x8000a0e9\n[ 229.214401] BUG: unable to handle kernel NULL pointer dereference at 0000000000000228\n[ 229.214401] IP:jdskirvi-testlbo [\u003cffffffffa00a58be\u003e] sci_request_completed_state_enter+0x50/0xafb [isci]\n[ 229.214401] PGD 13d19e067 PUD 13d104067 PMD 0\n[ 229.214401] Oops: 0000 [#1] SMP\n[ 229.214401] CPU 0 x kernel: [ 226\n[ 229.214401] Modules linked in: ipv6 dm_multipath uinput nouveau snd_hda_codec_realtek snd_hda_intel ttm drm_kms_helper drm snd_hda_codec snd_hwdep snd_pcm snd_timer i2c_algo_bit isci snd libsas ioatdma mxm_wmi iTCO_wdt soundcore snd_page_alloc scsi_transport_sas iTCO_vendor_support wmi dca video i2c_i801 i2c_core [last unloaded: speedstep_lib]\n[ 229.214401]\n[ 229.214401] Pid: 5, comm: kworker/u:0 Not tainted 3.0.0-isci-11.7.29+ #30.353196] Buffer Intel Corporation Stoakley/Pearlcity Workstation\n[ 229.214401] RIP: 0010:[\u003cffffffffa00a58be\u003e] I/O error on dev [\u003cffffffffa00a58be\u003e] sci_request_completed_state_enter+0x50/0xafb [isci]\n[ 229.214401] RSP: 0018:ffff88014fc03d20 EFLAGS: 00010046\n[ 229.214401] RAX: 0000000000000000 RBX: ffff88014741f000 RCX: 0000000000000000\n[ 229.214401] RDX: ffffffffa00b2c90 RSI: 0000000000000017 RDI: ffff88014741f0a0\n[ 229.214401] RBP: ffff88014fc03d90 R08: 0000000000000018 R09: 0000000000000000\n[ 229.214401] R10: 0000000000000000 R11: ffffffff81a17d98 R12: 000000000000001d\n[ 229.214401] R13: ffff8801470f46c0 R14: 0000000000000000 R15: 0000000000008000\n[ 229.214401] FS: 0000000000000000(0000) GS:ffff88014fc00000(0000) knlGS:0000000000000000\n[ 229.214401] CS: 0010 DS: 0000 ES: 0000 CR0: 000000008005003b\n[ 229.214401] CR2: 0000000000000228 CR3: 000000013ceaa000 CR4: 00000000000406f0\n[ 229.214401] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000\n[ 229.214401] DR3: 0000000000000000 DR6: 00000000ffff0ff0 DR7: 0000000000000400\n[ 229.214401] Process kworker/u:0 (pid: 5, threadinfo ffff880149714000, task ffff880149718000)\n[ 229.214401] Call Trace:\n[ 229.214401] \u003cIRQ\u003e\n[ 229.214401] [\u003cffffffffa00aa6ce\u003e] sci_change_state+0x4a/0x4f [isci]\n[ 229.214401] [\u003cffffffffa00a4ca6\u003e] sci_io_request_tc_completion+0x79c/0x7a0 [isci]\n[ 229.214401] [\u003cffffffffa00acf35\u003e] sci_controller_process_completions+0x14f/0x396 [isci]\n[ 229.214401] [\u003cffffffffa00abbda\u003e] ? spin_lock_irq+0xe/0x10 [isci]\n[ 229.214401] [\u003cffffffffa00ad2cf\u003e] isci_host_completion_routine+0x71/0x2be [isci]\n[ 229.214401] [\u003cffffffff8107c6b3\u003e] ? mark_held_locks+0x52/0x70\n[ 229.214401] [\u003cffffffff810538e8\u003e] tasklet_action+0x90/0xf1\n[ 229.214401] [\u003cffffffff81054050\u003e] __do_softirq+0xe5/0x1bf\n[ 229.214401] [\u003cffffffff8106d9d1\u003e] ? hrtimer_interrupt+0x129/0x1bb\n[ 229.214401] [\u003cffffffff814ff69c\u003e] call_softirq+0x1c/0x30\n[ 229.214401] [\u003cffffffff8100bb67\u003e] do_softirq+0x4b/0xa3\n[ 229.214401] [\u003cffffffff81053d84\u003e] irq_exit+0x53/0xb4\n[ 229.214401] [\u003cffffffff814fffe7\u003e] smp_apic_timer_interrupt+0x83/0x91\n[ 229.214401] [\u003cffffffff814fee53\u003e] apic_timer_interrupt+0x13/0x20\n[ 229.214401] \u003cEOI\u003e\n[ 229.214401] [\u003cffffffff814f7ad4\u003e] ? retint_restore_args+0x13/0x13\n[ 229.214401] [\u003cffffffff8107af29\u003e] ? trace_hardirqs_off+0xd/0xf\n[ 229.214401] [\u003cffffffff8104ea71\u003e] ? vprintk+0x40b/0x452\n[ 229.214401] [\u003cffffffff814f4b5a\u003e] printk+0x41/0x47\n[ 229.214401] [\u003cffffffff81314484\u003e] __dev_printk+0x78/0x7a\n[ 229.214401] [\u003cffffffff8131471e\u003e] dev_printk+0x45/0x47\n[ 229.214401] [\u003cffffffffa00ae2a3\u003e] isci_terminate_request_core+0x15d/0x317 [isci]\n[ 229.214401] [\u003cffffffffa00af1ad\u003e] isci_terminate_pending_requests+0x1a4/0x204 [isci]\n[ 229.214401] [\u003cffffffffa00229f6\u003e] ? sas_phye_oob_error+0xc3/0xc3 [libsas]\n[ 229.214401] [\u003cffffffffa00a7d9e\u003e] isci_remote_device_nuke_requests+0xa6/0xff [isci]\n[ 229.214401] [\u003cffffffffa00a811a\u003e] isci_remote_device_stop+0x7c/0x166 [isci]\n[ 229.214401] [\u003cffffffffa00229f6\u003e] ? sas_phye_oob_error+0xc3/0xc3 [libsas]\n[ 229.214401] [\u003cffffffffa00a827a\u003e] isci_remote_device_gone+0x76/0x7e [isci]\n[ 229.214401] [\u003cffffffffa002363e\u003e] sas_notify_lldd_dev_gone+0x34/0x36 [libsas]\n[ 229.214401] [\u003cffffffffa0023945\u003e] sas_unregister_dev+0x57/0x9c [libsas]\n[ 229.214401] [\u003cffffffffa00239c0\u003e] sas_unregister_domain_devices+0x36/0x65 [libsas]\n[ 229.214401] [\u003cffffffffa0022cb8\u003e] sas_deform_port+0x72/0x1ac [libsas]\n[ 229.214401] [\u003cffffffffa00229f6\u003e] ? sas_phye_oob_error+0xc3/0xc3 [libsas]\n[ 229.214401] [\u003cffffffffa0022a34\u003e] sas_phye_loss_of_signal+0x3e/0x42 [libsas]\n\nSigned-off-by: Jeff Skirvin \u003cjeffrey.d.skirvin@intel.com\u003e\nSigned-off-by: Dan Williams \u003cdan.j.williams@intel.com\u003e\nSigned-off-by: James Bottomley \u003cJBottomley@Parallels.com\u003e\n" } ], "next": "ec7ae517537ae5c7b0b2cd7f562dfa3e7a05b954" }