Hi, On Friday 09 March 2007, Uwe Bugla wrote: > Hello Bart, hello everybody, > > As requested I applied the following patch with parameter -R against 2.6.20: > http://kernel.org/pub/linux/kernel/people/bart/ide-2.6.20.patch > > Result: incompilable kernel! Reason is the following hunk: > > 99,113d98 > < diff --git a/drivers/ide/ide-probe.c b/drivers/ide/ide-probe.c > < index dad9c47..5a5c565 100644 > < --- a/drivers/ide/ide-probe.c > < +++ b/drivers/ide/ide-probe.c > < @@ -1000,10 +1000,6 @@ static int ide_init_queue(ide_drive_t *drive) > < /* needs drive->queue to be set */ > < ide_toggle_bounce(drive, 1); > < > < - /* enable led activity for disk drives only */ > < - if (drive->media == ide_disk && hwif->led_act) > < - blk_queue_activity_fn(q, hwif->led_act, drive); > < - > < return 0; > < } > < > > Would you please care for proper patch sets in future, just for efficiency reasons? Doh, I missed the chunk of the block layer patch, sorry for that. > Side note: One diode of my TEAC CDRW54 (/dev/hdd) flickers every 2 seconds. > That's normal and it does not do any harm. > > > I ripped out this hunk and applied the resulting patch > collection with parameter -R against 2.6.20. > Results: > a. compilable kernel > b. SUCCESS after endless hours of testing! 2.6.20 "good" kernel has newer build date then the "bad" 2.6.20.1 test kernel (with the IDE patchset reversed)? > 3 conclusions from that: > > 1. The buggy code responsible for the horrible reinitialization > and drive seek errors with my TEAC CDRW54 as /dev/hdd > is proven to reside in the patch collection mentioned above. Nope, unfortunately this only proves that all 2.6.20 changes modulo IDE changes doesn't seem to cause the problems. "Unfortunately" because it would be much easier for me to just find guilty change in the patchset, take the blame and fix it :-) but... There are only 6 changes in the patchset which influence the _code_ that is used by your hardware/driver configuration and none seems to actually influence the driver _behavior_ on your system: * [PATCH] ide-cd: Handle strange interrupt on the Intel ESB2 the quirk code is not executed on ICH4 chipset * [PATCH] PIIX: remove check for broken MW DMA mode 0 dead code removal * [PATCH] PIIX/SLC90E66: PIO mode fallback fix not used on your hardware as both hdc and hdd use DMA modes * [PATCH] remove blk_queue_activity_fn dead code removal * ide: unregister idepnp driver on unload ide-pnp is not used on this system * [PATCH] lockdep: fix ide/proc interaction shouldn't change anything - the code is executed only during device driver (ide-cd etc.) removal Besides the PCI configuration for IDE chipset is the same for both kernels. Something really fishy is going on here... I'm out of ideas for the moment. Maybe except retesting 2.6.19 + ide-2.6.20.patch applied... Thanks, Bart - 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
