* [PULL 01/15] qemu-img-cmds.hx: Update comment that mentions Texinfo
2020-10-13 6:52 [PULL 00/15] Trivial branch for 5.2 patches Laurent Vivier
@ 2020-10-13 6:52 ` Laurent Vivier
2020-10-13 6:53 ` [PULL 02/15] target/i386/cpu: " Laurent Vivier
` (14 subsequent siblings)
15 siblings, 0 replies; 21+ messages in thread
From: Laurent Vivier @ 2020-10-13 6:52 UTC (permalink / raw)
To: qemu-devel
Cc: qemu-trivial, Philippe Mathieu-Daudé, Michael Tokarev,
Markus Armbruster, Laurent Vivier
From: Markus Armbruster <armbru@redhat.com>
Missed in 3c95fdef94 "Update comments in .hx files that mention
Texinfo".
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20200929075824.1517969-2-armbru@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
qemu-img-cmds.hx | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/qemu-img-cmds.hx b/qemu-img-cmds.hx
index b89c019b76bf..cab82342358e 100644
--- a/qemu-img-cmds.hx
+++ b/qemu-img-cmds.hx
@@ -1,5 +1,5 @@
HXCOMM Keep the list of subcommands sorted by name.
-HXCOMM Use DEFHEADING() to define headings in both help text and texi
+HXCOMM Use DEFHEADING() to define headings in both help text and rST
HXCOMM Text between SRST and ERST are copied to rST version and
HXCOMM discarded from C version
HXCOMM DEF(command, callback, arg_string) is used to construct
--
2.26.2
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PULL 02/15] target/i386/cpu: Update comment that mentions Texinfo
2020-10-13 6:52 [PULL 00/15] Trivial branch for 5.2 patches Laurent Vivier
2020-10-13 6:52 ` [PULL 01/15] qemu-img-cmds.hx: Update comment that mentions Texinfo Laurent Vivier
@ 2020-10-13 6:53 ` Laurent Vivier
2020-10-13 6:53 ` [PULL 03/15] hw/rtc/twl92230: Silence warnings about missing fallthrough statements Laurent Vivier
` (13 subsequent siblings)
15 siblings, 0 replies; 21+ messages in thread
From: Laurent Vivier @ 2020-10-13 6:53 UTC (permalink / raw)
To: qemu-devel
Cc: qemu-trivial, Philippe Mathieu-Daudé, Michael Tokarev,
Markus Armbruster, Laurent Vivier
From: Markus Armbruster <armbru@redhat.com>
Missed in commit 41fba1618b "docs/system: convert the documentation of
deprecated features to rST."
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20200929075824.1517969-3-armbru@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
target/i386/cpu.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/target/i386/cpu.c b/target/i386/cpu.c
index f37eb7b675f4..9eafbe369075 100644
--- a/target/i386/cpu.c
+++ b/target/i386/cpu.c
@@ -4127,7 +4127,8 @@ static PropValue tcg_default_props[] = {
* We resolve CPU model aliases using -v1 when using "-machine
* none", but this is just for compatibility while libvirt isn't
* adapted to resolve CPU model versions before creating VMs.
- * See "Runnability guarantee of CPU models" at * qemu-deprecated.texi.
+ * See "Runnability guarantee of CPU models" at
+ * docs/system/deprecated.rst.
*/
X86CPUVersion default_cpu_version = 1;
--
2.26.2
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PULL 03/15] hw/rtc/twl92230: Silence warnings about missing fallthrough statements
2020-10-13 6:52 [PULL 00/15] Trivial branch for 5.2 patches Laurent Vivier
2020-10-13 6:52 ` [PULL 01/15] qemu-img-cmds.hx: Update comment that mentions Texinfo Laurent Vivier
2020-10-13 6:53 ` [PULL 02/15] target/i386/cpu: " Laurent Vivier
@ 2020-10-13 6:53 ` Laurent Vivier
2020-10-13 6:53 ` [PULL 04/15] hw/block/nvme: Simplify timestamp sum Laurent Vivier
` (12 subsequent siblings)
15 siblings, 0 replies; 21+ messages in thread
From: Laurent Vivier @ 2020-10-13 6:53 UTC (permalink / raw)
To: qemu-devel
Cc: qemu-trivial, Peter Maydell, Thomas Huth, Michael Tokarev,
Laurent Vivier
From: Thomas Huth <thuth@redhat.com>
When compiling with -Werror=implicit-fallthrough, gcc complains about
missing fallthrough annotations in this file. Looking at the code,
the fallthrough is indeed wanted here, but instead of adding the
annotations, it can be done more efficiently by simply calculating
the offset with a subtraction instead of increasing a local variable
one by one.
Signed-off-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Message-Id: <20201001182121.187122-1-thuth@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
hw/rtc/twl92230.c | 50 +++++++++++++++++++++++------------------------
1 file changed, 24 insertions(+), 26 deletions(-)
diff --git a/hw/rtc/twl92230.c b/hw/rtc/twl92230.c
index f838913b378d..499f421932e6 100644
--- a/hw/rtc/twl92230.c
+++ b/hw/rtc/twl92230.c
@@ -271,37 +271,36 @@ static void menelaus_gpio_set(void *opaque, int line, int level)
static uint8_t menelaus_read(void *opaque, uint8_t addr)
{
MenelausState *s = (MenelausState *) opaque;
- int reg = 0;
switch (addr) {
case MENELAUS_REV:
return 0x22;
- case MENELAUS_VCORE_CTRL5: reg ++;
- case MENELAUS_VCORE_CTRL4: reg ++;
- case MENELAUS_VCORE_CTRL3: reg ++;
- case MENELAUS_VCORE_CTRL2: reg ++;
+ case MENELAUS_VCORE_CTRL5:
+ case MENELAUS_VCORE_CTRL4:
+ case MENELAUS_VCORE_CTRL3:
+ case MENELAUS_VCORE_CTRL2:
case MENELAUS_VCORE_CTRL1:
- return s->vcore[reg];
+ return s->vcore[addr - MENELAUS_VCORE_CTRL1];
- case MENELAUS_DCDC_CTRL3: reg ++;
- case MENELAUS_DCDC_CTRL2: reg ++;
+ case MENELAUS_DCDC_CTRL3:
+ case MENELAUS_DCDC_CTRL2:
case MENELAUS_DCDC_CTRL1:
- return s->dcdc[reg];
-
- case MENELAUS_LDO_CTRL8: reg ++;
- case MENELAUS_LDO_CTRL7: reg ++;
- case MENELAUS_LDO_CTRL6: reg ++;
- case MENELAUS_LDO_CTRL5: reg ++;
- case MENELAUS_LDO_CTRL4: reg ++;
- case MENELAUS_LDO_CTRL3: reg ++;
- case MENELAUS_LDO_CTRL2: reg ++;
+ return s->dcdc[addr - MENELAUS_VCORE_CTRL1];
+
+ case MENELAUS_LDO_CTRL8:
+ case MENELAUS_LDO_CTRL7:
+ case MENELAUS_LDO_CTRL6:
+ case MENELAUS_LDO_CTRL5:
+ case MENELAUS_LDO_CTRL4:
+ case MENELAUS_LDO_CTRL3:
+ case MENELAUS_LDO_CTRL2:
case MENELAUS_LDO_CTRL1:
- return s->ldo[reg];
+ return s->ldo[addr - MENELAUS_LDO_CTRL1];
- case MENELAUS_SLEEP_CTRL2: reg ++;
+ case MENELAUS_SLEEP_CTRL2:
case MENELAUS_SLEEP_CTRL1:
- return s->sleep[reg];
+ return s->sleep[addr - MENELAUS_SLEEP_CTRL1];
case MENELAUS_DEVICE_OFF:
return 0;
@@ -395,10 +394,10 @@ static uint8_t menelaus_read(void *opaque, uint8_t addr)
case MENELAUS_S2_PULL_DIR:
return s->pull[3];
- case MENELAUS_MCT_CTRL3: reg ++;
- case MENELAUS_MCT_CTRL2: reg ++;
+ case MENELAUS_MCT_CTRL3:
+ case MENELAUS_MCT_CTRL2:
case MENELAUS_MCT_CTRL1:
- return s->mmc_ctrl[reg];
+ return s->mmc_ctrl[addr - MENELAUS_MCT_CTRL1];
case MENELAUS_MCT_PIN_ST:
/* TODO: return the real Card Detect */
return 0;
@@ -418,7 +417,6 @@ static void menelaus_write(void *opaque, uint8_t addr, uint8_t value)
{
MenelausState *s = (MenelausState *) opaque;
int line;
- int reg = 0;
struct tm tm;
switch (addr) {
@@ -496,9 +494,9 @@ static void menelaus_write(void *opaque, uint8_t addr, uint8_t value)
s->ldo[7] = value & 3;
break;
- case MENELAUS_SLEEP_CTRL2: reg ++;
+ case MENELAUS_SLEEP_CTRL2:
case MENELAUS_SLEEP_CTRL1:
- s->sleep[reg] = value;
+ s->sleep[addr - MENELAUS_SLEEP_CTRL1] = value;
break;
case MENELAUS_DEVICE_OFF:
--
2.26.2
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PULL 04/15] hw/block/nvme: Simplify timestamp sum
2020-10-13 6:52 [PULL 00/15] Trivial branch for 5.2 patches Laurent Vivier
` (2 preceding siblings ...)
2020-10-13 6:53 ` [PULL 03/15] hw/rtc/twl92230: Silence warnings about missing fallthrough statements Laurent Vivier
@ 2020-10-13 6:53 ` Laurent Vivier
2020-10-13 6:53 ` [PULL 05/15] tests/test-char: Use a proper fallthrough comment Laurent Vivier
` (11 subsequent siblings)
15 siblings, 0 replies; 21+ messages in thread
From: Laurent Vivier @ 2020-10-13 6:53 UTC (permalink / raw)
To: qemu-devel
Cc: qemu-trivial, Philippe Mathieu-Daudé, Michael Tokarev,
Laurent Vivier, Klaus Jensen
From: Philippe Mathieu-Daudé <philmd@redhat.com>
As the 'timestamp' variable is declared as a 48-bit bitfield,
we do not need to wrap the sum result.
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Klaus Jensen <k.jensen@samsung.com>
Message-Id: <20201002075716.1657849-1-philmd@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
hw/block/nvme.c | 7 +------
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/hw/block/nvme.c b/hw/block/nvme.c
index 63078f600920..44fa5b90769b 100644
--- a/hw/block/nvme.c
+++ b/hw/block/nvme.c
@@ -1280,12 +1280,7 @@ static inline uint64_t nvme_get_timestamp(const NvmeCtrl *n)
union nvme_timestamp ts;
ts.all = 0;
-
- /*
- * If the sum of the Timestamp value set by the host and the elapsed
- * time exceeds 2^48, the value returned should be reduced modulo 2^48.
- */
- ts.timestamp = (n->host_timestamp + elapsed_time) & 0xffffffffffff;
+ ts.timestamp = n->host_timestamp + elapsed_time;
/* If the host timestamp is non-zero, set the timestamp origin */
ts.origin = n->host_timestamp ? 0x01 : 0x00;
--
2.26.2
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PULL 05/15] tests/test-char: Use a proper fallthrough comment
2020-10-13 6:52 [PULL 00/15] Trivial branch for 5.2 patches Laurent Vivier
` (3 preceding siblings ...)
2020-10-13 6:53 ` [PULL 04/15] hw/block/nvme: Simplify timestamp sum Laurent Vivier
@ 2020-10-13 6:53 ` Laurent Vivier
2020-10-13 6:53 ` [PULL 06/15] vmdk: fix maybe uninitialized warnings Laurent Vivier
` (10 subsequent siblings)
15 siblings, 0 replies; 21+ messages in thread
From: Laurent Vivier @ 2020-10-13 6:53 UTC (permalink / raw)
To: qemu-devel
Cc: Thomas Huth, qemu-trivial, Michael Tokarev, Laurent Vivier,
Marc-André Lureau, Philippe Mathieu-Daudé
From: Thomas Huth <thuth@redhat.com>
For being able to compile with -Werror=implicit-fallthrough we need
to use comments that the compiler recognizes. Use "fallthrough" instead
of "no break" here.
Signed-off-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20201002171343.283426-1-thuth@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
tests/test-char.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/test-char.c b/tests/test-char.c
index d35cc839bc60..9196e566e9ba 100644
--- a/tests/test-char.c
+++ b/tests/test-char.c
@@ -70,7 +70,7 @@ static void fe_event(void *opaque, QEMUChrEvent event)
h->openclose_mismatch = true;
}
h->is_open = new_open_state;
- /* no break */
+ /* fallthrough */
default:
quit = true;
break;
--
2.26.2
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PULL 06/15] vmdk: fix maybe uninitialized warnings
2020-10-13 6:52 [PULL 00/15] Trivial branch for 5.2 patches Laurent Vivier
` (4 preceding siblings ...)
2020-10-13 6:53 ` [PULL 05/15] tests/test-char: Use a proper fallthrough comment Laurent Vivier
@ 2020-10-13 6:53 ` Laurent Vivier
2020-10-13 6:53 ` [PULL 07/15] hw/acpi/piix4: Rename piix4_pm_add_propeties() to piix4_pm_add_properties() Laurent Vivier
` (9 subsequent siblings)
15 siblings, 0 replies; 21+ messages in thread
From: Laurent Vivier @ 2020-10-13 6:53 UTC (permalink / raw)
To: qemu-devel
Cc: qemu-trivial, Christian Borntraeger, Fam Zheng, Michael Tokarev,
Laurent Vivier
From: Christian Borntraeger <borntraeger@de.ibm.com>
Fedora 32 gcc 10 seems to give false positives:
Compiling C object libblock.fa.p/block_vmdk.c.o
../block/vmdk.c: In function ‘vmdk_parse_extents’:
../block/vmdk.c:587:5: error: ‘extent’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
587 | g_free(extent->l1_table);
| ^~~~~~~~~~~~~~~~~~~~~~~~
../block/vmdk.c:754:17: note: ‘extent’ was declared here
754 | VmdkExtent *extent;
| ^~~~~~
../block/vmdk.c:620:11: error: ‘extent’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
620 | ret = vmdk_init_tables(bs, extent, errp);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../block/vmdk.c:598:17: note: ‘extent’ was declared here
598 | VmdkExtent *extent;
| ^~~~~~
../block/vmdk.c:1178:39: error: ‘extent’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
1178 | extent->flat_start_offset = flat_offset << 9;
| ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~
../block/vmdk.c: In function ‘vmdk_open_vmdk4’:
../block/vmdk.c:581:22: error: ‘extent’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
581 | extent->l2_cache =
| ~~~~~~~~~~~~~~~~~^
582 | g_malloc(extent->entry_size * extent->l2_size * L2_CACHE_SIZE);
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../block/vmdk.c:872:17: note: ‘extent’ was declared here
872 | VmdkExtent *extent;
| ^~~~~~
../block/vmdk.c: In function ‘vmdk_open’:
../block/vmdk.c:620:11: error: ‘extent’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
620 | ret = vmdk_init_tables(bs, extent, errp);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../block/vmdk.c:598:17: note: ‘extent’ was declared here
598 | VmdkExtent *extent;
| ^~~~~~
cc1: all warnings being treated as errors
make: *** [Makefile.ninja:884: libblock.fa.p/block_vmdk.c.o] Error 1
fix them by assigning a default value.
Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
Reviewed-by: Fam Zheng <fam@euphon.net>
Message-Id: <20200930155859.303148-2-borntraeger@de.ibm.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
block/vmdk.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/block/vmdk.c b/block/vmdk.c
index 8ec62c7ab798..a00dc00eb47a 100644
--- a/block/vmdk.c
+++ b/block/vmdk.c
@@ -595,7 +595,7 @@ static int vmdk_open_vmfs_sparse(BlockDriverState *bs,
int ret;
uint32_t magic;
VMDK3Header header;
- VmdkExtent *extent;
+ VmdkExtent *extent = NULL;
ret = bdrv_pread(file, sizeof(magic), &header, sizeof(header));
if (ret < 0) {
@@ -751,7 +751,7 @@ static int vmdk_open_se_sparse(BlockDriverState *bs,
int ret;
VMDKSESparseConstHeader const_header;
VMDKSESparseVolatileHeader volatile_header;
- VmdkExtent *extent;
+ VmdkExtent *extent = NULL;
ret = bdrv_apply_auto_read_only(bs,
"No write support for seSparse images available", errp);
@@ -869,7 +869,7 @@ static int vmdk_open_vmdk4(BlockDriverState *bs,
uint32_t magic;
uint32_t l1_size, l1_entry_sectors;
VMDK4Header header;
- VmdkExtent *extent;
+ VmdkExtent *extent = NULL;
BDRVVmdkState *s = bs->opaque;
int64_t l1_backup_offset = 0;
bool compressed;
@@ -1088,7 +1088,7 @@ static int vmdk_parse_extents(const char *desc, BlockDriverState *bs,
BdrvChild *extent_file;
BdrvChildRole extent_role;
BDRVVmdkState *s = bs->opaque;
- VmdkExtent *extent;
+ VmdkExtent *extent = NULL;
char extent_opt_prefix[32];
Error *local_err = NULL;
--
2.26.2
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PULL 07/15] hw/acpi/piix4: Rename piix4_pm_add_propeties() to piix4_pm_add_properties()
2020-10-13 6:52 [PULL 00/15] Trivial branch for 5.2 patches Laurent Vivier
` (5 preceding siblings ...)
2020-10-13 6:53 ` [PULL 06/15] vmdk: fix maybe uninitialized warnings Laurent Vivier
@ 2020-10-13 6:53 ` Laurent Vivier
2020-10-13 6:53 ` [PULL 08/15] softmmu/memory: Log invalid memory accesses Laurent Vivier
` (8 subsequent siblings)
15 siblings, 0 replies; 21+ messages in thread
From: Laurent Vivier @ 2020-10-13 6:53 UTC (permalink / raw)
To: qemu-devel
Cc: qemu-trivial, Michael Tokarev, Li Qiang, Greg Kurz,
Laurent Vivier, Philippe Mathieu-Daudé
From: Greg Kurz <groug@kaod.org>
Signed-off-by: Greg Kurz <groug@kaod.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Li Qiang <liq3ea@gmail.com>
Message-Id: <160165476743.57452.2128307974125615413.stgit@bahia.lan>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
hw/acpi/piix4.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/acpi/piix4.c b/hw/acpi/piix4.c
index 894d357f8c35..67a1ea41914f 100644
--- a/hw/acpi/piix4.c
+++ b/hw/acpi/piix4.c
@@ -437,7 +437,7 @@ static void piix4_pm_machine_ready(Notifier *n, void *opaque)
(memory_region_present(io_as, 0x2f8) ? 0x90 : 0);
}
-static void piix4_pm_add_propeties(PIIX4PMState *s)
+static void piix4_pm_add_properties(PIIX4PMState *s)
{
static const uint8_t acpi_enable_cmd = ACPI_ENABLE;
static const uint8_t acpi_disable_cmd = ACPI_DISABLE;
@@ -509,7 +509,7 @@ static void piix4_pm_realize(PCIDevice *dev, Error **errp)
pci_get_bus(dev), s);
qbus_set_hotplug_handler(BUS(pci_get_bus(dev)), OBJECT(s));
- piix4_pm_add_propeties(s);
+ piix4_pm_add_properties(s);
}
I2CBus *piix4_pm_init(PCIBus *bus, int devfn, uint32_t smb_io_base,
--
2.26.2
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PULL 08/15] softmmu/memory: Log invalid memory accesses
2020-10-13 6:52 [PULL 00/15] Trivial branch for 5.2 patches Laurent Vivier
` (6 preceding siblings ...)
2020-10-13 6:53 ` [PULL 07/15] hw/acpi/piix4: Rename piix4_pm_add_propeties() to piix4_pm_add_properties() Laurent Vivier
@ 2020-10-13 6:53 ` Laurent Vivier
2020-10-13 6:53 ` [PULL 09/15] hw/pci: Fix typo in PCI hot-plug error message Laurent Vivier
` (7 subsequent siblings)
15 siblings, 0 replies; 21+ messages in thread
From: Laurent Vivier @ 2020-10-13 6:53 UTC (permalink / raw)
To: qemu-devel
Cc: qemu-trivial, Philippe Mathieu-Daudé, Michael Tokarev,
Laurent Vivier, Michael S . Tsirkin
From: Philippe Mathieu-Daudé <philmd@redhat.com>
Log invalid memory accesses with as GUEST_ERROR.
This is particularly useful since commit 5d971f9e67 which reverted
("memory: accept mismatching sizes in memory_region_access_valid").
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Message-Id: <20201005152725.2143444-1-philmd@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
softmmu/memory.c | 16 ++++++++++++++++
1 file changed, 16 insertions(+)
diff --git a/softmmu/memory.c b/softmmu/memory.c
index fa280a19f7f7..403ff3abc99b 100644
--- a/softmmu/memory.c
+++ b/softmmu/memory.c
@@ -14,6 +14,7 @@
*/
#include "qemu/osdep.h"
+#include "qemu/log.h"
#include "qapi/error.h"
#include "cpu.h"
#include "exec/memory.h"
@@ -1353,10 +1354,18 @@ bool memory_region_access_valid(MemoryRegion *mr,
{
if (mr->ops->valid.accepts
&& !mr->ops->valid.accepts(mr->opaque, addr, size, is_write, attrs)) {
+ qemu_log_mask(LOG_GUEST_ERROR, "Invalid access at addr "
+ "0x%" HWADDR_PRIX ", size %u, "
+ "region '%s', reason: rejected\n",
+ addr, size, memory_region_name(mr));
return false;
}
if (!mr->ops->valid.unaligned && (addr & (size - 1))) {
+ qemu_log_mask(LOG_GUEST_ERROR, "Invalid access at addr "
+ "0x%" HWADDR_PRIX ", size %u, "
+ "region '%s', reason: unaligned\n",
+ addr, size, memory_region_name(mr));
return false;
}
@@ -1367,6 +1376,13 @@ bool memory_region_access_valid(MemoryRegion *mr,
if (size > mr->ops->valid.max_access_size
|| size < mr->ops->valid.min_access_size) {
+ qemu_log_mask(LOG_GUEST_ERROR, "Invalid access at addr "
+ "0x%" HWADDR_PRIX ", size %u, "
+ "region '%s', reason: invalid size "
+ "(min:%u max:%u)\n",
+ addr, size, memory_region_name(mr),
+ mr->ops->valid.min_access_size,
+ mr->ops->valid.max_access_size);
return false;
}
return true;
--
2.26.2
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PULL 09/15] hw/pci: Fix typo in PCI hot-plug error message
2020-10-13 6:52 [PULL 00/15] Trivial branch for 5.2 patches Laurent Vivier
` (7 preceding siblings ...)
2020-10-13 6:53 ` [PULL 08/15] softmmu/memory: Log invalid memory accesses Laurent Vivier
@ 2020-10-13 6:53 ` Laurent Vivier
2020-10-13 6:53 ` [PULL 10/15] block/blkdebug: fix memory leak Laurent Vivier
` (6 subsequent siblings)
15 siblings, 0 replies; 21+ messages in thread
From: Laurent Vivier @ 2020-10-13 6:53 UTC (permalink / raw)
To: qemu-devel
Cc: qemu-trivial, Julia Suvorova, Michael Tokarev, Laurent Vivier,
Philippe Mathieu-Daudé
From: Julia Suvorova <jusual@redhat.com>
'occupied' is spelled like 'ocuppied' in the message.
Signed-off-by: Julia Suvorova <jusual@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20201006133958.600932-1-jusual@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
hw/pci/pci.c | 2 +-
hw/ppc/spapr_pci.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/pci/pci.c b/hw/pci/pci.c
index 3c8f10b4618a..100c9381c2f1 100644
--- a/hw/pci/pci.c
+++ b/hw/pci/pci.c
@@ -1036,7 +1036,7 @@ static PCIDevice *do_pci_register_device(PCIDevice *pci_dev,
return NULL;
} else if (dev->hotplugged &&
pci_get_function_0(pci_dev)) {
- error_setg(errp, "PCI: slot %d function 0 already ocuppied by %s,"
+ error_setg(errp, "PCI: slot %d function 0 already occupied by %s,"
" new func %s cannot be exposed to guest.",
PCI_SLOT(pci_get_function_0(pci_dev)->devfn),
pci_get_function_0(pci_dev)->name,
diff --git a/hw/ppc/spapr_pci.c b/hw/ppc/spapr_pci.c
index 3999392b32c6..88ce87f130a5 100644
--- a/hw/ppc/spapr_pci.c
+++ b/hw/ppc/spapr_pci.c
@@ -1571,7 +1571,7 @@ static void spapr_pci_plug(HotplugHandler *plug_handler,
*/
if (plugged_dev->hotplugged && bus->devices[PCI_DEVFN(slotnr, 0)] &&
PCI_FUNC(pdev->devfn) != 0) {
- error_setg(errp, "PCI: slot %d function 0 already ocuppied by %s,"
+ error_setg(errp, "PCI: slot %d function 0 already occupied by %s,"
" additional functions can no longer be exposed to guest.",
slotnr, bus->devices[PCI_DEVFN(slotnr, 0)]->name);
return;
--
2.26.2
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PULL 10/15] block/blkdebug: fix memory leak
2020-10-13 6:52 [PULL 00/15] Trivial branch for 5.2 patches Laurent Vivier
` (8 preceding siblings ...)
2020-10-13 6:53 ` [PULL 09/15] hw/pci: Fix typo in PCI hot-plug error message Laurent Vivier
@ 2020-10-13 6:53 ` Laurent Vivier
2020-10-13 6:53 ` [PULL 11/15] hw/char/serial: remove duplicate .class_init in serial_mm_info Laurent Vivier
` (5 subsequent siblings)
15 siblings, 0 replies; 21+ messages in thread
From: Laurent Vivier @ 2020-10-13 6:53 UTC (permalink / raw)
To: qemu-devel; +Cc: qemu-trivial, Elena Afanasova, Michael Tokarev, Laurent Vivier
From: Elena Afanasova <eafanasova@gmail.com>
Spotted by PVS-Studio
Signed-off-by: Elena Afanasova <eafanasova@gmail.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <1e903f928eb3da332cc95e2a6f87243bd9fe66e4.camel@gmail.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
block/blkdebug.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/block/blkdebug.c b/block/blkdebug.c
index eecbf3e5c479..54da719dd151 100644
--- a/block/blkdebug.c
+++ b/block/blkdebug.c
@@ -215,6 +215,7 @@ static int add_rule(void *opaque, QemuOpts *opts, Error **errp)
BLKDEBUG_IO_TYPE__MAX, &local_error);
if (local_error) {
error_propagate(errp, local_error);
+ g_free(rule);
return -1;
}
if (iotype != BLKDEBUG_IO_TYPE__MAX) {
--
2.26.2
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PULL 11/15] hw/char/serial: remove duplicate .class_init in serial_mm_info
2020-10-13 6:52 [PULL 00/15] Trivial branch for 5.2 patches Laurent Vivier
` (9 preceding siblings ...)
2020-10-13 6:53 ` [PULL 10/15] block/blkdebug: fix memory leak Laurent Vivier
@ 2020-10-13 6:53 ` Laurent Vivier
2020-10-13 6:53 ` [PULL 12/15] goldfish_rtc: change MemoryRegionOps endianness to DEVICE_NATIVE_ENDIAN Laurent Vivier
` (4 subsequent siblings)
15 siblings, 0 replies; 21+ messages in thread
From: Laurent Vivier @ 2020-10-13 6:53 UTC (permalink / raw)
To: qemu-devel
Cc: Laurent Vivier, qemu-trivial, Michael Tokarev, Li Qiang,
Philippe Mathieu-Daudé, Laurent Vivier, Alistair Francis,
marcandre.lureau
From: Laurent Vivier <lvivier@redhat.com>
.class_init is already set to serial_mm_class_init.
Remove the duplicate entry.
Fixes: 17fd1a6490b1 ("serial-mm: add "regshift" property")
Cc: marcandre.lureau@redhat.com
Signed-off-by: Laurent Vivier <lvivier@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Li Qiang <liq3ea@gmail.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20201009113843.60995-1-lvivier@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
hw/char/serial.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/hw/char/serial.c b/hw/char/serial.c
index 4386adabd466..97f71879ff2a 100644
--- a/hw/char/serial.c
+++ b/hw/char/serial.c
@@ -1120,7 +1120,6 @@ static const TypeInfo serial_mm_info = {
.class_init = serial_mm_class_init,
.instance_init = serial_mm_instance_init,
.instance_size = sizeof(SerialMM),
- .class_init = serial_mm_class_init,
};
static void serial_register_types(void)
--
2.26.2
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PULL 12/15] goldfish_rtc: change MemoryRegionOps endianness to DEVICE_NATIVE_ENDIAN
2020-10-13 6:52 [PULL 00/15] Trivial branch for 5.2 patches Laurent Vivier
` (10 preceding siblings ...)
2020-10-13 6:53 ` [PULL 11/15] hw/char/serial: remove duplicate .class_init in serial_mm_info Laurent Vivier
@ 2020-10-13 6:53 ` Laurent Vivier
2020-10-13 6:53 ` [PULL 13/15] target/sparc/int32_helper: Remove duplicated 'Tag Overflow' entry Laurent Vivier
` (3 subsequent siblings)
15 siblings, 0 replies; 21+ messages in thread
From: Laurent Vivier @ 2020-10-13 6:53 UTC (permalink / raw)
To: qemu-devel
Cc: Laurent Vivier, qemu-trivial, Anup.Patel, Michael Tokarev,
Laurent Vivier, Alistair Francis
From: Laurent Vivier <lvivier@redhat.com>
The doc [1] doesn't define the endianness, but the kernel driver
uses readl() to access the registers, so we can guess it depends
on the architecture endianness.
As riscv architecture endianness is little it might not change anything
for it.
Moreover, android implementation uses DEVICE_NATIVE_ENDIAN [2]
[1] https://android.googlesource.com/platform/external/qemu/+/master/docs/GOLDFISH-VIRTUAL-HARDWARE.TXT
[2] https://android.googlesource.com/platform/external/qemu/+/refs/heads/emu-master-dev/hw/timer/goldfish_timer.c#177
Fixes: 9a5b40b84279 ("hw: rtc: Add Goldfish RTC device")
Cc: Anup.Patel@wdc.com
Signed-off-by: Laurent Vivier <lvivier@redhat.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20201009113843.60995-2-lvivier@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
hw/rtc/goldfish_rtc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/rtc/goldfish_rtc.c b/hw/rtc/goldfish_rtc.c
index 6ddd45cce039..0f4e8185a796 100644
--- a/hw/rtc/goldfish_rtc.c
+++ b/hw/rtc/goldfish_rtc.c
@@ -217,7 +217,7 @@ static int goldfish_rtc_post_load(void *opaque, int version_id)
static const MemoryRegionOps goldfish_rtc_ops = {
.read = goldfish_rtc_read,
.write = goldfish_rtc_write,
- .endianness = DEVICE_LITTLE_ENDIAN,
+ .endianness = DEVICE_NATIVE_ENDIAN,
.valid = {
.min_access_size = 4,
.max_access_size = 4
--
2.26.2
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PULL 13/15] target/sparc/int32_helper: Remove duplicated 'Tag Overflow' entry
2020-10-13 6:52 [PULL 00/15] Trivial branch for 5.2 patches Laurent Vivier
` (11 preceding siblings ...)
2020-10-13 6:53 ` [PULL 12/15] goldfish_rtc: change MemoryRegionOps endianness to DEVICE_NATIVE_ENDIAN Laurent Vivier
@ 2020-10-13 6:53 ` Laurent Vivier
2020-10-13 6:53 ` [PULL 14/15] mingw: fix error __USE_MINGW_ANSI_STDIO redefined Laurent Vivier
` (2 subsequent siblings)
15 siblings, 0 replies; 21+ messages in thread
From: Laurent Vivier @ 2020-10-13 6:53 UTC (permalink / raw)
To: qemu-devel
Cc: qemu-trivial, Mark Cave-Ayland, Michael Tokarev,
Philippe Mathieu-Daudé, Laurent Vivier
From: Philippe Mathieu-Daudé <f4bug@amsat.org>
Commit 0b09be2b2f ("Nicer debug output for exceptions") added
twice the same "Tag Overflow" entry, remove the extra one.
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Message-Id: <20201011200112.3222822-1-f4bug@amsat.org>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
target/sparc/int32_helper.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/target/sparc/int32_helper.c b/target/sparc/int32_helper.c
index 9a71e1abd871..ba63c739c1e8 100644
--- a/target/sparc/int32_helper.c
+++ b/target/sparc/int32_helper.c
@@ -50,7 +50,6 @@ static const char * const excp_names[0x80] = {
[TT_EXTINT | 0xd] = "External Interrupt 13",
[TT_EXTINT | 0xe] = "External Interrupt 14",
[TT_EXTINT | 0xf] = "External Interrupt 15",
- [TT_TOVF] = "Tag Overflow",
[TT_CODE_ACCESS] = "Instruction Access Error",
[TT_DATA_ACCESS] = "Data Access Error",
[TT_DIV_ZERO] = "Division By Zero",
--
2.26.2
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PULL 14/15] mingw: fix error __USE_MINGW_ANSI_STDIO redefined
2020-10-13 6:52 [PULL 00/15] Trivial branch for 5.2 patches Laurent Vivier
` (12 preceding siblings ...)
2020-10-13 6:53 ` [PULL 13/15] target/sparc/int32_helper: Remove duplicated 'Tag Overflow' entry Laurent Vivier
@ 2020-10-13 6:53 ` Laurent Vivier
2020-10-13 6:53 ` [PULL 15/15] meson.build: drop duplicate 'sparc64' entry Laurent Vivier
2020-10-13 11:06 ` [PULL 00/15] Trivial branch for 5.2 patches Peter Maydell
15 siblings, 0 replies; 21+ messages in thread
From: Laurent Vivier @ 2020-10-13 6:53 UTC (permalink / raw)
To: qemu-devel
Cc: Peter Maydell, qemu-trivial, Stefan Weil, Michael Tokarev,
Philippe Mathieu-Daudé, Laurent Vivier,
Marc-André Lureau
From: Marc-André Lureau <marcandre.lureau@redhat.com>
Always put osdep.h first, and remove redundant stdlib.h include.
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Stefan Weil <sw@weilnetz.de>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20201008165953.884599-1-marcandre.lureau@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
migration/dirtyrate.c | 3 ++-
tests/test-bitmap.c | 1 -
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/migration/dirtyrate.c b/migration/dirtyrate.c
index 68577ef25028..47f761e67acb 100644
--- a/migration/dirtyrate.c
+++ b/migration/dirtyrate.c
@@ -10,8 +10,9 @@
* See the COPYING file in the top-level directory.
*/
-#include <zlib.h>
#include "qemu/osdep.h"
+
+#include <zlib.h>
#include "qapi/error.h"
#include "cpu.h"
#include "qemu/config-file.h"
diff --git a/tests/test-bitmap.c b/tests/test-bitmap.c
index 2f5b71458a3d..8db4f67883fe 100644
--- a/tests/test-bitmap.c
+++ b/tests/test-bitmap.c
@@ -8,7 +8,6 @@
* Author: Peter Xu <peterx@redhat.com>
*/
-#include <stdlib.h>
#include "qemu/osdep.h"
#include "qemu/bitmap.h"
--
2.26.2
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PULL 15/15] meson.build: drop duplicate 'sparc64' entry
2020-10-13 6:52 [PULL 00/15] Trivial branch for 5.2 patches Laurent Vivier
` (13 preceding siblings ...)
2020-10-13 6:53 ` [PULL 14/15] mingw: fix error __USE_MINGW_ANSI_STDIO redefined Laurent Vivier
@ 2020-10-13 6:53 ` Laurent Vivier
2020-10-13 11:06 ` [PULL 00/15] Trivial branch for 5.2 patches Peter Maydell
15 siblings, 0 replies; 21+ messages in thread
From: Laurent Vivier @ 2020-10-13 6:53 UTC (permalink / raw)
To: qemu-devel
Cc: qemu-trivial, Michael Tokarev, Laurent Vivier, Sergei Trofimovich,
Philippe Mathieu-Daudé
From: Sergei Trofimovich <slyfox@gentoo.org>
CC: Laurent Vivier <laurent@vivier.eu>
CC: qemu-trivial@nongnu.org
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Sergei Trofimovich <slyfox@gentoo.org>
Message-Id: <20201012175719.2573367-1-slyfox@gentoo.org>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
meson.build | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meson.build b/meson.build
index 17c89c87c626..392d326f4563 100644
--- a/meson.build
+++ b/meson.build
@@ -41,7 +41,7 @@ have_block = have_system or have_tools
python = import('python').find_installation()
supported_oses = ['windows', 'freebsd', 'netbsd', 'openbsd', 'darwin', 'sunos', 'linux']
-supported_cpus = ['ppc', 'ppc64', 's390x', 'sparc64', 'riscv32', 'riscv64', 'x86', 'x86_64',
+supported_cpus = ['ppc', 'ppc64', 's390x', 'riscv32', 'riscv64', 'x86', 'x86_64',
'arm', 'aarch64', 'mips', 'mips64', 'sparc', 'sparc64']
cpu = host_machine.cpu_family()
--
2.26.2
^ permalink raw reply related [flat|nested] 21+ messages in thread* Re: [PULL 00/15] Trivial branch for 5.2 patches
2020-10-13 6:52 [PULL 00/15] Trivial branch for 5.2 patches Laurent Vivier
` (14 preceding siblings ...)
2020-10-13 6:53 ` [PULL 15/15] meson.build: drop duplicate 'sparc64' entry Laurent Vivier
@ 2020-10-13 11:06 ` Peter Maydell
2020-10-13 11:32 ` Laurent Vivier
2020-10-13 11:34 ` Thomas Huth
15 siblings, 2 replies; 21+ messages in thread
From: Peter Maydell @ 2020-10-13 11:06 UTC (permalink / raw)
To: Laurent Vivier; +Cc: QEMU Trivial, Michael Tokarev, QEMU Developers
On Tue, 13 Oct 2020 at 07:56, Laurent Vivier <laurent@vivier.eu> wrote:
>
> The following changes since commit 2387df497b4b4bcf754eb7398edca82889e2ef54:
>
> Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2020-10-10' int=
> o staging (2020-10-12 11:29:42 +0100)
>
> are available in the Git repository at:
>
> git://github.com/vivier/qemu.git tags/trivial-branch-for-5.2-pull-request
>
> for you to fetch changes up to ef3a1d9fc36140f8933fbced70ee21fd9740b33c:
>
> meson.build: drop duplicate 'sparc64' entry (2020-10-12 23:11:32 +0200)
>
> ----------------------------------------------------------------
> Trivial Patches Pull request 20201013
>
> ----------------------------------------------------------------
Compile failure on ppc64be:
../../hw/rtc/twl92230.c: In function ‘menelaus_rx’:
../../hw/rtc/twl92230.c:289:23: error: array subscript is above array
bounds [-Werror=array-bounds]
return s->dcdc[addr - MENELAUS_VCORE_CTRL1];
^
cc1: all warnings being treated as errors
thanks
-- PMM
^ permalink raw reply [flat|nested] 21+ messages in thread* Re: [PULL 00/15] Trivial branch for 5.2 patches
2020-10-13 11:06 ` [PULL 00/15] Trivial branch for 5.2 patches Peter Maydell
@ 2020-10-13 11:32 ` Laurent Vivier
2020-10-13 11:34 ` Thomas Huth
1 sibling, 0 replies; 21+ messages in thread
From: Laurent Vivier @ 2020-10-13 11:32 UTC (permalink / raw)
To: Peter Maydell; +Cc: QEMU Trivial, Thomas Huth, Michael Tokarev, QEMU Developers
Le 13/10/2020 à 13:06, Peter Maydell a écrit :
> On Tue, 13 Oct 2020 at 07:56, Laurent Vivier <laurent@vivier.eu> wrote:
>>
>> The following changes since commit 2387df497b4b4bcf754eb7398edca82889e2ef54:
>>
>> Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2020-10-10' int=
>> o staging (2020-10-12 11:29:42 +0100)
>>
>> are available in the Git repository at:
>>
>> git://github.com/vivier/qemu.git tags/trivial-branch-for-5.2-pull-request
>>
>> for you to fetch changes up to ef3a1d9fc36140f8933fbced70ee21fd9740b33c:
>>
>> meson.build: drop duplicate 'sparc64' entry (2020-10-12 23:11:32 +0200)
>>
>> ----------------------------------------------------------------
>> Trivial Patches Pull request 20201013
>>
>> ----------------------------------------------------------------
>
> Compile failure on ppc64be:
>
> ../../hw/rtc/twl92230.c: In function ‘menelaus_rx’:
> ../../hw/rtc/twl92230.c:289:23: error: array subscript is above array
> bounds [-Werror=array-bounds]
> return s->dcdc[addr - MENELAUS_VCORE_CTRL1];
> ^
> cc1: all warnings being treated as errors
Thank you.
I'm removing the following patch patch from the pull request:
hw/rtc/twl92230: Silence warnings about missing fallthrough statements
Thanks,
Laurent
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PULL 00/15] Trivial branch for 5.2 patches
2020-10-13 11:06 ` [PULL 00/15] Trivial branch for 5.2 patches Peter Maydell
2020-10-13 11:32 ` Laurent Vivier
@ 2020-10-13 11:34 ` Thomas Huth
2020-10-13 11:40 ` Laurent Vivier
1 sibling, 1 reply; 21+ messages in thread
From: Thomas Huth @ 2020-10-13 11:34 UTC (permalink / raw)
To: Laurent Vivier
Cc: QEMU Trivial, Peter Maydell, Michael Tokarev, QEMU Developers
On 13/10/2020 13.06, Peter Maydell wrote:
> On Tue, 13 Oct 2020 at 07:56, Laurent Vivier <laurent@vivier.eu> wrote:
>>
>> The following changes since commit 2387df497b4b4bcf754eb7398edca82889e2ef54:
>>
>> Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2020-10-10' int=
>> o staging (2020-10-12 11:29:42 +0100)
>>
>> are available in the Git repository at:
>>
>> git://github.com/vivier/qemu.git tags/trivial-branch-for-5.2-pull-request
>>
>> for you to fetch changes up to ef3a1d9fc36140f8933fbced70ee21fd9740b33c:
>>
>> meson.build: drop duplicate 'sparc64' entry (2020-10-12 23:11:32 +0200)
>>
>> ----------------------------------------------------------------
>> Trivial Patches Pull request 20201013
>>
>> ----------------------------------------------------------------
>
> Compile failure on ppc64be:
>
> ../../hw/rtc/twl92230.c: In function ‘menelaus_rx’:
> ../../hw/rtc/twl92230.c:289:23: error: array subscript is above array
> bounds [-Werror=array-bounds]
> return s->dcdc[addr - MENELAUS_VCORE_CTRL1];
Uh, oh, mea culpa, copy-n-paste error... that should have been
MENELAUS_DCDC_CTRL1 instead.
Laurent, could you fix it up, or shall I send a v2 of the patch?
Thomas
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PULL 00/15] Trivial branch for 5.2 patches
2020-10-13 11:34 ` Thomas Huth
@ 2020-10-13 11:40 ` Laurent Vivier
2020-10-13 11:47 ` Thomas Huth
0 siblings, 1 reply; 21+ messages in thread
From: Laurent Vivier @ 2020-10-13 11:40 UTC (permalink / raw)
To: Thomas Huth; +Cc: QEMU Trivial, Peter Maydell, Michael Tokarev, QEMU Developers
Le 13/10/2020 à 13:34, Thomas Huth a écrit :
> On 13/10/2020 13.06, Peter Maydell wrote:
>> On Tue, 13 Oct 2020 at 07:56, Laurent Vivier <laurent@vivier.eu> wrote:
>>>
>>> The following changes since commit 2387df497b4b4bcf754eb7398edca82889e2ef54:
>>>
>>> Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2020-10-10' int=
>>> o staging (2020-10-12 11:29:42 +0100)
>>>
>>> are available in the Git repository at:
>>>
>>> git://github.com/vivier/qemu.git tags/trivial-branch-for-5.2-pull-request
>>>
>>> for you to fetch changes up to ef3a1d9fc36140f8933fbced70ee21fd9740b33c:
>>>
>>> meson.build: drop duplicate 'sparc64' entry (2020-10-12 23:11:32 +0200)
>>>
>>> ----------------------------------------------------------------
>>> Trivial Patches Pull request 20201013
>>>
>>> ----------------------------------------------------------------
>>
>> Compile failure on ppc64be:
>>
>> ../../hw/rtc/twl92230.c: In function ‘menelaus_rx’:
>> ../../hw/rtc/twl92230.c:289:23: error: array subscript is above array
>> bounds [-Werror=array-bounds]
>> return s->dcdc[addr - MENELAUS_VCORE_CTRL1];
>
> Uh, oh, mea culpa, copy-n-paste error... that should have been
> MENELAUS_DCDC_CTRL1 instead.
> Laurent, could you fix it up, or shall I send a v2 of the patch?
I prefere a v2, I don't like to play with patches in a pull request.
Thanks,
Laurent
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PULL 00/15] Trivial branch for 5.2 patches
2020-10-13 11:40 ` Laurent Vivier
@ 2020-10-13 11:47 ` Thomas Huth
0 siblings, 0 replies; 21+ messages in thread
From: Thomas Huth @ 2020-10-13 11:47 UTC (permalink / raw)
To: Laurent Vivier
Cc: QEMU Trivial, Peter Maydell, Michael Tokarev, QEMU Developers
On 13/10/2020 13.40, Laurent Vivier wrote:
> Le 13/10/2020 à 13:34, Thomas Huth a écrit :
>> On 13/10/2020 13.06, Peter Maydell wrote:
>>> On Tue, 13 Oct 2020 at 07:56, Laurent Vivier <laurent@vivier.eu> wrote:
>>>>
>>>> The following changes since commit 2387df497b4b4bcf754eb7398edca82889e2ef54:
>>>>
>>>> Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2020-10-10' int=
>>>> o staging (2020-10-12 11:29:42 +0100)
>>>>
>>>> are available in the Git repository at:
>>>>
>>>> git://github.com/vivier/qemu.git tags/trivial-branch-for-5.2-pull-request
>>>>
>>>> for you to fetch changes up to ef3a1d9fc36140f8933fbced70ee21fd9740b33c:
>>>>
>>>> meson.build: drop duplicate 'sparc64' entry (2020-10-12 23:11:32 +0200)
>>>>
>>>> ----------------------------------------------------------------
>>>> Trivial Patches Pull request 20201013
>>>>
>>>> ----------------------------------------------------------------
>>>
>>> Compile failure on ppc64be:
>>>
>>> ../../hw/rtc/twl92230.c: In function ‘menelaus_rx’:
>>> ../../hw/rtc/twl92230.c:289:23: error: array subscript is above array
>>> bounds [-Werror=array-bounds]
>>> return s->dcdc[addr - MENELAUS_VCORE_CTRL1];
>>
>> Uh, oh, mea culpa, copy-n-paste error... that should have been
>> MENELAUS_DCDC_CTRL1 instead.
>> Laurent, could you fix it up, or shall I send a v2 of the patch?
>
> I prefere a v2, I don't like to play with patches in a pull request.
Sure, I'll send a v2 when I've got some spare time (i.e. for a future pull
request, not for this one).
Thomas
^ permalink raw reply [flat|nested] 21+ messages in thread