On Wed, Jun 15, 2016 at 10:52:55AM -0600, Alex Williamson wrote: > On Wed, 15 Jun 2016 15:03:15 +1000 > David Gibson <[email protected]> wrote: > > > On Wed, Jun 15, 2016 at 02:46:23PM +1000, David Gibson wrote: > > > On Wed, Jun 15, 2016 at 02:28:27PM +1000, Gavin Shan wrote: > > > > vfio_eeh_container_op() is the backend that communicates with > > > > host kernel to support EEH functionality in QEMU. However, the > > > > functon should return the value from host kernel instead of 0 > > > > unconditionally. > > > > > > > > Signed-off-by: Gavin Shan <[email protected]> > > > > > > Applied to ppc-for-2.7, thanks. > > > > Hang on, wait.. forgot I should get an ack for this from you, Alex. > > I'll keep it in my tree for now, unless you tell me you'd prefer to > > take it through yours. > > Hmm, clearly this patch should have cc'd qemu-devel from the start.
Yeah. Gavin's not usually a qemu developer and I forgot a bunch of things in my 15s version of how-to-submit-qemu-patches. > Go > ahead and take it through your tree David. Will do, thanks. > > Acked-by: Alex Williamson <[email protected]> > > > > > > > > > We should probably look at applying this to the 2.6 stable branch as well. > > > > > > > --- > > > > hw/vfio/common.c | 2 +- > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > diff --git a/hw/vfio/common.c b/hw/vfio/common.c > > > > index e51ed3a..e51db46 100644 > > > > --- a/hw/vfio/common.c > > > > +++ b/hw/vfio/common.c > > > > @@ -1258,7 +1258,7 @@ static int vfio_eeh_container_op(VFIOContainer > > > > *container, uint32_t op) > > > > return -errno; > > > > } > > > > > > > > - return 0; > > > > + return ret; > > > > } > > > > > > > > static VFIOContainer *vfio_eeh_as_container(AddressSpace *as) > > > > > > > > > > -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
