src/sys/dev/ata/atapi-cd.c, branch releng/8.0

Revert revisions 188839 and 188868. Use of the ioctl in geom_dev.c

2009-07-08T05:56:14+00:00

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)

Revert my ata_identify()/ata_reinit() related changes: r189166, r189091

2009-02-28T22:07:15+00:00

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.

Rework device probing by moving ata_getparam() call from ata_identify() to

2009-02-28T11:25:05+00:00

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.

Teach device drivers' ata_reinit() methods, that there can be more then two

2009-02-21T16:39:26+00:00

devices per channel.

Don't read the TOC for DIOCGPROVIDERALIAS. This causes the

2009-02-20T23:01:27+00:00

acd code to create new provider (?), but not from an expected
context, which results in a panic (GEOM topology not held).

Fix shutdown routine to return 0 and change signature from void return

2009-02-04T20:26:27+00:00

to int.

Fix panic and breakage for non-DMA ATA devices e.g. powermac macio cells.

2008-05-08T17:55:44+00:00

Handle cases where dma function pointers may be NULL, and where
the max_iosize can't be derived from a DMA data structure. For
the latter, revert to the prior behaviour of using DFLTPHYS for
the max i/o size when there is no other data.

Reviewed by:		marcel
No objection by:	sos

Go back to preallocating everything possible on init.

2008-04-17T12:29:35+00:00

This avoids calling busdma in the request processing path which caused a traumatic performance degradation.
Allocation has be postponed to after we know how many devices we possible can have on portmulitpliers to save some space.

Fix the brokenness in the former commit, sorry for the mess.

2008-04-11T11:30:27+00:00

The problem is that the PM support is part of a much larger WIP here, but due to popular demand I decided to get some of it imported.

Also I forgot the mention:

HW sponsored by: Vitsch Electronics / VEHosting

Add experimental support for SATA Port Multipliers

2008-04-10T13:05:05+00:00

Support is working on the Silicon Image SiI3124/3132.
Support is working on some AHCI chips but far from all.

Remember this is WIP, so test reports and (constructive) suggestions are welcome!