From: Peter Xu <peterx@redhat.com>
To: Leonardo Bras Soares Passos <leobras@redhat.com>
Cc: "Elena Ufimtseva" <elena.ufimtseva@oracle.com>,
"John G Johnson" <john.g.johnson@oracle.com>,
"Jagannathan Raman" <jag.raman@oracle.com>,
qemu-block@nongnu.org, "Juan Quintela" <quintela@redhat.com>,
qemu-devel <qemu-devel@nongnu.org>,
"Daniel P. Berrangé" <berrange@redhat.com>,
"Dr. David Alan Gilbert" <dgilbert@redhat.com>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Marc-André Lureau" <marcandre.lureau@redhat.com>,
"Fam Zheng" <fam@euphon.net>
Subject: Re: [PATCH v1 2/3] io: Add zerocopy and errqueue
Date: Tue, 7 Sep 2021 12:44:52 -0400 [thread overview]
Message-ID: <YTeXBCAUOpy0HagV@t490s> (raw)
In-Reply-To: <CAJ6HWG72311UK9Nvh+JDw-zo1Days0u8N_QHbeGGdfKR+hALCA@mail.gmail.com>
On Thu, Sep 02, 2021 at 03:59:25AM -0300, Leonardo Bras Soares Passos wrote:
> I also suggested something like that, but I thought it could be good if we could
> fall back to io_writev() if we didn't have the zerocopy feature (or
> the async feature).
>
> What do you think?
That fallback looks safe and ok, I'm just not sure whether it'll be of great
help. E.g. if we provide an QIO api that allows both sync write and zero-copy
write (then we do the fallback when necessary), it means the buffer implication
applies too to this api, so it's easier to me to just detect the zero copy
capability and use one alternative. Thanks,
--
Peter Xu
next prev parent reply other threads:[~2021-09-07 16:47 UTC|newest]
Thread overview: 62+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-31 11:02 [PATCH v1 0/3] QIOChannel flags + multifd zerocopy Leonardo Bras
2021-08-31 11:02 ` [PATCH v1 1/3] io: Enable write flags for QIOChannel Leonardo Bras
2021-09-01 20:54 ` Eric Blake
2021-09-02 8:26 ` Leonardo Bras Soares Passos
2021-08-31 11:02 ` [PATCH v1 2/3] io: Add zerocopy and errqueue Leonardo Bras
2021-08-31 12:57 ` Daniel P. Berrangé
2021-08-31 20:27 ` Peter Xu
2021-09-01 8:50 ` Daniel P. Berrangé
2021-09-01 15:52 ` Peter Xu
2021-09-01 15:59 ` Daniel P. Berrangé
2021-09-02 7:07 ` Leonardo Bras Soares Passos
2021-09-02 6:59 ` Leonardo Bras Soares Passos
2021-09-07 16:44 ` Peter Xu [this message]
2021-09-08 20:13 ` Leonardo Bras Soares Passos
2021-09-08 21:04 ` Peter Xu
2021-09-02 6:38 ` Leonardo Bras Soares Passos
2021-09-02 8:47 ` Daniel P. Berrangé
2021-09-02 9:34 ` Leonardo Bras Soares Passos
2021-09-02 9:49 ` Daniel P. Berrangé
2021-09-02 10:19 ` Leonardo Bras Soares Passos
2021-09-02 10:28 ` Daniel P. Berrangé
2021-09-07 11:06 ` Dr. David Alan Gilbert
2021-09-07 18:09 ` Peter Xu
2021-09-08 8:30 ` Dr. David Alan Gilbert
2021-09-08 15:24 ` Peter Xu
2021-09-09 8:49 ` Dr. David Alan Gilbert
2021-09-08 20:25 ` Leonardo Bras Soares Passos
2021-09-08 21:09 ` Peter Xu
2021-09-08 21:57 ` Daniel P. Berrangé
2021-09-09 2:05 ` Peter Xu
2021-09-09 4:58 ` Leonardo Bras Soares Passos
2021-09-09 16:40 ` Peter Xu
2021-08-31 11:02 ` [PATCH v1 3/3] migration: multifd: Enable zerocopy Leonardo Bras
2021-08-31 13:16 ` Daniel P. Berrangé
2021-08-31 20:29 ` Peter Xu
2021-09-01 8:53 ` Daniel P. Berrangé
2021-09-01 15:35 ` Peter Xu
2021-09-01 15:44 ` Daniel P. Berrangé
2021-09-01 16:01 ` Peter Xu
2021-09-02 7:57 ` Leonardo Bras Soares Passos
2021-09-07 11:13 ` Dr. David Alan Gilbert
2021-09-08 15:26 ` Daniel P. Berrangé
2021-09-02 7:23 ` Jason Wang
2021-09-02 8:08 ` Leonardo Bras Soares Passos
2021-09-02 7:27 ` Leonardo Bras Soares Passos
2021-09-02 7:22 ` Leonardo Bras Soares Passos
2021-09-02 8:20 ` Daniel P. Berrangé
2021-09-02 8:52 ` Leonardo Bras Soares Passos
2021-09-02 9:20 ` Daniel P. Berrangé
2021-09-02 9:49 ` Leonardo Bras Soares Passos
2021-09-02 9:59 ` Daniel P. Berrangé
2021-09-02 10:25 ` Leonardo Bras Soares Passos
2021-09-07 11:17 ` Dr. David Alan Gilbert
2021-09-07 18:32 ` Peter Xu
2021-09-08 2:59 ` Jason Wang
2021-09-08 3:24 ` Peter Xu
2021-09-08 3:26 ` Jason Wang
2021-09-08 8:19 ` Dr. David Alan Gilbert
2021-09-08 15:19 ` Peter Xu
2021-09-09 1:10 ` Jason Wang
2021-08-31 21:24 ` [PATCH v1 0/3] QIOChannel flags + multifd zerocopy Peter Xu
2021-09-01 19:21 ` Leonardo Bras Soares Passos
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=YTeXBCAUOpy0HagV@t490s \
--to=peterx@redhat.com \
--cc=berrange@redhat.com \
--cc=dgilbert@redhat.com \
--cc=elena.ufimtseva@oracle.com \
--cc=fam@euphon.net \
--cc=jag.raman@oracle.com \
--cc=john.g.johnson@oracle.com \
--cc=leobras@redhat.com \
--cc=marcandre.lureau@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=quintela@redhat.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).