FreeBSD source tree https://cgit-dev.freebsd.org/src-test2/atom?h=release%2F8.1.0 2009-12-20T01:38:01Z 2009-12-20T01:38:01Z Marius Strobl marius@FreeBSD.org 2009-12-20T01:38:01Z urn:sha1:3759df8ec67b85fb3ac5d3df8fef992dc9b4bfa3 Specify the capability and media bits of the capabilities page in native, i.e. big-endian, format and convert as appropriate like we also do with the multibyte fields of the other pages. This fixes the output of acd_describe() to match reality on big-endian machines without breaking it on little-endian ones. While at it, also convert the remaining multibyte fields of the pages read although they are currently unused for consistency and in order to prevent possible similar bugs in the future. 2009-12-12T10:37:31Z Alexander Motin mav@FreeBSD.org 2009-12-12T10:37:31Z urn:sha1:71e7360ed94d5ffb14ab957e0bfb620760c841e7 Introduce ATA_CAM kernel option, turning ata(4) controller drivers into cam(4) interface modules. When enabled, this option deprecates all ata(4) peripheral drivers (ad, acd, ...) and interfaces and allows cam(4) drivers (ada, cd, ...) and interfaces to be natively used instead. As side effect of this, ata(4) mode setting code was completely rewritten to make controller API more strict and permit above change. While doing this, SATA revision was separated from PATA mode. It allows DMA-incapable SATA devices to operate and makes hw.ata.(ata|atapi)_dma tunable work again. Also allow ata(4) controller drivers (except some specific or broken ones) to handle larger data transfers. Previous constraint of 64K was artificial and is not really required by PCI ATA BM specification or hardware. Submitted by: nwitehorn (powerpc part) 2009-11-17T12:18:14Z Alexander Motin mav@FreeBSD.org 2009-11-17T12:18:14Z urn:sha1:ba0e41c3ccb4c019c11e1019a5a5bd924204793e Don't ignore the return value of g_modevent() in acd_modevent(). 2009-11-10T22:37:44Z Alexander Motin mav@FreeBSD.org 2009-11-10T22:37:44Z urn:sha1:58ad43feb10d9c06e68659be993cdd5a01d2cdc3 Increase ATA command timeouts. Introduce define and kernel option ATA_REQUEST_TIMEOUT to control it. PR: kern/111023 2009-11-06T10:48:44Z Alexander Motin mav@FreeBSD.org 2009-11-06T10:48:44Z urn:sha1:cdcdbef48e088722f4a75d823f1542b45e534bc8 Do not differentiate 12/16 bytes ATAPI CCB formats when it is not needed. 2009-07-08T05:56:14Z Marcel Moolenaar marcel@FreeBSD.org 2009-07-08T05:56:14Z urn:sha1:f43b57e32a9b8b21482eba2b3c870270f7556c2f is invalid because the ioctl happens without prior open. The ioctl got introduced to provide backward compatibility for extended partitions, but it ended up not being used because it didn't work as expected. Since there are no consumers of the ioctl and the implementation is broken, the best fix is to remove the code entirely. Spotted by: phk Approved by: re (kensmith) 2009-02-28T22:07:15Z Alexander Motin mav@FreeBSD.org 2009-02-28T22:07:15Z urn:sha1:ea74abd5f54699f69263b2cce18f3dd1dff23c23 and partially r188903. Revert breaks new drives detection on reinit to the state as it was before me, but fixes series of new bugs reported by some people. Unconditional queueing of ata_completed() calls can lead to deadlock if due to timeout ata_reinit() was called at the same thread by previous ata_completed(). Calling of ata_identify() on ata_reinit() in current implementation opens numerous races and deadlocks. Problems I was touching here are still exist and should be addresed, but probably in different way. 2009-02-28T11:25:05Z Alexander Motin mav@FreeBSD.org 2009-02-28T11:25:05Z urn:sha1:84b59262f102e564cafed54b3f4f2a9a893b04dc drivers' probe routines. It allows not to sleep and so not drop Giant inside ata_identify() critical section and so avoid crash if it reentered on request timeout. Reentering of probe call checked inside of it. Give device own knowledge about it's type (ata/atapi/atapicam). It is not a good idea to ask channel status for device type inside ata_getparam(). Add softc memory deallocation on device destruction. 2009-02-21T16:39:26Z Alexander Motin mav@FreeBSD.org 2009-02-21T16:39:26Z urn:sha1:7c84b0bd945e07aed5a4ba62b8c5944150b75dd8 devices per channel. 2009-02-20T23:01:27Z Marcel Moolenaar marcel@FreeBSD.org 2009-02-20T23:01:27Z urn:sha1:2571133f87fded303ffe1a81c0f02be9fe619ed8 acd code to create new provider (?), but not from an expected context, which results in a panic (GEOM topology not held).