aboutsummaryrefslogtreecommitdiff
path: root/hw/ppc/spapr_iommu.c
diff options
context:
space:
mode:
authorThomas Huth <thuth@redhat.com>2015-05-07 15:33:38 +1000
committerAlexander Graf <agraf@suse.de>2015-06-03 23:56:51 +0200
commitf9ce8e0aa3fb55ae7a8ea34d3169e73e87feb337 (patch)
tree8c1d69c87ca9c907d335f79e71a48d4a64fbbd2f /hw/ppc/spapr_iommu.c
parentdea1b3ce756d7242d4212c22b7d6e6a896495154 (diff)
downloadqemu-f9ce8e0aa3fb55ae7a8ea34d3169e73e87feb337.zip
qemu-f9ce8e0aa3fb55ae7a8ea34d3169e73e87feb337.tar.gz
qemu-f9ce8e0aa3fb55ae7a8ea34d3169e73e87feb337.tar.bz2
hw/ppc/spapr_iommu: Fix the check for invalid upper bits in liobn
The check "liobn & 0xFFFFFFFF00000000ULL" in spapr_tce_find_by_liobn() is completely useless since liobn is only declared as an uint32_t parameter. Fix this by using target_ulong instead (this is what most of the callers of this function are using, too). Signed-off-by: Thomas Huth <thuth@redhat.com> Signed-off-by: David Gibson <david@gibson.dropbear.id.au> Signed-off-by: Alexander Graf <agraf@suse.de>
Diffstat (limited to 'hw/ppc/spapr_iommu.c')
-rw-r--r--hw/ppc/spapr_iommu.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/hw/ppc/spapr_iommu.c b/hw/ppc/spapr_iommu.c
index a14cdc4..8cd9dba 100644
--- a/hw/ppc/spapr_iommu.c
+++ b/hw/ppc/spapr_iommu.c
@@ -41,7 +41,7 @@ enum sPAPRTCEAccess {
static QLIST_HEAD(spapr_tce_tables, sPAPRTCETable) spapr_tce_tables;
-sPAPRTCETable *spapr_tce_find_by_liobn(uint32_t liobn)
+sPAPRTCETable *spapr_tce_find_by_liobn(target_ulong liobn)
{
sPAPRTCETable *tcet;
@@ -52,7 +52,7 @@ sPAPRTCETable *spapr_tce_find_by_liobn(uint32_t liobn)
}
QLIST_FOREACH(tcet, &spapr_tce_tables, list) {
- if (tcet->liobn == liobn) {
+ if (tcet->liobn == (uint32_t)liobn) {
return tcet;
}
}