Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   Matt Lupfer <mlupfer@ddn.com>
To:     "Michael S . Tsirkin" <mst@redhat.com>,
        Jason Wang <jasowang@redhat.com>,
        Paolo Bonzini <pbonzini@redhat.com>,
        Stefan Hajnoczi <stefanha@redhat.com>,
        "James E . J . Bottomley" <jejb@linux.ibm.com>,
        "Martin K . Petersen" <martin.petersen@oracle.com>
CC:     "virtualization@lists.linux-foundation.org" 
        <virtualization@lists.linux-foundation.org>,
        "linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Matt Lupfer <mlupfer@ddn.com>
Subject: [PATCH] scsi: virtio_scsi: unplug LUNs when events missed
Thread-Topic: [PATCH] scsi: virtio_scsi: unplug LUNs when events missed
Thread-Index: AQHVYnmgUCG1mM4ObEKGSGCIQ1xBzg==
Date:   Tue, 3 Sep 2019 17:04:20 +0000
Message-ID: <20190903170408.32286-1-mlupfer@ddn.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-messagesentrepresentingtype: 1
x-mailer: git-send-email 2.23.0
x-originating-ip: [107.128.241.61]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: cc3df57e-7d38-49e1-eba6-08d73090c2ff
x-microsoft-antispam: BCL:0;PCL:0;RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600166)(711020)(4605104)(1401327)(2017052603328)(7193020);SRVR:BY5PR19MB3221;
x-ms-traffictypediagnostic: BY5PR19MB3221:
x-ms-exchange-transport-forked: True
x-microsoft-antispam-prvs: <BY5PR19MB322198CDECF53BC0BF569074AEB90@BY5PR19MB3221.namprd19.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:7219;
x-forefront-prvs: 01494FA7F7
x-forefront-antispam-report: SFV:NSPM;SFS:(10009020)(4636009)(39850400004)(376002)(136003)(366004)(346002)(396003)(189003)(199004)(36756003)(71190400001)(71200400001)(14454004)(25786009)(4326008)(256004)(478600001)(5024004)(66556008)(66476007)(66446008)(64756008)(66946007)(476003)(6486002)(2616005)(86362001)(486006)(8676002)(81156014)(102836004)(81166006)(8936002)(50226002)(386003)(7736002)(6506007)(305945005)(6116002)(3846002)(26005)(5660300002)(2906002)(316002)(186003)(110136005)(54906003)(1076003)(52116002)(99286004)(6512007)(66066001)(6436002)(53936002);DIR:OUT;SFP:1101;SCL:1;SRVR:BY5PR19MB3221;H:BY5PR19MB3176.namprd19.prod.outlook.com;FPR:;SPF:None;LANG:en;PTR:InfoNoRecords;MX:1;A:1;
received-spf: None (protection.outlook.com: ddn.com does not designate
 permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam-message-info: RUmvvb9EMpVdSOP5wT+r+RINhpPJNUTNk4b/DHOnu/M9IaKFyUuNdjjERa6bKk9Dlx7GAd/jpk+NfTC1NMisgz1/mRYDu88+JzI6xKs0bzTtIDN6z75g4F+4KINMOilhmac2lvQTI9JNkrG/AuJlorJQpAWs/VmufEgi+1MF7gm2PL8nKxRItYFYrAII8cjyZ63KzlJy7NrG7i+bnqZZ7jeyAMAjBQLAYopZwt58iONTQ8gzKDzdjEJz5rMgPePiTLnUErMmRrAQPHgZfOvTPblMYn+Ex9PjJ7pdsQliJTjN1cc7MX8Htfa9fXXFEm1Daa5eiQyLzn3roSXB/Ed9sHjYpOnbftguKWZalCSPcCGKgMuh1nG2vJWhClcL7Ex4iZ5sPeUhnsoe5mDBpc78zl0kfnlxDmUggZ3OQ5BzJR8=
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: ddn.com
X-MS-Exchange-CrossTenant-Network-Message-Id: cc3df57e-7d38-49e1-eba6-08d73090c2ff
X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Sep 2019 17:04:20.2709
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 753b6e26-6fd3-43e6-8248-3f1735d59bb4
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: Vt/yFrolFeFtItMqa33m8bCgerKq3VBWspQM6MbtqDFP9VNcOVTyV6OM4vFl5w1v
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR19MB3221
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

The event handler calls scsi_scan_host() when events are missed, which
will hotplug new LUNs.  However, this function won't remove any
unplugged LUNs.  The result is that hotunplug doesn't work properly when
the number of unplugged LUNs exceeds the event queue size (currently 8).

Scan existing LUNs when events are missed to check if they are still
present.  If not, remove them.

Signed-off-by: Matt Lupfer <mlupfer@ddn.com>
---
 drivers/scsi/virtio_scsi.c | 31 +++++++++++++++++++++++++++++++
 1 file changed, 31 insertions(+)

diff --git a/drivers/scsi/virtio_scsi.c b/drivers/scsi/virtio_scsi.c
index 297e1076e571..18df77bf371b 100644
--- a/drivers/scsi/virtio_scsi.c
+++ b/drivers/scsi/virtio_scsi.c
@@ -324,6 +324,36 @@ static void virtscsi_handle_param_change(struct virtio=
_scsi *vscsi,
 	scsi_device_put(sdev);
 }
=20
+static void virtscsi_rescan_hotunplug(struct virtio_scsi *vscsi)
+{
+	struct scsi_device *sdev;
+	struct Scsi_Host *shost =3D virtio_scsi_host(vscsi->vdev);
+	unsigned char scsi_cmd[MAX_COMMAND_SIZE];
+	int result, inquiry_len, inq_result_len =3D 256;
+	char *inq_result =3D kmalloc(inq_result_len, GFP_KERNEL);
+
+	shost_for_each_device(sdev, shost) {
+		inquiry_len =3D sdev->inquiry_len ? sdev->inquiry_len : 36;
+
+		memset(scsi_cmd, 0, sizeof(scsi_cmd));
+		scsi_cmd[0] =3D INQUIRY;
+		scsi_cmd[4] =3D (unsigned char) inquiry_len;
+
+		memset(inq_result, 0, inq_result_len);
+
+		result =3D scsi_execute_req(sdev, scsi_cmd, DMA_FROM_DEVICE,
+					  inq_result, inquiry_len, NULL,
+					  2, 3, NULL);
+
+		if (result =3D=3D 0 && inq_result[0] >> 5) {
+			/* PQ indicates the LUN is not attached */
+			scsi_remove_device(sdev);
+		}
+	}
+
+	kfree(inq_result);
+}
+
 static void virtscsi_handle_event(struct work_struct *work)
 {
 	struct virtio_scsi_event_node *event_node =3D
@@ -335,6 +365,7 @@ static void virtscsi_handle_event(struct work_struct *w=
ork)
 	    cpu_to_virtio32(vscsi->vdev, VIRTIO_SCSI_T_EVENTS_MISSED)) {
 		event->event &=3D ~cpu_to_virtio32(vscsi->vdev,
 						   VIRTIO_SCSI_T_EVENTS_MISSED);
+		virtscsi_rescan_hotunplug(vscsi);
 		scsi_scan_host(virtio_scsi_host(vscsi->vdev));
 	}
=20
--=20
2.23.0