From: Paolo Bonzini <pbonzini@redhat.com>
To: Boaz Harrosh <bharrosh@panasas.com>
Cc: Wang Sen <senwang@linux.vnet.ibm.com>,
linux-scsi@vger.kernel.org, JBottomley@parallels.com,
stefanha@linux.vnet.ibm.com, mc@linux.vnet.ibm.com,
linux-kernel@vger.kernel.org,
"kvm@vger.kernel.org" <kvm@vger.kernel.org>,
Rusty Russell <rusty@rustcorp.com.au>
Subject: Re: virtio(-scsi) vs. chained sg_lists (was Re: [PATCH] scsi: virtio-scsi: Fix address translation failure of HighMem pages used by sg list)
Date: Thu, 26 Jul 2012 09:58:14 +0200 [thread overview]
Message-ID: <5010F896.8090409@redhat.com> (raw)
In-Reply-To: <5010F831.9030300@panasas.com>
Il 26/07/2012 09:56, Boaz Harrosh ha scritto:
>> > In the meanwhile, we still have a bug to fix, and we need to choose
>> > between Sen Wang's v1 (sg_set_page) or v2 (value assignment). I'm still
>> > leaning more towards v2, if only because I already tested that one myself.
>
> It's your call, you know what I think ;-)
>
> I Agree none of them are bugs in current code, they will both work
> just the same.
Cool. Then, Sen, please fix the commit message in v2 and repost.
> Please CC me on the "convert to sg copy-less" patches, It looks interesting
Sure.
Paolo
next prev parent reply other threads:[~2012-07-26 7:58 UTC|newest]
Thread overview: 47+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-25 8:29 [PATCH] scsi: virtio-scsi: Fix address translation failure of HighMem pages used by sg list Wang Sen
2012-07-25 8:44 ` Paolo Bonzini
2012-07-25 9:22 ` Boaz Harrosh
2012-07-25 9:22 ` Boaz Harrosh
2012-07-25 9:41 ` Paolo Bonzini
2012-07-25 12:34 ` Boaz Harrosh
2012-07-25 12:34 ` Boaz Harrosh
2012-07-25 12:49 ` Paolo Bonzini
2012-07-25 13:26 ` Boaz Harrosh
2012-07-25 13:26 ` Boaz Harrosh
2012-07-25 13:36 ` Paolo Bonzini
2012-07-25 14:36 ` Boaz Harrosh
2012-07-25 14:36 ` Boaz Harrosh
2012-07-25 15:09 ` performance improvements for the sglist API (Re: [PATCH] scsi: virtio-scsi: Fix address translation failure of HighMem pages used by sg list) Paolo Bonzini
2012-07-25 15:16 ` Paolo Bonzini
2012-07-25 14:17 ` virtio(-scsi) vs. chained sg_lists (was " Paolo Bonzini
2012-07-25 15:28 ` Boaz Harrosh
2012-07-25 15:28 ` Boaz Harrosh
2012-07-25 17:43 ` Paolo Bonzini
2012-07-25 19:16 ` Boaz Harrosh
2012-07-25 19:16 ` Boaz Harrosh
2012-07-25 20:06 ` Paolo Bonzini
2012-07-25 21:04 ` Boaz Harrosh
2012-07-25 21:04 ` Boaz Harrosh
2012-07-26 7:23 ` Paolo Bonzini
2012-07-26 7:56 ` Boaz Harrosh
2012-07-26 7:56 ` Boaz Harrosh
2012-07-26 7:58 ` Paolo Bonzini [this message]
2012-07-26 13:05 ` Paolo Bonzini
2012-07-27 6:27 ` Rusty Russell
2012-07-27 6:27 ` Rusty Russell
2012-07-27 8:11 ` Paolo Bonzini
2012-07-29 23:50 ` Rusty Russell
2012-07-29 23:50 ` Rusty Russell
2012-07-30 7:12 ` Paolo Bonzini
2012-07-30 8:56 ` Boaz Harrosh
2012-07-30 8:56 ` Boaz Harrosh
2012-07-25 10:41 ` [PATCH] scsi: virtio-scsi: Fix address translation failure of HighMem pages used by sg list Stefan Hajnoczi
2012-07-25 11:48 ` Sen Wang
2012-07-25 11:44 ` Sen Wang
2012-07-25 12:40 ` Boaz Harrosh
2012-07-25 12:40 ` Boaz Harrosh
2012-07-27 3:12 ` Wang Sen
2012-07-27 6:50 ` Paolo Bonzini
2012-07-25 10:04 ` Rolf Eike Beer
2012-07-25 10:04 ` Rolf Eike Beer
2012-07-25 11:46 ` Sen Wang
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=5010F896.8090409@redhat.com \
--to=pbonzini@redhat.com \
--cc=JBottomley@parallels.com \
--cc=bharrosh@panasas.com \
--cc=kvm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=mc@linux.vnet.ibm.com \
--cc=rusty@rustcorp.com.au \
--cc=senwang@linux.vnet.ibm.com \
--cc=stefanha@linux.vnet.ibm.com \
/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.