[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 07/14] register: fix incorrect read mask
From: |
Michael Tokarev |
Subject: |
[Qemu-devel] [PULL 07/14] register: fix incorrect read mask |
Date: |
Tue, 28 Feb 2017 09:44:50 +0300 |
From: Philippe Mathieu-Daudé <address@hidden>
The register_read() and register_write() functions expect a bitmask argument.
To avoid duplicated code, a new inlined function register_enabled_mask() is
introduced.
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Alistair Francis <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>
---
hw/core/register.c | 21 +++++++++++++++------
1 file changed, 15 insertions(+), 6 deletions(-)
diff --git a/hw/core/register.c b/hw/core/register.c
index be170dc..dc335a7 100644
--- a/hw/core/register.c
+++ b/hw/core/register.c
@@ -59,6 +59,15 @@ static inline uint64_t register_read_val(RegisterInfo *reg)
return 0; /* unreachable */
}
+static inline uint64_t register_enabled_mask(int data_size, unsigned size)
+{
+ if (data_size < size) {
+ size = data_size;
+ }
+
+ return MAKE_64BIT_MASK(0, size * 8);
+}
+
void register_write(RegisterInfo *reg, uint64_t val, uint64_t we,
const char *prefix, bool debug)
{
@@ -192,11 +201,7 @@ void register_write_memory(void *opaque, hwaddr addr,
}
/* Generate appropriate write enable mask */
- if (reg->data_size < size) {
- we = MAKE_64BIT_MASK(0, reg->data_size * 8);
- } else {
- we = MAKE_64BIT_MASK(0, size * 8);
- }
+ we = register_enabled_mask(reg->data_size, size);
register_write(reg, value, we, reg_array->prefix,
reg_array->debug);
@@ -208,6 +213,7 @@ uint64_t register_read_memory(void *opaque, hwaddr addr,
RegisterInfoArray *reg_array = opaque;
RegisterInfo *reg = NULL;
uint64_t read_val;
+ uint64_t re;
int i;
for (i = 0; i < reg_array->num_elements; i++) {
@@ -223,7 +229,10 @@ uint64_t register_read_memory(void *opaque, hwaddr addr,
return 0;
}
- read_val = register_read(reg, size * 8, reg_array->prefix,
+ /* Generate appropriate read enable mask */
+ re = register_enabled_mask(reg->data_size, size);
+
+ read_val = register_read(reg, re, reg_array->prefix,
reg_array->debug);
return extract64(read_val, 0, size * 8);
--
2.1.4
- [Qemu-devel] [PULL 01/14] target/s390x: Fix typo, (continued)
- [Qemu-devel] [PULL 01/14] target/s390x: Fix typo, Michael Tokarev, 2017/02/28
- [Qemu-devel] [PULL 13/14] hw/acpi/tco.c: fix tco timer stop, Michael Tokarev, 2017/02/28
- [Qemu-devel] [PULL 06/14] ide: remove undefined behavior in ide-test, Michael Tokarev, 2017/02/28
- [Qemu-devel] [PULL 14/14] syscall: fixed mincore(2) not failing with ENOMEM, Michael Tokarev, 2017/02/28
- [Qemu-devel] [PULL 05/14] CODING_STYLE: Mention preferred comment form, Michael Tokarev, 2017/02/28
- [Qemu-devel] [PULL 09/14] tests/prom-env: Enable the test for the sun4u machine, too, Michael Tokarev, 2017/02/28
- [Qemu-devel] [PULL 11/14] qemu-options.hx: add missing id=chr0 chardev argument in vhost-user example, Michael Tokarev, 2017/02/28
- [Qemu-devel] [PULL 10/14] Update copyright year, Michael Tokarev, 2017/02/28
- [Qemu-devel] [PULL 08/14] cadence_gem: Remove unused parameter debug message, Michael Tokarev, 2017/02/28
- [Qemu-devel] [PULL 04/14] hw/core/register: Mark the device with cannot_instantiate_with_device_add_yet, Michael Tokarev, 2017/02/28
- [Qemu-devel] [PULL 07/14] register: fix incorrect read mask,
Michael Tokarev <=
- [Qemu-devel] [PULL 02/14] softfloat: Use correct type in float64_to_uint64_round_to_zero(), Michael Tokarev, 2017/02/28
- [Qemu-devel] [PULL 12/14] lm32: milkymist-tmu2: fix a third integer overflow, Michael Tokarev, 2017/02/28
- [Qemu-devel] [PULL 03/14] hw/core/or-irq: Mark the device with cannot_instantiate_with_device_add_yet, Michael Tokarev, 2017/02/28
- Re: [Qemu-devel] [PULL 00/14] Trivial patches for 2017-02-28, Peter Maydell, 2017/02/28