* Testing IDE-TCQ and Taskfile - doesn't work nicely:) @ 2003-06-23 19:45 Jan-Benedict Glaw 2003-06-23 21:18 ` Bartlomiej Zolnierkiewicz 0 siblings, 1 reply; 23+ messages in thread From: Jan-Benedict Glaw @ 2003-06-23 19:45 UTC (permalink / raw) To: linux-kernel [-- Attachment #1: Type: text/plain, Size: 4163 bytes --] Hi! I've played a bit with my "mirror" machine - 200MHz Pentium-MMX - 64MB RAM - jbglaw@mirror:~$ cat /proc/ide/hd*/model WDC AC2850F # System drive IC35L040AVER07-0 # \ IC35L120AVV207-0 # > Storage (with DM/LVM) Maxtor 4W100H6 # / - jbglaw@mirror:~$ lspci |grep IDE 00:07.1 IDE interface: Intel Corp. 82371AB/EB/MB PIIX4 IDE (rev 01) - Linux v2.5.73 - <*> Intel PIIXn chipsets support Basically, if I enable Taskfile I/O, the box refuses to boot (basically, the first drive sounds really broken like "clack clack clack" and no data comes off the drive so there's no partition table, no root FS, but a panic:) Is anybody interested in nailing this bug down? Disabling Taskfile lets me boot the box, but hdc doesn't like TCQ and gives errors: ide_tcq_intr_timeout: timeout waiting for service interrupt ide_tcq_intr_timeout: missing isr! hdc: invalidating tag queue (0 commands) hdc: drive_cmd: status=0x51 { DriveReady SeekComplete Error } hdc: drive_cmd: error=0x04 { DriveStatusError } [above messages repeat...] IDE's boot log: Uniform Multi-Platform E-IDE driver Revision: 7.00alpha2 ide: Assuming 33MHz system bus speed for PIO modes; override with idebus=xx PIIX4: IDE controller at PCI slot 00:07.1 PIIX4: chipset revision 1 PIIX4: not 100% native mode: will probe irqs later ide0: BM-DMA at 0xf000-0xf007, BIOS settings: hda:pio, hdb:pio ide1: BM-DMA at 0xf008-0xf00f, BIOS settings: hdc:pio, hdd:pio hda: WDC AC2850F, ATA DISK drive hdb: IC35L040AVER07-0, ATA DISK drive hda: tcq forbidden by blacklist hdb: only one drive on a channel supported for tcq ide0 at 0x1f0-0x1f7,0x3f6 on irq 14 hdc: IC35L120AVV207-0, ATA DISK drive hdd: Maxtor 4W100H6, ATA DISK drive hdc: tagged command queueing enabled, command queue depth 32 hdd: only one drive on a channel supported for tcq ide1 at 0x170-0x177,0x376 on irq 15 hda: max request size: 128KiB hda: task_no_data_intr: status=0x51 { DriveReady SeekComplete Error } hda: task_no_data_intr: error=0x04 { DriveStatusError } hda: 1667232 sectors (854 MB) w/64KiB Cache, CHS=1654/16/63, DMA hda: hda1 hda2 hda3 hdb: max request size: 128KiB hdb: host protected area => 1 hdb: 80418240 sectors (41174 MB) w/1916KiB Cache, CHS=79780/16/63, UDMA(33) hdb: hdb1 hdb2 hdb3 hdb4 hdc: max request size: 1024KiB hdc: host protected area => 1 hdc: 241254720 sectors (123522 MB) w/1821KiB Cache, CHS=15017/255/63, UDMA(33) hdc: hdc1 < hdc5 hdc6 hdc7 hdc8 hdc9 hdc10 hdc11 hdc12 hdc13 hdc14 hdc15 hdc16 hdc17 > hdd: max request size: 128KiB hdd: host protected area => 1 hdd: 195711264 sectors (100204 MB) w/2048KiB Cache, CHS=194158/16/63, UDMA(33) hdd: hdd1 < hdd5 hdd6 hdd7 hdd8 hdd9 hdd10 hdd11 hdd12 hdd13 hdd14 > IDE settings: mirror:~# lspci -xxx -v -d 8086:7111 00:07.1 IDE interface: Intel Corp. 82371AB/EB/MB PIIX4 IDE (rev 01) (prog-if 80 [Master]) Flags: bus master, medium devsel, latency 64 I/O ports at f000 [size=16] 00: 86 80 11 71 05 00 80 02 01 80 01 01 00 40 00 00 10: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 20: 01 f0 00 00 00 00 00 00 00 00 00 00 00 00 00 00 30: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 40: 77 e1 77 e3 bb 00 00 00 0e 00 20 22 00 00 00 00 50: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 60: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 70: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 80: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 90: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 a0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 b0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 c0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 d0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 e0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 f0: 00 00 00 00 00 00 00 00 28 0f 00 00 00 00 00 00 MfG, JBG -- Jan-Benedict Glaw jbglaw@lug-owl.de . +49-172-7608481 "Eine Freie Meinung in einem Freien Kopf | Gegen Zensur | Gegen Krieg fuer einen Freien Staat voll Freier Bürger" | im Internet! | im Irak! ret = do_actions((curr | FREE_SPEECH) & ~(IRAQ_WAR_2 | DRM | TCPA)); [-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --] ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-23 19:45 Testing IDE-TCQ and Taskfile - doesn't work nicely:) Jan-Benedict Glaw @ 2003-06-23 21:18 ` Bartlomiej Zolnierkiewicz 2003-06-23 22:19 ` Jan-Benedict Glaw ` (3 more replies) 0 siblings, 4 replies; 23+ messages in thread From: Bartlomiej Zolnierkiewicz @ 2003-06-23 21:18 UTC (permalink / raw) To: Jan-Benedict Glaw; +Cc: axboe, linux-kernel [-- Attachment #1: Type: TEXT/PLAIN, Size: 4646 bytes --] On Mon, 23 Jun 2003, Jan-Benedict Glaw wrote: > Hi! > > I've played a bit with my "mirror" machine > > - 200MHz Pentium-MMX > - 64MB RAM > - jbglaw@mirror:~$ cat /proc/ide/hd*/model > WDC AC2850F # System drive > IC35L040AVER07-0 # \ > IC35L120AVV207-0 # > Storage (with DM/LVM) > Maxtor 4W100H6 # / > - jbglaw@mirror:~$ lspci |grep IDE > 00:07.1 IDE interface: Intel Corp. 82371AB/EB/MB PIIX4 IDE (rev 01) > - Linux v2.5.73 > - <*> Intel PIIXn chipsets support > > Basically, if I enable Taskfile I/O, the box refuses to boot (basically, > the first drive sounds really broken like "clack clack clack" and no > data comes off the drive so there's no partition table, no root FS, but > a panic:) Is anybody interested in nailing this bug down? YES > Disabling Taskfile lets me boot the box, but hdc doesn't like TCQ and > gives errors: > > ide_tcq_intr_timeout: timeout waiting for service interrupt > ide_tcq_intr_timeout: missing isr! > hdc: invalidating tag queue (0 commands) > hdc: drive_cmd: status=0x51 { DriveReady SeekComplete Error } > hdc: drive_cmd: error=0x04 { DriveStatusError } > [above messages repeat...] TCQ shouldn't be enabled on hdc, you have two drives on second ide channel and current TCQ driver design allows only one drive per channel, so proper fix is to not enable TCQ :-). Can you try attached patch? Regards, -- Bartlomiej > IDE's boot log: > > Uniform Multi-Platform E-IDE driver Revision: 7.00alpha2 > ide: Assuming 33MHz system bus speed for PIO modes; override with idebus=xx > PIIX4: IDE controller at PCI slot 00:07.1 > PIIX4: chipset revision 1 > PIIX4: not 100% native mode: will probe irqs later > ide0: BM-DMA at 0xf000-0xf007, BIOS settings: hda:pio, hdb:pio > ide1: BM-DMA at 0xf008-0xf00f, BIOS settings: hdc:pio, hdd:pio > hda: WDC AC2850F, ATA DISK drive > hdb: IC35L040AVER07-0, ATA DISK drive > hda: tcq forbidden by blacklist > hdb: only one drive on a channel supported for tcq > ide0 at 0x1f0-0x1f7,0x3f6 on irq 14 > hdc: IC35L120AVV207-0, ATA DISK drive > hdd: Maxtor 4W100H6, ATA DISK drive > hdc: tagged command queueing enabled, command queue depth 32 > hdd: only one drive on a channel supported for tcq > ide1 at 0x170-0x177,0x376 on irq 15 > hda: max request size: 128KiB > hda: task_no_data_intr: status=0x51 { DriveReady SeekComplete Error } > hda: task_no_data_intr: error=0x04 { DriveStatusError } > hda: 1667232 sectors (854 MB) w/64KiB Cache, CHS=1654/16/63, DMA > hda: hda1 hda2 hda3 > hdb: max request size: 128KiB > hdb: host protected area => 1 > hdb: 80418240 sectors (41174 MB) w/1916KiB Cache, CHS=79780/16/63, UDMA(33) > hdb: hdb1 hdb2 hdb3 hdb4 > hdc: max request size: 1024KiB > hdc: host protected area => 1 > hdc: 241254720 sectors (123522 MB) w/1821KiB Cache, CHS=15017/255/63, UDMA(33) > hdc: hdc1 < hdc5 hdc6 hdc7 hdc8 hdc9 hdc10 hdc11 hdc12 hdc13 hdc14 hdc15 hdc16 hdc17 > > hdd: max request size: 128KiB > hdd: host protected area => 1 > hdd: 195711264 sectors (100204 MB) w/2048KiB Cache, CHS=194158/16/63, UDMA(33) > hdd: hdd1 < hdd5 hdd6 hdd7 hdd8 hdd9 hdd10 hdd11 hdd12 hdd13 hdd14 > > > IDE settings: > mirror:~# lspci -xxx -v -d 8086:7111 > 00:07.1 IDE interface: Intel Corp. 82371AB/EB/MB PIIX4 IDE (rev 01) (prog-if 80 [Master]) > Flags: bus master, medium devsel, latency 64 > I/O ports at f000 [size=16] > 00: 86 80 11 71 05 00 80 02 01 80 01 01 00 40 00 00 > 10: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > 20: 01 f0 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > 30: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > 40: 77 e1 77 e3 bb 00 00 00 0e 00 20 22 00 00 00 00 > 50: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > 60: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > 70: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > 80: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > 90: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > a0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > b0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > c0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > d0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > e0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > f0: 00 00 00 00 00 00 00 00 28 0f 00 00 00 00 00 00 > > MfG, JBG > > -- > Jan-Benedict Glaw jbglaw@lug-owl.de . +49-172-7608481 > "Eine Freie Meinung in einem Freien Kopf | Gegen Zensur | Gegen Krieg > fuer einen Freien Staat voll Freier Bürger" | im Internet! | im Irak! > ret = do_actions((curr | FREE_SPEECH) & ~(IRAQ_WAR_2 | DRM | TCPA)); [-- Attachment #2: Type: TEXT/PLAIN, Size: 2457 bytes --] [ide] TCQ initialization fixes - do not enable TCQ in ide_init_drive(), its too early - enable TCQ in __ide_dma_on() only if CONFIG_BLK_DEV_IDE_TCQ_DEFAULT=y drivers/ide/ide-dma.c | 8 ++++---- drivers/ide/ide-probe.c | 5 ----- include/linux/ide.h | 1 - 3 files changed, 4 insertions(+), 10 deletions(-) diff -puN drivers/ide/ide-probe.c~ide-tcq-init-fixes drivers/ide/ide-probe.c --- linux-2.5.73/drivers/ide/ide-probe.c~ide-tcq-init-fixes Mon Jun 23 23:10:11 2003 +++ linux-2.5.73-root/drivers/ide/ide-probe.c Mon Jun 23 23:10:11 2003 @@ -983,7 +983,6 @@ static void ide_init_queue(ide_drive_t * blk_init_queue(q, do_ide_request, &ide_lock); q->queuedata = HWGROUP(drive); - drive->queue_setup = 1; blk_queue_segment_boundary(q, 0xffff); if (!hwif->rqsize) @@ -1005,10 +1004,6 @@ static void ide_init_queue(ide_drive_t * static void ide_init_drive(ide_drive_t *drive) { ide_toggle_bounce(drive, 1); - -#ifdef CONFIG_BLK_DEV_IDE_TCQ_DEFAULT - HWIF(drive)->ide_dma_queued_on(drive); -#endif } /* diff -puN drivers/ide/ide-dma.c~ide-tcq-init-fixes drivers/ide/ide-dma.c --- linux-2.5.73/drivers/ide/ide-dma.c~ide-tcq-init-fixes Mon Jun 23 23:10:11 2003 +++ linux-2.5.73-root/drivers/ide/ide-dma.c Mon Jun 23 23:10:11 2003 @@ -523,8 +523,7 @@ int __ide_dma_off_quietly (ide_drive_t * if (HWIF(drive)->ide_dma_host_off(drive)) return 1; - if (drive->queue_setup) - HWIF(drive)->ide_dma_queued_off(drive); + HWIF(drive)->ide_dma_queued_off(drive); return 0; } @@ -585,8 +584,9 @@ int __ide_dma_on (ide_drive_t *drive) if (HWIF(drive)->ide_dma_host_on(drive)) return 1; - if (drive->queue_setup) - HWIF(drive)->ide_dma_queued_on(drive); +#ifdef BLK_DEV_IDE_TCQ_DEFAULT + HWIF(drive)->ide_dma_queued_on(drive); +#endif return 0; } diff -puN include/linux/ide.h~ide-tcq-init-fixes include/linux/ide.h --- linux-2.5.73/include/linux/ide.h~ide-tcq-init-fixes Mon Jun 23 23:10:11 2003 +++ linux-2.5.73-root/include/linux/ide.h Mon Jun 23 23:10:11 2003 @@ -726,7 +726,6 @@ typedef struct ide_drive_s { unsigned ata_flash : 1; /* 1=present, 0=default */ unsigned blocked : 1; /* 1=powermanagment told us not to do anything, so sleep nicely */ unsigned vdma : 1; /* 1=doing PIO over DMA 0=doing normal DMA */ - unsigned queue_setup : 1; unsigned addressing; /* : 3; * 0=28-bit * 1=48-bit _ ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-23 21:18 ` Bartlomiej Zolnierkiewicz @ 2003-06-23 22:19 ` Jan-Benedict Glaw 2003-06-23 23:50 ` Jan-Benedict Glaw ` (2 subsequent siblings) 3 siblings, 0 replies; 23+ messages in thread From: Jan-Benedict Glaw @ 2003-06-23 22:19 UTC (permalink / raw) To: Bartlomiej Zolnierkiewicz; +Cc: axboe, linux-kernel [-- Attachment #1: Type: text/plain, Size: 2156 bytes --] On Mon, 2003-06-23 23:18:00 +0200, Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl> wrote in message <Pine.SOL.4.30.0306232315480.8078-200000@mion.elka.pw.edu.pl>: > On Mon, 23 Jun 2003, Jan-Benedict Glaw wrote: > > I've played a bit with my "mirror" machine > > > > - 200MHz Pentium-MMX > > - 64MB RAM > > - jbglaw@mirror:~$ cat /proc/ide/hd*/model > > WDC AC2850F # System drive > > IC35L040AVER07-0 # \ > > IC35L120AVV207-0 # > Storage (with DM/LVM) > > Maxtor 4W100H6 # / > > - jbglaw@mirror:~$ lspci |grep IDE > > 00:07.1 IDE interface: Intel Corp. 82371AB/EB/MB PIIX4 IDE (rev 01) > > - Linux v2.5.73 > > - <*> Intel PIIXn chipsets support > > > > Basically, if I enable Taskfile I/O, the box refuses to boot (basically, > > the first drive sounds really broken like "clack clack clack" and no > > data comes off the drive so there's no partition table, no root FS, but > > a panic:) Is anybody interested in nailing this bug down? > > YES What can I do about this one? I'll try to capture the lines... Taskfile I/O seems to hit my drive(s) really hard:) > > Disabling Taskfile lets me boot the box, but hdc doesn't like TCQ and > > gives errors: > > > > ide_tcq_intr_timeout: timeout waiting for service interrupt > > ide_tcq_intr_timeout: missing isr! > > hdc: invalidating tag queue (0 commands) > > hdc: drive_cmd: status=0x51 { DriveReady SeekComplete Error } > > hdc: drive_cmd: error=0x04 { DriveStatusError } > > [above messages repeat...] > > TCQ shouldn't be enabled on hdc, you have two drives on second ide > channel and current TCQ driver design allows only one drive per channel, > so proper fix is to not enable TCQ :-). > > Can you try attached patch? I'll rebuild with your patch and give it a try (TCQ enabled, Taskfile disabled...). MfG, JBG -- Jan-Benedict Glaw jbglaw@lug-owl.de . +49-172-7608481 "Eine Freie Meinung in einem Freien Kopf | Gegen Zensur | Gegen Krieg fuer einen Freien Staat voll Freier Bürger" | im Internet! | im Irak! ret = do_actions((curr | FREE_SPEECH) & ~(IRAQ_WAR_2 | DRM | TCPA)); [-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --] ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-23 21:18 ` Bartlomiej Zolnierkiewicz 2003-06-23 22:19 ` Jan-Benedict Glaw @ 2003-06-23 23:50 ` Jan-Benedict Glaw 2003-06-24 0:53 ` Bartlomiej Zolnierkiewicz 2003-06-24 8:06 ` Jens Axboe 2003-06-24 14:28 ` Jeff Garzik 3 siblings, 1 reply; 23+ messages in thread From: Jan-Benedict Glaw @ 2003-06-23 23:50 UTC (permalink / raw) To: Bartlomiej Zolnierkiewicz; +Cc: axboe, linux-kernel [-- Attachment #1: Type: text/plain, Size: 3647 bytes --] On Mon, 2003-06-23 23:18:00 +0200, Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl> wrote in message <Pine.SOL.4.30.0306232315480.8078-200000@mion.elka.pw.edu.pl>: > On Mon, 23 Jun 2003, Jan-Benedict Glaw wrote: > > I've played a bit with my "mirror" machine > > > > - 200MHz Pentium-MMX > > - 64MB RAM > > - jbglaw@mirror:~$ cat /proc/ide/hd*/model > > WDC AC2850F # System drive > > IC35L040AVER07-0 # \ > > IC35L120AVV207-0 # > Storage (with DM/LVM) > > Maxtor 4W100H6 # / > > - jbglaw@mirror:~$ lspci |grep IDE > > 00:07.1 IDE interface: Intel Corp. 82371AB/EB/MB PIIX4 IDE (rev 01) > > - Linux v2.5.73 > > - <*> Intel PIIXn chipsets support > > > > Basically, if I enable Taskfile I/O, the box refuses to boot (basically, > > the first drive sounds really broken like "clack clack clack" and no > > data comes off the drive so there's no partition table, no root FS, but > > a panic:) Is anybody interested in nailing this bug down? > > YES I'll note down this one after having had a little sleep... > > Disabling Taskfile lets me boot the box, but hdc doesn't like TCQ and > > gives errors: > > > > ide_tcq_intr_timeout: timeout waiting for service interrupt > > ide_tcq_intr_timeout: missing isr! > > hdc: invalidating tag queue (0 commands) > > hdc: drive_cmd: status=0x51 { DriveReady SeekComplete Error } > > hdc: drive_cmd: error=0x04 { DriveStatusError } > > [above messages repeat...] > > TCQ shouldn't be enabled on hdc, you have two drives on second ide > channel and current TCQ driver design allows only one drive per channel, > so proper fix is to not enable TCQ :-). Your patch works for me - TCQ gets no longer stitched on while I've configured to default ON, queue depth 32: Uniform Multi-Platform E-IDE driver Revision: 7.00alpha2 ide: Assuming 33MHz system bus speed for PIO modes; override with idebus=xx PIIX4: IDE controller at PCI slot 00:07.1 PIIX4: chipset revision 1 PIIX4: not 100% native mode: will probe irqs later ide0: BM-DMA at 0xf000-0xf007, BIOS settings: hda:pio, hdb:pio ide1: BM-DMA at 0xf008-0xf00f, BIOS settings: hdc:pio, hdd:pio hda: WDC AC2850F, ATA DISK drive hdb: IC35L040AVER07-0, ATA DISK drive ide0 at 0x1f0-0x1f7,0x3f6 on irq 14 hdc: IC35L120AVV207-0, ATA DISK drive hdd: Maxtor 4W100H6, ATA DISK drive ide1 at 0x170-0x177,0x376 on irq 15 hda: max request size: 128KiB hda: task_no_data_intr: status=0x51 { DriveReady SeekComplete Error } hda: task_no_data_intr: error=0x04 { DriveStatusError } hda: 1667232 sectors (854 MB) w/64KiB Cache, CHS=1654/16/63, DMA hda: hda1 hda2 hda3 hdb: max request size: 128KiB hdb: host protected area => 1 hdb: 80418240 sectors (41174 MB) w/1916KiB Cache, CHS=79780/16/63, UDMA(33) hdb: hdb1 hdb2 hdb3 hdb4 hdc: max request size: 1024KiB hdc: host protected area => 1 hdc: 241254720 sectors (123522 MB) w/1821KiB Cache, CHS=15017/255/63, UDMA(33) hdc: hdc1 < hdc5 hdc6 hdc7 hdc8 hdc9 hdc10 hdc11 hdc12 hdc13 hdc14 hdc15 hdc16 hdc17 > hdd: max request size: 128KiB hdd: host protected area => 1 hdd: 195711264 sectors (100204 MB) w/2048KiB Cache, CHS=194158/16/63, UDMA(33) hdd: hdd1 < hdd5 hdd6 hdd7 hdd8 hdd9 hdd10 hdd11 hdd12 hdd13 hdd14 > The box is now running for a couple of minuted hitting it's HDDs with rsync... MfG, JBG -- Jan-Benedict Glaw jbglaw@lug-owl.de . +49-172-7608481 "Eine Freie Meinung in einem Freien Kopf | Gegen Zensur | Gegen Krieg fuer einen Freien Staat voll Freier Bürger" | im Internet! | im Irak! ret = do_actions((curr | FREE_SPEECH) & ~(IRAQ_WAR_2 | DRM | TCPA)); [-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --] ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-23 23:50 ` Jan-Benedict Glaw @ 2003-06-24 0:53 ` Bartlomiej Zolnierkiewicz 2003-06-24 6:31 ` Jan-Benedict Glaw 0 siblings, 1 reply; 23+ messages in thread From: Bartlomiej Zolnierkiewicz @ 2003-06-24 0:53 UTC (permalink / raw) To: Jan-Benedict Glaw; +Cc: axboe, linux-kernel On Tue, 24 Jun 2003, Jan-Benedict Glaw wrote: <...> > > TCQ shouldn't be enabled on hdc, you have two drives on second ide > > channel and current TCQ driver design allows only one drive per channel, > > so proper fix is to not enable TCQ :-). > > Your patch works for me - TCQ gets no longer stitched on while I've > configured to default ON, queue depth 32: > > Uniform Multi-Platform E-IDE driver Revision: 7.00alpha2 > ide: Assuming 33MHz system bus speed for PIO modes; override with idebus=xx > PIIX4: IDE controller at PCI slot 00:07.1 > PIIX4: chipset revision 1 > PIIX4: not 100% native mode: will probe irqs later > ide0: BM-DMA at 0xf000-0xf007, BIOS settings: hda:pio, hdb:pio > ide1: BM-DMA at 0xf008-0xf00f, BIOS settings: hdc:pio, hdd:pio > hda: WDC AC2850F, ATA DISK drive > hdb: IC35L040AVER07-0, ATA DISK drive > ide0 at 0x1f0-0x1f7,0x3f6 on irq 14 > hdc: IC35L120AVV207-0, ATA DISK drive > hdd: Maxtor 4W100H6, ATA DISK drive > ide1 at 0x170-0x177,0x376 on irq 15 > hda: max request size: 128KiB > hda: task_no_data_intr: status=0x51 { DriveReady SeekComplete Error } > hda: task_no_data_intr: error=0x04 { DriveStatusError } > hda: 1667232 sectors (854 MB) w/64KiB Cache, CHS=1654/16/63, DMA > hda: hda1 hda2 hda3 > hdb: max request size: 128KiB > hdb: host protected area => 1 > hdb: 80418240 sectors (41174 MB) w/1916KiB Cache, CHS=79780/16/63, UDMA(33) > hdb: hdb1 hdb2 hdb3 hdb4 > hdc: max request size: 1024KiB > hdc: host protected area => 1 > hdc: 241254720 sectors (123522 MB) w/1821KiB Cache, CHS=15017/255/63, UDMA(33) > hdc: hdc1 < hdc5 hdc6 hdc7 hdc8 hdc9 hdc10 hdc11 hdc12 hdc13 hdc14 hdc15 hdc16 hdc17 > > hdd: max request size: 128KiB > hdd: host protected area => 1 > hdd: 195711264 sectors (100204 MB) w/2048KiB Cache, CHS=194158/16/63, UDMA(33) > hdd: hdd1 < hdd5 hdd6 hdd7 hdd8 hdd9 hdd10 hdd11 hdd12 hdd13 hdd14 > There is no info about IDE trying (then failing) to enable TCQ ? -- Bartlomiej ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-24 0:53 ` Bartlomiej Zolnierkiewicz @ 2003-06-24 6:31 ` Jan-Benedict Glaw 2003-06-24 13:44 ` Bartlomiej Zolnierkiewicz 0 siblings, 1 reply; 23+ messages in thread From: Jan-Benedict Glaw @ 2003-06-24 6:31 UTC (permalink / raw) To: Bartlomiej Zolnierkiewicz; +Cc: axboe, linux-kernel [-- Attachment #1: Type: text/plain, Size: 4632 bytes --] On Tue, 2003-06-24 02:53:29 +0200, Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl> wrote in message <Pine.SOL.4.30.0306240249410.5865-100000@mion.elka.pw.edu.pl>: > On Tue, 24 Jun 2003, Jan-Benedict Glaw wrote: > > > TCQ shouldn't be enabled on hdc, you have two drives on second ide > > > channel and current TCQ driver design allows only one drive per channel, > > > so proper fix is to not enable TCQ :-). > > > > Your patch works for me - TCQ gets no longer stitched on while I've > > configured to default ON, queue depth 32: > > > > Uniform Multi-Platform E-IDE driver Revision: 7.00alpha2 > > ide: Assuming 33MHz system bus speed for PIO modes; override with idebus=xx > > PIIX4: IDE controller at PCI slot 00:07.1 > > PIIX4: chipset revision 1 > > PIIX4: not 100% native mode: will probe irqs later > > ide0: BM-DMA at 0xf000-0xf007, BIOS settings: hda:pio, hdb:pio > > ide1: BM-DMA at 0xf008-0xf00f, BIOS settings: hdc:pio, hdd:pio > > hda: WDC AC2850F, ATA DISK drive > > hdb: IC35L040AVER07-0, ATA DISK drive > > ide0 at 0x1f0-0x1f7,0x3f6 on irq 14 > > hdc: IC35L120AVV207-0, ATA DISK drive > > hdd: Maxtor 4W100H6, ATA DISK drive > > ide1 at 0x170-0x177,0x376 on irq 15 > > hda: max request size: 128KiB > > hda: task_no_data_intr: status=0x51 { DriveReady SeekComplete Error } > > hda: task_no_data_intr: error=0x04 { DriveStatusError } > > hda: 1667232 sectors (854 MB) w/64KiB Cache, CHS=1654/16/63, DMA > > hda: hda1 hda2 hda3 > > hdb: max request size: 128KiB > > hdb: host protected area => 1 > > hdb: 80418240 sectors (41174 MB) w/1916KiB Cache, CHS=79780/16/63, UDMA(33) > > hdb: hdb1 hdb2 hdb3 hdb4 > > hdc: max request size: 1024KiB > > hdc: host protected area => 1 > > hdc: 241254720 sectors (123522 MB) w/1821KiB Cache, CHS=15017/255/63, UDMA(33) > > hdc: hdc1 < hdc5 hdc6 hdc7 hdc8 hdc9 hdc10 hdc11 hdc12 hdc13 hdc14 hdc15 hdc16 hdc17 > > > hdd: max request size: 128KiB > > hdd: host protected area => 1 > > hdd: 195711264 sectors (100204 MB) w/2048KiB Cache, CHS=194158/16/63, UDMA(33) > > hdd: hdd1 < hdd5 hdd6 hdd7 hdd8 hdd9 hdd10 hdd11 hdd12 hdd13 hdd14 > > > There is no info about IDE trying (then failing) to enable TCQ ? No, there isn't. The box is now up'n'running for nearly 7 hours - that should have been enough:) That's with your patch applied and the IDE config as attached below. Up to now, there's not a single IDE failure of any kind *knock on wood*. MfG, JBG # # Please see Documentation/ide.txt for help/info on IDE drives # # CONFIG_BLK_DEV_HD_IDE is not set # CONFIG_BLK_DEV_HD is not set CONFIG_BLK_DEV_IDEDISK=y CONFIG_IDEDISK_MULTI_MODE=y # CONFIG_IDEDISK_STROKE is not set CONFIG_BLK_DEV_IDECD=m CONFIG_BLK_DEV_IDEFLOPPY=m CONFIG_BLK_DEV_IDESCSI=m # CONFIG_IDE_TASK_IOCTL is not set # CONFIG_IDE_TASKFILE_IO is not set # # IDE chipset support/bugfixes # # CONFIG_BLK_DEV_CMD640 is not set # CONFIG_BLK_DEV_IDEPNP is not set CONFIG_BLK_DEV_IDEPCI=y CONFIG_BLK_DEV_GENERIC=y CONFIG_IDEPCI_SHARE_IRQ=y CONFIG_BLK_DEV_IDEDMA_PCI=y CONFIG_BLK_DEV_IDE_TCQ=y CONFIG_BLK_DEV_IDE_TCQ_DEFAULT=y CONFIG_BLK_DEV_IDE_TCQ_DEPTH=32 # CONFIG_BLK_DEV_OFFBOARD is not set # CONFIG_BLK_DEV_IDEDMA_FORCED is not set CONFIG_IDEDMA_PCI_AUTO=y # CONFIG_IDEDMA_ONLYDISK is not set CONFIG_BLK_DEV_IDEDMA=y # CONFIG_IDEDMA_PCI_WIP is not set CONFIG_BLK_DEV_ADMA=y # CONFIG_BLK_DEV_AEC62XX is not set # CONFIG_BLK_DEV_ALI15X3 is not set # CONFIG_BLK_DEV_AMD74XX is not set # CONFIG_BLK_DEV_CMD64X is not set # CONFIG_BLK_DEV_TRIFLEX is not set # CONFIG_BLK_DEV_CY82C693 is not set # CONFIG_BLK_DEV_CS5520 is not set # CONFIG_BLK_DEV_HPT34X is not set # CONFIG_BLK_DEV_HPT366 is not set # CONFIG_BLK_DEV_SC1200 is not set CONFIG_BLK_DEV_PIIX=y # CONFIG_BLK_DEV_NS87415 is not set # CONFIG_BLK_DEV_OPTI621 is not set # CONFIG_BLK_DEV_PDC202XX_OLD is not set # CONFIG_BLK_DEV_PDC202XX_NEW is not set # CONFIG_BLK_DEV_RZ1000 is not set # CONFIG_BLK_DEV_SVWKS is not set # CONFIG_BLK_DEV_SIIMAGE is not set # CONFIG_BLK_DEV_SIS5513 is not set # CONFIG_BLK_DEV_SLC90E66 is not set # CONFIG_BLK_DEV_TRM290 is not set # CONFIG_BLK_DEV_VIA82CXXX is not set # CONFIG_IDE_CHIPSETS is not set CONFIG_IDEDMA_AUTO=y # CONFIG_IDEDMA_IVB is not set CONFIG_BLK_DEV_IDE_MODES=y -- Jan-Benedict Glaw jbglaw@lug-owl.de . +49-172-7608481 "Eine Freie Meinung in einem Freien Kopf | Gegen Zensur | Gegen Krieg fuer einen Freien Staat voll Freier Bürger" | im Internet! | im Irak! ret = do_actions((curr | FREE_SPEECH) & ~(IRAQ_WAR_2 | DRM | TCPA)); [-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --] ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-24 6:31 ` Jan-Benedict Glaw @ 2003-06-24 13:44 ` Bartlomiej Zolnierkiewicz 2003-06-24 22:40 ` Jan-Benedict Glaw 0 siblings, 1 reply; 23+ messages in thread From: Bartlomiej Zolnierkiewicz @ 2003-06-24 13:44 UTC (permalink / raw) To: Jan-Benedict Glaw; +Cc: axboe, linux-kernel Corrected patch below... -- Bartlomiej [ide] TCQ initialization fixes - do not enable TCQ in ide_init_drive(), its too early - enable TCQ in __ide_dma_on() only if CONFIG_BLK_DEV_IDE_TCQ_DEFAULT=y - try to enable TCQ only on disk drives - correct check for two drives on one channel drivers/ide/ide-dma.c | 8 ++++---- drivers/ide/ide-probe.c | 5 ----- drivers/ide/ide-tcq.c | 14 +++++++++++--- include/linux/ide.h | 1 - 4 files changed, 15 insertions(+), 13 deletions(-) diff -puN drivers/ide/ide-probe.c~ide-tcq-init-fixes drivers/ide/ide-probe.c --- linux-2.5.73/drivers/ide/ide-probe.c~ide-tcq-init-fixes Tue Jun 24 14:56:57 2003 +++ linux-2.5.73-root/drivers/ide/ide-probe.c Tue Jun 24 14:56:57 2003 @@ -983,7 +983,6 @@ static void ide_init_queue(ide_drive_t * blk_init_queue(q, do_ide_request, &ide_lock); q->queuedata = HWGROUP(drive); - drive->queue_setup = 1; blk_queue_segment_boundary(q, 0xffff); if (!hwif->rqsize) @@ -1005,10 +1004,6 @@ static void ide_init_queue(ide_drive_t * static void ide_init_drive(ide_drive_t *drive) { ide_toggle_bounce(drive, 1); - -#ifdef CONFIG_BLK_DEV_IDE_TCQ_DEFAULT - HWIF(drive)->ide_dma_queued_on(drive); -#endif } /* diff -puN drivers/ide/ide-dma.c~ide-tcq-init-fixes drivers/ide/ide-dma.c --- linux-2.5.73/drivers/ide/ide-dma.c~ide-tcq-init-fixes Tue Jun 24 14:56:57 2003 +++ linux-2.5.73-root/drivers/ide/ide-dma.c Tue Jun 24 15:16:34 2003 @@ -523,8 +523,7 @@ int __ide_dma_off_quietly (ide_drive_t * if (HWIF(drive)->ide_dma_host_off(drive)) return 1; - if (drive->queue_setup) - HWIF(drive)->ide_dma_queued_off(drive); + HWIF(drive)->ide_dma_queued_off(drive); return 0; } @@ -585,8 +584,9 @@ int __ide_dma_on (ide_drive_t *drive) if (HWIF(drive)->ide_dma_host_on(drive)) return 1; - if (drive->queue_setup) - HWIF(drive)->ide_dma_queued_on(drive); +#ifdef CONFIG_BLK_DEV_IDE_TCQ_DEFAULT + HWIF(drive)->ide_dma_queued_on(drive); +#endif return 0; } diff -puN drivers/ide/ide-tcq.c~ide-tcq-init-fixes drivers/ide/ide-tcq.c --- linux-2.5.73/drivers/ide/ide-tcq.c~ide-tcq-init-fixes Tue Jun 24 14:57:10 2003 +++ linux-2.5.73-root/drivers/ide/ide-tcq.c Tue Jun 24 15:40:26 2003 @@ -501,8 +501,10 @@ static int ide_tcq_configure(ide_drive_t * bit 14 and 1 must be set in word 83 of the device id to indicate * support for dma queued protocol, and bit 15 must be cleared */ - if ((drive->id->command_set_2 & tcq_bits) ^ tcq_mask) + if ((drive->id->command_set_2 & tcq_bits) ^ tcq_mask) { + printk(KERN_INFO "%s: TCQ not supported\n", drive->name); return -EIO; + } args = kmalloc(sizeof(*args), GFP_ATOMIC); if (!args) @@ -655,16 +657,20 @@ static int ide_tcq_check_blacklist(ide_d int __ide_dma_queued_on(ide_drive_t *drive) { + ide_hwif_t *hwif = HWIF(drive); + + if (drive->media != ide_disk) + return 1; if (!drive->using_dma) return 1; - if (HWIF(drive)->chipset == ide_pdc4030) + if (hwif->chipset == ide_pdc4030) return 1; if (ide_tcq_check_blacklist(drive)) { printk(KERN_WARNING "%s: tcq forbidden by blacklist\n", drive->name); return 1; } - if (drive->next != drive) { + if (hwif->drives[0].present && hwif->drives[1].present) { printk(KERN_WARNING "%s: only one drive on a channel supported" " for tcq\n", drive->name); return 1; @@ -681,6 +687,8 @@ int __ide_dma_queued_on(ide_drive_t *dri int __ide_dma_queued_off(ide_drive_t *drive) { + if (drive->media != ide_disk) + return 1; if (ata_pending_commands(drive)) { printk("ide-tcq; can't toggle tcq feature on busy drive\n"); return 1; diff -puN include/linux/ide.h~ide-tcq-init-fixes include/linux/ide.h --- linux-2.5.73/include/linux/ide.h~ide-tcq-init-fixes Tue Jun 24 14:56:57 2003 +++ linux-2.5.73-root/include/linux/ide.h Tue Jun 24 14:56:57 2003 @@ -726,7 +726,6 @@ typedef struct ide_drive_s { unsigned ata_flash : 1; /* 1=present, 0=default */ unsigned blocked : 1; /* 1=powermanagment told us not to do anything, so sleep nicely */ unsigned vdma : 1; /* 1=doing PIO over DMA 0=doing normal DMA */ - unsigned queue_setup : 1; unsigned addressing; /* : 3; * 0=28-bit * 1=48-bit _ ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-24 13:44 ` Bartlomiej Zolnierkiewicz @ 2003-06-24 22:40 ` Jan-Benedict Glaw 2003-06-24 23:08 ` Bartlomiej Zolnierkiewicz 0 siblings, 1 reply; 23+ messages in thread From: Jan-Benedict Glaw @ 2003-06-24 22:40 UTC (permalink / raw) To: Bartlomiej Zolnierkiewicz; +Cc: axboe, linux-kernel [-- Attachment #1: Type: text/plain, Size: 2360 bytes --] On Tue, 2003-06-24 15:44:36 +0200, Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl> wrote in message <Pine.SOL.4.30.0306241543050.23584-100000@mion.elka.pw.edu.pl>: > Corrected patch below... Taking this 2nd IDE patch, my boot-up looks as shown below. I still have to test the taskfile thing again, but I cannot find my girlfriends digital camera right now:) MfG, JBG Uniform Multi-Platform E-IDE driver Revision: 7.00alpha2 ide: Assuming 33MHz system bus speed for PIO modes; override with idebus=xx PIIX4: IDE controller at PCI slot 00:07.1 PIIX4: chipset revision 1 PIIX4: not 100% native mode: will probe irqs later ide0: BM-DMA at 0xf000-0xf007, BIOS settings: hda:pio, hdb:pio ide1: BM-DMA at 0xf008-0xf00f, BIOS settings: hdc:pio, hdd:pio hda: WDC AC2850F, ATA DISK drive hdb: IC35L040AVER07-0, ATA DISK drive hda: tcq forbidden by blacklist hdb: only one drive on a channel supported for tcq ide0 at 0x1f0-0x1f7,0x3f6 on irq 14 hdc: IC35L120AVV207-0, ATA DISK drive hdd: Maxtor 4W100H6, ATA DISK drive hdc: only one drive on a channel supported for tcq hdd: only one drive on a channel supported for tcq ide1 at 0x170-0x177,0x376 on irq 15 hda: max request size: 128KiB hda: task_no_data_intr: status=0x51 { DriveReady SeekComplete Error } hda: task_no_data_intr: error=0x04 { DriveStatusError } hda: 1667232 sectors (854 MB) w/64KiB Cache, CHS=1654/16/63, DMA hda: hda1 hda2 hda3 hdb: max request size: 128KiB hdb: host protected area => 1 hdb: 80418240 sectors (41174 MB) w/1916KiB Cache, CHS=79780/16/63, UDMA(33) hdb: hdb1 hdb2 hdb3 hdb4 hdc: max request size: 1024KiB hdc: host protected area => 1 hdc: 241254720 sectors (123522 MB) w/1821KiB Cache, CHS=15017/255/63, UDMA(33) hdc: hdc1 < hdc5 hdc6 hdc7 hdc8 hdc9 hdc10 hdc11 hdc12 hdc13 hdc14 hdc15 hdc16 hdc17 > hdd: max request size: 128KiB hdd: host protected area => 1 hdd: 195711264 sectors (100204 MB) w/2048KiB Cache, CHS=194158/16/63, UDMA(33) hdd: hdd1 < hdd5 hdd6 hdd7 hdd8 hdd9 hdd10 hdd11 hdd12 hdd13 hdd14 > -- Jan-Benedict Glaw jbglaw@lug-owl.de . +49-172-7608481 "Eine Freie Meinung in einem Freien Kopf | Gegen Zensur | Gegen Krieg fuer einen Freien Staat voll Freier Bürger" | im Internet! | im Irak! ret = do_actions((curr | FREE_SPEECH) & ~(IRAQ_WAR_2 | DRM | TCPA)); [-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --] ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-24 22:40 ` Jan-Benedict Glaw @ 2003-06-24 23:08 ` Bartlomiej Zolnierkiewicz 2003-06-25 18:22 ` SOLVED - " Jan-Benedict Glaw 0 siblings, 1 reply; 23+ messages in thread From: Bartlomiej Zolnierkiewicz @ 2003-06-24 23:08 UTC (permalink / raw) To: Jan-Benedict Glaw; +Cc: linux-kernel On Wed, 25 Jun 2003, Jan-Benedict Glaw wrote: > On Tue, 2003-06-24 15:44:36 +0200, Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl> > wrote in message <Pine.SOL.4.30.0306241543050.23584-100000@mion.elka.pw.edu.pl>: > > > Corrected patch below... > > Taking this 2nd IDE patch, my boot-up looks as shown below. I still have Thanks for testing, it looks okay now. > to test the taskfile thing again, but I cannot find my girlfriends > digital camera right now:) :-) > MfG, JBG > > Uniform Multi-Platform E-IDE driver Revision: 7.00alpha2 > ide: Assuming 33MHz system bus speed for PIO modes; override with idebus=xx > PIIX4: IDE controller at PCI slot 00:07.1 > PIIX4: chipset revision 1 > PIIX4: not 100% native mode: will probe irqs later > ide0: BM-DMA at 0xf000-0xf007, BIOS settings: hda:pio, hdb:pio > ide1: BM-DMA at 0xf008-0xf00f, BIOS settings: hdc:pio, hdd:pio > hda: WDC AC2850F, ATA DISK drive > hdb: IC35L040AVER07-0, ATA DISK drive > hda: tcq forbidden by blacklist > hdb: only one drive on a channel supported for tcq > ide0 at 0x1f0-0x1f7,0x3f6 on irq 14 > hdc: IC35L120AVV207-0, ATA DISK drive > hdd: Maxtor 4W100H6, ATA DISK drive > hdc: only one drive on a channel supported for tcq > hdd: only one drive on a channel supported for tcq > ide1 at 0x170-0x177,0x376 on irq 15 > hda: max request size: 128KiB > hda: task_no_data_intr: status=0x51 { DriveReady SeekComplete Error } > hda: task_no_data_intr: error=0x04 { DriveStatusError } > hda: 1667232 sectors (854 MB) w/64KiB Cache, CHS=1654/16/63, DMA > hda: hda1 hda2 hda3 > hdb: max request size: 128KiB > hdb: host protected area => 1 > hdb: 80418240 sectors (41174 MB) w/1916KiB Cache, CHS=79780/16/63, UDMA(33) > hdb: hdb1 hdb2 hdb3 hdb4 > hdc: max request size: 1024KiB > hdc: host protected area => 1 > hdc: 241254720 sectors (123522 MB) w/1821KiB Cache, CHS=15017/255/63, UDMA(33) > hdc: hdc1 < hdc5 hdc6 hdc7 hdc8 hdc9 hdc10 hdc11 hdc12 hdc13 hdc14 hdc15 hdc16 hdc17 > > hdd: max request size: 128KiB > hdd: host protected area => 1 > hdd: 195711264 sectors (100204 MB) w/2048KiB Cache, CHS=194158/16/63, UDMA(33) > hdd: hdd1 < hdd5 hdd6 hdd7 hdd8 hdd9 hdd10 hdd11 hdd12 hdd13 hdd14 > > -- > Jan-Benedict Glaw jbglaw@lug-owl.de . +49-172-7608481 > "Eine Freie Meinung in einem Freien Kopf | Gegen Zensur | Gegen Krieg > fuer einen Freien Staat voll Freier Bürger" | im Internet! | im Irak! > ret = do_actions((curr | FREE_SPEECH) & ~(IRAQ_WAR_2 | DRM | TCPA)); ^ permalink raw reply [flat|nested] 23+ messages in thread
* SOLVED - Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-24 23:08 ` Bartlomiej Zolnierkiewicz @ 2003-06-25 18:22 ` Jan-Benedict Glaw 2003-06-25 18:44 ` Bartlomiej Zolnierkiewicz 0 siblings, 1 reply; 23+ messages in thread From: Jan-Benedict Glaw @ 2003-06-25 18:22 UTC (permalink / raw) To: linux-kernel; +Cc: Bartlomiej Zolnierkiewicz [-- Attachment #1: Type: text/plain, Size: 1585 bytes --] On Wed, 2003-06-25 01:08:13 +0200, Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl> wrote in message <Pine.SOL.4.30.0306250107180.17106-100000@mion.elka.pw.edu.pl>: > On Wed, 25 Jun 2003, Jan-Benedict Glaw wrote: > > On Tue, 2003-06-24 15:44:36 +0200, Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl> > > wrote in message <Pine.SOL.4.30.0306241543050.23584-100000@mion.elka.pw.edu.pl>: > > > > > Corrected patch below... > > > > Taking this 2nd IDE patch, my boot-up looks as shown below. I still have > > Thanks for testing, it looks okay now. > > > to test the taskfile thing again, but I cannot find my girlfriends > > digital camera right now:) Found the cam, downloaded any old pics, recompiled with taskfile I/O and - nothing. 2.5.73 + your second IDE patch do work flawlessly on my old box. The "clack clack clack" went away is if had never been there... So - thanks a lot for looking at my problem. Please feed your patch to Linus as it seems to actually solve (at least my) problems. However, allow me to ask why this occured never bevore (for other people)? Do they all have only one drive? Does nobody use TCQ? Nobody with old hardware (though, your patch hasn't touched the core PIIX parts...)? MfG, JBG -- Jan-Benedict Glaw jbglaw@lug-owl.de . +49-172-7608481 "Eine Freie Meinung in einem Freien Kopf | Gegen Zensur | Gegen Krieg fuer einen Freien Staat voll Freier Bürger" | im Internet! | im Irak! ret = do_actions((curr | FREE_SPEECH) & ~(IRAQ_WAR_2 | DRM | TCPA)); [-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --] ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: SOLVED - Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-25 18:22 ` SOLVED - " Jan-Benedict Glaw @ 2003-06-25 18:44 ` Bartlomiej Zolnierkiewicz 2003-06-25 19:05 ` Jan-Benedict Glaw 0 siblings, 1 reply; 23+ messages in thread From: Bartlomiej Zolnierkiewicz @ 2003-06-25 18:44 UTC (permalink / raw) To: Jan-Benedict Glaw; +Cc: linux-kernel On Wed, 25 Jun 2003, Jan-Benedict Glaw wrote: > On Wed, 2003-06-25 01:08:13 +0200, Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl> > wrote in message <Pine.SOL.4.30.0306250107180.17106-100000@mion.elka.pw.edu.pl>: > > On Wed, 25 Jun 2003, Jan-Benedict Glaw wrote: > > > On Tue, 2003-06-24 15:44:36 +0200, Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl> > > > wrote in message <Pine.SOL.4.30.0306241543050.23584-100000@mion.elka.pw.edu.pl>: > > > > > > > Corrected patch below... > > > > > > Taking this 2nd IDE patch, my boot-up looks as shown below. I still have > > > > Thanks for testing, it looks okay now. > > > > > to test the taskfile thing again, but I cannot find my girlfriends > > > digital camera right now:) > > Found the cam, downloaded any old pics, recompiled with taskfile I/O and > - nothing. 2.5.73 + your second IDE patch do work flawlessly on my old > box. The "clack clack clack" went away is if had never been > there... :-) > So - thanks a lot for looking at my problem. Please feed your patch to > Linus as it seems to actually solve (at least my) problems. > > However, allow me to ask why this occured never bevore (for other > people)? Do they all have only one drive? Does nobody use TCQ? Nobody > with old hardware (though, your patch hasn't touched the core PIIX > parts...)? nobody is stupid^H^Hbrave enough to use TCQ ;-) Thanks, -- Bartlomiej > MfG, JBG > > -- > Jan-Benedict Glaw jbglaw@lug-owl.de . +49-172-7608481 > "Eine Freie Meinung in einem Freien Kopf | Gegen Zensur | Gegen Krieg > fuer einen Freien Staat voll Freier Bürger" | im Internet! | im Irak! > ret = do_actions((curr | FREE_SPEECH) & ~(IRAQ_WAR_2 | DRM | TCPA)); ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: SOLVED - Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-25 18:44 ` Bartlomiej Zolnierkiewicz @ 2003-06-25 19:05 ` Jan-Benedict Glaw 2003-06-25 19:30 ` Bartlomiej Zolnierkiewicz 0 siblings, 1 reply; 23+ messages in thread From: Jan-Benedict Glaw @ 2003-06-25 19:05 UTC (permalink / raw) To: Bartlomiej Zolnierkiewicz; +Cc: linux-kernel [-- Attachment #1: Type: text/plain, Size: 2013 bytes --] On Wed, 2003-06-25 20:44:19 +0200, Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl> wrote in message <Pine.SOL.4.30.0306252040570.11992-100000@mion.elka.pw.edu.pl>: > On Wed, 25 Jun 2003, Jan-Benedict Glaw wrote: > > On Wed, 2003-06-25 01:08:13 +0200, Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl> > > wrote in message <Pine.SOL.4.30.0306250107180.17106-100000@mion.elka.pw.edu.pl>: > > > On Wed, 25 Jun 2003, Jan-Benedict Glaw wrote: > > > > On Tue, 2003-06-24 15:44:36 +0200, Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl> > > > > wrote in message <Pine.SOL.4.30.0306241543050.23584-100000@mion.elka.pw.edu.pl>: > > However, allow me to ask why this occured never bevore (for other > > people)? Do they all have only one drive? Does nobody use TCQ? Nobody > > with old hardware (though, your patch hasn't touched the core PIIX > > parts...)? > > nobody is stupid^H^Hbrave enough to use TCQ ;-) Well... I knew that it was, erm, brave to use it at some time (this box isn't really important - if I kill its filesystems, I'll simply re-install and recover my mirror scripts:) , but new, where 2.5.x starts to be really useable, I'd suggest to face users with upcoming problems: CONFIG_IDE_TASKFILE_IO It is safe to say Y to this question, in most cases. (By the way, exactly which cases are ment?) CONFIG_BLK_DEV_IDE_TCQ If you have such a drive, say Y here. CONFIG_BLK_DEV_IDE_TCQ_DEFAULT Generally say Y here. CONFIG_BLK_DEV_IDE_TCQ_DEPTH You probably just want the default of 32 here. (Default queue depth is 8, not 32...) Right now, I'm building 2.5.73-bk3 + your 2nd patch. Let's face it:) MfG, JBG -- Jan-Benedict Glaw jbglaw@lug-owl.de . +49-172-7608481 "Eine Freie Meinung in einem Freien Kopf | Gegen Zensur | Gegen Krieg fuer einen Freien Staat voll Freier Bürger" | im Internet! | im Irak! ret = do_actions((curr | FREE_SPEECH) & ~(IRAQ_WAR_2 | DRM | TCPA)); [-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --] ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: SOLVED - Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-25 19:05 ` Jan-Benedict Glaw @ 2003-06-25 19:30 ` Bartlomiej Zolnierkiewicz 2003-06-26 3:21 ` Zwane Mwaikambo 2003-06-26 5:58 ` Jan-Benedict Glaw 0 siblings, 2 replies; 23+ messages in thread From: Bartlomiej Zolnierkiewicz @ 2003-06-25 19:30 UTC (permalink / raw) To: Jan-Benedict Glaw; +Cc: linux-kernel On Wed, 25 Jun 2003, Jan-Benedict Glaw wrote: > On Wed, 2003-06-25 20:44:19 +0200, Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl> > wrote in message <Pine.SOL.4.30.0306252040570.11992-100000@mion.elka.pw.edu.pl>: > > On Wed, 25 Jun 2003, Jan-Benedict Glaw wrote: > > > On Wed, 2003-06-25 01:08:13 +0200, Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl> > > > wrote in message <Pine.SOL.4.30.0306250107180.17106-100000@mion.elka.pw.edu.pl>: > > > > On Wed, 25 Jun 2003, Jan-Benedict Glaw wrote: > > > > > On Tue, 2003-06-24 15:44:36 +0200, Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl> > > > > > wrote in message <Pine.SOL.4.30.0306241543050.23584-100000@mion.elka.pw.edu.pl>: > > > > However, allow me to ask why this occured never bevore (for other > > > people)? Do they all have only one drive? Does nobody use TCQ? Nobody > > > with old hardware (though, your patch hasn't touched the core PIIX > > > parts...)? > > > > nobody is stupid^H^Hbrave enough to use TCQ ;-) > > Well... I knew that it was, erm, brave to use it at some time (this box > isn't really important - if I kill its filesystems, I'll simply > re-install and recover my mirror scripts:) , but new, where 2.5.x > starts to be really useable, I'd suggest to face users with upcoming > problems: I think 2.5.x is really useable. > CONFIG_IDE_TASKFILE_IO > It is safe to say Y to this question, in most cases. > > (By the way, exactly which cases are ment?) Ehh... 3 weird bugreports (all are oopses): - on x86_64 with 32 bit userspace when reading /proc/ide/hdx/identify - in task_mulout_intr() on access to freed rq->bio - "bad: scheduling while atomic" I don't have any ideas how to find&fix these bugs... > CONFIG_BLK_DEV_IDE_TCQ > If you have such a drive, say Y here. TCQ is marked (EXPERIMENTAL) but there should be warning about potential risk... :-) > CONFIG_BLK_DEV_IDE_TCQ_DEFAULT > Generally say Y here. > > > CONFIG_BLK_DEV_IDE_TCQ_DEPTH > You probably just want the default of 32 here. > > (Default queue depth is 8, not 32...) Jens did some testing ages ago and 8 was optimal... > Right now, I'm building 2.5.73-bk3 + your 2nd patch. Let's face it:) Okay. :-) -- Bartlomiej > MfG, JBG > > -- > Jan-Benedict Glaw jbglaw@lug-owl.de . +49-172-7608481 > "Eine Freie Meinung in einem Freien Kopf | Gegen Zensur | Gegen Krieg > fuer einen Freien Staat voll Freier Bürger" | im Internet! | im Irak! > ret = do_actions((curr | FREE_SPEECH) & ~(IRAQ_WAR_2 | DRM | TCPA)); ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: SOLVED - Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-25 19:30 ` Bartlomiej Zolnierkiewicz @ 2003-06-26 3:21 ` Zwane Mwaikambo 2003-06-26 5:58 ` Jan-Benedict Glaw 1 sibling, 0 replies; 23+ messages in thread From: Zwane Mwaikambo @ 2003-06-26 3:21 UTC (permalink / raw) To: Bartlomiej Zolnierkiewicz; +Cc: Jan-Benedict Glaw, linux-kernel On Wed, 25 Jun 2003, Bartlomiej Zolnierkiewicz wrote: > Ehh... 3 weird bugreports (all are oopses): > - on x86_64 with 32 bit userspace when reading /proc/ide/hdx/identify > - in task_mulout_intr() on access to freed rq->bio > - "bad: scheduling while atomic" Perhaps if you can point me towards that one, i can try and give a hand Zwane -- function.linuxpower.ca ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: SOLVED - Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-25 19:30 ` Bartlomiej Zolnierkiewicz 2003-06-26 3:21 ` Zwane Mwaikambo @ 2003-06-26 5:58 ` Jan-Benedict Glaw 1 sibling, 0 replies; 23+ messages in thread From: Jan-Benedict Glaw @ 2003-06-26 5:58 UTC (permalink / raw) To: Bartlomiej Zolnierkiewicz; +Cc: linux-kernel [-- Attachment #1: Type: text/plain, Size: 700 bytes --] On Wed, 2003-06-25 21:30:02 +0200, Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl> wrote in message <Pine.SOL.4.30.0306252117240.25993-100000@mion.elka.pw.edu.pl>: > On Wed, 25 Jun 2003, Jan-Benedict Glaw wrote: > > Right now, I'm building 2.5.73-bk3 + your 2nd patch. Let's face it:) > Okay. :-) 2.5.73-bk3 with TCQ and Taskfile-I/O [X] Works Good job! MfG, JBG -- Jan-Benedict Glaw jbglaw@lug-owl.de . +49-172-7608481 "Eine Freie Meinung in einem Freien Kopf | Gegen Zensur | Gegen Krieg fuer einen Freien Staat voll Freier Bürger" | im Internet! | im Irak! ret = do_actions((curr | FREE_SPEECH) & ~(IRAQ_WAR_2 | DRM | TCPA)); [-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --] ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-23 21:18 ` Bartlomiej Zolnierkiewicz 2003-06-23 22:19 ` Jan-Benedict Glaw 2003-06-23 23:50 ` Jan-Benedict Glaw @ 2003-06-24 8:06 ` Jens Axboe 2003-06-24 14:28 ` Jeff Garzik 3 siblings, 0 replies; 23+ messages in thread From: Jens Axboe @ 2003-06-24 8:06 UTC (permalink / raw) To: Bartlomiej Zolnierkiewicz; +Cc: linux-kernel On Mon, Jun 23 2003, Bartlomiej Zolnierkiewicz wrote: > [ide] TCQ initialization fixes > > - do not enable TCQ in ide_init_drive(), its too early > - enable TCQ in __ide_dma_on() only if CONFIG_BLK_DEV_IDE_TCQ_DEFAULT=y > > drivers/ide/ide-dma.c | 8 ++++---- > drivers/ide/ide-probe.c | 5 ----- > include/linux/ide.h | 1 - > 3 files changed, 4 insertions(+), 10 deletions(-) > > diff -puN drivers/ide/ide-probe.c~ide-tcq-init-fixes drivers/ide/ide-probe.c > --- linux-2.5.73/drivers/ide/ide-probe.c~ide-tcq-init-fixes Mon Jun 23 23:10:11 2003 > +++ linux-2.5.73-root/drivers/ide/ide-probe.c Mon Jun 23 23:10:11 2003 > @@ -983,7 +983,6 @@ static void ide_init_queue(ide_drive_t * > > blk_init_queue(q, do_ide_request, &ide_lock); > q->queuedata = HWGROUP(drive); > - drive->queue_setup = 1; > blk_queue_segment_boundary(q, 0xffff); > > if (!hwif->rqsize) > @@ -1005,10 +1004,6 @@ static void ide_init_queue(ide_drive_t * > static void ide_init_drive(ide_drive_t *drive) > { > ide_toggle_bounce(drive, 1); > - > -#ifdef CONFIG_BLK_DEV_IDE_TCQ_DEFAULT > - HWIF(drive)->ide_dma_queued_on(drive); > -#endif > } > > /* > diff -puN drivers/ide/ide-dma.c~ide-tcq-init-fixes drivers/ide/ide-dma.c > --- linux-2.5.73/drivers/ide/ide-dma.c~ide-tcq-init-fixes Mon Jun 23 23:10:11 2003 > +++ linux-2.5.73-root/drivers/ide/ide-dma.c Mon Jun 23 23:10:11 2003 > @@ -523,8 +523,7 @@ int __ide_dma_off_quietly (ide_drive_t * > if (HWIF(drive)->ide_dma_host_off(drive)) > return 1; > > - if (drive->queue_setup) > - HWIF(drive)->ide_dma_queued_off(drive); > + HWIF(drive)->ide_dma_queued_off(drive); > > return 0; > } > @@ -585,8 +584,9 @@ int __ide_dma_on (ide_drive_t *drive) > if (HWIF(drive)->ide_dma_host_on(drive)) > return 1; > > - if (drive->queue_setup) > - HWIF(drive)->ide_dma_queued_on(drive); > +#ifdef BLK_DEV_IDE_TCQ_DEFAULT > + HWIF(drive)->ide_dma_queued_on(drive); > +#endif > > return 0; > } > diff -puN include/linux/ide.h~ide-tcq-init-fixes include/linux/ide.h > --- linux-2.5.73/include/linux/ide.h~ide-tcq-init-fixes Mon Jun 23 23:10:11 2003 > +++ linux-2.5.73-root/include/linux/ide.h Mon Jun 23 23:10:11 2003 > @@ -726,7 +726,6 @@ typedef struct ide_drive_s { > unsigned ata_flash : 1; /* 1=present, 0=default */ > unsigned blocked : 1; /* 1=powermanagment told us not to do anything, so sleep nicely */ > unsigned vdma : 1; /* 1=doing PIO over DMA 0=doing normal DMA */ > - unsigned queue_setup : 1; > unsigned addressing; /* : 3; > * 0=28-bit > * 1=48-bit > > _ Patch is nice and kills the ->queue_setup abomination, great. -- Jens Axboe ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-23 21:18 ` Bartlomiej Zolnierkiewicz ` (2 preceding siblings ...) 2003-06-24 8:06 ` Jens Axboe @ 2003-06-24 14:28 ` Jeff Garzik 2003-06-24 14:47 ` Jens Axboe 2003-06-29 2:17 ` Andre Hedrick 3 siblings, 2 replies; 23+ messages in thread From: Jeff Garzik @ 2003-06-24 14:28 UTC (permalink / raw) To: Bartlomiej Zolnierkiewicz; +Cc: Jan-Benedict Glaw, axboe, linux-kernel Bartlomiej Zolnierkiewicz wrote: > TCQ shouldn't be enabled on hdc, you have two drives on second ide > channel and current TCQ driver design allows only one drive per channel, > so proper fix is to not enable TCQ :-). IMO the best rule is "enable TCQ if and only if 100% of the channel supports TCQ". So, two drives on the same channel can do TCQ, if and only if they both support TCQ. That's a big benefit of bus release, after all, simultaneously servicing multiple drives. The device-select and service interrupt semantics are annoying but doable. Jeff ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-24 14:28 ` Jeff Garzik @ 2003-06-24 14:47 ` Jens Axboe 2003-06-24 15:39 ` Jeff Garzik 2003-06-29 2:17 ` Andre Hedrick 1 sibling, 1 reply; 23+ messages in thread From: Jens Axboe @ 2003-06-24 14:47 UTC (permalink / raw) To: Jeff Garzik; +Cc: Bartlomiej Zolnierkiewicz, Jan-Benedict Glaw, linux-kernel On Tue, Jun 24 2003, Jeff Garzik wrote: > Bartlomiej Zolnierkiewicz wrote: > >TCQ shouldn't be enabled on hdc, you have two drives on second ide > >channel and current TCQ driver design allows only one drive per channel, > >so proper fix is to not enable TCQ :-). > > > IMO the best rule is "enable TCQ if and only if 100% of the channel > supports TCQ". > > So, two drives on the same channel can do TCQ, if and only if they both > support TCQ. That's a big benefit of bus release, after all, > simultaneously servicing multiple drives. The device-select and service > interrupt semantics are annoying but doable. ide-tcq is even more restrictive now, it only enables TCQ if the drive is alone on the channel. Feel free to write the device select code if you want, I'm not mucking more with the bastard that is ide tcq. -- Jens Axboe ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-24 14:47 ` Jens Axboe @ 2003-06-24 15:39 ` Jeff Garzik 0 siblings, 0 replies; 23+ messages in thread From: Jeff Garzik @ 2003-06-24 15:39 UTC (permalink / raw) To: Jens Axboe; +Cc: Bartlomiej Zolnierkiewicz, Jan-Benedict Glaw, linux-kernel Jens Axboe wrote: > ide-tcq is even more restrictive now, it only enables TCQ if the drive > is alone on the channel. Feel free to write the device select code if > you want, I'm not mucking more with the bastard that is ide tcq. hehe! I don't blame you ;-) I plan to write the devsel code, but for my ata-scsi driver... :/ It will go into libata, which I would love to have sharing code with drivers/ide... Jeff ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-24 14:28 ` Jeff Garzik 2003-06-24 14:47 ` Jens Axboe @ 2003-06-29 2:17 ` Andre Hedrick 2003-06-29 8:14 ` Jan-Benedict Glaw 1 sibling, 1 reply; 23+ messages in thread From: Andre Hedrick @ 2003-06-29 2:17 UTC (permalink / raw) To: Jeff Garzik Cc: Bartlomiej Zolnierkiewicz, Jan-Benedict Glaw, axboe, linux-kernel The best rule is to default it off and let the end user enable regardless. Thus the default will NEVER encounter the issues seen now. Andre Hedrick LAD Storage Consulting Group On Tue, 24 Jun 2003, Jeff Garzik wrote: > Bartlomiej Zolnierkiewicz wrote: > > TCQ shouldn't be enabled on hdc, you have two drives on second ide > > channel and current TCQ driver design allows only one drive per channel, > > so proper fix is to not enable TCQ :-). > > > IMO the best rule is "enable TCQ if and only if 100% of the channel > supports TCQ". > > So, two drives on the same channel can do TCQ, if and only if they both > support TCQ. That's a big benefit of bus release, after all, > simultaneously servicing multiple drives. The device-select and service > interrupt semantics are annoying but doable. > > Jeff > > > > - > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ > ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-29 2:17 ` Andre Hedrick @ 2003-06-29 8:14 ` Jan-Benedict Glaw 2003-06-29 9:30 ` Andre Hedrick 0 siblings, 1 reply; 23+ messages in thread From: Jan-Benedict Glaw @ 2003-06-29 8:14 UTC (permalink / raw) To: Andre Hedrick; +Cc: Jeff Garzik, Bartlomiej Zolnierkiewicz, axboe, linux-kernel [-- Attachment #1: Type: text/plain, Size: 1132 bytes --] On Sat, 2003-06-28 19:17:53 -0700, Andre Hedrick <andre@linux-ide.org> wrote in message <Pine.LNX.4.10.10306281917040.1116-100000@master.linux-ide.org>: > > The best rule is to default it off and let the end user enable regardless. > Thus the default will NEVER encounter the issues seen now. While I was the one who initially had the "trouble", I was thankful for having a "Switch It On by Default" option. I don't exactly know what the patch did to the IDE subsystem (personally I'm not really into IDE...), but I'd unhide a problem (either my dumb drive/controller setup or a bug in Linux' IDE code) and so - one more bug down (at least from my point of view). That's a good thing, IMHO. This *is* why I'm using 2.5.x - there's actually a chance to easily find bugs with everybody's hardware:) MfG, JBG -- Jan-Benedict Glaw jbglaw@lug-owl.de . +49-172-7608481 "Eine Freie Meinung in einem Freien Kopf | Gegen Zensur | Gegen Krieg fuer einen Freien Staat voll Freier Bürger" | im Internet! | im Irak! ret = do_actions((curr | FREE_SPEECH) & ~(IRAQ_WAR_2 | DRM | TCPA)); [-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --] ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-29 8:14 ` Jan-Benedict Glaw @ 2003-06-29 9:30 ` Andre Hedrick 2003-06-29 10:45 ` Jan-Benedict Glaw 0 siblings, 1 reply; 23+ messages in thread From: Andre Hedrick @ 2003-06-29 9:30 UTC (permalink / raw) To: Jan-Benedict Glaw Cc: Jeff Garzik, Bartlomiej Zolnierkiewicz, axboe, linux-kernel You missed a point. Defaulting to off means stablity for the masses. Enduser enable is for testing to get it correct. Regardless, it was my $0.02 for the thread. Cheers, Andre Hedrick LAD Storage Consulting Group On Sun, 29 Jun 2003, Jan-Benedict Glaw wrote: > On Sat, 2003-06-28 19:17:53 -0700, Andre Hedrick <andre@linux-ide.org> > wrote in message <Pine.LNX.4.10.10306281917040.1116-100000@master.linux-ide.org>: > > > > The best rule is to default it off and let the end user enable regardless. > > Thus the default will NEVER encounter the issues seen now. > > While I was the one who initially had the "trouble", I was thankful for > having a "Switch It On by Default" option. I don't exactly know what the > patch did to the IDE subsystem (personally I'm not really into IDE...), > but I'd unhide a problem (either my dumb drive/controller setup or a bug > in Linux' IDE code) and so - one more bug down (at least from my point > of view). That's a good thing, IMHO. > > This *is* why I'm using 2.5.x - there's actually a chance to easily find > bugs with everybody's hardware:) > > MfG, JBG > > -- > Jan-Benedict Glaw jbglaw@lug-owl.de . +49-172-7608481 > "Eine Freie Meinung in einem Freien Kopf | Gegen Zensur | Gegen Krieg > fuer einen Freien Staat voll Freier Bürger" | im Internet! | im Irak! > ret = do_actions((curr | FREE_SPEECH) & ~(IRAQ_WAR_2 | DRM | TCPA)); > ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: Testing IDE-TCQ and Taskfile - doesn't work nicely:) 2003-06-29 9:30 ` Andre Hedrick @ 2003-06-29 10:45 ` Jan-Benedict Glaw 0 siblings, 0 replies; 23+ messages in thread From: Jan-Benedict Glaw @ 2003-06-29 10:45 UTC (permalink / raw) To: Andre Hedrick; +Cc: Jeff Garzik, Bartlomiej Zolnierkiewicz, axboe, linux-kernel [-- Attachment #1: Type: text/plain, Size: 1509 bytes --] On Sun, 2003-06-29 02:30:29 -0700, Andre Hedrick <andre@linux-ide.org> wrote in message <Pine.LNX.4.10.10306290229280.2711-100000@master.linux-ide.org>: > > You missed a point. > > Defaulting to off means stablity for the masses. > Enduser enable is for testing to get it correct. > Regardless, it was my $0.02 for the thread. No, I didn't. We're in 2.5.x timeline and doing early testing (esp. with "exotic" or rarely used features) is IMHO best done at early times. THis is why I did it. For now, I don't exactly care for pure stability _if_ a user can easily choose really dangerous code, esp. if it's advised as experimental, but "you should say yes" in it's help text. This is, I like to compile with many features to see where it breaks. Not doing this means to walk through a mines field later, at 2.6.x times. That is playing minesweeper with some poor end user. No good... MfG, JBG Btw., that patch got merged to Linus and I'm currently running 2.5.73-bk6 with TF and TCQ enabled (in config, but gets somewhat disabled because of missing hardware of too many drives per channel) on a laptop and my (quite a bit loaded) mirror box. No problems / corruption so far:) -- Jan-Benedict Glaw jbglaw@lug-owl.de . +49-172-7608481 "Eine Freie Meinung in einem Freien Kopf | Gegen Zensur | Gegen Krieg fuer einen Freien Staat voll Freier Bürger" | im Internet! | im Irak! ret = do_actions((curr | FREE_SPEECH) & ~(IRAQ_WAR_2 | DRM | TCPA)); [-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --] ^ permalink raw reply [flat|nested] 23+ messages in thread
end of thread, other threads:[~2003-06-29 10:31 UTC | newest] Thread overview: 23+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2003-06-23 19:45 Testing IDE-TCQ and Taskfile - doesn't work nicely:) Jan-Benedict Glaw 2003-06-23 21:18 ` Bartlomiej Zolnierkiewicz 2003-06-23 22:19 ` Jan-Benedict Glaw 2003-06-23 23:50 ` Jan-Benedict Glaw 2003-06-24 0:53 ` Bartlomiej Zolnierkiewicz 2003-06-24 6:31 ` Jan-Benedict Glaw 2003-06-24 13:44 ` Bartlomiej Zolnierkiewicz 2003-06-24 22:40 ` Jan-Benedict Glaw 2003-06-24 23:08 ` Bartlomiej Zolnierkiewicz 2003-06-25 18:22 ` SOLVED - " Jan-Benedict Glaw 2003-06-25 18:44 ` Bartlomiej Zolnierkiewicz 2003-06-25 19:05 ` Jan-Benedict Glaw 2003-06-25 19:30 ` Bartlomiej Zolnierkiewicz 2003-06-26 3:21 ` Zwane Mwaikambo 2003-06-26 5:58 ` Jan-Benedict Glaw 2003-06-24 8:06 ` Jens Axboe 2003-06-24 14:28 ` Jeff Garzik 2003-06-24 14:47 ` Jens Axboe 2003-06-24 15:39 ` Jeff Garzik 2003-06-29 2:17 ` Andre Hedrick 2003-06-29 8:14 ` Jan-Benedict Glaw 2003-06-29 9:30 ` Andre Hedrick 2003-06-29 10:45 ` Jan-Benedict Glaw
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).