qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PULL 0/1] RDMA queue
@ 2020-12-18 19:00 Marcel Apfelbaum
  2020-12-18 19:00 ` [Qemu-devel] [PULL 1/1] contrib/rdmacm-mux: Fix error condition in hash_tbl_search_fd_by_ifid() Marcel Apfelbaum
  2021-01-01 14:32 ` [Qemu-devel] [PULL 0/1] RDMA queue Peter Maydell
  0 siblings, 2 replies; 5+ messages in thread
From: Marcel Apfelbaum @ 2020-12-18 19:00 UTC (permalink / raw)
  To: qemu-devel, peter.maydell; +Cc: alex.chen, yuval.shaia.ml, euler.robot

The following changes since commit a05f8ecd88f15273d033b6f044b850a8af84a5b8:

  Merge remote-tracking branch 'remotes/alistair/tags/pull-riscv-to-apply-20201217-1' into staging (2020-12-18 11:12:35 +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 eed31bc5e2e9cc68626d51e7a10914dc52c370f8:

  contrib/rdmacm-mux: Fix error condition in hash_tbl_search_fd_by_ifid() (2020-12-18 20:15:42 +0200)

----------------------------------------------------------------
RDMA queue

*  bug fix in contrib/rdmacm-mux

----------------------------------------------------------------
AlexChen (1):
      contrib/rdmacm-mux: Fix error condition in hash_tbl_search_fd_by_ifid()

 contrib/rdmacm-mux/main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.17.2



^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Qemu-devel] [PULL 1/1] contrib/rdmacm-mux: Fix error condition in hash_tbl_search_fd_by_ifid()
  2020-12-18 19:00 [Qemu-devel] [PULL 0/1] RDMA queue Marcel Apfelbaum
@ 2020-12-18 19:00 ` Marcel Apfelbaum
  2021-01-01 14:32 ` [Qemu-devel] [PULL 0/1] RDMA queue Peter Maydell
  1 sibling, 0 replies; 5+ messages in thread
From: Marcel Apfelbaum @ 2020-12-18 19:00 UTC (permalink / raw)
  To: qemu-devel, peter.maydell; +Cc: alex.chen, yuval.shaia.ml, euler.robot

From: AlexChen <alex.chen@huawei.com>

When fd is not found according to ifid, the _hash_tbl_search_fd_by_ifid()
returns 0 and assigns the result to *fd, so We have to check that *fd is 0,
not that fd is 0.

Reported-by: Euler Robot <euler.robot@huawei.com>
Signed-off-by: AlexChen <alex.chen@huawei.com>
Message-Id: <5F9AC6FF.4000301@huawei.com>
Reviewed-by: Marcel Apfelbaum <marcel.apfelbaum@gmail.com>
Signed-off-by: Marcel Apfelbaum <marcel.apfelbaum@gmail.com>
---
 contrib/rdmacm-mux/main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/contrib/rdmacm-mux/main.c b/contrib/rdmacm-mux/main.c
index bd82abbad3..771ca01e03 100644
--- a/contrib/rdmacm-mux/main.c
+++ b/contrib/rdmacm-mux/main.c
@@ -186,7 +186,7 @@ static int hash_tbl_search_fd_by_ifid(int *fd, __be64 *gid_ifid)
     *fd = _hash_tbl_search_fd_by_ifid(gid_ifid);
     pthread_rwlock_unlock(&server.lock);
 
