From: Michael Tokarev <mjt@tls.msk.ru>
To: "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 1/3] 9pfs-proxy: simplify error handling
Date: Wed, 11 Mar 2015 23:05:57 +0300 [thread overview]
Message-ID: <5500A025.4050305@msgid.tls.msk.ru> (raw)
In-Reply-To: <8761a7rkf7.fsf@linux.vnet.ibm.com>
11.03.2015 11:46, Aneesh Kumar K.V wrote:
[]
> ... Rather than saying
>
> "Oh well. I've no idea how this code has been accepted.
> It is absolute crap."
>
> it would be nice if we take up specific issues, w.r.t the os/cpu
> combination where we are running into issues, we may be able to fix this
> easily.
The specific issues were outlined and partially fixed by the v3
patchset. This is, for example, the way v9fs_request() is called
with arguments for v9fs_marshal function, which are ignored and
new arguments are constructed inside _request. That's what I
refer to when saying it is a complete crap. Do you not agree? :)
[]
> Note: At this point i haven't picked any of the patches posted, so if
> you have anything specific you want me to take up, please repost then
> with correct version number (v1, v2...)
I sent a v3 (last) with correct version number, it has just 3 (rather
big) patches: http://thread.gmane.org/gmane.comp.emulators.qemu/324157 .
Thanks,
/mjt
next prev parent reply other threads:[~2015-03-11 20:06 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-06 21:43 [Qemu-devel] [PATCH v3 0/3] RFC: 9pfs-proxy: simplify/cleanup Michael Tokarev
2015-03-06 21:43 ` [Qemu-devel] [PATCH 1/3] 9pfs-proxy: simplify error handling Michael Tokarev
2015-03-07 20:37 ` Eric Blake
2015-03-07 22:10 ` Michael Tokarev
2015-03-07 22:11 ` Michael Tokarev
2015-03-08 16:27 ` Aneesh Kumar K.V
2015-03-10 4:23 ` Michael Tokarev
2015-03-10 17:41 ` Aneesh Kumar K.V
2015-03-11 5:58 ` Michael Tokarev
2015-03-11 8:46 ` Aneesh Kumar K.V
2015-03-11 20:05 ` Michael Tokarev [this message]
2015-03-11 20:08 ` Michael Tokarev
2015-03-08 16:21 ` Aneesh Kumar K.V
2015-03-06 21:43 ` [Qemu-devel] [PATCH 2/3] fsdev: introduce v9fs_vmarshal() and v9fs_vunmarshal() Michael Tokarev
2015-03-06 21:43 ` [Qemu-devel] [PATCH 3/3] 9pfs-proxy: remove one half of redundrand code Michael Tokarev
2015-03-06 21:46 ` Michael Tokarev
2015-03-07 20:39 ` Eric Blake
2015-03-10 4:49 ` [Qemu-devel] [PATCH v3 0/3] RFC: 9pfs-proxy: simplify/cleanup Michael Tokarev
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=5500A025.4050305@msgid.tls.msk.ru \
--to=mjt@tls.msk.ru \
--cc=aneesh.kumar@linux.vnet.ibm.com \
--cc=qemu-devel@nongnu.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.