Hi Shem, "Shem Multinymous" <multinymous@xxxxxxxxx> wrote: > Hi Elias, > > On 10/9/06, Elias Oltmanns <oltmanns@xxxxxxxxxxx> wrote: > >> A slightly stripped version of the patch is available too, which has >> been verified to trigger the described problem in exactly the same way >> as the original but lacks the IDLE IMMEDIATE feature (leaving the >> STANDBY IMMEDIATE option only) in order to make it (hopefully) more >> readable and easier to understand. > > What happens if you strip away *all* the head parking code, leaving > only the queue freeze code? Conversely, what happens if you issue the > head park command but don't freeze the queue? In all setups the loop # while true; do hdparm -q -y /dev/hda; hdparm -q -C /dev/hda; done didn't trigger the freeze. This should be very close to the effect of the patch with all queue freezing stuff removed. Regarding your question, it seems rather difficult to test the right thing. Just issuing the commands and leaving the queue alone is easy to implement and I couldn't reproduce the problem running such a kernel. Testing the queue handling stuff without actually issuing any commands seems rather difficult to me as its the callback mechanism used to freeze the queue after command completion which I'd really like to test. If I don't issue any command, I don't know how to test whether the callback procedure and all the rest works as expected. Regards, Elias - To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
