All of lore.kernel.org
 help / color / mirror / Atom feed
From: bugzilla-daemon@bugzilla.kernel.org
To: linux-scsi@vger.kernel.org
Subject: [Bug 15007] SCSI host adapter's scatter-gather list size Issue
Date: Thu, 14 Jan 2010 12:49:56 GMT	[thread overview]
Message-ID: <201001141249.o0ECnuTH008110@demeter.kernel.org> (raw)
In-Reply-To: <bug-15007-11613@http.bugzilla.kernel.org/>

http://bugzilla.kernel.org/show_bug.cgi?id=15007





--- Comment #5 from Anonymous Emailer <anonymous@kernel-bugs.osdl.org>  2010-01-14 12:49:54 ---
Reply-To: fujita.tomonori@lab.ntt.co.jp

On Thu, 14 Jan 2010 12:27:12 GMT
bugzilla-daemon@bugzilla.kernel.org wrote:

> http://bugzilla.kernel.org/show_bug.cgi?id=15007
> 
> 
> 
> 
> 
> --- Comment #4 from nick cheng <nick.cheng@areca.com.tw>  2010-01-14 12:27:11 ---
> Sorry, is it a really bug?

Yes because if sg entries are larger than 127, they are not
continuous. So sg++ doesn't work. We chain sg entries. The last sg
entry is a pointer to the next sg list.

For example, a command has 128 entries, it has two sg lists. The first
sg list has 127 sg entries. The last entry in the first sg list is a
pointer to the second sg list. The second sg list has one sg
entry. The command has 128 entries in total.

My previous mail is partially wrong. As I explained above, if sg
entries is 128, this bug leads to bogus 128th sg entry. So I think
that you hit this bug.


> Does it bug anything? I can't figure it out.
> I ever used scsi_for_each_sg() and found the performance is inferior to the
> original version. I tracked the source code for scsi_for_each_sg()and there is
> no two much difference to each other. I doubt it is compiler issue to make it
> worst.
> But it is my guess. Do you know why? 
> Anyway, will you or anybody take care of this issue?

-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

  parent reply	other threads:[~2010-01-14 12:49 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-01-08 11:07 [Bug 15007] New: SCSI host adapter's scatter-gather list size Issue bugzilla-daemon
2010-01-08 11:11 ` [Bug 15007] " bugzilla-daemon
2010-01-14  8:25 ` [Bug 15007] New: " FUJITA Tomonori
2010-01-14  8:25 ` [Bug 15007] " bugzilla-daemon
2010-01-14 10:54 ` bugzilla-daemon
2010-01-14 11:12   ` FUJITA Tomonori
2010-01-14 11:12 ` bugzilla-daemon
2010-01-14 12:27 ` bugzilla-daemon
2010-01-14 12:49   ` FUJITA Tomonori
2010-01-14 12:49 ` bugzilla-daemon [this message]
2010-01-15  2:49 ` bugzilla-daemon
2010-01-18  4:18   ` FUJITA Tomonori
2010-01-15 12:23 ` bugzilla-daemon
2010-01-18  4:18   ` FUJITA Tomonori
2010-01-18  4:18 ` bugzilla-daemon
2010-01-18  4:18 ` bugzilla-daemon
2010-01-18  9:42 ` bugzilla-daemon
2010-01-18  9:45 ` bugzilla-daemon

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=201001141249.o0ECnuTH008110@demeter.kernel.org \
    --to=bugzilla-daemon@bugzilla.kernel.org \
    --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.