)]}' { "log": [ { "commit": "ea3c64506ea7965f86f030155e6fdef381de10e2", "tree": "e98d1fbe34fd0c1c81d0fb08b3314f6f059eb494", "parents": [ "e2f3d75fc0e4a0d03c61872bad39ffa2e74a04ff" ], "author": { "name": "Gwendal Grignou", "email": "gwendal@google.com", "time": "Tue Aug 31 16:20:36 2010 -0700" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Sep 09 22:31:55 2010 -0400" }, "message": "libata-sff: Reenable Port Multiplier after libata-sff remodeling.\n\nKeep track of the link on the which the current request is in progress.\nIt allows support of links behind port multiplier.\n\nNot all libata-sff is PMP compliant. Code for native BMDMA controller\ndoes not take in accound PMP.\n\nTested on Marvell 7042 and Sil7526.\n\nSigned-off-by: Gwendal Grignou \u003cgwendal@google.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "e2f3d75fc0e4a0d03c61872bad39ffa2e74a04ff", "tree": "b98d70dadfe9143453494bd145a53c06910df4f7", "parents": [ "992b3fb9b5391bc4de5b42bb810dc6dd583a6c4a" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue Sep 07 14:05:31 2010 +0200" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Sep 09 22:27:59 2010 -0400" }, "message": "libata: skip EH autopsy and recovery during suspend\n\nFor some mysterious reason, certain hardware reacts badly to usual EH\nactions while the system is going for suspend. As the devices won\u0027t\nbe needed until the system is resumed, ask EH to skip usual autopsy\nand recovery and proceed directly to suspend.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nTested-by: Stephan Diestelhorst \u003cstephan.diestelhorst@amd.com\u003e\nCc: stable@kernel.org\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "ad72cf9885c536e3adae03f8337557ac9dd1e4bb", "tree": "e93af7f241987ffe365792c0130d182b0ac890d1", "parents": [ "d313dd85ad846bc768d58e9ceb28588f917f4c9a" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Fri Jul 02 10:03:52 2010 +0200" }, "committer": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Fri Jul 02 10:59:24 2010 +0200" }, "message": "libata: take advantage of cmwq and remove concurrency limitations\n\nlibata has two concurrency related limitations.\n\na. ata_wq which is used for polling PIO has single thread per CPU. If\n there are multiple devices doing polling PIO on the same CPU, they\n can\u0027t be executed simultaneously.\n\nb. ata_aux_wq which is used for SCSI probing has single thread. In\n cases where SCSI probing is stalled for extended period of time\n which is possible for ATAPI devices, this will stall all probing.\n\n#a is solved by increasing maximum concurrency of ata_wq. Please note\nthat polling PIO might be used under allocation path and thus needs to\nbe served by a separate wq with a rescuer.\n\n#b is solved by using the default wq instead and achieving exclusion\nvia per-port mutex.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nAcked-by: Jeff Garzik \u003cjgarzik@pobox.com\u003e\n" }, { "commit": "d8d9129ea28e2177749627c82962feb26e8d11e9", "tree": "f930703c8e3ba66cadcaf712ee7ce9b36a6d0823", "parents": [ "68939ce5fc17ee9c03ef6e543d4f82bd9f5583d4" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Sat May 15 20:09:34 2010 +0200" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Wed Jun 02 13:50:10 2010 -0400" }, "message": "libata: implement on-demand HPA unlocking\n\nImplement ata_scsi_unlock_native_capacity() which will be called\nthrough SCSI layer when block layer notices that partitions on a\ndevice extend beyond the end of the device. It requests EH to unlock\nHPA, waits for completion and returns the current device capacity.\n\nThis allows libata to unlock HPA on demand instead of having to decide\nwhether to unlock upfront. Unlocking on demand is safer than\nunlocking by upfront because some BIOSes write private data to the\narea beyond HPA limit. This was suggested by Ben Hutchings.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSuggested-by: Ben Hutchings \u003cben@decadent.org.uk\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "43c9c59185eec7caaff6e9dd8d4c93a4d9836a86", "tree": "6c627f018b6d83c448d059d61fff0fafd8c55b86", "parents": [ "e7ecd435692ca9bde9d124be30b3a26e672ea6c2" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Sun May 23 12:59:11 2010 +0200" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue May 25 19:41:19 2010 -0400" }, "message": "libata: implement dump_id force param\n\nAdd dump_id libata.force parameter. If specified, libata dumps full\nIDENTIFY data during device configuration. This is to aid debugging.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Larry Baker \u003cbaker@usgs.gov\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "9a7780c9acb821fe1c2b6fc53f74cc2556ff5364", "tree": "9590f79a1a0631120322ecfa8a86f610859d7aec", "parents": [ "d6b0de8c28ef32d081cfee938e443ce2a2429cfe" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed May 19 22:10:24 2010 +0200" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue May 25 19:41:12 2010 -0400" }, "message": "libata-sff: make BMDMA optional\n\nMake BMDMA optional depending on new config variable CONFIG_ATA_BMDMA.\nIn Kconfig, drivers are grouped into five groups - non-SFF native, SFF\nw/ custom DMA interface, SFF w/ BMDMA, PIO-only SFF, and generic\nfallback / legacy ones. Kconfig and Makefile are reorganized\naccording to the groups and ordered alphabetically inside each group.\n\nata_ioports.bmdma_addr and ata_port.bmdma_prd[_dma] are put into\nCONFIG_ATA_BMDMA, as are all bmdma related ops, variables and\nfunctions.\n\nThis increase the binary size slightly when BMDMA is enabled but on\nboth native-only and PIO-only configurations the size is slightly\nreduced. Either way, the size difference is insignificant. This\nchange is more meaningful to signify the separation between SFF and\nBMDMA and as a tool to verify the separation.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "1c5afdf7a629d2e77de8dd043b97a33dcd7e6dfa", "tree": "9f29bfdd32dd1bf497167cd7a8f596c7680059dd", "parents": [ "c3b2889424c26f3b42962b6f39aabb4f1fd1b576" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed May 19 22:10:22 2010 +0200" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue May 25 19:40:30 2010 -0400" }, "message": "libata-sff: separate out BMDMA init\n\nSeparate out ata_pci_bmdma_prepare_host() and ata_pci_bmdma_init_one()\nfrom their SFF counterparts. SFF ones no longer try to initialize\nBMDMA or set PCI master.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "c3b2889424c26f3b42962b6f39aabb4f1fd1b576", "tree": "c131d260c18815cb84331357db75c17dabf083d8", "parents": [ "37f65b8bc262a5ae4c8e58be92fe3032f0aaaf04" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed May 19 22:10:21 2010 +0200" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue May 25 19:40:24 2010 -0400" }, "message": "libata-sff: separate out BMDMA irq handler\n\nSeparate out BMDMA irq handler from SFF irq handler. The misnamed\nhost_intr() functions are renamed to ata_sff_port_intr() and\nata_bmdma_port_intr(). Common parts are factored into\n__ata_sff_port_intr() and __ata_sff_interrupt() and used by sff and\nbmdma interrupt routines.\n\nAll BMDMA drivers now use ata_bmdma_interrupt() or\nata_bmdma_port_intr() while all non-BMDMA SFF ones use\nata_sff_interrupt() or ata_sff_port_intr().\n\nFor now, ata_pci_sff_init_one() uses ata_bmdma_interrupt() as it\u0027s\nused by both SFF and BMDMA drivers.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "37f65b8bc262a5ae4c8e58be92fe3032f0aaaf04", "tree": "0a7507b26bdc61024d01b538ad7d3d25bc4f7811", "parents": [ "2f20ccf8190b064468cb2c7ba92e757ae37e61b1" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed May 19 22:10:20 2010 +0200" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue May 25 19:40:19 2010 -0400" }, "message": "libata-sff: ata_sff_irq_clear() is BMDMA specific\n\nata_sff_irq_clear() is BMDMA specific. Rename it to\nata_bmdma_irq_clear(), move it to ata_bmdma_port_ops and make\n-\u003esff_irq_clear() optional.\n\nNote: ata_bmdma_irq_clear() is actually only needed by ata_piix and\n possibly by sata_sil. This should be moved to respective low\n level drivers later.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "360ff7833098e944e5003618b03894251e937802", "tree": "aaa8c8f771cf35b2a0dab58ea6bad35d88f4ac8a", "parents": [ "f60d70113fa04e32aee2dedbf304a48469c9c45c" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon May 10 21:41:42 2010 +0200" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Wed May 19 13:38:55 2010 -0400" }, "message": "libata-sff: separate out BMDMA qc_issue\n\nSeparate out ata_bmdma_qc_issue() from ata_sff_qc_issue() such that\nata_sff_qc_issue() only deals with non-BMDMA SFF protocols (PIO and\nnodata) while ata_bmdma_qc_issue() deals with the BMDMA protocols and\nuses ata_sff_qc_issue() for non-DMA commands. All the users are\nupdated accordingly.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "f60d70113fa04e32aee2dedbf304a48469c9c45c", "tree": "6e74f483773dfae0adda0faa8754b5ce6129d8f7", "parents": [ "f47451c45fe0032ef491aaf3e0623fa0154e156d" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon May 10 21:41:41 2010 +0200" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Wed May 19 13:38:54 2010 -0400" }, "message": "libata-sff: prd is BMDMA specific\n\nstruct ata_prd and ap-\u003eprd are BMDMA specific. Add bmdma_ prefix to\nthem and move them inside CONFIG_ATA_SFF.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "f47451c45fe0032ef491aaf3e0623fa0154e156d", "tree": "3e2901991480836be75f75288c2220ef086fe694", "parents": [ "fe06e5f9b7c61dc567edace3f4909672067f7d7e" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon May 10 21:41:40 2010 +0200" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Wed May 19 13:36:50 2010 -0400" }, "message": "libata-sff: ata_sff_[dumb_]qc_prep are BMDMA specific\n\nBoth qc_prep functions deal only with BMDMA PRD setup and PIO only SFF\ndrivers don\u0027t need them. Rename to ata_bmdma_[dumb_]qc_prep() and\nrelocate.\n\nAll usages are renamed except for pdc_adma and sata_qstor. Those two\ndrivers are not BMDMA drivers and don\u0027t need to call BMDMA qc_prep\nfunctions. Calls to ata_sff_qc_prep() in the two drivers are removed.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "fe06e5f9b7c61dc567edace3f4909672067f7d7e", "tree": "b2242169e8e3b32c63925ed9901fff9d49c26192", "parents": [ "c429137a67b82788d24682153bb9c96501a9ef34" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon May 10 21:41:39 2010 +0200" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Wed May 19 13:36:46 2010 -0400" }, "message": "libata-sff: separate out BMDMA EH\n\nSome of error handling logic in ata_sff_error_handler() and all of\nata_sff_post_internal_cmd() are for BMDMA. Create\nata_bmdma_error_handler() and ata_bmdma_post_internal_cmd() and move\nBMDMA part into those.\n\nWhile at it, change DMA protocol check to ata_is_dma(), fix\npost_internal_cmd to call ap-\u003eops-\u003ebmdma_stop instead of directly\ncalling ata_bmdma_stop() and open code hardreset selection so that\nata_std_error_handler() doesn\u0027t have to know about sff hardreset.\n\nAs these two functions are BMDMA specific, there\u0027s no reason to check\nfor bmdma_addr before calling bmdma methods if the protocol of the\nfailed command is DMA. sata_mv and pata_mpc52xx now don\u0027t need to set\n.post_internal_cmd to ATA_OP_NULL and pata_icside and sata_qstor don\u0027t\nneed to set it to their bmdma_stop routines.\n\nata_sff_post_internal_cmd() becomes noop and is removed.\n\nThis fixes p3 described in clean-up-BMDMA-initialization patch.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "c429137a67b82788d24682153bb9c96501a9ef34", "tree": "b24ed75908f08cc9cddb19efc74fcb46be75de8f", "parents": [ "5fe7454aa9c6ef5fcf506b0f2dfc20f696891f1a" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon May 10 21:41:38 2010 +0200" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Wed May 19 13:35:49 2010 -0400" }, "message": "libata-sff: port_task is SFF specific\n\nport_task is tightly bound to the standard SFF PIO HSM implementation.\nUsing it for any other purpose would be error-prone and there\u0027s no\nsuch user and if some drivers need such feature, it would be much\nbetter off using its own. Move it inside CONFIG_ATA_SFF and rename it\nto sff_pio_task.\n\nThe only function which is exposed to the core layer is\nata_sff_flush_pio_task() which is renamed from ata_port_flush_task()\nand now also takes care of resetting hsm_task_state to HSM_ST_IDLE,\nwhich is possible as it\u0027s now specific to PIO HSM.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "5fe7454aa9c6ef5fcf506b0f2dfc20f696891f1a", "tree": "94e01d83671ed916dd6c931333722702bf46c7ef", "parents": [ "8244cd05979ef924787aa70fd80304f1773976a1" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon May 10 21:41:37 2010 +0200" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Wed May 19 13:35:47 2010 -0400" }, "message": "libata-sff: ap-\u003e[last_]ctl are SFF specific\n\nap-\u003e[last_]ctl are specific to SFF controllers. Put them inside\nCONFIG_ATA_SFF and move initialization into ata_sff_port_init().\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "8244cd05979ef924787aa70fd80304f1773976a1", "tree": "1e6f2434807eb3adbc097b3f4d7a4240237bc105", "parents": [ "270390e1ae1818b111543b8bfffa08095d73c1a5" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon May 10 21:41:36 2010 +0200" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Wed May 19 13:35:44 2010 -0400" }, "message": "libata-sff: rename ap-\u003eops-\u003edrain_fifo() to sff_drain_fifo()\n\n-\u003edrain_fifo() is SFF specific. Rename and relocate it.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "c7087652e1890a3feef35b30ee1d4be68e1932cd", "tree": "04aeca0f44e87a531d7cb5f894b6790391c2c552", "parents": [ "8930ff254a3a80d4477c3391ade07d6dd2a036c7" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon May 10 21:41:34 2010 +0200" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Wed May 19 13:32:19 2010 -0400" }, "message": "libata-sff: clean up BMDMA initialization\n\nWhen BMDMA initialization failed or BMDMA was not available for\nwhatever reason, bmdma_addr was left at zero and used as an indication\nthat BMDMA shouldn\u0027t be used. This leads to the following problems.\n\np1. For BMDMA drivers which don\u0027t use traditional BMDMA register,\n ata_bmdma_mode_filter() incorrectly inhibits DMA modes. Those\n drivers either have to inherit from ata_sff_port_ops or clear\n -\u003emode_filter explicitly.\n\np2. non-BMDMA drivers call into BMDMA PRD table allocation. It\n doesn\u0027t actually allocate PRD table if bmdma_addr is not\n initialized but is still confusing.\n\np3. For BMDMA drivers which don\u0027t use traditional BMDMA register, some\n methods might not be invoked as expected (e.g. bmdma_stop from\n ata_sff_post_internal_cmd()).\n\np4. SFF drivers w/ custom DMA interface implement noop BMDMA ops\n worrying libata core might call into one of them.\n\nThese problems are caused by the muddy line between SFF and BMDMA and\nthe assumption that all BMDMA controllers initialize bmdma_addr.\n\nThis patch fixes p1 and p2 by removing the bmdma_addr assumption and\nmoving prd allocation to BMDMA port start. Later patches will fix the\nremaining issues.\n\nThis patch improves BMDMA initialization such that\n\n* When BMDMA register initialization fails, falls back to PIO instead\n of failing. ata_pci_bmdma_init() never fails now.\n\n* When ata_pci_bmdma_init() falls back to PIO, it clears\n ap-\u003emwdma_mask and udma_mask instead of depending on\n ata_bmdma_mode_filter(). This makes ata_bmdma_mode_filter()\n unnecessary thus resolving p1.\n\n* ata_port_start() which actually is BMDMA specific is moved to\n ata_bmdma_port_start(). ata_port_start() and ata_sff_port_start()\n are killed.\n\n* ata_sff_port_start32() is moved and renamed to\n ata_bmdma_port_start32().\n\nDrivers which no longer call into PRD table allocation are...\n\n pdc_adma, sata_inic162x, sata_qstor, sata_sx4, pata_cmd640 and all\n drivers which inherit from ata_sff_port_ops.\n\npata_icside sets -\u003eport_start to ATA_OP_NULL as it doesn\u0027t need PRD\nbut is a BMDMA controller and doesn\u0027t have custom port_start like\nother such controllers.\n\nNote that with the previous patch which makes all and only BMDMA\ndrivers inherit from ata_bmdma_port_ops, this change doesn\u0027t break\ndrivers which need PRD table.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "9f2f72107ff621fdf3066e5a1b5ecb03ee587ebc", "tree": "266c1371cb706c941ebaebd097380beccd530041", "parents": [ "6bc0d390dd3dfb7be4325ad0685ae8ec2baf15a4" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon May 10 21:41:32 2010 +0200" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Mon May 17 22:49:07 2010 -0400" }, "message": "libata-sff: reorder SFF/BMDMA functions\n\nReorder functions such that SFF and BMDMA functions are grouped.\nWhile at it, s/BMDMA/SFF in a few comments where it actually meant\nSFF.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "3e4ec3443f70fbe144799ccf0b1c3797f78d1715", "tree": "cfb297e42075baacbbabfb6034e3e7c9a44c73b2", "parents": [ "c7a8209f766961eea4cfc6f22d2d6e06ef63546c" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon May 10 21:41:30 2010 +0200" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Mon May 17 22:49:02 2010 -0400" }, "message": "libata: kill ATA_FLAG_DISABLED\n\nATA_FLAG_DISABLED is only used by drivers which don\u0027t use\n-\u003eerror_handler framework and is largely broken. Its only meaningful\nfunction is to make irq handlers skip processing if the flag is set,\nwhich is largely useless and even harmful as it makes those ports more\nlikely to cause IRQ storms.\n\nKill ATA_FLAG_DISABLED and makes the callers disable attached devices\ninstead. ata_port_probe() and ata_port_disable() which manipulate the\nflag are also killed.\n\nThis simplifies condition check in IRQ handlers. While updating IRQ\nhandlers, remove ap NULL check as libata guarantees consecutive port\nallocation (unoccupied ports are initialized with dummies) and\nlong-obsolete ATA_QCFLAG_ACTIVE check (checked by ata_qc_from_tag()).\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "e42a542ba9cca594897176020445023c54d903d6", "tree": "b2a136581d6fcea9a2a6c534888319b70e15c2af", "parents": [ "41dec29bcb05eb8ec396f70ce791c6e3e4ce4712" ], "author": { "name": "Sergei Shtylyov", "email": "sshtylyov@ru.mvista.com", "time": "Fri May 07 22:49:02 2010 +0400" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Fri May 14 17:35:52 2010 -0400" }, "message": "libata: make sff_irq_on() method optional\n\nNow, with the introduction of the sff_set_devctl() method, we can\nuse it in sff_irq_on() method too -- that way its implementations\nin \u0027pata_bf54x\u0027 and \u0027pata_scc\u0027 become virtually identical to\nata_sff_irq_on(). The sff_irq_on() method now becomes quite\nsuperfluous, and the only reason not to remove it completely is\nthe existence of the \u0027pata_octeon_cf\u0027 driver which implements it\nas an empty function. Just make the method optional then, with\nata_sff_irq_on() becoming generic taskfile-bound function, still\nglobal for the \u0027pata_bf54x\u0027 driver to be able to call it from its\nthaw() and postreset() methods.\n\nWhile at it, make the sff_irq_on() method and ata_sff_irq_on() return\n\u0027void\u0027 as the result is always ignored anyway.\n\nSigned-off-by: Sergei Shtylyov \u003csshtylyov@ru.mvista.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "41dec29bcb05eb8ec396f70ce791c6e3e4ce4712", "tree": "e76f63519031769e848dcc6eba63c9207cf4d84e", "parents": [ "55787183ade44c4f826f581a068f52a1a80c6a2e" ], "author": { "name": "Sergei Shtylyov", "email": "sshtylyov@ru.mvista.com", "time": "Fri May 07 22:47:50 2010 +0400" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Fri May 14 17:35:52 2010 -0400" }, "message": "libata: introduce sff_set_devctl() method\n\nThe set of libata\u0027s taskfile access methods is clearly incomplete as\nit lacks a method to write to the device control register -- which\nforces drivers like \u0027pata_bf54x\u0027 and \u0027pata_scc\u0027 to implement more\n\"high level\" (and more weighty) methods like freeze() and postreset().\n\nSo, introduce the optional sff_set_devctl() method which the drivers\nonly have to implement if the standard iowrite8() can\u0027t be used (just\nlike the existing sff_check_altstatus() method) and make use of it\nin the freeze() and postreset() method implementations (I could also\nhave used it in softreset() method but it also reads other taskfile\nregisters without using tf_read() making that quite pointless);\nthis makes freeze() method implementations in the \u0027pata_bf54x\u0027 and\n\u0027pata_scc\u0027 methods virtually identical to ata_sff_freeze(), so we\ncan get rid of them completely.\n\nSigned-off-by: Sergei Shtylyov \u003csshtylyov@ru.mvista.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "b48d58f55aa1d2d0d12378e45663842d4021916e", "tree": "6fe1afb742eef755f4d09022548b6d2d2f8e919e", "parents": [ "294440887b32c58d220fb54b73b7a58079b78f20" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Fri Apr 09 19:46:38 2010 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Fri May 14 17:08:02 2010 -0400" }, "message": "libata: use longer 0xff wait if parallel scan is enabled\n\nThere are some SATA devices which take relatively long to get out of\n0xff status after reset. In libata, this timeout is determined by\nATA_TMOUT_FF_WAIT. Quantum GoVault is the worst requring about 2s for\nreliable detection. However, because 2s 0xff timeout can introduce\nrather long spurious delay during boot, libata has been compromising\nat the next longest timeout of 800ms for HHD424020F7SV00 iVDR drive.\n\nNow that parallel scan is in place for common drivers, libata can\nafford 2s 0xff timeout. Use 2s 0xff timeout if parallel scan is\nenabled.\n\nPlease note that the chance of spurious wait is pretty slim w/ working\nSCR access so this will only affect SATA controllers w/o SCR access\nwhich isn\u0027t too common these days.\n\nPlease read the following thread for more information on the GoVault\ndrive.\n\n http://thread.gmane.org/gmane.linux.ide/14545/focus\u003d14663\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Gary Hade \u003cgaryhade@us.ibm.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "294440887b32c58d220fb54b73b7a58079b78f20", "tree": "83ad03746ca4e7e34fb74cdb2e42b075e085d4da", "parents": [ "4f2c774856708bccecb74c0e6296e9e2c9136ee1" ], "author": { "name": "Sergei Shtylyov", "email": "sshtylyov@ru.mvista.com", "time": "Thu Apr 22 20:03:35 2010 +0400" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Fri May 14 17:08:02 2010 -0400" }, "message": "libata-sff: kill unused ata_bus_reset()\n\n... since I see no callers of it.\n\nSigned-off-by: Sergei Shtylyov \u003csshtylyov@ru.mvista.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "445d211b0da4e9a6e6d576edff85085c2aaf53df", "tree": "039ade5b49cb0bbd388babc369f28f541c76ec96", "parents": [ "68b0ddb289220b6d4d865be128939663be34959d" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon Apr 05 10:33:13 2010 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue Apr 06 10:55:33 2010 -0400" }, "message": "libata: unlock HPA if device shrunk\n\nSome BIOSes don\u0027t configure HPA during boot but do so while resuming.\nThis causes harddrives to shrink during resume making libata detach\nand reattach them. This can be worked around by unlocking HPA if old\nsize equals native size.\n\nAdd ATA_DFLAG_UNLOCK_HPA so that HPA unlocking can be controlled\nper-device and update ata_dev_revalidate() such that it sets\nATA_DFLAG_UNLOCK_HPA and fails with -EIO when the above condition is\ndetected.\n\nThis patch fixes the following bug.\n\n https://bugzilla.kernel.org/show_bug.cgi?id\u003d15396\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nReported-by: Oleksandr Yermolenko \u003cyaa.bta@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "16ea0fc98d53c72cb4e1a9edcb685a87e3a81430", "tree": "1a799ce01f8ea2b3a425631e791b946cb94285b1", "parents": [ "cb6643e1c38b6bd5c1594f0a45d8cf6943a6f934" ], "author": { "name": "Alan Cox", "email": "alan@linux.intel.com", "time": "Tue Feb 23 02:26:06 2010 -0500" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Mon Mar 01 14:58:46 2010 -0500" }, "message": "libata: Pass host flags into the pci helper\n\nThis allows parallel scan and the like to be set without having to stop\nusing the existing full helper functions. This patch merely adds the argument\nand fixes up the callers. It doesn\u0027t undo the special cases already in the\ntree or add any new parallel callers.\n\nSigned-off-by: Alan Cox \u003calan@linux.intel.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "27943620cbd960f710a385ff4a538e14ed3f1922", "tree": "fae161f49c10fa488f7a5891085961f284021544", "parents": [ "d88ec2e5c13261cf317b46832a7de216f6d06537" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Tue Jan 19 10:49:19 2010 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Mon Mar 01 14:58:44 2010 -0500" }, "message": "libata: implement spurious irq handling for SFF and apply it to piix\n\nTraditional IDE interface sucks in that it doesn\u0027t have a reliable IRQ\npending bit, so if the controller raises IRQ while the driver is\nexpecting it not to, the IRQ won\u0027t be cleared and eventually the IRQ\nline will be killed by interrupt subsystem. Some controllers have\nnon-standard mechanism to indicate IRQ pending so that this condition\ncan be detected and worked around.\n\nThis patch adds an optional operation -\u003esff_irq_check() which will be\ncalled for each port from the ata_sff_interrupt() if an unexpected\ninterrupt is received. If the operation returns %true,\n-\u003esff_check_status() and -\u003esff_irq_clear() will be cleared for the\nport. Note that this doesn\u0027t mark the interrupt as handled so it\nwon\u0027t prevent IRQ subsystem from killing the IRQ if this mechanism\nfails to clear the spurious IRQ.\n\nThis patch also implements -\u003esff_irq_check() for ata_piix. Note that\nthis adds slight overhead to shared IRQ operation as IRQs which are\ndestined for other controllers will trigger extra register accesses to\ncheck whether IDE interrupt is pending but this solves rare screaming\nIRQ cases and for some curious reason also helps weird BIOS related\nglitch on Samsung n130 as reported in bko#14314.\n\n http://bugzilla.kernel.org/show_bug.cgi?id\u003d14314\n\n* piix_base_ops dropped as suggested by Sergei.\n\n* Spurious IRQ detection doesn\u0027t kick in anymore if polling qc is in\n progress. This provides less protection but some controllers have\n possible data corruption issues if the wrong register is accessed\n while a command is in progress.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nReported-by: Johannes Stezenbach \u003cjs@sig21.net\u003e\nReported-by: Hans Werner \u003chwerner4@gmx.de\u003e\nCc: Alan Cox \u003calan@lxorguk.ukuu.org.uk\u003e\nCc: Sergei Shtylyov \u003csshtylyov@ru.mvista.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "5040ab67a2c6d5710ba497dc52a8f7035729d7b0", "tree": "dea2290e50111fde938bfcbb83de0377e7c4d94a", "parents": [ "0b67c7439fe2a5d76602de36854c88e2beab00b0" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon Jan 11 11:14:44 2010 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue Jan 12 14:34:14 2010 -0500" }, "message": "libata: retry link resume if necessary\n\nInterestingly, when SIDPR is used in ata_piix, writes to DET in\nSControl sometimes get ignored leading to detection failure. Update\nsata_link_resume() such that it reads back SControl after clearing DET\nand retry if it\u0027s not clear.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nReported-by: fengxiangjun \u003cfengxiangjun@neusoft.com\u003e\nReported-by: Jim Faulkner \u003cjfaulkne@ccs.neu.edu\u003e\nCc: stable@kernel.org\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "382f51fe2f2276344d8a21447656778cdf6583b6", "tree": "c2836a2cca4126c9c026ce5aa2fdf9f1c8ccded6", "parents": [ "701791cc3c8fc6dd83f6ec8af7e2541b4a316606", "54987386ee3790f3900de4df2ed4deb0e18dfc9f" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Dec 09 19:42:25 2009 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Dec 09 19:42:25 2009 -0800" }, "message": "Merge git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi-misc-2.6\n\n* git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi-misc-2.6: (222 commits)\n [SCSI] zfcp: Remove flag ZFCP_STATUS_FSFREQ_TMFUNCNOTSUPP\n [SCSI] zfcp: Activate fc4s attributes for zfcp in FC transport class\n [SCSI] zfcp: Block scsi_eh thread for rport state BLOCKED\n [SCSI] zfcp: Update FSF error reporting\n [SCSI] zfcp: Improve ELS ADISC handling\n [SCSI] zfcp: Simplify handling of ct and els requests\n [SCSI] zfcp: Remove ZFCP_DID_MASK\n [SCSI] zfcp: Move WKA port to zfcp FC code\n [SCSI] zfcp: Use common code definitions for FC CT structs\n [SCSI] zfcp: Use common code definitions for FC ELS structs\n [SCSI] zfcp: Update FCP protocol related code\n [SCSI] zfcp: Dont fail SCSI commands when transitioning to blocked fc_rport\n [SCSI] zfcp: Assign scheduled work to driver queue\n [SCSI] zfcp: Remove STATUS_COMMON_REMOVE flag as it is not required anymore\n [SCSI] zfcp: Implement module unloading\n [SCSI] zfcp: Merge trace code for fsf requests in one function\n [SCSI] zfcp: Access ports and units with container_of in sysfs code\n [SCSI] zfcp: Remove suspend callback\n [SCSI] zfcp: Remove global config_mutex\n [SCSI] zfcp: Replace local reference counting with common kref\n ...\n" }, { "commit": "e881a172dac4d9ea3b2a1540041d872963c269bd", "tree": "9eb1f344b107806c0041c4e0a64192a055117289", "parents": [ "dbf9bfe615717d1145f263c0049fe2328e6ed395" ], "author": { "name": "Mike Christie", "email": "michaelc@cs.wisc.edu", "time": "Thu Oct 15 17:46:39 2009 -0700" }, "committer": { "name": "James Bottomley", "email": "James.Bottomley@suse.de", "time": "Fri Dec 04 12:00:41 2009 -0600" }, "message": "[SCSI] modify change_queue_depth to take in reason why it is being called\n\nThis patch modifies scsi_host_template-\u003echange_queue_depth so that\nit takes an argument indicating why it is being called. This will be\nused so that if a LLD needs to do some extra processing when\nhandling queue fulls or later ramp ups, it can do so.\n\nThis is a simple port of the drivers setting a change_queue_depth\ncallback. In the patch I just have these LLDs adjust the queue depth\nif the user was requesting it.\n\nSigned-off-by: Mike Christie \u003cmichaelc@cs.wisc.edu\u003e\n\n[Vasu.Dev: v2\n\tAlso converted pmcraid_change_queue_depth and then verified\nall modules compile using \"make allmodconfig\" for any new build\nwarnings on X86_64.\n\n\tUpdated original description after combing two original\npatches from Mike to make this patch git bisectable.]\nSigned-off-by: Vasu Dev \u003cvasu.dev@intel.com\u003e\n[jejb: fixed up 53c700]\nSigned-off-by: James Bottomley \u003cJames.Bottomley@suse.de\u003e\n" }, { "commit": "95514fd8ff0f30de7815950edfd84ef1e19fb1c8", "tree": "dc9ab63bb174ce7b1727efc5ad5836f89d046b99", "parents": [ "7a02267e0f3938a6e91c0a0a82211e4d7007210e" ], "author": { "name": "Bartlomiej Zolnierkiewicz", "email": "bzolnier@gmail.com", "time": "Wed Nov 25 18:12:48 2009 +0100" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Dec 03 14:36:18 2009 -0500" }, "message": "libata: add private driver field to struct ata_device\n\nThis brings struct ata_device in-line with struct ata_{port,host}.\n\nSigned-off-by: Bartlomiej Zolnierkiewicz \u003cbzolnier@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "6013efd8860bf15c1f86f365332642cfe557152f", "tree": "9d7fe6d92ed9e067cda3b2e62d69f50175450e97", "parents": [ "fd6c29e3dec9e44ecbcba3c57efa08af70a10f1e" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Thu Nov 19 15:36:45 2009 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Dec 03 02:46:35 2009 -0500" }, "message": "libata: retry failed FLUSH if device didn\u0027t fail it\n\nIf ATA device failed FLUSH, it means that the device failed to write\nout some amount of data and the error needs to be reported to upper\nlayers. As retries can\u0027t recover the lost data, FLUSH failures need to\nbe reported immediately in general.\n\nHowever, if FLUSH fails due to transmission errors, the FLUSH needs to\nbe retried; otherwise, filesystems may switch to RO mode and/or raid\narray may drop a drive for a random transmission glitch.\n\nThis condition can be rather easily reproduced on certain ahci\ncontrollers which go through a PHY event after powersave mode switch +\next4 combination. Powersave mode switch is often closely followed by\nflush from the filesystem failing the FLUSH with ATA bus error which\nmakes the filesystem code believe that data is lost and drop to RO\nmode. This was reported in the following bugzilla bug.\n\n http://bugzilla.kernel.org/show_bug.cgi?id\u003d14543\n\nThis patch makes libata EH retry FLUSH if it wasn\u0027t failed by the\ndevice.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nReported-by: Andrey Vihrov \u003candrey.vihrov@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "110f66d25c33c2259b1125255fa7063ab07b8340", "tree": "2d524b3f27aa168eb01cbb007c2bfa195bea42f4", "parents": [ "fa5b561c4ea170caf9759109acc2e961a7e83bea" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Sep 16 04:17:28 2009 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue Oct 06 00:26:27 2009 -0400" }, "message": "libata: make gtf_filter per-dev\n\nAdd -\u003egtf_filter to ata_device and set it to ata_acpi_gtf_filter when\ninitializing ata_link. This is to allow quirks which apply different\ngtf filters.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "fa5b561c4ea170caf9759109acc2e961a7e83bea", "tree": "55d872dc6bba8bce920f95d1a19bf441784c94ac", "parents": [ "f1bce7f80e3b400cf29787b0afa9c3042b959017" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Sep 16 04:17:02 2009 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue Oct 06 00:26:26 2009 -0400" }, "message": "libata: implement more acpi filtering options\n\nCurrently libata-acpi can only filter DIPM among SATA feature enables\nvia _GTF. This patch adds the capability to filter out FPDMA non-zero\noffset, in-order guarantee and auto-activation.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "f1bce7f80e3b400cf29787b0afa9c3042b959017", "tree": "fa0cec132984e3cd36890c2dfedb52dd0e64df24", "parents": [ "4c521c8ef0e802f88f1d80352dd1b3d6a6aa1cc8" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Sep 16 04:16:04 2009 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue Oct 06 00:25:03 2009 -0400" }, "message": "libata: cosmetic updates\n\nWe\u0027re about to add more SATA_* and ATA_ACPI_FILTER_* constants.\nReformat them in preparation.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "051d9fbdd1d1ec85ea18ba20581234cf23f1c217", "tree": "7ad5adda0326170be068578782129ce59c41f491", "parents": [ "6521148c6449724c3b707820b9c535c7e8b8afcd" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Fri Jul 03 11:46:12 2009 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue Sep 01 19:47:20 2009 -0400" }, "message": "libata: remove spindown skipping and warning\n\nThis was a hack to give userland shutdown tools time to drop manual\nspindown. All popular distros updated quite some time ago and the due\nis well passed. Drop it.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Jaswinder Singh Rajput \u003cjaswinder@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "388539f3ff0cf1de926b03f94e1eec112358f74d", "tree": "6840b76ede7e27e27eb9aaf516f827beba16c5ad", "parents": [ "2fc37adba0fb05760b8635c6706773af828ccf3c" ], "author": { "name": "Shaohua Li", "email": "shaohua.li@intel.com", "time": "Mon Jul 27 09:24:35 2009 +0800" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue Sep 01 19:47:19 2009 -0400" }, "message": "[libata] add DMA setup FIS auto-activate feature\n\nHopefully results in fewer on-the-wire FIS\u0027s and no breakage. We\u0027ll see!\n\nSigned-off-by: Shaohua Li \u003cshaohua.li@intel.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "5920dadfb4aec6c1372c5570e71bcd3b4837e63c", "tree": "4d4e1c5a47d23b1578bfde9ab223b07206a77c64", "parents": [ "7d084d96fdf1d791cb171da57efc1ca89d68dd6c" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Jul 15 17:11:41 2009 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue Jul 28 21:07:09 2009 -0400" }, "message": "libata: accept late unlocking of HPA\n\nOn certain configurations, HPA isn\u0027t or can\u0027t be unlocked during\nprobing but it somehow ends up unlocked afterwards. In the following\nthread, the problem can be reliably reproduced after resuming from\nSTR. The BIOS turns on HPA during boot but forgets to do it during\nresume.\n\n http://thread.gmane.org/gmane.linux.kernel/858310\n\nThis patch updates libata revalidation such that it considers native\nn_sectors. If the device size has increased to match native\nn_sectors, it\u0027s assumed that HPA has been unlocked involuntarily and\nthe device is recognized as the same one. This should be fairly safe\nwhile nicely working around the problem.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nReported-by: Christof Warlich \u003cchristof@warlich.name\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "d0cb43b35d64877b2944bd37719708be5d7bbf99", "tree": "7b282b1070613c552143d143f74e407c2ccb4b6e", "parents": [ "fe2c4d018fc6127610fef677e020b3bb41cfaaaf" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Thu Jul 09 09:27:50 2009 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue Jul 14 22:41:46 2009 -0400" }, "message": "libata: implement and use HORKAGE_NOSETXFER, take#2\n\nPIONEER DVD-RW DVRTD08 times out SETXFER if no media is present. The\ndevice is SATA and simply skipping SETXFER works around the problem.\nImplement ATA_HORKAGE_NOSETXFER and apply it to the device.\n\nReported by Moritz Rigler in the following thread.\n\n http://thread.gmane.org/gmane.linux.ide/36790\n\nand by Lars in bko#9540.\n\nUpdated to whine and ignore NOSETXFER if PATA component is detected as\nsuggested by Alan Cox.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nReported-by: Moritz Rigler \u003clinux-ide@momail.e4ward.com\u003e\nReported-by: Lars \u003clars21ce@gmx.de\u003e\nCc: Alan Cox \u003calan@lxorguk.ukuu.org.uk\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "e3cf95dd6d352954b663d2934110d6e30af2406d", "tree": "f3245e91c30e0c1e12a169af637373a8b5ff670a", "parents": [ "62afe5d744047df8ff15a369f4c1ebad71c937d4" ], "author": { "name": "Alan Cox", "email": "alan@lxorguk.ukuu.org.uk", "time": "Thu Apr 09 17:31:17 2009 +0100" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 16 15:28:23 2009 -0400" }, "message": "ata: Report 16/32bit PIO as best we can\n\nThe legacy old IDE ioctl API for this is a bit primitive so we try\nand map stuff sensibly onto it.\n\n- Set PIO over DMA devices to report 32bit\n- Add ability to change the PIO32 settings if the controller permits it\n- Add that functionality into the sff drivers\n- Add that functionality into the VLB legacy driver\n- Turn on the 32bit PIO on the ninja32 and add support there\n\nSigned-off-by: Alan Cox \u003calan@linux.intel.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "877d03105d04b2c13e241130277fa69c8d2564f0", "tree": "79234721145f9dab3e0970027ed0b9b9f8929725", "parents": [ "caa790ba6cb88dccfab356960d93e2f4e0bd8704" ], "author": { "name": "Nick Andrew", "email": "nick@nick-andrew.net", "time": "Mon Jan 26 11:06:57 2009 +0100" }, "committer": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Mon Mar 30 15:21:59 2009 +0200" }, "message": "trivial: Fix misspelling of firmware\n\nFix misspelling of firmware.\n\nSigned-off-by: Nick Andrew \u003cnick@nick-andrew.net\u003e\nSigned-off-by: Jiri Kosina \u003cjkosina@suse.cz\u003e\n" }, { "commit": "c96f1732e25362d10ee7bcac1df8412a2e6b7d23", "tree": "66e24eddb174d6751579ec5952f72cbbac0fb038", "parents": [ "3d47aa8e7e7b2aa09256590388aa8dddc79280f9" ], "author": { "name": "Alan Cox", "email": "alan@redhat.com", "time": "Tue Mar 24 10:23:46 2009 +0000" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue Mar 24 22:52:39 2009 -0400" }, "message": "[libata] Improve timeout handling\n\nOn a timeout call a device specific handler early in the recovery so that\nwe can complete and process successful commands which timed out due to IRQ\nloss or the like rather more elegantly.\n\n[Revised to exclude the timeout handling on a few devices that inherit from\n SFF but are not SFF enough to use the default timeout handler]\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "3d47aa8e7e7b2aa09256590388aa8dddc79280f9", "tree": "82f4c85842e5e02489eb0533609dabb865e55198", "parents": [ "c0f2ee34a5a0b79fd98d965ad8ae765d4639bfa5" ], "author": { "name": "Alan Cox", "email": "alan@redhat.com", "time": "Tue Mar 24 10:23:19 2009 +0000" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue Mar 24 22:48:26 2009 -0400" }, "message": "[libata] Drain data on errors\n\nIf the device is signalling that there is data to drain after an error we\nshould read the bytes out and throw them away. Without this some devices\nand controllers get wedged and don\u0027t recover.\n\nBased on earlier work by Mark Lord\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "1a660164c291f41b2aa853a7269b310933574ef9", "tree": "ba113e3cb6579c45d4e11aede7d8714e9dde2b05", "parents": [ "d16ab3f633b75aac1cf42b00355cd9aa65033dcc" ], "author": { "name": "Mark Lord", "email": "mlord@pobox.com", "time": "Wed Feb 25 15:18:32 2009 -0500" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue Mar 24 22:02:41 2009 -0400" }, "message": "[libata] Export ata_pio_queue_task() so that it can be used from sata_mv.\n\nSigned-off-by: Mark Lord \u003cmlord@pobox.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "84bda12af31f930e4200c5244aa111de2485d7b0", "tree": "08d214bf7a1ce56e8c994e7970513bfb4b215e3d", "parents": [ "5825627c9463581fd9e70f8285685889ae5bb9bb" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon Mar 02 18:53:26 2009 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Mar 05 07:25:02 2009 -0500" }, "message": "libata: align ap-\u003esector_buf\n\nap-\u003esector_buf is used as DMA target and should at least be aligned on\ncacheline. This caused problems on some embedded machines.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "5825627c9463581fd9e70f8285685889ae5bb9bb", "tree": "321a20a0e25aee8f32254b13730f423f6b434486", "parents": [ "e7d3ef13d52a126438f687a1a32da65ff926ed57" ], "author": { "name": "FUJITA Tomonori", "email": "fujita.tomonori@lab.ntt.co.jp", "time": "Fri Feb 27 17:35:43 2009 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Mar 05 07:24:57 2009 -0500" }, "message": "libata: fix dma_unmap_sg misuse\n\nlibata passes the returned value of dma_map_sg() to\ndma_unmap_sg(),which is the misuse of dma_unmap_sg().\n\nDMA-mapping.txt says:\n\nTo unmap a scatterlist, just call:\n\n\tpci_unmap_sg(pdev, sglist, nents, direction);\n\nAgain, make sure DMA activity has already finished.\n\nPLEASE NOTE: The \u0027nents\u0027 argument to the pci_unmap_sg call must be\n the _same_ one you passed into the pci_map_sg call,\n\t it should _NOT_ be the \u0027count\u0027 value _returned_ from the\n pci_map_sg call.\n\nSigned-off-by: FUJITA Tomonori \u003cfujita.tomonori@lab.ntt.co.jp\u003e\nAcked-by: Bartlomiej Zolnierkiewicz \u003cbzolnier@gmail.com\u003e\nAcked-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "e7d3ef13d52a126438f687a1a32da65ff926ed57", "tree": "b4eb90e422f33fe6bb8756fcaadbd19863eae649", "parents": [ "559595a985e106d2fa9f0c79b7f5805453fed593" ], "author": { "name": "Stuart Hayes", "email": "stuart_hayes@dell.com", "time": "Wed Mar 04 11:59:46 2009 -0800" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Mar 05 07:24:42 2009 -0500" }, "message": "libata: change drive ready wait after hard reset to 5s\n\nThis fixes problems during resume with drives that take longer than 1s to\nbe ready. The ATA-6 spec appears to allow 5 seconds for a drive to be\nready.\n\nOn one affected system, this patch changes \"PM: resume devices took...\"\nmessage from 17 seconds to 4 seconds, and gets rid of a lot of ugly\ntimeout/error messages.\n\nWithout this patch, the libata code moves on after 1s, tries to send a\nsoft reset (which the drive doesn\u0027t see because it isn\u0027t ready) which also\ntimes out, then an IDENTIFY command is sent to the drive which times out,\nand finally the error handler will try to send another hard reset which\nwill finally get things working.\n\nSigned-off-by: Stuart Hayes \u003cstuart_hayes@dell.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "9062712fa9ed13b531dfc2228086650b8bd6a255", "tree": "d1bde6caeb53936859943bb1866ac65794cae5cb", "parents": [ "cf9a590a9eae3b99ca77d8db17afd2d7dbdd0986" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Thu Jan 29 20:31:36 2009 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Mon Feb 02 23:04:31 2009 -0500" }, "message": "libata: implement HORKAGE_1_5_GBPS and apply it to WD My Book\n\n3Gbps is often much more prone to transmission failures. It\u0027s usually\nokay to let EH handle speed down after transmission failures but some\nWD My Book drives completely shutdown after certain transmission\nfailures and after it only power cycling can revive them. Combined\nwith the fact that external drives often end up with cable assembly\nwhich is longer than usual and more likely to have intervening gender,\nthis makes these drives very likely to shutdown under certain\nconfigurations virtually rendering them unusable.\n\nThis patch implements HOARKGE_1_5_GBPS and applies it to WD My Book\nsuch that 1.5Gbps is forced once the device is identified.\n\nPlease take a look at the following bz for related reports.\n\n http://bugzilla.kernel.org/show_bug.cgi?id\u003d9913\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "99cf610aa4840d822cdc67d194b23b55010ca9bd", "tree": "68bc3d6c1f13849731bb7b4b11414b4b0e5fbffb", "parents": [ "9913ff8abf1c70a8d52560dc931e1901d025ad27" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Thu Jan 29 20:31:32 2009 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Mon Feb 02 23:03:17 2009 -0500" }, "message": "libata: clear dev-\u003eering in smarter way\n\ndev-\u003eering used to be cleared together with the rest of ata_device in\nata_dev_init() which is called whenever a probing event occurs.\ndev-\u003eering is about to be used to track probing failures so it needs\nto remain persistent over multiple porbing events. This patch\nachieves this by doing the following.\n\n* Instead of CLEAR_OFFSET, define CLEAR_BEGIN and CLEAR_END and only\n clear between BEGIN and END. ering is moved after END. The split\n of persistent area is to allow hotter items remain at the head.\n\n* ering is explicitly cleared on ata_dev_disable() and when device\n attach succeeds. So, ering is persistent throug a device\u0027s life\n time (unless explicitly cleared of course) and also through periods\n inbetween disablement of an attached device and successful detection\n of the next one.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "2a6e58d2731dcc05dafa7f976d935e0f0627fcd7", "tree": "836d0ec85dccde709aeaaedb3b7686ac51805ad1", "parents": [ "d7b1956fed33d30c4815e848fd7a143722916868" ], "author": { "name": "Rafael J. Wysocki", "email": "rjw@sisk.pl", "time": "Mon Jan 19 20:56:43 2009 +0100" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue Jan 27 02:15:49 2009 -0500" }, "message": "SATA: Blacklisting of systems that spin off disks during ACPI power off\n\nIntroduce new libata flags ATA_FLAG_NO_POWEROFF_SPINDOWN and\nATA_FLAG_NO_HIBERNATE_SPINDOWN that, if set, will prevent disks from\nbeing spun off during system power off and hibernation, respectively\n(to handle the hibernation case we need the new system state\nSYSTEM_HIBERNATE_ENTER that can be checked against by libata, in\nanalogy with SYSTEM_POWER_OFF).\n\nSigned-off-by: Rafael J. Wysocki \u003crjw@sisk.pl\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "3ada9c126499dd4700dcdbd5b9fe8110ad17f578", "tree": "aec4fc6d574589eec606cf234eb7f5df529b05f3", "parents": [ "6813952021a7820a505002de260bda36978671f7" ], "author": { "name": "David Daney", "email": "ddaney@caviumnetworks.com", "time": "Thu Jan 15 17:45:31 2009 -0800" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Fri Jan 16 10:23:37 2009 -0500" }, "message": "libata: Add another column to the ata_timing table.\n\nThe forthcoming OCTEON SOC Compact Flash driver needs an additional\ntiming value that was not available in the ata_timing table. I add a\nnew column for dmack_hold time. The values were obtained from the\nCompact Flash specification Rev 4.1.\n\nSigned-off-by: David Daney \u003cddaney@caviumnetworks.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "94be9a58d7e683ac3c1df1858a17f09ebade8da0", "tree": "2f346bb4d179caf1cf0f61012f35c8277504c55a", "parents": [ "5393f780277165f282a37ed82dd878159ec9dad5" ], "author": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri Jan 16 10:17:09 2009 -0500" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Fri Jan 16 10:17:09 2009 -0500" }, "message": "[libata] get-identity ioctl: Fix use of invalid memory pointer\nfor SAS drivers.\n\nCaught by Ke Wei (and team?) at Marvell.\n\nAlso, move the ata_scsi_ioctl export to libata-scsi.c, as that seems to be the\ngeneral trend.\n\nAcked-by: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "886ad09fc83342aa1c5a02a0b6d3298b78a8067f", "tree": "82a7818829b25db9c67f95c315521dac3da47fc6", "parents": [ "3d14bdad40315b54470cb7812293d14c8af2bf7d" ], "author": { "name": "Arjan van de Ven", "email": "arjan@linux.intel.com", "time": "Fri Jan 09 15:54:07 2009 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sat Jan 10 15:06:52 2009 -0800" }, "message": "libata: Add a per-host flag to opt-in into parallel port probes\n\nThis patch adds a per host flag that allows drivers to opt in into\nhaving its busses scanned in parallel.\n\nDrivers that do not set this flag get their ports scanned in\nthe \"original\" sequence.\n\nSigned-off-by: Arjan van de Ven \u003carjan@linux.intel.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "871af1210f13966ab911ed2166e4ab2ce775b99d", "tree": "f17f0016f6e966d54a379a3de6e6bbde3b9359fe", "parents": [ "e427fe042cf90c0652eed9a85e57a8fd8af89890" ], "author": { "name": "Alan Cox", "email": "alan@redhat.com", "time": "Mon Jan 05 14:16:39 2009 +0000" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Jan 08 16:34:27 2009 -0500" }, "message": "libata: Add 32bit PIO support\n\nThis matters for some controllers and in one or two cases almost doubles\nPIO performance. Add a bmdma32 operations set we can inherit and activate\nit for some controllers\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "ece180d1cfe5fa751eaa85bf796cf28b2150af15", "tree": "aca9d485036858ed3f1859e679473cebd3476845", "parents": [ "ad74e4c18d0962397314460d0da312e72c8bd02d" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon Nov 03 20:04:37 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Sun Dec 28 22:43:21 2008 -0500" }, "message": "libata: perform port detach in EH\n\nata_port_detach() first made sure EH saw ATA_PFLAG_UNLOADING and then\nassumed EH context belongs to it and performed detach operation\nitself. However, UNLOADING doesn\u0027t disable all of EH and this could\nlead to problems including triggering WARN_ON()\u0027s in EH path.\n\nThis patch makes port detach behave more like other EH actions such\nthat ata_port_detach() requests EH to detach and waits for completion.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "1eca4365be25c540650693e941bc06a66cf38f94", "tree": "e3ed82850da00308180bf166118f9f9e69d92898", "parents": [ "3c92ec8ae91ecf59d88c798301833d7cf83f2179" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon Nov 03 20:03:17 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Sun Dec 28 22:43:20 2008 -0500" }, "message": "libata: beef up iterators\n\nThere currently are the following looping constructs.\n\n* __ata_port_for_each_link() for all available links\n* ata_port_for_each_link() for edge links\n* ata_link_for_each_dev() for all devices\n* ata_link_for_each_dev_reverse() for all devices in reverse order\n\nNow there\u0027s a need for looping construct which is similar to\n__ata_port_for_each_link() but iterates over PMP links before the host\nlink. Instead of adding another one with long name, do the following\ncleanup.\n\n* Implement and export ata_link_next() and ata_dev_next() which take\n @mode parameter and can be used to build custom loop.\n* Implement ata_for_each_link() and ata_for_each_dev() which take\n looping mode explicitly.\n\nThe following iteration modes are implemented.\n\n* ATA_LITER_EDGE\t\t: loop over edge links\n* ATA_LITER_HOST_FIRST\t\t: loop over all links, host link first\n* ATA_LITER_PMP_FIRST\t\t: loop over all links, PMP links first\n\n* ATA_DITER_ENABLED\t\t: loop over enabled devices\n* ATA_DITER_ENABLED_REVERSE\t: loop over enabled devices in reverse order\n* ATA_DITER_ALL\t\t\t: loop over all devices\n* ATA_DITER_ALL_REVERSE\t\t: loop over all devices in reverse order\n\nThis change removes exlicit device enabledness checks from many loops\nand makes it clear which ones are iterated over in which direction.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "ac70a964b0e22a95af3628c344815857a01461b7", "tree": "c5711139bd95053b9b5c8897e3f7cd50b8331141", "parents": [ "03f60840fa462e92220b093f778b2426ceab23af" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Thu Nov 27 13:36:48 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Mon Dec 01 13:49:27 2008 -0500" }, "message": "libata: blacklist Seagate drives which time out FLUSH_CACHE when used with NCQ\n\nSome recent Seagate harddrives have firmware bug which causes FLUSH\nCACHE to timeout under certain circumstances if NCQ is being used.\nThis can be worked around by disabling NCQ and fixed by updating the\nfirmware. Implement ATA_HORKAGE_FIRMWARE_UPDATE and blacklist these\ndevices.\n\nThe wiki page has been updated to contain information on this issue.\n\n http://ata.wiki.kernel.org/index.php/Known_issues\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "8a8bc22332ee6ea49137508467a76aa7f4367719", "tree": "f41c62dc99c4249d592935b4659569081d4a190a", "parents": [ "f7160c7573615ec82c691e294cf80d920b5d588d" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon Nov 10 14:48:21 2008 +0900" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Nov 10 08:04:47 2008 -0800" }, "message": "libata: revert convert-to-block-tagging patches\n\nThis patch reverts the following three commits which convert libata to\nuse block layer tagging.\n\n 43a49cbdf31e812c0d8f553d433b09b421f5d52c\n e013e13bf605b9e6b702adffbe2853cfc60e7806\n 2fca5ccf97d2c28bcfce44f5b07d85e74e3cd18e\n\nAlthough using block layer tagging is the right direction, due to the\ntight coupling among tag number, data structure allocation and\nhardware command slot allocation, libata doesn\u0027t work correctly with\nthe current conversion.\n\nThe biggest problem is guaranteeing that tag 0 is always used for\nnon-NCQ commands. Due to the way blk-tag is implemented and how SCSI\nstarts and finishes requests, such guarantee can\u0027t be made. I\u0027m not\nsure whether this would actually break any low level driver but it\ndoesn\u0027t look like a good idea to break such assumption given the\nfrailty of ATA controllers.\n\nSo, for the time being, keep using the old dumb in-libata qc\nallocation.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Jens Axobe \u003cjens.axboe@oracle.com\u003e\nCc: Jeff Garzik \u003cjeff@garzik.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "6a87e42e955ff27e07a77f65f8f077dc7c4171e1", "tree": "e5d50b2b91c17c6719b75bbd88ea5cbed4130304", "parents": [ "a464189de350b050aa8f334bd4cc53ed406e56dd" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon Nov 03 19:01:09 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue Nov 04 01:08:27 2008 -0500" }, "message": "libata: implement ATA_HORKAGE_ATAPI_MOD16_DMA and apply it\n\nlibata always uses PIO for ATAPI commands when the number of bytes to\ntransfer isn\u0027t multiple of 16 but quantum DAT72 chokes on odd bytes\nPIO transfers. Implement a horkage to skip the mod16 check and apply\nit to the quantum device.\n\nThis is reported by John Clark in the following thread.\n\n http://thread.gmane.org/gmane.linux.ide/34748\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: John Clark \u003cclarkjc@runbox.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "9ce8e3073d9cfd6f859c22a25441db41b85cbf6e", "tree": "81ddeb3d7203677b541b132b4ed8a909137d7ccf", "parents": [ "b9d5b89b487517cbd4cb4702da829e07ef9e4432" ], "author": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Wed Aug 27 15:23:18 2008 +0200" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Fri Oct 31 01:45:06 2008 -0400" }, "message": "libata: add whitelist for devices with known good pata-sata bridges\n\nlibata currently imposes a UDMA5 max transfer rate and 200 sector max\ntransfer size for SATA devices that sit behind a pata-sata bridge. Lots\nof devices have known good bridges that don\u0027t need this limit applied.\nThe MTRON SSD disks are such devices. Transfer rates are increased by\n20-30% with the restriction removed.\n\nSo add a \"blacklist\" entry for the MTRON devices, with a flag indicating\nthat the bridge is known good.\n\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "2fca5ccf97d2c28bcfce44f5b07d85e74e3cd18e", "tree": "483dedd08d6ccbb56959eeb127e82ab8f84edf2e", "parents": [ "332edc2f7fa58b818dfed1cede60272eecc27c0a" ], "author": { "name": "Jens Axboe", "email": "jens.axboe@oracle.com", "time": "Wed Oct 22 09:34:49 2008 +0200" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Oct 23 16:05:26 2008 -0700" }, "message": "libata: switch to using block layer tagging support\n\nlibata currently has a pretty dumb ATA_MAX_QUEUE loop for finding\na free tag to use. Instead of fixing that up, convert libata to\nusing block layer tagging - gets rid of code in libata, and is also\nmuch faster.\n\nSigned-off-by: Jens Axboe \u003cjens.axboe@oracle.com\u003e\nAcked-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "848e4c68c4695beae563f9a3d59fce596b466a74", "tree": "90c5ea1ff1b016f72888641f96f014336e969434", "parents": [ "570106df6bdb4907ad7f70793079c762f34d561a" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Tue Oct 21 14:26:39 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Wed Oct 22 20:40:19 2008 -0400" }, "message": "libata: transfer EHI control flags to slave ehc.i\n\nATA_EHI_NO_AUTOPSY and ATA_EHI_QUIET are used to control the behavior\nof EH. As only the master link is visible outside EH, these flags are\nset only for the master link although they should also apply to the\nslave link, which causes spurious EH messages during probe and\nsuspend/resume.\n\nThis patch transfers those two flags to slave ehc.i before performing\nslave autopsy and reporting.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "6866e7bc83f13a1bc6de59099930e9db1ab0042f", "tree": "aa028533eb6a019d8c1bc7f57ddf945454453f6d", "parents": [ "67e3e221d61c0e70b2f244fd921e5e601d6c7339" ], "author": { "name": "Richard Kennedy", "email": "richard@rsk.demon.co.uk", "time": "Mon Sep 22 14:47:13 2008 -0700" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Mon Sep 29 00:28:36 2008 -0400" }, "message": "libata: reorder ata_device to remove 8 bytes of padding on 64 bits\n\nreduce size by 8 bytes from 1160 to 1152 allowing it to fit in 1 fewer\ncachelines.\n\nSigned-off-by: Richard Kennedy \u003crichard@rsk.demon.co.uk\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "45fabbb77bd95adff7a80bde1c7a0ace1075fde6", "tree": "0e98efc190b25a11f84b8ae7d1ee0a17c41d3da8", "parents": [ "ea6ce53cd5d005455ec0a3cc1d45d3af0cb90919" ], "author": { "name": "Elias Oltmanns", "email": "eo@nebensachen.de", "time": "Sun Sep 21 11:54:08 2008 +0200" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Mon Sep 29 00:27:54 2008 -0400" }, "message": "libata: Implement disk shock protection support\n\nOn user request (through sysfs), the IDLE IMMEDIATE command with UNLOAD\nFEATURE as specified in ATA-7 is issued to the device and processing of\nthe request queue is stopped thereafter until the specified timeout\nexpires or user space asks to resume normal operation. This is supposed\nto prevent the heads of a hard drive from accidentally crashing onto the\nplatter when a heavy shock is anticipated (like a falling laptop\nexpected to hit the floor). In fact, the whole port stops processing\ncommands until the timeout has expired in order to avoid any resets due\nto failed commands on another device.\n\nSigned-off-by: Elias Oltmanns \u003ceo@nebensachen.de\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "b1c72916abbdd0a55015c87358536ca0ebaf6735", "tree": "1064fe92f2c3600dd6587c880d907020896b3348", "parents": [ "b5b3fa386b8f96c7fa92e507e5deddc2637924b4" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Thu Jul 31 17:02:43 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Mon Sep 29 00:25:28 2008 -0400" }, "message": "libata: implement slave_link\n\nExplanation taken from the comment of ata_slave_link_init().\n\n In libata, a port contains links and a link contains devices. There\n is single host link but if a PMP is attached to it, there can be\n multiple fan-out links. On SATA, there\u0027s usually a single device\n connected to a link but PATA and SATA controllers emulating TF based\n interface can have two - master and slave.\n\n However, there are a few controllers which don\u0027t fit into this\n abstraction too well - SATA controllers which emulate TF interface\n with both master and slave devices but also have separate SCR\n register sets for each device. These controllers need separate links\n for physical link handling (e.g. onlineness, link speed) but should\n be treated like a traditional M/S controller for everything else\n (e.g. command issue, softreset).\n\n slave_link is libata\u0027s way of handling this class of controllers\n without impacting core layer too much. For anything other than\n physical link handling, the default host link is used for both master\n and slave. For physical link handling, separate @ap-\u003eslave_link is\n used. All dirty details are implemented inside libata core layer.\n From LLD\u0027s POV, the only difference is that prereset, hardreset and\n postreset are called once more for the slave link, so the reset\n sequence looks like the following.\n\n prereset(M) -\u003e prereset(S) -\u003e hardreset(M) -\u003e hardreset(S) -\u003e\n softreset(M) -\u003e postreset(M) -\u003e postreset(S)\n\n Note that softreset is called only for the master. Softreset resets\n both M/S by definition, so SRST on master should handle both (the\n standard method will work just fine).\n\nAs slave_link excludes PMP support and only code paths which deal with\nthe attributes of physical link are affected, all the changes are\nlocalized to libata.h, libata-core.c and libata-eh.c.\n\n * ata_is_host_link() updated so that slave_link is considered as host\n link too.\n\n * iterator extended to iterate over the slave_link when using the\n underbarred version.\n\n * force param handling updated such that devno 16 is mapped to the\n slave link/device.\n\n * ata_link_on/offline() updated to return the combined result from\n master and slave link. ata_phys_link_on/offline() are the direct\n versions.\n\n * EH autopsy and report are performed separately for master slave\n links. Reset is udpated to implement the above described reset\n sequence.\n\nExcept for reset update, most changes are minor, many of them just\nmodifying dev-\u003elink to ata_dev_phys_link(dev) or using phys online\ntest instead.\n\nAfter this update, LLDs can take full advantage of per-dev SCR\nregisters by simply turning on slave link.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "b5b3fa386b8f96c7fa92e507e5deddc2637924b4", "tree": "2d203860958d43b430974bcf365f4d8d3aa36b24", "parents": [ "aadffb682cc5572f48cc24883681db65530bd284" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Thu Jul 31 17:02:42 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Mon Sep 29 00:22:32 2008 -0400" }, "message": "libata: misc updates to prepare for slave link\n\n* Add ATA_EH_ALL_ACTIONS.\n\n* Make sata_link_{on|off}_line() return bool instead of int.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "aadffb682cc5572f48cc24883681db65530bd284", "tree": "24ab92ec9b0cdd9e6f4c0bf7a8a4c04635be03db", "parents": [ "82ef04fb4c82542b3eda81cca461f0594ce9cd0b" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Thu Jul 31 17:02:41 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Mon Sep 29 00:22:30 2008 -0400" }, "message": "libata: reimplement link iterator\n\nImplement __ata_port_next_link() and reimplement\n__ata_port_for_each_link() and ata_port_for_each_link() using it.\nThis removes relatively large inlined code and makes iteration easier\nto extend.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "82ef04fb4c82542b3eda81cca461f0594ce9cd0b", "tree": "e513df5ad8dc9f7400830bfc8786afb6bec90fb6", "parents": [ "6ef190cc92e33565accff6a320f0e7d90480bfe7" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Thu Jul 31 17:02:40 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Mon Sep 29 00:22:28 2008 -0400" }, "message": "libata: make SCR access ops per-link\n\nLogically, SCR access ops should take @link; however, there was no\ncompelling reason to convert all SCR access ops when adding @link\nabstraction as there\u0027s one-to-one mapping between a port and a non-PMP\nlink. However, that assumption won\u0027t hold anymore with the scheduled\naddition of slave link.\n\nMake SCR access ops per-link.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "b15b3ebae102f89c25ccbcae0b2099af312f2e82", "tree": "73f64d0e578d1c6bc6a3dc43f940a31f24daad8f", "parents": [ "20e2de4a505aa02131a95665e8920eb053fce686" ], "author": { "name": "Alan Cox", "email": "alan@redhat.com", "time": "Fri Aug 01 09:18:34 2008 +0100" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Fri Aug 22 02:27:49 2008 -0400" }, "message": "libata: Fix a large collection of DMA mode mismatches\n\nDave Müller sent a diff for the pata_oldpiix that highlighted a problem\nwhere a lot of the ATA drivers assume dma_mode \u003d\u003d 0 means \"no DMA\" while\nthe core code uses 0xFF.\n\nThis turns out to have other consequences such as code doing \u003e\u003d XFER_UDMA_0\nalso catching 0xFF as UDMAlots. Fortunately it doesn\u0027t generally affect\nset_dma_mode, although some drivers call back into their own set mode code\nfrom other points.\n\nHaving been through the drivers I\u0027ve added helpers for using_udma/using_mwdma\ndma_enabled so that people don\u0027t open code ranges that may change (eg if UDMA8\nappears somewhere)\n\nThanks to David for the initial bits\n[and added fix for pata_oldpiix from and signed-off-by Dave Mueller\n \u003cdave.mueller@gmx.ch\u003e -jg]\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "d127ea7b8643a93d14d1f3c542974407f14d3663", "tree": "2edbaf46167c86faad6b7645f3d3d2ef52bba84a", "parents": [ "da0e21d3fa2340114fe24821718a1b57123e4664" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Thu Jul 31 16:09:34 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Fri Aug 22 02:19:46 2008 -0400" }, "message": "libata: restore SControl on detach\n\nSave SControl during probing and restore it on detach. This prevents\nadjustments made by libata drivers to seep into the next driver which\ngets attached (be it a libata one or not).\n\nIt\u0027s not clear whether SControl also needs to be restored on suspend.\nThe next system to have control (ACPI or kexec\u0027d kernel) would\nprobably like to see the original SControl value but there\u0027s no\nguarantee that a link is gonna keep working after SControl is adjusted\nwithout a reset and adding a reset and modified recovery cycle soley\nfor this is an overkill. For now, do it only for detach.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "05944bdf6fadb5394710269df6770dde447b23ca", "tree": "7dd0435f44518b01e5d3a229167e7122425e2255", "parents": [ "6a55617ed5d1aa62b850de2cf66f5ede2eef4825" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Aug 13 20:19:09 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Fri Aug 22 02:07:43 2008 -0400" }, "message": "libata: implement no[hs]rst force params\n\nImplement force params nohrst, nosrst and norst. This is to work\naround reset related problems and ease debugging.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "963e4975c6f93c148ca809d986d412201df9af89", "tree": "41cb6ec55e48a2186dbe4c13531fd05439efbade", "parents": [ "1f938d060a7bc01b5f82d46db3e38cd501b445a6" ], "author": { "name": "Alan Cox", "email": "alan@lxorguk.ukuu.org.uk", "time": "Thu Jul 24 17:16:06 2008 +0100" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Jul 31 02:04:50 2008 -0400" }, "message": "pata_it821x: Driver updates and reworking\n\n- Add support for the RDC 1010 variant\n- Rework the core library to have a read_id method. This allows the hacky\n bits of it821x to go and prepares us for pata_hd\n- Switch from WARN to BUG in ata_id_string as it will reboot if you get\n it wrong so WARN won\u0027t be seen\n- Allow the issue of command 0xFC on the 821x. This is needed to query\n rebuild status.\n- Tidy up printk formatting\n- Do more ident rewriting on RAID volumes to handle firmware provided\n ident data which is rather wonky\n- Report the firmware revision and device layout in RAID mode\n- Don\u0027t try and disable raid on the 8211 or RDC - they don\u0027t have the\n relevant bits\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "1f938d060a7bc01b5f82d46db3e38cd501b445a6", "tree": "f88f748671ac616742b41f202b4f56da1e577423", "parents": [ "487eff68e42287fd45cf178063f1ce1bad23c612" ], "author": { "name": "Alexander Beregalov", "email": "a.beregalov@gmail.com", "time": "Mon Jul 21 00:06:19 2008 +0400" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Jul 31 01:48:16 2008 -0400" }, "message": "libata.h: replace __FUNCTION__ with __func__\n\nSigned-off-by: Alexander Beregalov \u003ca.beregalov@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "18f7ba4c2f4be6b37d925931f04d6cc28d88d1ee", "tree": "4f127510c378cba33e79d5fb71bd9fc14a28e1cb", "parents": [ "87fbc5a060faf2394bee88a93519f9b9d434727c" ], "author": { "name": "Kristen Carlson Accardi", "email": "kristen.c.accardi@intel.com", "time": "Tue Jun 03 10:33:55 2008 -0700" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Mon Jul 14 15:59:33 2008 -0400" }, "message": "libata/ahci: enclosure management support\n\nAdd Enclosure Management support to libata and ahci.\n\nSigned-off-by: Kristen Carlson Accardi \u003ckristen.c.accardi@intel.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "87fbc5a060faf2394bee88a93519f9b9d434727c", "tree": "6f90cacb1471e64051473c4e1e664b2dea8603e0", "parents": [ "d8af0eb6046c56e7238171ca420622541db24926" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue May 20 02:17:54 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Mon Jul 14 15:59:32 2008 -0400" }, "message": "libata: improve EH internal command timeout handling\n\nATA_TMOUT_INTERNAL which was 30secs were used for all internal\ncommands which is way too long when something goes wrong. This patch\nimplements command type based stepped timeouts. Different command\ntypes can use different timeouts and each command type can use\ndifferent timeout values after timeouts.\n\nie. the initial timeout is set to a value which should cover most of\nthe cases but not too long so that run away cases don\u0027t delay things\ntoo much. After the first try times out, the second try can use\nlonger timeout and if that one times out too, it can go for full 30sec\ntimeout.\n\nIDENTIFYs use 5s - 10s - 30s timeout and all other commands use 5s -\n10s timeouts.\n\nThis patch significantly cuts down the needed time to handle failure\ncases while still allowing libata to work with nut job devices through\nretries.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "0a2c0f56159999e20015241d3b8fa89b1ab14309", "tree": "2c516452d3b3f85e9a4092b6092160e123d5f5d4", "parents": [ "341c2c958ec7bdd9f54733a8b0b432fe76842a82" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue May 20 02:17:52 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Mon Jul 14 15:59:32 2008 -0400" }, "message": "libata: improve EH retry delay handling\n\nEH retries were delayed by 5 seconds to ensure that resets don\u0027t occur\nback-to-back. However, this 5 second delay is superflous or excessive\nin many cases. For example, after IDENTIFY times out, there\u0027s no\nreason to wait five more seconds before retrying.\n\nThis patch adds ehc-\u003elast_reset timestamp and record the timestamp for\nthe last reset trial or success and uses it to space resets by\nATA_EH_RESET_COOL_DOWN which is 5 secs and removes unconditional 5 sec\nsleeps.\n\nAs this change makes inter-try waits often shorter and they\u0027re\nredundant in nature, this patch also removes the \"retrying...\"\nmessages.\n\nWhile at it, convert explicit rounding up division to DIV_ROUND_UP().\n\nThis change speeds up EH in many cases w/o sacrificing robustness.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "341c2c958ec7bdd9f54733a8b0b432fe76842a82", "tree": "1af1983fe82ad032de6142229fe6c70f04466918", "parents": [ "bce7f793daec3e65ec5c5705d2457b81fe7b5725" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue May 20 02:17:51 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Mon Jul 14 15:59:32 2008 -0400" }, "message": "libata: consistently use msecs for time durations\n\nlibata has been using mix of jiffies and msecs for time druations.\nThis is getting confusing. As writing sub HZ values in jiffies is\nPITA and msecs_to_jiffies() can\u0027t be used as initializer, unify unit\nfor all time durations to msecs. So, durations are in msecs and\ndeadlines are in jiffies. ata_deadline() is added to compute deadline\nfrom a start time and duration in msecs.\n\nWhile at it, drop now superflous _msec suffix from arguments and\nrename @timeout to @deadline if it represents a fixed point in time\nrather than duration.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "a57c1bade5a0ee5cd8b74502db9cbebb7f5780b2", "tree": "7919f7e4d85d512442698819db2ef98ed9121574", "parents": [ "4f0ebe3cc57f18ba26317b56b80b108c2848b1de" ], "author": { "name": "Alan Cox", "email": "alan@lxorguk.ukuu.org.uk", "time": "Thu May 29 22:10:58 2008 +0100" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Wed Jun 04 06:40:41 2008 -0400" }, "message": "libata-sff: Fix oops reported in kerneloops.org for pnp devices with no ctl\n\n- Make ata_sff_altstatus private so nobody uses it by mistake\n- Drop the 400nS delay from it\n\nAdd\n\nata_sff_irq_status\t-\tencapsulates the IRQ check logic\n\nThis function keeps the existing behaviour for altstatus using devices. I\nactually suspect the logic was wrong before the changes but -rc isn\u0027t the\ntime to play with that\n\nata_sff_sync\t\t-\tensure writes hit the device\n\nReally we want an io* operation for \u0027is posted\u0027 eg ioisposted(ioaddr) so\nthat we can fix the nasty delay this causes on most systems.\n\n- ata_sff_pause\t\t-\t400nS delay\n\nEnsure the command hit the device and delay 400nS\n\n- ata_sff_dma_pause\n\nEnsure the I/O hit the device and enforce an HDMA1:0 transition delay.\nRequires altstatus register exists, BUG if not so we don\u0027t risk\ncorruption in MWDMA modes. (UDMA the checksum will save your backside in\ntheory)\n\nThe only other complication then is devices with their own handlers.\nrb532 can use dma_pause but scc needs to access its own altstatus\nregister for internal errata workarounds so directly call the drivers own\naltstatus function.\n\nSigned-off-by: Alan Cox \u003calan@redhat.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "4f0ebe3cc57f18ba26317b56b80b108c2848b1de", "tree": "5e4e028b968c86dad23f17f636637eb73034819d", "parents": [ "ba069e376cc0801cd28352ca5986ce20413acb21" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Tue May 20 02:17:50 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Wed Jun 04 06:29:14 2008 -0400" }, "message": "libata: kill unused constants\n\nKill a few unused constants.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "50af2fa1e18d0ab411d06bf727ecadb7e01721e9", "tree": "16f2b3b74c51a02ee182f29e9de5752088aaadbe", "parents": [ "906c1ff44a81aaad96a9feb40ea13d73bbf3662a" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon May 19 01:15:14 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Mon May 19 17:51:48 2008 -0400" }, "message": "libata: ignore SIMG4726 config pseudo device\n\nI was hoping ATA_HORKAGE_NODMA | ATA_HORKAGE_SKIP_PM could keep it\nhappy but no even this doesn\u0027t work under certain configurations and\nit\u0027s not like we can do anything useful with the cofig device anyway.\nReplace ATA_HORKAGE_SKIP_PM with ATA_HORKAGE_DISABLE and use it for\nthe config device. This makes the device completely ignored by\nlibata.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "bf1bff6fa9fdd4e92e57d80a5434fd5201c051fc", "tree": "0ebd09fbf4e7f9f5e81e479685930c180eaece76", "parents": [ "e0614db2a398d4d0dc5fb47fe2c2783141262a3e" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon May 19 01:15:10 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Mon May 19 17:51:47 2008 -0400" }, "message": "libata: increase PMP register access timeout to 3s\n\nThis timeout was set low because previously PMP register access was\ndone via polling and register access timeouts could stack up. This is\nno longer the case. One timeout will make all following accesses fail\nimmediately.\n\nIn rare cases both marvell and SIMG PMPs need almost a second. Bump\nit to 3s.\n\nWhile at it, rename it to SATA_PMP_RW_TIMEOUT. It\u0027s not specific to\nSCR access.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "07633b5d0723ce2ec31262e1096dcf61311bf078", "tree": "d7df0c597f64b23d684e9cc492c8ee803acf21a2", "parents": [ "06aaca3f6301d04463b1ee0eb75c0352147159f2" ], "author": { "name": "Harvey Harrison", "email": "harvey.harrison@gmail.com", "time": "Wed May 14 16:17:00 2008 -0700" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Mon May 19 17:30:32 2008 -0400" }, "message": "ata: remove FIT() macro\n\nUse the kernel-provided clamp_val() macro.\n\nFIT was always applied to a member of struct ata_timing (unsigned short)\nand two constants. clamp_val will not cast to short anymore.\n\nSigned-off-by: Harvey Harrison \u003charvey.harrison@gmail.com\u003e\nCc: Jeff Garzik \u003cjeff@garzik.org\u003e\nCc: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "005b1f7495e812b99b73de5adbc73afd7a1cbcaf", "tree": "d0ee13a35bbda1f53577db2a79eb288c71f3e6c2", "parents": [ "28a4acb48586dc21d2d14a75a7aab7be78b7c83b" ], "author": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Fri May 09 15:00:55 2008 -0400" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Fri May 09 15:00:55 2008 -0400" }, "message": "[libata] revert new check-ready Status register logic\n\nThis behavior differs across multiple controllers, so we cannot use\ncommon logic for all controllers.\n\nRevert back to the basic common behavior, and specific drivers will\nbe updated from here to take into account the unusual Status return\nvalues.\n\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "10acf3b0d3b46c6ef5d6f0722f72ad9b743ea848", "tree": "08aec05645770a3c0b90fb083c2e67e6aab4d572", "parents": [ "a90103298fd5ccd9a9df6d47bde9a3f371707037" ], "author": { "name": "Mark Lord", "email": "liml@rtr.ca", "time": "Fri May 02 02:14:53 2008 -0400" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue May 06 11:37:58 2008 -0400" }, "message": "libata: export ata_eh_analyze_ncq_error\n\nExport ata_eh_analyze_ncq_error() for subsequent use by sata_mv,\nas suggested by Tejun.\n\nSigned-off-by: Mark Lord \u003cmlord@pobox.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "78ab88f04f44bed566d51dce0c7cbfeff6449a06", "tree": "f60d9ebf37fca7af191cc16665c9025bc5cf56f4", "parents": [ "a15306365a16380f3bafee9e181ba01231d4acd7" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Thu May 01 23:41:41 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue May 06 11:32:02 2008 -0400" }, "message": "libata: improve post-reset device ready test\n\nSome controllers (jmb and inic162x) use 0x77 and 0x7f to indicate that\nthe device isn\u0027t ready yet. It looks like they use 0xff if device\npresence is detected but connection isn\u0027t established. 0x77 or 0x7f\nafter connection is established and use the value from signature FIS\nafter receiving it.\n\nThis patch implements ata_check_ready(), which takes TF status value\nand determines whether the port is ready or not considering the above\nand other conditions, and use it in @check_ready() functions. This is\nsafe as both 0x77 and 0x7f aren\u0027t valid ready status value even though\nthey have BSY bit cleared.\n\nThis fixes hot plug detection failures which can be triggered with\ncertain drives if they aren\u0027t already spun up when the data connector\nis hot plugged.\n\nTested on sil, sil24, ahci (jmb/ich), piix and inic162x combined with\neight drives from all major vendors.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "f7e989301b6c232dec5489e94ee7741c85cb11ba", "tree": "c92a575b14714ae2f57edb6b5f5e0df5b409b453", "parents": [ "22b5e7a74280deae560c20ee1a9b502b35181327" ], "author": { "name": "Jeff Garzik", "email": "jeff@garzik.org", "time": "Tue Apr 29 17:47:34 2008 -0400" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Tue Apr 29 17:47:34 2008 -0400" }, "message": "[libata] linux/libata.h: reorganize ata_device struct members a bit\n\nPut the big stuff at the end, to prepare for upcoming changes (and\nalso hopefully achieve nicer packing of remaining members).\n\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "6bdb4fc9f9e5307012f6f2afb8642b52dad9c186", "tree": "32de6cd571b74e74374c42c30c1fc241c42eb2e3", "parents": [ "66a9099e02e3fca5198ab52b4bb7088f03dee42e" ], "author": { "name": "Adrian Bunk", "email": "bunk@kernel.org", "time": "Mon Apr 21 11:51:11 2008 +0300" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Fri Apr 25 00:46:09 2008 -0400" }, "message": "make sata_print_link_status() static\n\nsata_print_link_status() can now become static.\n\nSigned-off-by: Adrian Bunk \u003cbunk@kernel.org\u003e\nSigned-off-by: Jeff Garzik \u003cjgarzik@redhat.com\u003e\n" }, { "commit": "ee959b00c335d7780136c5abda37809191fe52c3", "tree": "7775f3b274fd8caf5e7e5154fea89e96f2babd94", "parents": [ "56d110e852b0b1c85ad6c9bfe1cb4473ceb16402" ], "author": { "name": "Tony Jones", "email": "tonyj@suse.de", "time": "Fri Feb 22 00:13:36 2008 +0100" }, "committer": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Sat Apr 19 19:10:33 2008 -0700" }, "message": "SCSI: convert struct class_device to struct device\n\nIt\u0027s big, but there doesn\u0027t seem to be a way to split it up smaller...\n\nSigned-off-by: Tony Jones \u003ctonyj@suse.de\u003e\nSigned-off-by: Kay Sievers \u003ckay.sievers@vrfy.org\u003e\nCc: Roland Dreier \u003crolandd@cisco.com\u003e\nCc: Sean Hefty \u003csean.hefty@intel.com\u003e\nCc: Hal Rosenstock \u003chal.rosenstock@gmail.com\u003e\nCc: James Bottomley \u003cJames.Bottomley@HansenPartnership.com\u003e\nSigned-off-by: Greg Kroah-Hartman \u003cgregkh@suse.de\u003e\n\n" }, { "commit": "88fcd5627563722483427a55113c0a83f56e8080", "tree": "6a596e33d6497fbed58147e8fdb59b611d956c12", "parents": [ "071f44b1d2c051641b62a3571223314737ccbe59" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:22 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:25 2008 -0400" }, "message": "libata: make PMP support optional\n\nMake PMP support optional by adding CONFIG_SATA_PMP and leaving out\nlibata-pmp.c if it isn\u0027t set. PMP helpers return constant values if\nPMP support is not enabled and PMP declarations alias non-PMP\ncounterparts. This makes the compiler to leave out PMP related part\nout and LLDs to use non-PMP counterparts automatically.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "071f44b1d2c051641b62a3571223314737ccbe59", "tree": "a07794c8109e5d82a78223ae0159eadbf862c463", "parents": [ "48515f6c006c2a9d7b624ee8ad068018c2d3fe0e" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:22 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:25 2008 -0400" }, "message": "libata: implement PMP helpers\n\nImplement helpers to test whether PMP is supported, attached and\ndetermine pmp number to use when issuing SRST to a link. While at it,\nmove ata_is_host_link() so that it\u0027s together with the two new PMP\nhelpers.\n\nThis change simplifies LLDs and helps making PMP support optional.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "48515f6c006c2a9d7b624ee8ad068018c2d3fe0e", "tree": "0ade033552ccb297ac7736a6b2d77dd249be4eb8", "parents": [ "127102aea2ea9ec4e9ca233e2b1a75c8d3b058c4" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:21 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:25 2008 -0400" }, "message": "libata: separate PMP support code from core code\n\nMost of PMP support code is already in libata-pmp.c. All that are in\nlibata-core.c are sata_pmp_port_ops and EXPORTs. Move them to\nlibata-pmp.c. Also, collect PMP related prototypes and declarations\nin header files and move them right above of SFF stuff.\n\nThis change is to make PMP support optional.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "127102aea2ea9ec4e9ca233e2b1a75c8d3b058c4", "tree": "92fb528eac658adbc7307a009bd8a7558db2658a", "parents": [ "350756f6dab6d37ef9ed3f18dec520e88969ddac" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:21 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:24 2008 -0400" }, "message": "libata: make SFF support optional\n\nNow that SFF support is completely separated out from the core layer,\nit can be made optional. Add CONFIG_ATA_SFF and let SFF drivers\ndepend on it. If CONFIG_ATA_SFF isn\u0027t set, all codes in libata-sff.c\nand data structures for SFF support are disabled. This saves good\nnumber of bytes for small systems.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "c9f75b04ed5ed65a058d18a8a8dda50632a96de8", "tree": "e0100eae3b10006a8618ddb32bdf971e20db0dd8", "parents": [ "520d06f92b32d7abe5127d7cc46a819db0f384e6" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:21 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:24 2008 -0400" }, "message": "libata: kill ata_noop_dev_select()\n\nNow that SFF assumptions are separated out from non-SFF reset\nsequence, port_ops-\u003esff_dev_select() is no longer necessary for\nnon-SFF controllers. Kill ata_noop_dev_select() and -\u003esff_dev_select\ninitialization from base and other non-SFF port_ops.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "79f97dadfe9b4b561634d202225ba2fa910dc225", "tree": "7bea39a135c7c87b7430d6c071288a05990b9309", "parents": [ "22183bf569c8600ff414ac25f23134044e0ef453" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:20 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:23 2008 -0400" }, "message": "libata: drop @finish_qc from ata_qc_complete_multiple()\n\nata_qc_complete_multiple() took @finish_qc and called it on every qc\nbefore completing it. This was to give opportunity to update TF cache\nbefore ata_qc_complete() tries to fill result_tf. Now that result TF\nis a separate operation, this is no longer necessary.\n\nUpdate sata_sil24, which was the only user of this mechanism, such\nthat it implements its own ops-\u003eqc_fill_rtf() and drop @finish_qc from\nata_qc_complete_multiple().\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "22183bf569c8600ff414ac25f23134044e0ef453", "tree": "1dea79cdbe439637477e1c4214b600b6823cb866", "parents": [ "305d2a1ab137d11d573319c315748a87060fe82d" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:20 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:23 2008 -0400" }, "message": "libata: add qc_fill_rtf port operation\n\nOn command completion, ata_qc_complete() directly called ops-\u003etf_read\nto fill qc-\u003eresult_tf. This patch adds ops-\u003eqc_fill_rtf to replace\nhardcoded ops-\u003etf_read usage.\n\nata_sff_qc_fill_rtf() which uses ops-\u003etf_read to fill result_tf is\nimplemented and set in ata_base_port_ops and other ops tables which\ndon\u0027t inherit from ata_base_port_ops, so this patch doesn\u0027t introduce\nany behavior change.\n\nops-\u003eqc_fill_rtf() is similar to ops-\u003esff_tf_read() but can only be\ncalled when a command finishes. As some non-SFF controllers don\u0027t\nhave TF registers defined unless they\u0027re associated with in-flight\ncommands, this limited operation makes life easier for those drivers\nand help lifting SFF assumptions from libata core layer.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "5958e3025fd9d97429163e074d9cfa3848f51f28", "tree": "f20492dc35fc2651452c871fbc812bd4f8379b40", "parents": [ "ac371987a81c61c2efbd6931245cdcaf43baad89" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:20 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:23 2008 -0400" }, "message": "libata: move PMP SCR access failure during reset to ata_eh_reset()\n\nIf PMP fan-out reset fails and SCR isn\u0027t accessible, PMP should be\nreset. This used to be tested by sata_pmp_std_hardreset() and\ncommunicated to EH by -ERESTART. However, this logic is generic and\ndoesn\u0027t really have much to do with specific hardreset implementation.\n\nThis patch moves SCR access failure detection logic to ata_eh_reset()\nwhere it belongs. As this makes sata_pmp_std_hardreset() identical to\nsata_std_hardreset(), the function is killed and replaced with the\nstandard method.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "ac371987a81c61c2efbd6931245cdcaf43baad89", "tree": "f88970931b26d2ad344d7d67ddabc64d9b48181d", "parents": [ "57c9efdfb3cee5d4564fcb5f70555e2edb1bc52a" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:19 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:23 2008 -0400" }, "message": "libata: clear SError after link resume\n\nSError used to be cleared in -\u003epostreset. This has small hotplug race\ncondition. If a device is plugged in after reset is complete but\npostreset hasn\u0027t run yet, its hotplug event gets lost when SError is\ncleared. This patch makes sata_link_resume() clear SError. This\nkills the race condition and makes a lot of sense as some PMP and host\nPHYs don\u0027t work properly without SError cleared.\n\nThis change makes sata_pmp_std_{pre|post}_reset()\u0027s unnecessary as\nthey become identical to ata_std counterparts. It also simplifies\nsata_pmp_hardreset() and ahci_vt8251_hardreset().\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "57c9efdfb3cee5d4564fcb5f70555e2edb1bc52a", "tree": "c2289500f093736853a94d2d9577036658676498", "parents": [ "9dadd45b24145d6aee2fabb28d7aef972301892b" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:19 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:23 2008 -0400" }, "message": "libata: implement and use sata_std_hardreset()\n\nImplement sata_std_hardreset(), which simply wraps around\nsata_link_hardreset(). sata_std_hardreset() becomes new standard\nhardreset method for sata_port_ops and sata_sff_hardreset() moves from\nata_base_port_ops to ata_sff_port_ops, which is where it really\nbelongs.\n\nata_is_builtin_hardreset() is added so that both\nata_std_error_handler() and ata_sff_error_handler() skip both builtin\nhardresets if SCR isn\u0027t accessible.\n\npiix_sidpr_hardreset() in ata_piix.c is identical to\nsata_std_hardreset() in functionality and got replaced with the\nstandard function.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "9dadd45b24145d6aee2fabb28d7aef972301892b", "tree": "c97c323e2edd400bc94eaceddf20f84e9a6da005", "parents": [ "a89611e8489ac24f371c9fd6fef6605b170b16ba" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:19 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:22 2008 -0400" }, "message": "libata: move generic hardreset code from sata_sff_hardreset() to sata_link_hardreset()\n\nsata_sff_hardreset() contains link readiness wait logic which isn\u0027t\nSFF specific. Move that part into sata_link_hardreset(), which now\ntakes two more parameters - @online and @check_ready. Both are\noptional. The former is out parameter for link onlineness after\nreset. The latter is used to wait for link readiness after hardreset.\n\nUsers of sata_link_hardreset() is updated to use new funtionality and\nahci_hardreset() is updated to use sata_link_hardreset() instead of\nsata_sff_hardreset(). This doesn\u0027t really cause any behavior change.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "aa2731ad9ad80ac3fca48bd1c4cf0eceede4810e", "tree": "db6d3327017b9b84355aba49a5ccd5c322d3f248", "parents": [ "705e76beb90b97421e1f61e857c4246799781bb5" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:19 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:22 2008 -0400" }, "message": "libata: separate out ata_wait_ready() and implement ata_wait_after_reset()\n\nFactor out waiting logic (which is common to all ATA controllers) from\nata_sff_wait_ready() into ata_wait_ready(). ata_wait_ready() takes\n@check_ready function pointer and uses it to poll for readiness. This\nallows non-SFF controllers to use ata_wait_ready() to wait for link\nreadiness.\n\nThis patch also implements ata_wait_after_reset() - generic version of\nata_sff_wait_after_reset() - using ata_wait_ready().\n\nata_sff_wait_ready() is reimplemented using ata_wait_ready() and\nata_sff_check_ready(). Functionality remains the same.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" }, { "commit": "705e76beb90b97421e1f61e857c4246799781bb5", "tree": "e571ad9229d469cd73d1388c76823922400823d5", "parents": [ "203c75b8245c5386044721d9c5eda5c6b71b3d14" ], "author": { "name": "Tejun Heo", "email": "htejun@gmail.com", "time": "Mon Apr 07 22:47:19 2008 +0900" }, "committer": { "name": "Jeff Garzik", "email": "jgarzik@redhat.com", "time": "Thu Apr 17 15:44:22 2008 -0400" }, "message": "libata: restructure SFF post-reset readiness waits\n\nPreviously, post-softreset readiness is waited as follows.\n\n1. ata_sff_wait_after_reset() waits for 150ms and then for\n ATA_TMOUT_FF_WAIT if status is 0xff and other conditions meet.\n\n2. ata_bus_softreset() finishes with -ENODEV if status is still 0xff.\n If not, continue to #3.\n\n3. ata_bus_post_reset() waits readiness of dev0 and/or dev1 depending\n on devmask using ata_sff_wait_ready().\n\nAnd for post-hardreset readiness,\n\n1. ata_sff_wait_after_reset() waits for 150ms and then for\n ATA_TMOUT_FF_WAIT if status is 0xff and other conditions meet.\n\n2. sata_sff_hardreset waits for device readiness using\n ata_sff_wait_ready().\n\nThis patch merges and unifies post-reset readiness waits into\nata_sff_wait_ready() and ata_sff_wait_after_reset().\n\nATA_TMOUT_FF_WAIT handling is merged into ata_sff_wait_ready(). If TF\nstatus is 0xff, link status is unknown and the port is SATA, it will\ncontinue polling till ATA_TMOUT_FF_WAIT.\n\nata_sff_wait_after_reset() is updated to perform the following steps.\n\n1. waits for 150ms.\n\n2. waits for dev0 readiness using ata_sff_wait_ready(). Note that\n this is done regardless of devmask, as ata_sff_wait_ready() handles\n 0xff status correctly, this preserves the original behavior except\n that it may wait longer after softreset if link is online but\n status is 0xff. This behavior change is very unlikely to cause any\n actual difference and is intended. It brings softreset behavior to\n that of hardreset.\n\n3. waits for dev1 readiness just the same way ata_bus_post_reset() did.\n\nNow both soft and hard resets call ata_sff_wait_after_reset() after\nreset to wait for readiness after resets. As\nata_sff_wait_after_reset() contains calls to -\u003esff_dev_select(),\nexplicit call near the end of sata_sff_hardreset() is removed.\n\nThis change makes reset implementation simpler and more consistent.\n\nWhile at it, make the magical 150ms wait post-reset wait duration a\nconstant and ata_sff_wait_ready() and ata_sff_wait_after_reset() take\n@link instead of @ap. This is to make them consistent with other\nreset helpers and ease core changes.\n\npata_scc is updated accordingly.\n\nSigned-off-by: Tejun Heo \u003chtejun@gmail.com\u003e\n" } ], "next": "203c75b8245c5386044721d9c5eda5c6b71b3d14" }