qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Marcel Apfelbaum <marcel.apfelbaum@gmail.com>
To: 李强 <liq3ea@163.com>
Cc: qemu-devel@nongnu.org, peter.maydell@linaro.org,
	yuval.shaia@oracle.com, kamalheib1@gmail.com, cohuck@redhat.com,
	philmd@redhat.com
Subject: Re: [Qemu-devel] [PATCH PULL 00/10] RDMA queue
Date: Sun, 20 Jan 2019 08:18:42 +0200	[thread overview]
Message-ID: <0d91276f-e9d4-4ac1-43ba-210c6e154fb1@gmail.com> (raw)
In-Reply-To: <7c5fdf65.41c2.16869e6f924.Coremail.liq3ea@163.com>



On 1/20/19 8:16 AM, 李强 wrote:
> At 2019-01-20 14:08:57, "Marcel Apfelbaum" <marcel.apfelbaum@gmail.com> wrote:
> >Hi,
> >
> >On 1/20/19 8:04 AM, 李强 wrote:
> >>
> >> Hi Marcel,
> >> Seems you lost another:
> >>
> >
> >No,  I did not, please see below.
> Yes, I just read the patch.
> >> hw: pvrdma: fix memory leak in error path
> >> -->http://lists.gnu.org/archive/html/qemu-devel/2019-01/msg01217.html
> >>
> >
> >Please see this mail thread, you were cc--ed:
> >https://www.mail-archive.com/qemu-devel@nongnu.org/msg588766.html   <https://www.mail-archive.com/qemu-devel@nongnu.org/msg588766.html
> >>
> > <https://www.mail-archive.com/qemu-devel@nongnu.org/msg588766.html >>
> I think you're right my patch "hw: pvrdma: fix memory leak in error path"
> is redundant.
> Sorry for the late reply.

No problem, thanks for the contribution!
Marcel

