summaryrefslogtreecommitdiffstats
path: root/drivers/virtio
diff options
context:
space:
mode:
authorMarc Zyngier <marc.zyngier@arm.com>2013-11-05 21:21:28 +1030
committerRusty Russell <rusty@rustcorp.com.au>2013-11-07 12:13:04 +1030
commit4ae85370720156025e9cb873c13a0afb06ca1612 (patch)
tree3f2fc075e48cdfb64df2dd659a3acc656308f18a /drivers/virtio
parent2342d6a6512ce5a3d2433bf77e6580e738cfd709 (diff)
downloadop-kernel-dev-4ae85370720156025e9cb873c13a0afb06ca1612.zip
op-kernel-dev-4ae85370720156025e9cb873c13a0afb06ca1612.tar.gz
virtio: mmio: fix signature checking for BE guests
As virtio-mmio config registers are specified to be little-endian, using readl() to read the magic value and then memcmp() to check it fails on BE (as readl() has an implicit swab). Fix it by encoding the magic value as an integer instead of a string. Cc: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Marc Zyngier <marc.zyngier@arm.com> Acked-by: Pawel Moll <pawel.moll@arm.com> Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Diffstat (limited to 'drivers/virtio')
-rw-r--r--drivers/virtio/virtio_mmio.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/virtio/virtio_mmio.c b/drivers/virtio/virtio_mmio.c
index e9fdeb8..c600ccf 100644
--- a/drivers/virtio/virtio_mmio.c
+++ b/drivers/virtio/virtio_mmio.c
@@ -471,7 +471,7 @@ static int virtio_mmio_probe(struct platform_device *pdev)
/* Check magic value */
magic = readl(vm_dev->base + VIRTIO_MMIO_MAGIC_VALUE);
- if (memcmp(&magic, "virt", 4) != 0) {
+ if (magic != ('v' | 'i' << 8 | 'r' << 16 | 't' << 24)) {
dev_warn(&pdev->dev, "Wrong magic value 0x%08lx!\n", magic);
return -ENODEV;
}
OpenPOWER on IntegriCloud