From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:42349)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <quintela@redhat.com>) id 1Yzm7E-0008U1-1O
	for qemu-devel@nongnu.org; Tue, 02 Jun 2015 09:20:24 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <quintela@redhat.com>) id 1Yzm7A-00085a-SK
	for qemu-devel@nongnu.org; Tue, 02 Jun 2015 09:20:23 -0400
Received: from mx1.redhat.com ([209.132.183.28]:39053)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <quintela@redhat.com>) id 1Yzm7A-00085U-M2
	for qemu-devel@nongnu.org; Tue, 02 Jun 2015 09:20:20 -0400
From: Juan Quintela <quintela@redhat.com>
In-Reply-To: <1433286850-12753-1-git-send-email-padmanabh.ratnakar@avagotech.com>
	(Padmanabh Ratnakar's message of "Wed, 3 Jun 2015 04:44:10 +0530")
References: <1433286850-12753-1-git-send-email-padmanabh.ratnakar@avagotech.com>
Date: Tue, 02 Jun 2015 15:20:18 +0200
Message-ID: <87bngy9rhp.fsf@neno.neno>
MIME-Version: 1.0
Content-Type: text/plain
Subject: Re: [Qemu-devel] [PATCH] rdma: Fix qemu crash when IPv6 address is
	used for migration
Reply-To: quintela@redhat.com
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Padmanabh Ratnakar <padmanabh.ratnakar@avagotech.com>
Cc: amit.shah@redhat.com, Meghana Cheripady <meghana.cheripady@avagotech.com>, qemu-devel@nongnu.org, mrhines@us.ibm.com

Padmanabh Ratnakar <padmanabh.ratnakar@avagotech.com> wrote:
> Qemu crashes when IPv6 address is specified for migration and access
> to any RDMA uverbs device available on the system is blocked using cgroups.
> Fix the crash by checking the return value of ibv_open_device routine.
>
> Signed-off-by: Meghana Cheripady <meghana.cheripady@avagotech.com>
> Signed-off-by: Padmanabh Ratnakar <padmanabh.ratnakar@avagotech.com>

Reviewed-by: Juan Quintela <quintela@redhat.com>

Applied, thanks.

Sorry for the previous comment, I just mixed up context and my mail made
zero sense.


> ---
>  migration/rdma.c |    7 +++++++
>  1 files changed, 7 insertions(+), 0 deletions(-)
>
> diff --git a/migration/rdma.c b/migration/rdma.c
> index 77e3444..3671903 100644
> --- a/migration/rdma.c
> +++ b/migration/rdma.c
> @@ -790,6 +790,13 @@ static int qemu_rdma_broken_ipv6_kernel(Error **errp, struct ibv_context *verbs)
>  
>          for (x = 0; x < num_devices; x++) {
>              verbs = ibv_open_device(dev_list[x]);
> +            if (!verbs) {
> +                if (errno == EPERM) {
> +                    continue;
> +                } else {
> +                    return -EINVAL;
> +                }
> +            }
>  
>              if (ibv_query_port(verbs, 1, &port_attr)) {
>                  ibv_close_device(verbs);