summaryrefslogtreecommitdiff
path: root/sys/dev
diff options
context:
space:
mode:
authoryamaguchi <yamaguchi@NetBSD.org>2023-03-25 02:59:23 +0000
committeryamaguchi <yamaguchi@NetBSD.org>2023-03-25 02:59:23 +0000
commitf8f39d50cab0eca1fcabc3bb34b9040d9d85f31b (patch)
tree3f46611a7579dc33b118b34d3f17ca30ccc203ce /sys/dev
parent41768b338a658f1058f03cfb2e1b33ef3b256b0c (diff)
Setup virtqueues after registering them to virtio_softc
Diffstat (limited to 'sys/dev')
-rw-r--r--sys/dev/pci/virtio.c25
1 files changed, 14 insertions, 11 deletions
diff --git a/sys/dev/pci/virtio.c b/sys/dev/pci/virtio.c
index 3d7f32abe3c..65023618ddf 100644
--- a/sys/dev/pci/virtio.c
+++ b/sys/dev/pci/virtio.c
@@ -1,4 +1,4 @@
-/* $NetBSD: virtio.c,v 1.68 2023/03/24 13:32:19 yamaguchi Exp $ */
+/* $NetBSD: virtio.c,v 1.69 2023/03/25 02:59:23 yamaguchi Exp $ */
/*
* Copyright (c) 2020 The NetBSD Foundation, Inc.
@@ -28,7 +28,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: virtio.c,v 1.68 2023/03/24 13:32:19 yamaguchi Exp $");
+__KERNEL_RCSID(0, "$NetBSD: virtio.c,v 1.69 2023/03/25 02:59:23 yamaguchi Exp $");
#include <sys/param.h>
#include <sys/systm.h>
@@ -865,10 +865,6 @@ virtio_alloc_vq(struct virtio_softc *sc, struct virtqueue *vq,
virtio_reset_vq(sc, vq);
- /* set the vq address */
- sc->sc_ops->setup_queue(sc, vq->vq_index,
- vq->vq_dmamap->dm_segs[0].ds_addr);
-
aprint_verbose_dev(sc->sc_dev,
"allocated %zu byte for virtqueue %d for %s, size %d\n",
allocsize, vq->vq_index, name, vq_num);
@@ -1301,6 +1297,7 @@ virtio_child_attach_finish(struct virtio_softc *sc,
virtio_callback config_change,
int req_flags)
{
+ size_t i;
int r;
#ifdef DIAGNOSTIC
@@ -1309,11 +1306,11 @@ virtio_child_attach_finish(struct virtio_softc *sc,
KASSERT((req_flags & VIRTIO_ASSERT_FLAGS) != VIRTIO_ASSERT_FLAGS);
#undef VIRTIO_ASSERT_FLAGS
- for (size_t _i = 0; _i < nvqs; _i++){
- KASSERT(vqs[_i].vq_index == _i);
- KASSERT(vqs[_i].vq_intrhand != NULL);
- KASSERT(vqs[_i].vq_done == NULL ||
- vqs[_i].vq_intrhand == virtio_vq_done);
+ for (i = 0; i < nvqs; i++){
+ KASSERT(vqs[i].vq_index == i);
+ KASSERT(vqs[i].vq_intrhand != NULL);
+ KASSERT(vqs[i].vq_done == NULL ||
+ vqs[i].vq_intrhand == virtio_vq_done);
}
#endif
@@ -1325,6 +1322,12 @@ virtio_child_attach_finish(struct virtio_softc *sc,
sc->sc_intrhand = virtio_vq_intr;
sc->sc_flags = req_flags;
+ /* set the vq address */
+ for (i = 0; i < nvqs; i++) {
+ sc->sc_ops->setup_queue(sc, vqs[i].vq_index,
+ vqs[i].vq_dmamap->dm_segs[0].ds_addr);
+ }
+
r = sc->sc_ops->alloc_interrupts(sc);
if (r != 0) {
aprint_error_dev(sc->sc_dev,