Peter Maydell (9):

hw/display/pl110: Remove dead code for non-32-bpp surfaces

hw/display/pl110: Pull included-once parts of template header into pl110.c

hw/display/pl110: Remove use of BITS from pl110_template.h

hw/display/pxa2xx_lcd: Remove dead code for non-32-bpp surfaces

hw/display/pxa2xx_lcd: Remove dest_width state field

hw/display/pxa2xx: Remove use of BITS in pxa2xx_template.h

hw/display/pxa2xx: Apply brace-related coding style fixes to template header

hw/display/pxa2xx: Apply whitespace-only coding style fixes to template header

hw/display/pxa2xx: Inline template header

Philippe Mathieu-Daudé (1):

hw/timer/sse-timer: Propagate eventual error in sse_timer_realize()

Richard Henderson (8):

target/arm: Fix sve_uzp_p vs odd vector lengths

target/arm: Fix sve_zip_p vs odd vector lengths

target/arm: Fix sve_punpk_p vs odd vector lengths

target/arm: Update find_last_active for PREDDESC

target/arm: Update BRKA, BRKB, BRKN for PREDDESC

target/arm: Update CNTP for PREDDESC

target/arm: Update WHILE for PREDDESC

target/arm: Update sve reduction vs simd_desc

docs/system/arm/nuvoton.rst | 2 +-

docs/system/arm/xlnx-versal-virt.rst | 1 +

hw/arm/smmu-internal.h | 5 +

hw/display/pl110_template.h | 120 +-------

hw/display/pxa2xx_template.h | 447 ---------------------------

include/hw/arm/npcm7xx.h | 13 +-

include/hw/arm/xlnx-versal.h | 13 +

include/hw/boards.h | 1 +

include/hw/misc/npcm7xx_mft.h | 70 +++++

include/hw/misc/npcm7xx_pwm.h | 4 +-

include/hw/misc/xlnx-versal-xramc.h | 97 ++++++

include/sysemu/dma.h | 12 +

target/arm/kvm_arm.h | 6 +-

accel/kvm/kvm-all.c | 2 +

hw/arm/npcm7xx.c | 45 ++-

hw/arm/npcm7xx_boards.c | 99 ++++++

hw/arm/smmu-common.c | 32 +-

hw/arm/smmuv3.c | 58 ++--

hw/arm/virt.c | 23 +-

hw/arm/xlnx-versal.c | 36 +++

hw/display/pl110.c | 123 +++++---

hw/display/pxa2xx_lcd.c | 520 ++++++++++++++++++++++++++-----

hw/i386/intel_iommu.c | 32 +-

hw/misc/npcm7xx_mft.c | 540 +++++++++++++++++++++++++++++++++

hw/misc/npcm7xx_pwm.c | 4 +

hw/misc/xlnx-versal-xramc.c | 253 +++++++++++++++

hw/net/allwinner-sun8i-emac.c | 62 ++--

hw/timer/sse-timer.c | 1 +

hw/virtio/virtio-iommu.c | 19 +-

softmmu/dma-helpers.c | 26 ++

target/arm/kvm.c | 4 +-

target/arm/sve_helper.c | 107 ++++---

target/arm/translate-sve.c | 26 +-

tests/qtest/npcm7xx_pwm-test.c | 205 ++++++++++++-

hw/arm/trace-events | 24 +-

hw/misc/meson.build | 2 +

hw/misc/trace-events | 8 +

tests/acceptance/boot_linux_console.py | 120 +++-----

tests/acceptance/replay_kernel.py | 10 +-

39 files changed, 2235 insertions(+), 937 deletions(-)

delete mode 100644 hw/display/pxa2xx_template.h

create mode 100644 include/hw/misc/npcm7xx_mft.h

create mode 100644 include/hw/misc/xlnx-versal-xramc.h

create mode 100644 hw/misc/npcm7xx_mft.c

create mode 100644 hw/misc/xlnx-versal-xramc.c

From: "Edgar E. Iglesias" <edgar.iglesias@xilinx.com>

Connect the support for the Versal Accelerator RAMs (XRAMs).

Reviewed-by: Luc Michel <luc@lmichel.fr>

Acked-by: Alistair Francis <alistair.francis@wdc.com>

