From mboxrd@z Thu Jan  1 00:00:00 1970
From: Will Deacon <will.deacon@arm.com>
Subject: Re: [PATCH -next] vfio/type1: fix error return code in
 vfio_iommu_type1_attach_group()
Date: Fri, 10 Feb 2017 11:56:47 +0000
Message-ID: <20170210115647.GC32547@arm.com>
References: <20170209160158.24497-1-weiyj.lk@gmail.com>
 <98b720c2-3f2a-0418-abb0-77b45b50a9ff@redhat.com>
 <20170209112311.4d4616a9@t450s.home>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: Joerg Roedel <jroedel@suse.de>, Auger Eric <eric.auger@redhat.com>,
        Wei Yongjun <weiyj.lk@gmail.com>,
        Wei Yongjun <weiyongjun1@huawei.com>, kvm@vger.kernel.org
To: Alex Williamson <alex.williamson@redhat.com>
Return-path: <kvm-owner@vger.kernel.org>
Received: from foss.arm.com ([217.140.101.70]:33506 "EHLO foss.arm.com"
        rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
        id S1752176AbdBJL5n (ORCPT <rfc822;kvm@vger.kernel.org>);
        Fri, 10 Feb 2017 06:57:43 -0500
Content-Disposition: inline
In-Reply-To: <20170209112311.4d4616a9@t450s.home>
Sender: kvm-owner@vger.kernel.org
List-ID: <kvm.vger.kernel.org>

On Thu, Feb 09, 2017 at 11:23:11AM -0700, Alex Williamson wrote:
> On Thu, 9 Feb 2017 18:03:47 +0100
> Auger Eric <eric.auger@redhat.com> wrote:
> > On 09/02/2017 17:01, Wei Yongjun wrote:
> > > From: Wei Yongjun <weiyongjun1@huawei.com>
> > > 
> > > Fix to return a negative error code from the error handling
> > > case instead of 0, as done elsewhere in this function.
> > > 
> > > Fixes: 5d704992189f ("vfio/type1: Allow transparent MSI IOVA allocation")
> > > Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>  
> > 
> > Reviewed-by: Eric Auger <eric.auger@redhat.com>
> 
> Acked-by: Alex Williamson <alex.williamson@redhat.com>
> 
> Joerg/Will,
> 
> The original commit comes in through iommu/next (arm/core), please feel
> free to also include this fix.  Thanks,

Patch looks fine to me. Joerg, can you pick this up please?

Will