All of lore.kernel.org
 help / color / mirror / Atom feed
From: Steven Pratt <slpratt@austin.ibm.com>
To: linux-scsi@vger.kernel.org
Subject: MPTSAS Driver and scatter gather lists
Date: Tue, 04 Dec 2007 09:45:04 -0600	[thread overview]
Message-ID: <47557600.5040902@austin.ibm.com> (raw)

I was doing some performance comparisons between the aic94xx sas adapter 
and the mptsas (lsi1078 chip) and saw that performance was slightly 
lower on lsi using the same drive. One thing I noticed the mptsas driver 
was only setting sg_tablesize to 40 which restricts the maximum size of 
the IO to 160kb.  This is inefficient and so I changed it to 255 (max 
value) for testing purposes.  The results were not what I expected.  
While CPU efficiency was improved at the larger block sizes (like I had 
hopped), there were drastic improvements at small (<16k) block sizes 
which is completely surprising).  So I decided to see where all the time 
was being spent.  When doing oprofile, loss of idle ticks was confusing 
things so I booted with idle=poll, and magically the original mptsas 
driver now performs like the modified version at the small block sizes.
My suspicion is that we have some some sleep delay which is minimized by 
going into poll mode, but I have no idea why changing the sg_tablesize 
gives the same results regardless of poll mode.  I should reiterate that 
I am not talking about small difference. This is 2x performance at 1/3 cpu.

Also, is there any reason we can't increase sg_tablesize for mptsas?

Any thoughts or experiments to try are welcome.
Steve

             reply	other threads:[~2007-12-04 15:45 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-12-04 15:45 Steven Pratt [this message]
2007-12-04 19:58 ` MPTSAS Driver and scatter gather lists Moore, Eric
2007-12-04 22:48   ` Steven Pratt
2007-12-11  8:18 ` Zhao Forrest
2007-12-11 14:49   ` Steven Pratt
2007-12-11 14:59   ` Steven Pratt

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=47557600.5040902@austin.ibm.com \
    --to=slpratt@austin.ibm.com \
    --cc=linux-scsi@vger.kernel.org \
    /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.