Signed-off-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>

Message-id: 20210308224637.2949533-3-edgar.iglesias@gmail.com

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

---

docs/system/arm/xlnx-versal-virt.rst | 1 +

include/hw/arm/xlnx-versal.h | 13 ++++++++++

hw/arm/xlnx-versal.c | 36 ++++++++++++++++++++++++++++

3 files changed, 50 insertions(+)

diff --git a/docs/system/arm/xlnx-versal-virt.rst b/docs/system/arm/xlnx-versal-virt.rst

index XXXXXXX..XXXXXXX 100644

--- a/docs/system/arm/xlnx-versal-virt.rst

+++ b/docs/system/arm/xlnx-versal-virt.rst

@@ -XXX,XX +XXX,XX @@ Implemented devices:

- 8 ADMA (Xilinx zDMA) channels

- 2 SD Controllers

- OCM (256KB of On Chip Memory)

+- XRAM (4MB of on chip Accelerator RAM)

- DDR memory

QEMU does not yet model any other devices, including the PL and the AI Engine.

diff --git a/include/hw/arm/xlnx-versal.h b/include/hw/arm/xlnx-versal.h

index XXXXXXX..XXXXXXX 100644

--- a/include/hw/arm/xlnx-versal.h

+++ b/include/hw/arm/xlnx-versal.h

@@ -XXX,XX +XXX,XX @@

#include "hw/sysbus.h"

#include "hw/arm/boot.h"

+#include "hw/or-irq.h"

#include "hw/sd/sdhci.h"

#include "hw/intc/arm_gicv3.h"

#include "hw/char/pl011.h"

@@ -XXX,XX +XXX,XX @@

#include "hw/rtc/xlnx-zynqmp-rtc.h"

#include "qom/object.h"

#include "hw/usb/xlnx-usb-subsystem.h"

+#include "hw/misc/xlnx-versal-xramc.h"

#define TYPE_XLNX_VERSAL "xlnx-versal"

OBJECT_DECLARE_SIMPLE_TYPE(Versal, XLNX_VERSAL)

@@ -XXX,XX +XXX,XX @@ OBJECT_DECLARE_SIMPLE_TYPE(Versal, XLNX_VERSAL)

#define XLNX_VERSAL_NR_GEMS 2

#define XLNX_VERSAL_NR_ADMAS 8

#define XLNX_VERSAL_NR_SDS 2

+#define XLNX_VERSAL_NR_XRAM 4

#define XLNX_VERSAL_NR_IRQS 192

struct Versal {

@@ -XXX,XX +XXX,XX @@ struct Versal {

XlnxZDMA adma[XLNX_VERSAL_NR_ADMAS];

VersalUsb2 usb;

} iou;

+

+ struct {

+ qemu_or_irq irq_orgate;

+ XlnxXramCtrl ctrl[XLNX_VERSAL_NR_XRAM];

+ } xram;

} lpd;

/* The Platform Management Controller subsystem. */

