Dexuan Cui <decui@xxxxxxxxxxxxx> writes: >> -----Original Message----- >> From: Vitaly Kuznetsov [mailto:vkuznets@xxxxxxxxxx] >> Sent: Wednesday, April 22, 2015 0:22 >> To: KY Srinivasan >> Cc: Haiyang Zhang; devel@xxxxxxxxxxxxxxxxxxxxxx; linux- >> kernel@xxxxxxxxxxxxxxx; Dexuan Cui; Dan Carpenter >> Subject: [PATCH v2 0/3] Drivers: hv: vmbus: additional fixes for the >> setup/teardown path >> >> Changes since v1 (PATCH 1/3): >> - Return -EAGAIN instead of open_info->response.open_result.status >> [Dexuan Cui] >> - Avoid 'if (ret != 0)' statement [Dan Carpenter] >> >> Original description: >> K. Y., >> >> Here are 3 additional patches for your "[PATCH 0/5] Drivers: hv: vmbus: >> Cleanup >> the vmbus unload path" series (with the fix I suggested). I tested the whole >> set on Gen2 Win2012R2 guest, load/unload path seems being stable. Can >> you >> please take a look? >> >> Thanks, >> >> Vitaly Kuznetsov (3): >> Drivers: hv: vmbus: do cleanup on all vmbus_open() failure paths >> Drivers: hv: vmbus: kill tasklets on module unload >> Drivers: hv: vmbus: setup irq after synic is initialized >> >> drivers/hv/channel.c | 13 ++++++------- >> drivers/hv/vmbus_drv.c | 10 +++++++--- >> 2 files changed, 13 insertions(+), 10 deletions(-) > > The patchset seems good to me. > > Reviewed-by: Dexuan Cui <decui@xxxxxxxxxxxxx> > Thanks! -- Vitaly _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