-    if (!fd) {
+    if (!*fd) {
         syslog(LOG_WARNING, "Can't find matching for ifid 0x%llx\n", *gid_ifid);
         return -ENOENT;
     }
-- 
2.17.2



^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [Qemu-devel] [PULL 0/1] RDMA queue
  2020-12-18 19:00 [Qemu-devel] [PULL 0/1] RDMA queue Marcel Apfelbaum
  2020-12-18 19:00 ` [Qemu-devel] [PULL 1/1] contrib/rdmacm-mux: Fix error condition in hash_tbl_search_fd_by_ifid() Marcel Apfelbaum
@ 2021-01-01 14:32 ` Peter Maydell
  1 sibling, 0 replies; 5+ messages in thread
From: Peter Maydell @ 2021-01-01 14:32 UTC (permalink / raw)
  To: Marcel Apfelbaum; +Cc: AlexChen, Yuval Shaia, QEMU Developers, Euler Robot

On Fri, 18 Dec 2020 at 19:00, Marcel Apfelbaum
<marcel.apfelbaum@gmail.com> wrote:
>
> The following changes since commit a05f8ecd88f15273d033b6f044b850a8af84a5b8:
>
>   Merge remote-tracking branch 'remotes/alistair/tags/pull-riscv-to-apply-20201217-1' into staging (2020-12-18 11:12:35 +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 eed31bc5e2e9cc68626d51e7a10914dc52c370f8:
>
>   contrib/rdmacm-mux: Fix error condition in hash_tbl_search_fd_by_ifid() (2020-12-18 20:15:42 +0200)
>
> ----------------------------------------------------------------
> RDMA queue
>
> *  bug fix in contrib/rdmacm-mux
>


Applied, thanks.

Please update the changelog at https://wiki.qemu.org/ChangeLog/6.0
for any user-visible changes.

-- PMM


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Qemu-devel] [PULL 0/1] RDMA queue
  2020-05-02 19:18 Marcel Apfelbaum
@ 2020-05-03 14:03 ` Peter Maydell
  0 siblings, 0 replies; 5+ messages in thread
From: Peter Maydell @ 2020-05-03 14:03 UTC (permalink / raw)
  To: Marcel Apfelbaum; +Cc: QEMU Developers, Yuval Shaia

On Sat, 2 May 2020 at 20:18, Marcel Apfelbaum
<marcel.apfelbaum@gmail.com> wrote:
>
> The following changes since commit 1c47613588ccff44422d4bdeea0dc36a0a308ec7:
>
>   Merge remote-tracking branch 'remotes/kevin/tags/for-upstream' into staging (2020-04-30 19:25:41 +0100)
>
> are available in the Git repository at:
>
>   https://github.com/marcel-apf/qemu tags/rdma-pull-request
>
> for you to fetch changes up to a5cde048e865da81fdc9077f3af28a43bff78d35:
>
>   hw/rdma: Destroy list mutex when list is destroyed (2020-05-02 21:31:17 +0300)
>
> ----------------------------------------------------------------
> RDMA queue
>
> * hw/rdma: Destroy list mutex when list is destroyed
>
> ----------------------------------------------------------------


Applied, thanks.

Please update the changelog at https://wiki.qemu.org/ChangeLog/5.1
for any user-visible changes.

-- PMM


^ permalink raw reply	[flat|nested] 5+ messages in thread

* [Qemu-devel] [PULL 0/1] RDMA queue
@ 2020-05-02 19:18 Marcel Apfelbaum
  2020-05-03 14:03 ` Peter Maydell
  0 siblings, 1 reply; 5+ messages in thread
From: Marcel Apfelbaum @ 2020-05-02 19:18 UTC (permalink / raw)
  To: qemu-devel, peter.maydell; +Cc: yuval.shaia.ml

The following changes since commit 1c47613588ccff44422d4bdeea0dc36a0a308ec7:

  Merge remote-tracking branch 'remotes/kevin/tags/for-upstream' into staging (2020-04-30 19:25:41 +0100)

are available in the Git repository at:

  https://github.com/marcel-apf/qemu tags/rdma-pull-request

for you to fetch changes up to a5cde048e865da81fdc9077f3af28a43bff78d35:

  hw/rdma: Destroy list mutex when list is destroyed (2020-05-02 21:31:17 +0300)

----------------------------------------------------------------
RDMA queue

* hw/rdma: Destroy list mutex when list is destroyed

----------------------------------------------------------------
Yuval Shaia (1):
      hw/rdma: Destroy list mutex when list is destroyed

 hw/rdma/rdma_utils.c | 1 +
 1 file changed, 1 insertion(+)

-- 
2.17.2



^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2021-01-01 14:34 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-18 19:00 [Qemu-devel] [PULL 0/1] RDMA queue Marcel Apfelbaum
2020-12-18 19:00 ` [Qemu-devel] [PULL 1/1] contrib/rdmacm-mux: Fix error condition in hash_tbl_search_fd_by_ifid() Marcel Apfelbaum
2021-01-01 14:32 ` [Qemu-devel] [PULL 0/1] RDMA queue Peter Maydell
  -- strict thread matches above, loose matches on Subject: below --
2020-05-02 19:18 Marcel Apfelbaum
2020-05-03 14:03 ` Peter Maydell

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).