All of lore.kernel.org
 help / color / mirror / Atom feed
From: Soeren Sonnenburg <kernel@nn7.de>
To: Tejun Heo <htejun@gmail.com>
Cc: Root Kit <rootkit85@yahoo.it>, linux-ide@vger.kernel.org
Subject: Re: [PATCH] AHCI PCI ID for MacBook Pro
Date: Fri, 09 Feb 2007 16:45:49 +0100	[thread overview]
Message-ID: <1171035949.23440.4.camel@localhost> (raw)
In-Reply-To: <45CC3FE5.2010408@gmail.com>

On Fri, 2007-02-09 at 04:33 -0500, Tejun Heo wrote:
> Root Kit wrote:
> > hi,
> > i have a MacBook pro and i noticed that linux
> > (2.6.20)
> > binds the ata_piix driver
> > to the SATA controller.
> > being my SATA controller an AHCI (ICH7M) i added the
> > PCI ID to ahci.c
> > then tested the system.
> > i inline the patch here:
> 
> Great.  Care to post a proper patch with leading description and SOB? 
> Please take a look at the following patch.  You can omit diffstat result.

Hmmhh I cannot really reproduce this benchmark. Matteo, with which
benchmarking program did you produce this ?

Below is what I get when compiling shogun with make -j 4 (src from
www.shogun-toolbox.org) and a bonnie++ run:

piix:

$ /usr/sbin/bonnie++ -d ./
Writing with putc()...done
Writing intelligently...done
Rewriting...done
Reading with getc()...done
Reading intelligently...done
start 'em...done...done...done...
Create files in sequential order...done.
Stat files in sequential order...done.
Delete files in sequential order...done.
Create files in random order...done.
Stat files in random order...done.
Delete files in random order...done.
Version  1.03       ------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
no               4G 23865  57 24221   6 11588   2 26972  60 27942   2  88.5   0
                    ------Sequential Create------ --------Random Create--------
                    -Create-- --Read--- -Delete-- -Create-- --Read--- -Delete--
              files  /sec %CP  /sec %CP  /sec %CP  /sec %CP  /sec %CP  /sec %CP
                 16 +++++ +++ +++++ +++ +++++ +++ +++++ +++ +++++ +++ +++++ +++
no,4G,23865,57,24221,6,11588,2,26972,60,27942,2,88.5,0,16,+++++,+++,+++++,+++,+++++,+++,+++++,+++,+++++,+++,+++++,+++

$ time make -j4
real    2m40.799s
user    4m9.374s
sys     0m8.719s

ahci:
$ /usr/sbin/bonnie++ -d ./
Writing with putc()...done
Writing intelligently...done
Rewriting...done
Reading with getc()...done
Reading intelligently...done
start 'em...done...done...done...
Create files in sequential order...done.
Stat files in sequential order...done.
Delete files in sequential order...done.
Create files in random order...done.
Stat files in random order...done.
Delete files in random order...done.
Version  1.03       ------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
no               4G 20872  48 29348   7 11642   2 28080  60 27293   2 126.5   0
                    ------Sequential Create------ --------Random Create--------
                    -Create-- --Read--- -Delete-- -Create-- --Read--- -Delete--
              files  /sec %CP  /sec %CP  /sec %CP  /sec %CP  /sec %CP  /sec %CP
                 16 +++++ +++ +++++ +++ +++++ +++ +++++ +++ +++++ +++ +++++ +++
no,4G,20872,48,29348,7,11642,2,28080,60,27293,2,126.5,0,16,+++++,+++,+++++,+++,+++++,+++,+++++,+++,+++++,+++,+++++,+++

$ time make -j4
real    2m33.293s
user    4m1.954s
sys     0m8.533s

-- 
Sometimes, there's a moment as you're waking, when you become aware of
the real world around you, but you're still dreaming.

  parent reply	other threads:[~2007-02-09 15:52 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-02-08 17:33 [PATCH] AHCI PCI ID for MacBook Pro Root Kit
2007-02-09  9:33 ` Tejun Heo
2007-02-09 14:27   ` Matteo Croce
2007-02-09 14:54     ` David M. Lloyd
2007-02-09 17:39       ` Matteo Croce
2007-02-09 18:15         ` David M. Lloyd
2007-02-09 18:51           ` Matteo Croce
2007-02-10 10:22           ` Soeren Sonnenburg
2007-02-10 14:03         ` Tejun Heo
2007-02-10 16:07           ` Soeren Sonnenburg
2007-02-10 17:10             ` Tejun Heo
2007-02-11 15:40           ` David M. Lloyd
2007-02-09 15:45   ` Soeren Sonnenburg [this message]
2007-02-09 17:41     ` Matteo Croce
2007-02-10 10:19       ` Soeren Sonnenburg
2007-02-10 13:17         ` Tejun Heo
2007-02-10 16:03           ` Soeren Sonnenburg
2007-02-10 17:17             ` Tejun Heo

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1171035949.23440.4.camel@localhost \
    --to=kernel@nn7.de \
    --cc=htejun@gmail.com \
    --cc=linux-ide@vger.kernel.org \
    --cc=rootkit85@yahoo.it \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.