> Thanks,
> Li Qiang
> >Basically the following patch makes the the mentioned one obsolete.
> >https://patchew.org/QEMU/20190109201559.3906-1-yuval.shaia@oracle.com/
> >
> >Thanks,
> >Marcel
> >
> >> At 2019-01-19 18:03:05, "Marcel Apfelbaum" <marcel.apfelbaum@gmail.com> wrote:
> >> >The following changes since commit a8d2b0685681e2f291faaa501efbbd76875f8ec8:
> >> >
> >> >  Merge remote-tracking branch 'remotes/cohuck/tags/s390x-20190118' into staging (2019-01-18 16:56:15 +0000)
> >> >
> >> >are available in the Git repository at:
> >> >
> >> >  https://github.com/marcel-apf/qemu tags/rdma-pull-request
> >> >
> >> >for you to fetch changes up to 0f645ba16c6b76ccf2076d38460aa998198893bc:
> >> >
> >> >  contrib/rdmacm-mux: fix clang compilation (2019-01-19 11:01:57 +0200)
> >> >
> >> >----------------------------------------------------------------
> >> >RDMA queue
> >> > * Clang compilation fix
> >> > * Coverity fix
> >> > * Various fixes for the pvrdma device
> >> >
> >> >----------------------------------------------------------------
> >> >Kamal Heib (1):
> >> >      docs/pvrdma: Update rdmacm-mux documentation
> >> >
> >> >Li Qiang (1):
> >> >      hw: rdma: fix an off-by-one issue
> >> >
> >> >Marcel Apfelbaum (3):
> >> >      contrib/rdmacm-mux: remove Wno-format-truncation flag
> >> >      hw/rdma: modify struct initialization
> >> >      contrib/rdmacm-mux: fix clang compilation
> >> >
> >> >Yuval Shaia (5):
> >> >      hw/pvrdma: Remove max-sge command-line param
> >> >      hw/rdma: Delete unused struct member
> >> >      hw/pvrdma: Post CQE when receive invalid gid index
> >> >      hw/pvrdma: Make function pvrdma_qp_send/recv return void.
> >> >      hw/rdma: Verify that ptr is not NULL before freeing
> >> >
> >> > Makefile                         |  2 ++
> >> > contrib/rdmacm-mux/Makefile.objs |  1 -
> >> > contrib/rdmacm-mux/main.c        | 12 ++++----
> >> > docs/pvrdma.txt                  |  4 ++-
> >> > hw/rdma/rdma_backend.c           | 63 +++++++++++++++-------------------------
> >> > hw/rdma/rdma_backend.h           | 12 ++++++++
> >> > hw/rdma/rdma_backend_defs.h      |  1 -
> >> > hw/rdma/rdma_rm.c                |  9 ++++--
> >> > hw/rdma/vmw/pvrdma_main.c        | 10 +++----
> >> > hw/rdma/vmw/pvrdma_qp_ops.c      | 44 +++++++++++++++++++++-------
> >> > hw/rdma/vmw/pvrdma_qp_ops.h      |  4 +--
> >> > 11 files changed, 96 insertions(+), 66 deletions(-)
> >> >-- 
> >> >2.17.1
> >>
> >>
>
>

  parent reply	other threads:[~2019-01-20  6:18 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-01-19 10:03 [Qemu-devel] [PATCH PULL 00/10] RDMA queue Marcel Apfelbaum
2019-01-19 10:03 ` [Qemu-devel] [PATCH PULL 01/10] docs/pvrdma: Update rdmacm-mux documentation Marcel Apfelbaum
2019-01-19 10:03 ` [Qemu-devel] [PATCH PULL 02/10] hw/pvrdma: Remove max-sge command-line param Marcel Apfelbaum
2019-01-19 10:03 ` [Qemu-devel] [PATCH PULL 03/10] hw/rdma: Delete unused struct member Marcel Apfelbaum
2019-01-19 10:03 ` [Qemu-devel] [PATCH PULL 04/10] hw/pvrdma: Post CQE when receive invalid gid index Marcel Apfelbaum
2019-01-19 10:03 ` [Qemu-devel] [PATCH PULL 05/10] hw/pvrdma: Make function pvrdma_qp_send/recv return void Marcel Apfelbaum
2019-01-19 10:03 ` [Qemu-devel] [PATCH PULL 06/10] hw/rdma: Verify that ptr is not NULL before freeing Marcel Apfelbaum
2019-01-19 10:03 ` [Qemu-devel] [PATCH PULL 07/10] hw: rdma: fix an off-by-one issue Marcel Apfelbaum
2019-01-19 10:03 ` [Qemu-devel] [PATCH PULL 08/10] contrib/rdmacm-mux: remove Wno-format-truncation flag Marcel Apfelbaum
2019-01-19 10:03 ` [Qemu-devel] [PATCH PULL 09/10] hw/rdma: modify struct initialization Marcel Apfelbaum
2019-01-19 10:03 ` [Qemu-devel] [PATCH PULL 10/10] contrib/rdmacm-mux: fix clang compilation Marcel Apfelbaum
2019-01-20  6:04 ` [Qemu-devel] [PATCH PULL 00/10] RDMA queue 李强
2019-01-20  6:08   ` Marcel Apfelbaum
     [not found]     ` <7c5fdf65.41c2.16869e6f924.Coremail.liq3ea@163.com>
2019-01-20  6:18       ` Marcel Apfelbaum [this message]
2019-01-22  9:48 ` Peter Maydell

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=0d91276f-e9d4-4ac1-43ba-210c6e154fb1@gmail.com \
    --to=marcel.apfelbaum@gmail.com \
    --cc=cohuck@redhat.com \
    --cc=kamalheib1@gmail.com \
    --cc=liq3ea@163.com \
    --cc=peter.maydell@linaro.org \
    --cc=philmd@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=yuval.shaia@oracle.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).