[Qemu-devel] [PATCH] scsi: Fix SCSI bus reset

[Qemu-devel] [PATCH] scsi: Fix SCSI bus reset

[Jan Kiszka]

When the controller raises the SCSI reset line, we have to perform the
requested reset on all disks attached to the controller's bus. Moreover,
reset is edge triggered, so avoid repeating it if the line was already
high.

Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
---
 hw/lsi53c895a.c |   15 +++++++++++++--
 1 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/hw/lsi53c895a.c b/hw/lsi53c895a.c
index 9a37fed..249c6bf 100644
--- a/hw/lsi53c895a.c
+++ b/hw/lsi53c895a.c
@@ -1590,8 +1590,19 @@ static void lsi_reg_writeb(LSIState *s, int offset, uint8_t val)
             BADF("Immediate Arbritration not implemented\n");
         }
         if (val & LSI_SCNTL1_RST) {
-            s->sstat0 |= LSI_SSTAT0_RST;
-            lsi_script_scsi_interrupt(s, LSI_SIST0_RST, 0);
+            if (!(s->sstat0 & LSI_SSTAT0_RST)) {
+                DeviceState *dev;
+                int id;
+
+                for (id = 0; id < s->bus.ndev; id++) {
+                    if (s->bus.devs[id]) {
+                        dev = &s->bus.devs[id]->qdev;
+                        dev->info->reset(dev);
+                    }
+                }
+                s->sstat0 |= LSI_SSTAT0_RST;
+                lsi_script_scsi_interrupt(s, LSI_SIST0_RST, 0);
+            }
         } else {
             s->sstat0 &= ~LSI_SSTAT0_RST;
         }
-- 
1.7.1

Re: [Qemu-devel] [PATCH] scsi: Fix SCSI bus reset

[Anthony Liguori]

On 07/06/2010 03:58 AM, Jan Kiszka wrote:
> When the controller raises the SCSI reset line, we have to perform the
> requested reset on all disks attached to the controller's bus. Moreover,
> reset is edge triggered, so avoid repeating it if the line was already
> high.
>
> Signed-off-by: Jan Kiszka<jan.kiszka@siemens.com>
>    

Applied.  Thanks.

Regards,

Anthony Liguori
> ---
>   hw/lsi53c895a.c |   15 +++++++++++++--
>   1 files changed, 13 insertions(+), 2 deletions(-)
>
> diff --git a/hw/lsi53c895a.c b/hw/lsi53c895a.c
> index 9a37fed..249c6bf 100644
> --- a/hw/lsi53c895a.c
> +++ b/hw/lsi53c895a.c
> @@ -1590,8 +1590,19 @@ static void lsi_reg_writeb(LSIState *s, int offset, uint8_t val)
>               BADF("Immediate Arbritration not implemented\n");
>           }
>           if (val&  LSI_SCNTL1_RST) {
> -            s->sstat0 |= LSI_SSTAT0_RST;
> -            lsi_script_scsi_interrupt(s, LSI_SIST0_RST, 0);
> +            if (!(s->sstat0&  LSI_SSTAT0_RST)) {
> +                DeviceState *dev;
> +                int id;
> +
> +                for (id = 0; id<  s->bus.ndev; id++) {
> +                    if (s->bus.devs[id]) {
> +                        dev =&s->bus.devs[id]->qdev;
> +                        dev->info->reset(dev);
> +                    }
> +                }
> +                s->sstat0 |= LSI_SSTAT0_RST;
> +                lsi_script_scsi_interrupt(s, LSI_SIST0_RST, 0);
> +            }
>           } else {
>               s->sstat0&= ~LSI_SSTAT0_RST;
>           }
>