lkml.org 
[lkml]   [2012]   [Jul]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v4] virtio-scsi: hotplug support for virtio-scsi
From
Date
On Thu, 2012-07-05 at 11:19 +0200, Paolo Bonzini wrote:
> Il 05/07/2012 11:06, Cong Meng ha scritto:
> > This patch implements the hotplug support for virtio-scsi.
> > When there is a device attached/detached, the virtio-scsi driver will be
> > signaled via event virtual queue and it will add/remove the scsi device
> > in question automatically.
> >
> > v2: handle no_event event
> > v3: add handle event dropped, and typo fix
> > v4: Cancel event works when exit. Coding type fix.
> >
> > Signed-off-by: Sen Wang <senwang@linux.vnet.ibm.com>
> > Signed-off-by: Cong Meng <mc@linux.vnet.ibm.com>
>
> Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>

Actually, that should be Acked-by, since you're the maintainer of the
driver (it means I reviewed it + I approve it for upstream inclusion).

> > ---
> > drivers/scsi/virtio_scsi.c | 127 ++++++++++++++++++++++++++++++++++++++++++-
> > include/linux/virtio_scsi.h | 9 +++
> > 2 files changed, 135 insertions(+), 1 deletions(-)

The next time, please run checkpatch on this. You can ignore the
warnings about lines over 80 characters, but the braces issue is worth
paying attention to when that reflects the current style.

James




\
 
 \ /
  Last update: 2012-07-11 21:21    [from the cache]
©2003-2011 Jasper Spaans