* [PATCH 2/2] Revert "Revert "ide: try to use PIO Mode 0 during probe if possible""
@ 2009-10-06 22:27 Bartlomiej Zolnierkiewicz
2009-10-06 22:41 ` David Miller
0 siblings, 1 reply; 6+ messages in thread
From: Bartlomiej Zolnierkiewicz @ 2009-10-06 22:27 UTC (permalink / raw)
To: Linus Torvalds
Cc: David S. Miller, Joao Ramos, David Fries, d.stussy, linux-ide,
linux-kernel
From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Subject: [PATCH] Revert "Revert "ide: try to use PIO Mode 0 during probe if possible""
This reverts commit 24df31acaff8465d797f0006437b45ad0f2a5cb1.
The root cause of reported system hangs was (now fixed) sis5513 bug
and not "ide: try to use PIO Mode 0 during probe if possible" change
(commit 6029336426a2b43e4bc6f4a84be8789a047d139e) so the revert was
incorrect (it simply replaced one regression with the other one).
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Cc: David S. Miller <davem@davemloft.net>
Cc: Joao Ramos <joao.ramos@inov.pt>
Cc: David Fries <david@fries.net>
Cc: d.stussy@yahoo.com
---
drivers/ide/ide-probe.c | 9 +++++++++
1 files changed, 9 insertions(+), 0 deletions(-)
diff --git a/drivers/ide/ide-probe.c b/drivers/ide/ide-probe.c
index 4d76ba4..63c53d6 100644
--- a/drivers/ide/ide-probe.c
+++ b/drivers/ide/ide-probe.c
@@ -1046,6 +1046,15 @@ static void ide_port_init_devices(ide_hwif_t *hwif)
if (port_ops && port_ops->init_dev)
port_ops->init_dev(drive);
}
+
+ ide_port_for_each_dev(i, drive, hwif) {
+ /*
+ * default to PIO Mode 0 before we figure out
+ * the most suited mode for the attached device
+ */
+ if (port_ops && port_ops->set_pio_mode)
+ port_ops->set_pio_mode(drive, 0);
+ }
}
static void ide_init_port(ide_hwif_t *hwif, unsigned int port,
--
1.6.5.rc2
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH 2/2] Revert "Revert "ide: try to use PIO Mode 0 during probe if possible""
2009-10-06 22:27 [PATCH 2/2] Revert "Revert "ide: try to use PIO Mode 0 during probe if possible"" Bartlomiej Zolnierkiewicz
@ 2009-10-06 22:41 ` David Miller
2009-10-06 22:52 ` Bartlomiej Zolnierkiewicz
0 siblings, 1 reply; 6+ messages in thread
From: David Miller @ 2009-10-06 22:41 UTC (permalink / raw)
To: bzolnier; +Cc: torvalds, joao.ramos, david, d.stussy, linux-ide, linux-kernel
From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Date: Wed, 7 Oct 2009 00:27:45 +0200
> The root cause of reported system hangs was (now fixed) sis5513 bug
> and not "ide: try to use PIO Mode 0 during probe if possible" change
> (commit 6029336426a2b43e4bc6f4a84be8789a047d139e) so the revert was
> incorrect (it simply replaced one regression with the other one).
What is this older "regression" fixed by the PIO-0 change?
You're not explaining it and neither does the commit message for the
PIO-0 change.
Please, fill us in :-)
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] Revert "Revert "ide: try to use PIO Mode 0 during probe if possible""
2009-10-06 22:41 ` David Miller
@ 2009-10-06 22:52 ` Bartlomiej Zolnierkiewicz
2009-10-06 22:57 ` David Miller
0 siblings, 1 reply; 6+ messages in thread
From: Bartlomiej Zolnierkiewicz @ 2009-10-06 22:52 UTC (permalink / raw)
To: David Miller
Cc: torvalds, joao.ramos, david, d.stussy, linux-ide, linux-kernel
On Wednesday 07 October 2009 00:41:20 David Miller wrote:
> From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
> Date: Wed, 7 Oct 2009 00:27:45 +0200
>
> > The root cause of reported system hangs was (now fixed) sis5513 bug
> > and not "ide: try to use PIO Mode 0 during probe if possible" change
> > (commit 6029336426a2b43e4bc6f4a84be8789a047d139e) so the revert was
> > incorrect (it simply replaced one regression with the other one).
>
> What is this older "regression" fixed by the PIO-0 change?
>
> You're not explaining it and neither does the commit message for the
> PIO-0 change.
It does:
| Subject: [PATCH] sis5513: fix PIO setup for ATAPI devices
|
| Clear prefetch setting before potentially (re-)enabling it in
| config_drive_art_rwp() so the transition of the device type on
| the port from ATA to ATAPI (i.e. during warm-plug operation)
| is handled correctly.
and the empty port defaults to ATA-like behavior for the initial PIO setup.
> Please, fill us in :-)
Just connect the dots.. :)
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] Revert "Revert "ide: try to use PIO Mode 0 during probe if possible""
2009-10-06 22:52 ` Bartlomiej Zolnierkiewicz
@ 2009-10-06 22:57 ` David Miller
2009-10-06 23:13 ` Bartlomiej Zolnierkiewicz
0 siblings, 1 reply; 6+ messages in thread
From: David Miller @ 2009-10-06 22:57 UTC (permalink / raw)
To: bzolnier; +Cc: torvalds, joao.ramos, david, d.stussy, linux-ide, linux-kernel
From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Date: Wed, 7 Oct 2009 00:52:37 +0200
> On Wednesday 07 October 2009 00:41:20 David Miller wrote:
>> From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
>> Date: Wed, 7 Oct 2009 00:27:45 +0200
>>
>> > The root cause of reported system hangs was (now fixed) sis5513 bug
>> > and not "ide: try to use PIO Mode 0 during probe if possible" change
>> > (commit 6029336426a2b43e4bc6f4a84be8789a047d139e) so the revert was
>> > incorrect (it simply replaced one regression with the other one).
>>
>> What is this older "regression" fixed by the PIO-0 change?
>>
>> You're not explaining it and neither does the commit message for the
>> PIO-0 change.
>
> It does:
>
> | Subject: [PATCH] sis5513: fix PIO setup for ATAPI devices
Sorry, I wasn't clear.
I'm talking about commit 6029336426a2b43e4bc6f4a84be8789a047d139e, it
doesn't say why we want to try PIO 0 mode during probe.
What was the regression fixed by that change?
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] Revert "Revert "ide: try to use PIO Mode 0 during probe if possible""
2009-10-06 22:57 ` David Miller
@ 2009-10-06 23:13 ` Bartlomiej Zolnierkiewicz
2009-10-06 23:34 ` David Miller
0 siblings, 1 reply; 6+ messages in thread
From: Bartlomiej Zolnierkiewicz @ 2009-10-06 23:13 UTC (permalink / raw)
To: David Miller
Cc: torvalds, joao.ramos, david, d.stussy, linux-ide, linux-kernel
On Wednesday 07 October 2009 00:57:05 David Miller wrote:
> From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
> Date: Wed, 7 Oct 2009 00:52:37 +0200
>
> > On Wednesday 07 October 2009 00:41:20 David Miller wrote:
> >> From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
> >> Date: Wed, 7 Oct 2009 00:27:45 +0200
> >>
> >> > The root cause of reported system hangs was (now fixed) sis5513 bug
> >> > and not "ide: try to use PIO Mode 0 during probe if possible" change
> >> > (commit 6029336426a2b43e4bc6f4a84be8789a047d139e) so the revert was
> >> > incorrect (it simply replaced one regression with the other one).
> >>
> >> What is this older "regression" fixed by the PIO-0 change?
> >>
> >> You're not explaining it and neither does the commit message for the
> >> PIO-0 change.
> >
> > It does:
> >
> > | Subject: [PATCH] sis5513: fix PIO setup for ATAPI devices
>
> Sorry, I wasn't clear.
>
> I'm talking about commit 6029336426a2b43e4bc6f4a84be8789a047d139e, it
> doesn't say why we want to try PIO 0 mode during probe.
For the same reason that libata wants it.
Many embedded controllers/platforms will have junk PIO timings set initially.
Now that your manager's ambition is satisfied could you please go help
some other subsystem for the moment so we can fix ide in the meantime? :)
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2009-10-06 23:34 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-10-06 22:27 [PATCH 2/2] Revert "Revert "ide: try to use PIO Mode 0 during probe if possible"" Bartlomiej Zolnierkiewicz
2009-10-06 22:41 ` David Miller
2009-10-06 22:52 ` Bartlomiej Zolnierkiewicz
2009-10-06 22:57 ` David Miller
2009-10-06 23:13 ` Bartlomiej Zolnierkiewicz
2009-10-06 23:34 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox