* [PATCH] make PIIX4 ide driver available for MIPS
@ 2002-07-24 0:09 Jun Sun
2002-07-24 0:40 ` Ralf Baechle
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Jun Sun @ 2002-07-24 0:09 UTC (permalink / raw)
To: Ralf Baechle, linux-mips
[-- Attachment #1: Type: text/plain, Size: 190 bytes --]
Malta uses this chip. The native driver does provide significant gain in
performance. See attached bonnie++ test results.
Two separate patches for both linux_2_4 branch and trunk.
Jun
[-- Attachment #2: 2002.07.23-make-PIIX-IDE-avaiable-for-mips.patch --]
[-- Type: text/plain, Size: 919 bytes --]
diff -Nru link/drivers/ide/Config.in.orig link/drivers/ide/Config.in
--- link/drivers/ide/Config.in.orig Wed Jun 26 15:35:44 2002
+++ link/drivers/ide/Config.in Tue Jul 23 17:03:44 2002
@@ -72,7 +72,7 @@
dep_bool ' HPT34X chipset support' CONFIG_BLK_DEV_HPT34X $CONFIG_BLK_DEV_IDEDMA_PCI
dep_mbool ' HPT34X AUTODMA support (WIP)' CONFIG_HPT34X_AUTODMA $CONFIG_BLK_DEV_HPT34X $CONFIG_IDEDMA_PCI_WIP
dep_bool ' HPT366/368/370 chipset support' CONFIG_BLK_DEV_HPT366 $CONFIG_BLK_DEV_IDEDMA_PCI
- if [ "$CONFIG_X86" = "y" -o "$CONFIG_IA64" = "y" ]; then
+ if [ "$CONFIG_X86" = "y" -o "$CONFIG_IA64" = "y" -o "$CONFIG_MIPS" = "y" -o "$CONFIG_MIPS64" = "y" ]; then
dep_mbool ' Intel PIIXn chipsets support' CONFIG_BLK_DEV_PIIX $CONFIG_BLK_DEV_IDEDMA_PCI
dep_mbool ' PIIXn Tuning support' CONFIG_PIIX_TUNING $CONFIG_BLK_DEV_PIIX $CONFIG_IDEDMA_PCI_AUTO
fi
[-- Attachment #3: 2002.07.23-2.5-make-PIIX-IDE-avaiable-for-mips.patch --]
[-- Type: text/plain, Size: 911 bytes --]
diff -Nru link/drivers/ide/Config.in.orig link/drivers/ide/Config.in
--- link/drivers/ide/Config.in.orig Wed Jul 10 14:04:49 2002
+++ link/drivers/ide/Config.in Tue Jul 23 17:06:55 2002
@@ -70,7 +70,7 @@
dep_bool ' HPT34X chipset support' CONFIG_BLK_DEV_HPT34X $CONFIG_BLK_DEV_IDEDMA_PCI
dep_mbool ' HPT34X AUTODMA support (WIP)' CONFIG_HPT34X_AUTODMA $CONFIG_BLK_DEV_HPT34X $CONFIG_IDEDMA_PCI_WIP
dep_bool ' HPT366 chipset support' CONFIG_BLK_DEV_HPT366 $CONFIG_BLK_DEV_IDEDMA_PCI
- if [ "$CONFIG_X86" = "y" -o "$CONFIG_IA64" = "y" ]; then
+ if [ "$CONFIG_X86" = "y" -o "$CONFIG_IA64" = "y" -o "$CONFIG_MIPS" = "y" -o "$CONFIG_MIPS64" = "y" ]; then
dep_mbool ' Intel PIIXn chipsets support' CONFIG_BLK_DEV_PIIX $CONFIG_BLK_DEV_IDEDMA_PCI
dep_mbool ' PIIXn Tuning support' CONFIG_PIIX_TUNING $CONFIG_BLK_DEV_PIIX $CONFIG_IDEDMA_PCI_AUTO
fi
[-- Attachment #4: junk --]
[-- Type: text/plain, Size: 2659 bytes --]
==========================
. bonnie++ test:
With PIIX4 controller code and turnning
---------------------------------------
Version 1.92 ------Sequential Output------ --Sequential Input- --Random-
-Per Chr- --Block-- -Rewrite- -Per Chr- --Block-- --Seeks--
Machine Size K/sec %CP K/sec %CP K/sec %CP K/sec %CP K/sec %CP /sec %CP
10.0.18.6 300M 44 99 4969 73 2875 65 65 99 5219 70 117.0 20
Latency 186ms 342ms 476ms 138ms 12186us 2550ms
Version 1.92 ------Sequential Create------ --------Random Create--------
10.0.18.6 -Create-- --Read--- -Delete-- -Create-- --Read--- -Delete--
files /sec %CP /sec %CP /sec %CP /sec %CP /sec %CP /sec %CP
16 83 99 1677 100 1158 98 87 99 1849 100 302 98
Latency 25192us 2257us 2009us 45175us 1163us 9985us
With PIIX4 controller code:
--------------------------
Version 1.92 ------Sequential Output------ --Sequential Input- --Random-
-Per Chr- --Block-- -Rewrite- -Per Chr- --Block-- --Seeks--
Machine Size K/sec %CP K/sec %CP K/sec %CP K/sec %CP K/sec %CP /sec %CP
10.0.18.6 300M 44 99 4548 67 2517 56 65 99 5019 69 114.1 20
Latency 186ms 436ms 1652ms 147ms 14697us 2418ms
Version 1.92 ------Sequential Create------ --------Random Create--------
10.0.18.6 -Create-- --Read--- -Delete-- -Create-- --Read--- -Delete--
files /sec %CP /sec %CP /sec %CP /sec %CP /sec %CP /sec %CP
16 83 99 1663 99 1141 98 87 99 1819 100 304 98
Latency 32274us 1690us 2826us 37199us 1157us 10074us
Use PCI generic code
--------------------
Version 1.92 ------Sequential Output------ --Sequential Input- --Random-
-Per Chr- --Block-- -Rewrite- -Per Chr- --Block-- --Seeks--
Machine Size K/sec %CP K/sec %CP K/sec %CP K/sec %CP K/sec %CP /sec %CP
10.0.18.6 300M 44 99 2433 35 1201 26 64 98 3101 44 85.5 14
Latency 186ms 4190ms 2722ms 168ms 40544us 1922ms
Version 1.92 ------Sequential Create------ --------Random Create--------
10.0.18.6 -Create-- --Read--- -Delete-- -Create-- --Read--- -Delete--
files /sec %CP /sec %CP /sec %CP /sec %CP /sec %CP /sec %CP
16 83 99 1607 99 1115 95 87 99 1788 100 305 98
Latency 24155us 2249us 2496us 33010us 1021us 10074us
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] make PIIX4 ide driver available for MIPS
2002-07-24 0:09 [PATCH] make PIIX4 ide driver available for MIPS Jun Sun
@ 2002-07-24 0:40 ` Ralf Baechle
2002-07-24 1:00 ` Zhang Fuxin
2002-07-24 14:43 ` Maciej W. Rozycki
2 siblings, 0 replies; 6+ messages in thread
From: Ralf Baechle @ 2002-07-24 0:40 UTC (permalink / raw)
To: Jun Sun; +Cc: linux-mips
On Tue, Jul 23, 2002 at 05:09:50PM -0700, Jun Sun wrote:
> Malta uses this chip. The native driver does provide significant gain in
> performance. See attached bonnie++ test results.
>
> Two separate patches for both linux_2_4 branch and trunk.
CONFIG_MIPS is defined for both 32-bit and 64-bit kernel.
Ralf
> diff -Nru link/drivers/ide/Config.in.orig link/drivers/ide/Config.in
> --- link/drivers/ide/Config.in.orig Wed Jun 26 15:35:44 2002
> +++ link/drivers/ide/Config.in Tue Jul 23 17:03:44 2002
> @@ -72,7 +72,7 @@
> dep_bool ' HPT34X chipset support' CONFIG_BLK_DEV_HPT34X $CONFIG_BLK_DEV_IDEDMA_PCI
> dep_mbool ' HPT34X AUTODMA support (WIP)' CONFIG_HPT34X_AUTODMA $CONFIG_BLK_DEV_HPT34X $CONFIG_IDEDMA_PCI_WIP
> dep_bool ' HPT366/368/370 chipset support' CONFIG_BLK_DEV_HPT366 $CONFIG_BLK_DEV_IDEDMA_PCI
> - if [ "$CONFIG_X86" = "y" -o "$CONFIG_IA64" = "y" ]; then
> + if [ "$CONFIG_X86" = "y" -o "$CONFIG_IA64" = "y" -o "$CONFIG_MIPS" = "y" -o "$CONFIG_MIPS64" = "y" ]; then
> dep_mbool ' Intel PIIXn chipsets support' CONFIG_BLK_DEV_PIIX $CONFIG_BLK_DEV_IDEDMA_PCI
> dep_mbool ' PIIXn Tuning support' CONFIG_PIIX_TUNING $CONFIG_BLK_DEV_PIIX $CONFIG_IDEDMA_PCI_AUTO
> fi
> diff -Nru link/drivers/ide/Config.in.orig link/drivers/ide/Config.in
> --- link/drivers/ide/Config.in.orig Wed Jul 10 14:04:49 2002
> +++ link/drivers/ide/Config.in Tue Jul 23 17:06:55 2002
> @@ -70,7 +70,7 @@
> dep_bool ' HPT34X chipset support' CONFIG_BLK_DEV_HPT34X $CONFIG_BLK_DEV_IDEDMA_PCI
> dep_mbool ' HPT34X AUTODMA support (WIP)' CONFIG_HPT34X_AUTODMA $CONFIG_BLK_DEV_HPT34X $CONFIG_IDEDMA_PCI_WIP
> dep_bool ' HPT366 chipset support' CONFIG_BLK_DEV_HPT366 $CONFIG_BLK_DEV_IDEDMA_PCI
> - if [ "$CONFIG_X86" = "y" -o "$CONFIG_IA64" = "y" ]; then
> + if [ "$CONFIG_X86" = "y" -o "$CONFIG_IA64" = "y" -o "$CONFIG_MIPS" = "y" -o "$CONFIG_MIPS64" = "y" ]; then
> dep_mbool ' Intel PIIXn chipsets support' CONFIG_BLK_DEV_PIIX $CONFIG_BLK_DEV_IDEDMA_PCI
> dep_mbool ' PIIXn Tuning support' CONFIG_PIIX_TUNING $CONFIG_BLK_DEV_PIIX $CONFIG_IDEDMA_PCI_AUTO
> fi
>
> ==========================
> . bonnie++ test:
>
> With PIIX4 controller code and turnning
> ---------------------------------------
>
> Version 1.92 ------Sequential Output------ --Sequential Input- --Random-
> -Per Chr- --Block-- -Rewrite- -Per Chr- --Block-- --Seeks--
> Machine Size K/sec %CP K/sec %CP K/sec %CP K/sec %CP K/sec %CP /sec %CP
> 10.0.18.6 300M 44 99 4969 73 2875 65 65 99 5219 70 117.0 20
> Latency 186ms 342ms 476ms 138ms 12186us 2550ms
> Version 1.92 ------Sequential Create------ --------Random Create--------
> 10.0.18.6 -Create-- --Read--- -Delete-- -Create-- --Read--- -Delete--
> files /sec %CP /sec %CP /sec %CP /sec %CP /sec %CP /sec %CP
> 16 83 99 1677 100 1158 98 87 99 1849 100 302 98
> Latency 25192us 2257us 2009us 45175us 1163us 9985us
>
> With PIIX4 controller code:
> --------------------------
>
> Version 1.92 ------Sequential Output------ --Sequential Input- --Random-
> -Per Chr- --Block-- -Rewrite- -Per Chr- --Block-- --Seeks--
> Machine Size K/sec %CP K/sec %CP K/sec %CP K/sec %CP K/sec %CP /sec %CP
> 10.0.18.6 300M 44 99 4548 67 2517 56 65 99 5019 69 114.1 20
> Latency 186ms 436ms 1652ms 147ms 14697us 2418ms
> Version 1.92 ------Sequential Create------ --------Random Create--------
> 10.0.18.6 -Create-- --Read--- -Delete-- -Create-- --Read--- -Delete--
> files /sec %CP /sec %CP /sec %CP /sec %CP /sec %CP /sec %CP
> 16 83 99 1663 99 1141 98 87 99 1819 100 304 98
> Latency 32274us 1690us 2826us 37199us 1157us 10074us
>
> Use PCI generic code
> --------------------
> Version 1.92 ------Sequential Output------ --Sequential Input- --Random-
> -Per Chr- --Block-- -Rewrite- -Per Chr- --Block-- --Seeks--
> Machine Size K/sec %CP K/sec %CP K/sec %CP K/sec %CP K/sec %CP /sec %CP
> 10.0.18.6 300M 44 99 2433 35 1201 26 64 98 3101 44 85.5 14
> Latency 186ms 4190ms 2722ms 168ms 40544us 1922ms
> Version 1.92 ------Sequential Create------ --------Random Create--------
> 10.0.18.6 -Create-- --Read--- -Delete-- -Create-- --Read--- -Delete--
> files /sec %CP /sec %CP /sec %CP /sec %CP /sec %CP /sec %CP
> 16 83 99 1607 99 1115 95 87 99 1788 100 305 98
> Latency 24155us 2249us 2496us 33010us 1021us 10074us
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] make PIIX4 ide driver available for MIPS
2002-07-24 0:09 [PATCH] make PIIX4 ide driver available for MIPS Jun Sun
2002-07-24 0:40 ` Ralf Baechle
@ 2002-07-24 1:00 ` Zhang Fuxin
2002-07-24 14:43 ` Maciej W. Rozycki
2 siblings, 0 replies; 6+ messages in thread
From: Zhang Fuxin @ 2002-07-24 1:00 UTC (permalink / raw)
To: Jun Sun; +Cc: Ralf Baechle, linux-mips
Algor p6032 board use it too. We have been using native driver for a
long time.
Jun Sun wrote:
>
> Malta uses this chip. The native driver does provide significant gain
> in performance. See attached bonnie++ test results.
>
> Two separate patches for both linux_2_4 branch and trunk.
>
> Jun
>
>
>------------------------------------------------------------------------
>
>diff -Nru link/drivers/ide/Config.in.orig link/drivers/ide/Config.in
>--- link/drivers/ide/Config.in.orig Wed Jun 26 15:35:44 2002
>+++ link/drivers/ide/Config.in Tue Jul 23 17:03:44 2002
>@@ -72,7 +72,7 @@
> dep_bool ' HPT34X chipset support' CONFIG_BLK_DEV_HPT34X $CONFIG_BLK_DEV_IDEDMA_PCI
> dep_mbool ' HPT34X AUTODMA support (WIP)' CONFIG_HPT34X_AUTODMA $CONFIG_BLK_DEV_HPT34X $CONFIG_IDEDMA_PCI_WIP
> dep_bool ' HPT366/368/370 chipset support' CONFIG_BLK_DEV_HPT366 $CONFIG_BLK_DEV_IDEDMA_PCI
>- if [ "$CONFIG_X86" = "y" -o "$CONFIG_IA64" = "y" ]; then
>+ if [ "$CONFIG_X86" = "y" -o "$CONFIG_IA64" = "y" -o "$CONFIG_MIPS" = "y" -o "$CONFIG_MIPS64" = "y" ]; then
> dep_mbool ' Intel PIIXn chipsets support' CONFIG_BLK_DEV_PIIX $CONFIG_BLK_DEV_IDEDMA_PCI
> dep_mbool ' PIIXn Tuning support' CONFIG_PIIX_TUNING $CONFIG_BLK_DEV_PIIX $CONFIG_IDEDMA_PCI_AUTO
> fi
>
>
>------------------------------------------------------------------------
>
>diff -Nru link/drivers/ide/Config.in.orig link/drivers/ide/Config.in
>--- link/drivers/ide/Config.in.orig Wed Jul 10 14:04:49 2002
>+++ link/drivers/ide/Config.in Tue Jul 23 17:06:55 2002
>@@ -70,7 +70,7 @@
> dep_bool ' HPT34X chipset support' CONFIG_BLK_DEV_HPT34X $CONFIG_BLK_DEV_IDEDMA_PCI
> dep_mbool ' HPT34X AUTODMA support (WIP)' CONFIG_HPT34X_AUTODMA $CONFIG_BLK_DEV_HPT34X $CONFIG_IDEDMA_PCI_WIP
> dep_bool ' HPT366 chipset support' CONFIG_BLK_DEV_HPT366 $CONFIG_BLK_DEV_IDEDMA_PCI
>- if [ "$CONFIG_X86" = "y" -o "$CONFIG_IA64" = "y" ]; then
>+ if [ "$CONFIG_X86" = "y" -o "$CONFIG_IA64" = "y" -o "$CONFIG_MIPS" = "y" -o "$CONFIG_MIPS64" = "y" ]; then
> dep_mbool ' Intel PIIXn chipsets support' CONFIG_BLK_DEV_PIIX $CONFIG_BLK_DEV_IDEDMA_PCI
> dep_mbool ' PIIXn Tuning support' CONFIG_PIIX_TUNING $CONFIG_BLK_DEV_PIIX $CONFIG_IDEDMA_PCI_AUTO
> fi
>
>
>------------------------------------------------------------------------
>
>
>==========================
>. bonnie++ test:
>
>With PIIX4 controller code and turnning
>---------------------------------------
>
>Version 1.92 ------Sequential Output------ --Sequential Input- --Random-
> -Per Chr- --Block-- -Rewrite- -Per Chr- --Block-- --Seeks--
>Machine Size K/sec %CP K/sec %CP K/sec %CP K/sec %CP K/sec %CP /sec %CP
>10.0.18.6 300M 44 99 4969 73 2875 65 65 99 5219 70 117.0 20
>Latency 186ms 342ms 476ms 138ms 12186us 2550ms
>Version 1.92 ------Sequential Create------ --------Random Create--------
>10.0.18.6 -Create-- --Read--- -Delete-- -Create-- --Read--- -Delete--
> files /sec %CP /sec %CP /sec %CP /sec %CP /sec %CP /sec %CP
> 16 83 99 1677 100 1158 98 87 99 1849 100 302 98
>Latency 25192us 2257us 2009us 45175us 1163us 9985us
>
>With PIIX4 controller code:
>--------------------------
>
>Version 1.92 ------Sequential Output------ --Sequential Input- --Random-
> -Per Chr- --Block-- -Rewrite- -Per Chr- --Block-- --Seeks--
>Machine Size K/sec %CP K/sec %CP K/sec %CP K/sec %CP K/sec %CP /sec %CP
>10.0.18.6 300M 44 99 4548 67 2517 56 65 99 5019 69 114.1 20
>Latency 186ms 436ms 1652ms 147ms 14697us 2418ms
>Version 1.92 ------Sequential Create------ --------Random Create--------
>10.0.18.6 -Create-- --Read--- -Delete-- -Create-- --Read--- -Delete--
> files /sec %CP /sec %CP /sec %CP /sec %CP /sec %CP /sec %CP
> 16 83 99 1663 99 1141 98 87 99 1819 100 304 98
>Latency 32274us 1690us 2826us 37199us 1157us 10074us
>
>Use PCI generic code
>--------------------
>Version 1.92 ------Sequential Output------ --Sequential Input- --Random-
> -Per Chr- --Block-- -Rewrite- -Per Chr- --Block-- --Seeks--
>Machine Size K/sec %CP K/sec %CP K/sec %CP K/sec %CP K/sec %CP /sec %CP
>10.0.18.6 300M 44 99 2433 35 1201 26 64 98 3101 44 85.5 14
>Latency 186ms 4190ms 2722ms 168ms 40544us 1922ms
>Version 1.92 ------Sequential Create------ --------Random Create--------
>10.0.18.6 -Create-- --Read--- -Delete-- -Create-- --Read--- -Delete--
> files /sec %CP /sec %CP /sec %CP /sec %CP /sec %CP /sec %CP
> 16 83 99 1607 99 1115 95 87 99 1788 100 305 98
>Latency 24155us 2249us 2496us 33010us 1021us 10074us
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] make PIIX4 ide driver available for MIPS
2002-07-24 0:09 [PATCH] make PIIX4 ide driver available for MIPS Jun Sun
2002-07-24 0:40 ` Ralf Baechle
2002-07-24 1:00 ` Zhang Fuxin
@ 2002-07-24 14:43 ` Maciej W. Rozycki
2002-07-24 14:46 ` Ralf Baechle
2 siblings, 1 reply; 6+ messages in thread
From: Maciej W. Rozycki @ 2002-07-24 14:43 UTC (permalink / raw)
To: Jun Sun; +Cc: Ralf Baechle, linux-mips
On Tue, 23 Jul 2002, Jun Sun wrote:
> Malta uses this chip. The native driver does provide significant gain in
> performance. See attached bonnie++ test results.
It's actually weird there are any CPU conditionals there at all. The
PIIX4 is a generic PCI-ISA bridge after all, so it can be used on any
system equipped with a PCI bus. DEC Alpha systems used to use Intel's
bridges for EISA and ISA busses as well.
--
+ Maciej W. Rozycki, Technical University of Gdansk, Poland +
+--------------------------------------------------------------+
+ e-mail: macro@ds2.pg.gda.pl, PGP key available +
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] make PIIX4 ide driver available for MIPS
2002-07-24 14:43 ` Maciej W. Rozycki
@ 2002-07-24 14:46 ` Ralf Baechle
2002-07-24 14:56 ` Maciej W. Rozycki
0 siblings, 1 reply; 6+ messages in thread
From: Ralf Baechle @ 2002-07-24 14:46 UTC (permalink / raw)
To: Maciej W. Rozycki; +Cc: Jun Sun, linux-mips
On Wed, Jul 24, 2002 at 04:43:39PM +0200, Maciej W. Rozycki wrote:
> > Malta uses this chip. The native driver does provide significant gain in
> > performance. See attached bonnie++ test results.
>
> It's actually weird there are any CPU conditionals there at all. The
> PIIX4 is a generic PCI-ISA bridge after all, so it can be used on any
> system equipped with a PCI bus. DEC Alpha systems used to use Intel's
> bridges for EISA and ISA busses as well.
I bet it's historical reasons and the whole architecture conditional
plain should go away ...
Ralf
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] make PIIX4 ide driver available for MIPS
2002-07-24 14:46 ` Ralf Baechle
@ 2002-07-24 14:56 ` Maciej W. Rozycki
0 siblings, 0 replies; 6+ messages in thread
From: Maciej W. Rozycki @ 2002-07-24 14:56 UTC (permalink / raw)
To: Ralf Baechle; +Cc: Jun Sun, linux-mips
On Wed, 24 Jul 2002, Ralf Baechle wrote:
> > It's actually weird there are any CPU conditionals there at all. The
> > PIIX4 is a generic PCI-ISA bridge after all, so it can be used on any
> > system equipped with a PCI bus. DEC Alpha systems used to use Intel's
> > bridges for EISA and ISA busses as well.
>
> I bet it's historical reasons and the whole architecture conditional
> plain should go away ...
The explicit support for the PIIX family is relatively new, IIRC. I
suppose it's rather the "it's an Intel, so obviously it works with an
Intel only" thinking. I'm for the removal as well.
--
+ Maciej W. Rozycki, Technical University of Gdansk, Poland +
+--------------------------------------------------------------+
+ e-mail: macro@ds2.pg.gda.pl, PGP key available +
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2002-07-24 14:55 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2002-07-24 0:09 [PATCH] make PIIX4 ide driver available for MIPS Jun Sun
2002-07-24 0:40 ` Ralf Baechle
2002-07-24 1:00 ` Zhang Fuxin
2002-07-24 14:43 ` Maciej W. Rozycki
2002-07-24 14:46 ` Ralf Baechle
2002-07-24 14:56 ` Maciej W. Rozycki
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.