@@ -XXX,XX +XXX,XX @@ struct Versal {

#define VERSAL_GEM1_IRQ_0 58

#define VERSAL_GEM1_WAKE_IRQ_0 59

#define VERSAL_ADMA_IRQ_0 60

+#define VERSAL_XRAM_IRQ_0 79

#define VERSAL_RTC_APB_ERR_IRQ 121

#define VERSAL_SD0_IRQ_0 126

#define VERSAL_RTC_ALARM_IRQ 142

@@ -XXX,XX +XXX,XX @@ struct Versal {

#define MM_OCM 0xfffc0000U

#define MM_OCM_SIZE 0x40000

+#define MM_XRAM 0xfe800000

+#define MM_XRAMC 0xff8e0000

+#define MM_XRAMC_SIZE 0x10000

+

#define MM_USB2_CTRL_REGS 0xFF9D0000

#define MM_USB2_CTRL_REGS_SIZE 0x10000

diff --git a/hw/arm/xlnx-versal.c b/hw/arm/xlnx-versal.c

index XXXXXXX..XXXXXXX 100644

--- a/hw/arm/xlnx-versal.c

+++ b/hw/arm/xlnx-versal.c

@@ -XXX,XX +XXX,XX @@

*/

#include "qemu/osdep.h"

+#include "qemu/units.h"

#include "qapi/error.h"

#include "qemu/log.h"

#include "qemu/module.h"

@@ -XXX,XX +XXX,XX @@ static void versal_create_rtc(Versal *s, qemu_irq *pic)

sysbus_connect_irq(sbd, 1, pic[VERSAL_RTC_APB_ERR_IRQ]);

}

+static void versal_create_xrams(Versal *s, qemu_irq *pic)

+{

+ int nr_xrams = ARRAY_SIZE(s->lpd.xram.ctrl);

+ DeviceState *orgate;

+ int i;

+

+ /* XRAM IRQs get ORed into a single line. */

+ object_initialize_child(OBJECT(s), "xram-irq-orgate",

+ &s->lpd.xram.irq_orgate, TYPE_OR_IRQ);

+ orgate = DEVICE(&s->lpd.xram.irq_orgate);

+ object_property_set_int(OBJECT(orgate),

+ "num-lines", nr_xrams, &error_fatal);

+ qdev_realize(orgate, NULL, &error_fatal);

+ qdev_connect_gpio_out(orgate, 0, pic[VERSAL_XRAM_IRQ_0]);

+

+ for (i = 0; i < ARRAY_SIZE(s->lpd.xram.ctrl); i++) {

+ SysBusDevice *sbd;

+ MemoryRegion *mr;

+

+ object_initialize_child(OBJECT(s), "xram[*]", &s->lpd.xram.ctrl[i],

+ TYPE_XLNX_XRAM_CTRL);

+ sbd = SYS_BUS_DEVICE(&s->lpd.xram.ctrl[i]);

+ sysbus_realize(sbd, &error_fatal);

+

+ mr = sysbus_mmio_get_region(sbd, 0);

+ memory_region_add_subregion(&s->mr_ps,

+ MM_XRAMC + i * MM_XRAMC_SIZE, mr);

+ mr = sysbus_mmio_get_region(sbd, 1);

+ memory_region_add_subregion(&s->mr_ps, MM_XRAM + i * MiB, mr);

+

+ sysbus_connect_irq(sbd, 0, qdev_get_gpio_in(orgate, i));

+ }

+}

+

/* This takes the board allocated linear DDR memory and creates aliases

* for each split DDR range/aperture on the Versal address map.

*/

@@ -XXX,XX +XXX,XX @@ static void versal_realize(DeviceState *dev, Error **errp)

versal_create_admas(s, pic);

versal_create_sds(s, pic);

versal_create_rtc(s, pic);

+ versal_create_xrams(s, pic);

versal_map_ddr(s);

versal_unimp(s);

--

2.20.1

From: Eric Auger <eric.auger@redhat.com>

With -Werror=maybe-uninitialized configuration we get

../hw/i386/intel_iommu.c: In function ‘vtd_context_device_invalidate’:

../hw/i386/intel_iommu.c:1888:10: error: ‘mask’ may be used

uninitialized in this function [-Werror=maybe-uninitialized]

1888 | mask = ~mask;

| ~~~~~^~~~~~~

Add a g_assert_not_reached() to avoid the error.

Signed-off-by: Eric Auger <eric.auger@redhat.com>

Reviewed-by: Peter Xu <peterx@redhat.com>

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Message-id: 20210309102742.30442-2-eric.auger@redhat.com

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

---

hw/i386/intel_iommu.c | 2 ++

1 file changed, 2 insertions(+)

diff --git a/hw/i386/intel_iommu.c b/hw/i386/intel_iommu.c

index XXXXXXX..XXXXXXX 100644

--- a/hw/i386/intel_iommu.c

+++ b/hw/i386/intel_iommu.c

@@ -XXX,XX +XXX,XX @@ static void vtd_context_device_invalidate(IntelIOMMUState *s,

case 3:

mask = 7; /* Mask bit 2:0 in the SID field */

break;

+ default:

+ g_assert_not_reached();

}

mask = ~mask;

--

2.20.1

From: Eric Auger <eric.auger@redhat.com>

Unmap notifiers work with an address mask assuming an

invalidation range of a power of 2. Nothing mandates this

in the VIRTIO-IOMMU spec.

So in case the range is not a power of 2, split it into

several invalidations.

Signed-off-by: Eric Auger <eric.auger@redhat.com>

Reviewed-by: Peter Xu <peterx@redhat.com>

Message-id: 20210309102742.30442-4-eric.auger@redhat.com

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

---

hw/virtio/virtio-iommu.c | 19 ++++++++++++++++---

1 file changed, 16 insertions(+), 3 deletions(-)

diff --git a/hw/virtio/virtio-iommu.c b/hw/virtio/virtio-iommu.c

index XXXXXXX..XXXXXXX 100644

--- a/hw/virtio/virtio-iommu.c

+++ b/hw/virtio/virtio-iommu.c

@@ -XXX,XX +XXX,XX @@ static void virtio_iommu_notify_unmap(IOMMUMemoryRegion *mr, hwaddr virt_start,

hwaddr virt_end)

{

IOMMUTLBEvent event;

+ uint64_t delta = virt_end - virt_start;

if (!(mr->iommu_notify_flags & IOMMU_NOTIFIER_UNMAP)) {

return;

@@ -XXX,XX +XXX,XX @@ static void virtio_iommu_notify_unmap(IOMMUMemoryRegion *mr, hwaddr virt_start,

event.type = IOMMU_NOTIFIER_UNMAP;

event.entry.target_as = &address_space_memory;

- event.entry.addr_mask = virt_end - virt_start;

- event.entry.iova = virt_start;

event.entry.perm = IOMMU_NONE;

event.entry.translated_addr = 0;

+ event.entry.addr_mask = delta;

+ event.entry.iova = virt_start;

- memory_region_notify_iommu(mr, 0, event);

+ if (delta == UINT64_MAX) {

+ memory_region_notify_iommu(mr, 0, event);

+ }

+

+

+ while (virt_start != virt_end + 1) {

+ uint64_t mask = dma_aligned_pow2_mask(virt_start, virt_end, 64);

+

+ event.entry.addr_mask = mask;

+ event.entry.iova = virt_start;

+ memory_region_notify_iommu(mr, 0, event);

+ virt_start += mask + 1;

+ }

}

static gboolean virtio_iommu_notify_unmap_cb(gpointer key, gpointer value,

--

2.20.1

From: Eric Auger <eric.auger@redhat.com>

If the asid is not set, do not attempt to locate the key directly

as all inserted keys have a valid asid.

Use g_hash_table_foreach_remove instead.

Signed-off-by: Eric Auger <eric.auger@redhat.com>

Message-id: 20210309102742.30442-5-eric.auger@redhat.com

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

---

hw/arm/smmu-common.c | 2 +-

1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hw/arm/smmu-common.c b/hw/arm/smmu-common.c

index XXXXXXX..XXXXXXX 100644

--- a/hw/arm/smmu-common.c

+++ b/hw/arm/smmu-common.c

@@ -XXX,XX +XXX,XX @@ inline void

smmu_iotlb_inv_iova(SMMUState *s, int asid, dma_addr_t iova,

uint8_t tg, uint64_t num_pages, uint8_t ttl)

{

- if (ttl && (num_pages == 1)) {

+ if (ttl && (num_pages == 1) && (asid >= 0)) {

SMMUIOTLBKey key = smmu_get_iotlb_key(asid, iova, tg, ttl);

g_hash_table_remove(s->iotlb, &key);

--

2.20.1

From: Eric Auger <eric.auger@redhat.com>

As of today, the driver can invalidate a number of pages that is

not a power of 2. However IOTLB unmap notifications and internal

IOTLB invalidations work with masks leading to erroneous

invalidations.

In case the range is not a power of 2, split invalidations into

power of 2 invalidations.

When looking for a single page entry in the vSMMU internal IOTLB,

let's make sure that if the entry is not found using a

g_hash_table_remove() we iterate over all the entries to find a

potential range that overlaps it.

Signed-off-by: Eric Auger <eric.auger@redhat.com>

Message-id: 20210309102742.30442-6-eric.auger@redhat.com

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

---

hw/arm/smmu-common.c | 30 ++++++++++++++++++------------

hw/arm/smmuv3.c | 24 ++++++++++++++++++++----

2 files changed, 38 insertions(+), 16 deletions(-)

diff --git a/hw/arm/smmu-common.c b/hw/arm/smmu-common.c

index XXXXXXX..XXXXXXX 100644

--- a/hw/arm/smmu-common.c

+++ b/hw/arm/smmu-common.c

@@ -XXX,XX +XXX,XX @@ inline void

smmu_iotlb_inv_iova(SMMUState *s, int asid, dma_addr_t iova,

uint8_t tg, uint64_t num_pages, uint8_t ttl)

{

+ /* if tg is not set we use 4KB range invalidation */

+ uint8_t granule = tg ? tg * 2 + 10 : 12;

+

if (ttl && (num_pages == 1) && (asid >= 0)) {

SMMUIOTLBKey key = smmu_get_iotlb_key(asid, iova, tg, ttl);

- g_hash_table_remove(s->iotlb, &key);

- } else {

- /* if tg is not set we use 4KB range invalidation */

- uint8_t granule = tg ? tg * 2 + 10 : 12;

-

- SMMUIOTLBPageInvInfo info = {

- .asid = asid, .iova = iova,

- .mask = (num_pages * 1 << granule) - 1};

-

- g_hash_table_foreach_remove(s->iotlb,

- smmu_hash_remove_by_asid_iova,

- &info);

+ if (g_hash_table_remove(s->iotlb, &key)) {

+ return;

+ }

+ /*

+ * if the entry is not found, let's see if it does not

+ * belong to a larger IOTLB entry

+ */

}

+

+ SMMUIOTLBPageInvInfo info = {

+ .asid = asid, .iova = iova,

+ .mask = (num_pages * 1 << granule) - 1};

+

+ g_hash_table_foreach_remove(s->iotlb,

+ smmu_hash_remove_by_asid_iova,

+ &info);

}

inline void smmu_iotlb_inv_asid(SMMUState *s, uint16_t asid)

diff --git a/hw/arm/smmuv3.c b/hw/arm/smmuv3.c

index XXXXXXX..XXXXXXX 100644

--- a/hw/arm/smmuv3.c

+++ b/hw/arm/smmuv3.c

@@ -XXX,XX +XXX,XX @@ static void smmuv3_s1_range_inval(SMMUState *s, Cmd *cmd)

uint16_t vmid = CMD_VMID(cmd);

bool leaf = CMD_LEAF(cmd);

uint8_t tg = CMD_TG(cmd);

- hwaddr num_pages = 1;

+ uint64_t first_page = 0, last_page;

+ uint64_t num_pages = 1;

int asid = -1;

if (tg) {

@@ -XXX,XX +XXX,XX @@ static void smmuv3_s1_range_inval(SMMUState *s, Cmd *cmd)

if (type == SMMU_CMD_TLBI_NH_VA) {

asid = CMD_ASID(cmd);

}

- trace_smmuv3_s1_range_inval(vmid, asid, addr, tg, num_pages, ttl, leaf);

- smmuv3_inv_notifiers_iova(s, asid, addr, tg, num_pages);

- smmu_iotlb_inv_iova(s, asid, addr, tg, num_pages, ttl);

+

+ /* Split invalidations into ^2 range invalidations */

+ last_page = num_pages - 1;

+ while (num_pages) {

+ uint8_t granule = tg * 2 + 10;

+ uint64_t mask, count;

+

+ mask = dma_aligned_pow2_mask(first_page, last_page, 64 - granule);

+ count = mask + 1;

+

+ trace_smmuv3_s1_range_inval(vmid, asid, addr, tg, count, ttl, leaf);

+ smmuv3_inv_notifiers_iova(s, asid, addr, tg, count);

+ smmu_iotlb_inv_iova(s, asid, addr, tg, count, ttl);

+

+ num_pages -= count;

+ first_page += count;

+ addr += count * BIT_ULL(granule);

+ }

}

static int smmuv3_cmdq_consume(SMMUv3State *s)

--

2.20.1

From: Eric Auger <eric.auger@redhat.com>

If the whole SID range (32b) is invalidated (SMMU_CMD_CFGI_ALL),

@end overflows and we fail to handle the command properly.

Once this gets fixed, the current code really is awkward in the

sense it loops over the whole range instead of removing the

currently cached configs through a hash table lookup.

Fix both the overflow and the lookup.

Signed-off-by: Eric Auger <eric.auger@redhat.com>

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

Message-id: 20210309102742.30442-7-eric.auger@redhat.com

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

---

hw/arm/smmu-internal.h | 5 +++++

hw/arm/smmuv3.c | 34 ++++++++++++++++++++--------------

2 files changed, 25 insertions(+), 14 deletions(-)

diff --git a/hw/arm/smmu-internal.h b/hw/arm/smmu-internal.h

index XXXXXXX..XXXXXXX 100644

--- a/hw/arm/smmu-internal.h

+++ b/hw/arm/smmu-internal.h

@@ -XXX,XX +XXX,XX @@ typedef struct SMMUIOTLBPageInvInfo {

uint64_t mask;

} SMMUIOTLBPageInvInfo;

+typedef struct SMMUSIDRange {

+ uint32_t start;

+ uint32_t end;

+} SMMUSIDRange;

+

#endif

diff --git a/hw/arm/smmuv3.c b/hw/arm/smmuv3.c

index XXXXXXX..XXXXXXX 100644

--- a/hw/arm/smmuv3.c

+++ b/hw/arm/smmuv3.c

@@ -XXX,XX +XXX,XX @@

#include "hw/arm/smmuv3.h"

#include "smmuv3-internal.h"

+#include "smmu-internal.h"

/**

* smmuv3_trigger_irq - pulse @irq if enabled and update

@@ -XXX,XX +XXX,XX @@ static void smmuv3_s1_range_inval(SMMUState *s, Cmd *cmd)

}

}

+static gboolean

+smmuv3_invalidate_ste(gpointer key, gpointer value, gpointer user_data)

+{

+ SMMUDevice *sdev = (SMMUDevice *)key;

+ uint32_t sid = smmu_get_sid(sdev);

+ SMMUSIDRange *sid_range = (SMMUSIDRange *)user_data;

+

+ if (sid < sid_range->start || sid > sid_range->end) {

+ return false;

+ }

+ trace_smmuv3_config_cache_inv(sid);

+ return true;

+}

+

static int smmuv3_cmdq_consume(SMMUv3State *s)

{

SMMUState *bs = ARM_SMMU(s);

@@ -XXX,XX +XXX,XX @@ static int smmuv3_cmdq_consume(SMMUv3State *s)

}

case SMMU_CMD_CFGI_STE_RANGE: /* same as SMMU_CMD_CFGI_ALL */

{

- uint32_t start = CMD_SID(&cmd), end, i;

+ uint32_t start = CMD_SID(&cmd);

uint8_t range = CMD_STE_RANGE(&cmd);

+ uint64_t end = start + (1ULL << (range + 1)) - 1;

+ SMMUSIDRange sid_range = {start, end};

if (CMD_SSEC(&cmd)) {

cmd_error = SMMU_CERROR_ILL;

break;

}

-

- end = start + (1 << (range + 1)) - 1;

trace_smmuv3_cmdq_cfgi_ste_range(start, end);

-

- for (i = start; i <= end; i++) {

- IOMMUMemoryRegion *mr = smmu_iommu_mr(bs, i);

- SMMUDevice *sdev;

-

- if (!mr) {

- continue;

- }

- sdev = container_of(mr, SMMUDevice, iommu);

- smmuv3_flush_config(sdev);

- }

+ g_hash_table_foreach_remove(bs->configs, smmuv3_invalidate_ste,

+ &sid_range);

break;

}

case SMMU_CMD_CFGI_CD:

--

2.20.1

From: Eric Auger <eric.auger@redhat.com>

Convert all sid printouts to sid=0x%x.

Signed-off-by: Eric Auger <eric.auger@redhat.com>

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Message-id: 20210309102742.30442-8-eric.auger@redhat.com

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

---

hw/arm/trace-events | 24 ++++++++++++------------

1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/hw/arm/trace-events b/hw/arm/trace-events

index XXXXXXX..XXXXXXX 100644

--- a/hw/arm/trace-events

+++ b/hw/arm/trace-events

@@ -XXX,XX +XXX,XX @@ smmuv3_cmdq_opcode(const char *opcode) "<--- %s"

smmuv3_cmdq_consume_out(uint32_t prod, uint32_t cons, uint8_t prod_wrap, uint8_t cons_wrap) "prod:%d, cons:%d, prod_wrap:%d, cons_wrap:%d "

smmuv3_cmdq_consume_error(const char *cmd_name, uint8_t cmd_error) "Error on %s command execution: %d"

smmuv3_write_mmio(uint64_t addr, uint64_t val, unsigned size, uint32_t r) "addr: 0x%"PRIx64" val:0x%"PRIx64" size: 0x%x(%d)"

-smmuv3_record_event(const char *type, uint32_t sid) "%s sid=%d"

-smmuv3_find_ste(uint16_t sid, uint32_t features, uint16_t sid_split) "SID:0x%x features:0x%x, sid_split:0x%x"

+smmuv3_record_event(const char *type, uint32_t sid) "%s sid=0x%x"

+smmuv3_find_ste(uint16_t sid, uint32_t features, uint16_t sid_split) "sid=0x%x features:0x%x, sid_split:0x%x"

smmuv3_find_ste_2lvl(uint64_t strtab_base, uint64_t l1ptr, int l1_ste_offset, uint64_t l2ptr, int l2_ste_offset, int max_l2_ste) "strtab_base:0x%"PRIx64" l1ptr:0x%"PRIx64" l1_off:0x%x, l2ptr:0x%"PRIx64" l2_off:0x%x max_l2_ste:%d"

smmuv3_get_ste(uint64_t addr) "STE addr: 0x%"PRIx64

-smmuv3_translate_disable(const char *n, uint16_t sid, uint64_t addr, bool is_write) "%s sid=%d bypass (smmu disabled) iova:0x%"PRIx64" is_write=%d"

-smmuv3_translate_bypass(const char *n, uint16_t sid, uint64_t addr, bool is_write) "%s sid=%d STE bypass iova:0x%"PRIx64" is_write=%d"

-smmuv3_translate_abort(const char *n, uint16_t sid, uint64_t addr, bool is_write) "%s sid=%d abort on iova:0x%"PRIx64" is_write=%d"

-smmuv3_translate_success(const char *n, uint16_t sid, uint64_t iova, uint64_t translated, int perm) "%s sid=%d iova=0x%"PRIx64" translated=0x%"PRIx64" perm=0x%x"

+smmuv3_translate_disable(const char *n, uint16_t sid, uint64_t addr, bool is_write) "%s sid=0x%x bypass (smmu disabled) iova:0x%"PRIx64" is_write=%d"

+smmuv3_translate_bypass(const char *n, uint16_t sid, uint64_t addr, bool is_write) "%s sid=0x%x STE bypass iova:0x%"PRIx64" is_write=%d"

+smmuv3_translate_abort(const char *n, uint16_t sid, uint64_t addr, bool is_write) "%s sid=0x%x abort on iova:0x%"PRIx64" is_write=%d"

+smmuv3_translate_success(const char *n, uint16_t sid, uint64_t iova, uint64_t translated, int perm) "%s sid=0x%x iova=0x%"PRIx64" translated=0x%"PRIx64" perm=0x%x"

smmuv3_get_cd(uint64_t addr) "CD addr: 0x%"PRIx64

smmuv3_decode_cd(uint32_t oas) "oas=%d"

smmuv3_decode_cd_tt(int i, uint32_t tsz, uint64_t ttb, uint32_t granule_sz, bool had) "TT[%d]:tsz:%d ttb:0x%"PRIx64" granule_sz:%d had:%d"

-smmuv3_cmdq_cfgi_ste(int streamid) "streamid =%d"

+smmuv3_cmdq_cfgi_ste(int streamid) "streamid= 0x%x"

smmuv3_cmdq_cfgi_ste_range(int start, int end) "start=0x%x - end=0x%x"

-smmuv3_cmdq_cfgi_cd(uint32_t sid) "streamid = %d"

-smmuv3_config_cache_hit(uint32_t sid, uint32_t hits, uint32_t misses, uint32_t perc) "Config cache HIT for sid %d (hits=%d, misses=%d, hit rate=%d)"

-smmuv3_config_cache_miss(uint32_t sid, uint32_t hits, uint32_t misses, uint32_t perc) "Config cache MISS for sid %d (hits=%d, misses=%d, hit rate=%d)"