* [Xen-devel] [PULL 0/4] Trivial branch patches
@ 2019-11-05 14:42 ` Laurent Vivier
0 siblings, 0 replies; 33+ messages in thread
From: Laurent Vivier @ 2019-11-05 14:42 UTC (permalink / raw)
To: qemu-devel
Cc: Peter Maydell, Stefano Stabellini, Daniel P. Berrangé,
Eduardo Habkost, Paolo Bonzini, Paul Durrant, qemu-trivial,
Claudio Fontana, Cleber Rosa, Michael Tokarev, Laurent Vivier,
Fabien Chouteau, KONRAD Frederic, qemu-arm, qemu-ppc,
Cédric Le Goater, Marc-André Lureau, Anthony Perard,
xen-devel, David Gibson
The following changes since commit 36609b4fa36f0ac934874371874416f7533a5408:
Merge remote-tracking branch 'remotes/palmer/tags/palmer-for-master-4.2-sf1' into staging (2019-11-02 17:59:03 +0000)
are available in the Git repository at:
git://github.com/vivier/qemu.git tags/trivial-branch-pull-request
for you to fetch changes up to a37a36a11b584e083b1c578f1d60e6e0f7878d5f:
global: Squash 'the the' (2019-11-05 15:06:09 +0100)
----------------------------------------------------------------
Trivial fixes (20191105)
----------------------------------------------------------------
Dr. David Alan Gilbert (1):
global: Squash 'the the'
Greg Kurz (1):
qom: Fix error message in object_class_property_add()
Philippe Mathieu-Daudé (2):
hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
disas/libvixl/vixl/invalset.h | 2 +-
docs/interop/pr-helper.rst | 2 +-
docs/specs/ppc-spapr-hotplug.txt | 2 +-
docs/specs/ppc-xive.rst | 2 +-
docs/specs/tpm.txt | 2 +-
hw/misc/grlib_ahb_apb_pnp.c | 12 ++++++++++++
include/hw/xen/interface/io/blkif.h | 2 +-
qom/object.c | 10 ++++------
scripts/dump-guest-memory.py | 2 +-
9 files changed, 23 insertions(+), 13 deletions(-)
--
2.21.0
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 33+ messages in thread
* [PULL 0/4] Trivial branch patches
@ 2019-11-05 14:42 ` Laurent Vivier
0 siblings, 0 replies; 33+ messages in thread
From: Laurent Vivier @ 2019-11-05 14:42 UTC (permalink / raw)
To: qemu-devel
Cc: Peter Maydell, Stefano Stabellini, Daniel P. Berrangé,
Eduardo Habkost, Paolo Bonzini, Paul Durrant, qemu-trivial,
Claudio Fontana, Cleber Rosa, Michael Tokarev, Laurent Vivier,
Fabien Chouteau, KONRAD Frederic, qemu-arm, qemu-ppc,
Cédric Le Goater, Marc-André Lureau, Anthony Perard,
xen-devel, David Gibson
The following changes since commit 36609b4fa36f0ac934874371874416f7533a5408:
Merge remote-tracking branch 'remotes/palmer/tags/palmer-for-master-4.2-sf1' into staging (2019-11-02 17:59:03 +0000)
are available in the Git repository at:
git://github.com/vivier/qemu.git tags/trivial-branch-pull-request
for you to fetch changes up to a37a36a11b584e083b1c578f1d60e6e0f7878d5f:
global: Squash 'the the' (2019-11-05 15:06:09 +0100)
----------------------------------------------------------------
Trivial fixes (20191105)
----------------------------------------------------------------
Dr. David Alan Gilbert (1):
global: Squash 'the the'
Greg Kurz (1):
qom: Fix error message in object_class_property_add()
Philippe Mathieu-Daudé (2):
hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
disas/libvixl/vixl/invalset.h | 2 +-
docs/interop/pr-helper.rst | 2 +-
docs/specs/ppc-spapr-hotplug.txt | 2 +-
docs/specs/ppc-xive.rst | 2 +-
docs/specs/tpm.txt | 2 +-
hw/misc/grlib_ahb_apb_pnp.c | 12 ++++++++++++
include/hw/xen/interface/io/blkif.h | 2 +-
qom/object.c | 10 ++++------
scripts/dump-guest-memory.py | 2 +-
9 files changed, 23 insertions(+), 13 deletions(-)
--
2.21.0
^ permalink raw reply [flat|nested] 33+ messages in thread
* [PULL 1/4] hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
2019-11-05 14:42 ` Laurent Vivier
(?)
@ 2019-11-05 14:42 ` Laurent Vivier
-1 siblings, 0 replies; 33+ messages in thread
From: Laurent Vivier @ 2019-11-05 14:42 UTC (permalink / raw)
To: qemu-devel
Cc: qemu-trivial, Stefano Stabellini, xen-devel,
Cédric Le Goater, Laurent Vivier, Daniel P. Berrangé,
Eduardo Habkost, David Gibson, qemu-arm, Marc-André Lureau,
Fabien Chouteau, qemu-ppc, Paul Durrant, Michael Tokarev,
KONRAD Frederic, Cleber Rosa, Claudio Fontana, Anthony Perard,
Paolo Bonzini, Peter Maydell, Philippe Mathieu-Daudé
From: Philippe Mathieu-Daudé <philmd@redhat.com>
Guests can crash QEMU when writting to PnP registers:
$ echo 'writeb 0x800ff042 69' | qemu-system-sparc -M leon3_generic -S -bios /etc/magic -qtest stdio
[I 1571938309.932255] OPENED
[R +0.063474] writeb 0x800ff042 69
Segmentation fault (core dumped)
(gdb) bt
#0 0x0000000000000000 in ()
#1 0x0000555f4bcdf0bc in memory_region_write_with_attrs_accessor (mr=0x555f4d7be8c0, addr=66, value=0x7fff07d00f08, size=1, shift=0, mask=255, attrs=...) at memory.c:503
#2 0x0000555f4bcdf185 in access_with_adjusted_size (addr=66, value=0x7fff07d00f08, size=1, access_size_min=1, access_size_max=4, access_fn=0x555f4bcdeff4 <memory_region_write_with_attrs_accessor>, mr=0x555f4d7be8c0, attrs=...) at memory.c:539
#3 0x0000555f4bce2243 in memory_region_dispatch_write (mr=0x555f4d7be8c0, addr=66, data=69, op=MO_8, attrs=...) at memory.c:1489
#4 0x0000555f4bc80b20 in flatview_write_continue (fv=0x555f4d92c400, addr=2148528194, attrs=..., buf=0x7fff07d01120 "E", len=1, addr1=66, l=1, mr=0x555f4d7be8c0) at exec.c:3161
#5 0x0000555f4bc80c65 in flatview_write (fv=0x555f4d92c400, addr=2148528194, attrs=..., buf=0x7fff07d01120 "E", len=1) at exec.c:3201
#6 0x0000555f4bc80fb0 in address_space_write (as=0x555f4d7aa460, addr=2148528194, attrs=..., buf=0x7fff07d01120 "E", len=1) at exec.c:3291
#7 0x0000555f4bc8101d in address_space_rw (as=0x555f4d7aa460, addr=2148528194, attrs=..., buf=0x7fff07d01120 "E", len=1, is_write=true) at exec.c:3301
#8 0x0000555f4bcdb388 in qtest_process_command (chr=0x555f4c2ed7e0 <qtest_chr>, words=0x555f4db0c5d0) at qtest.c:432
Instead of crashing, log the access as unimplemented.
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: KONRAD Frederic <frederic.konrad@adacore.com>
Message-Id: <20191025110114.27091-2-philmd@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
hw/misc/grlib_ahb_apb_pnp.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/hw/misc/grlib_ahb_apb_pnp.c b/hw/misc/grlib_ahb_apb_pnp.c
index 7338461694c9..f3c015d2c35f 100644
--- a/hw/misc/grlib_ahb_apb_pnp.c
+++ b/hw/misc/grlib_ahb_apb_pnp.c
@@ -22,6 +22,7 @@
*/
#include "qemu/osdep.h"
+#include "qemu/log.h"
#include "hw/sysbus.h"
#include "hw/misc/grlib_ahb_apb_pnp.h"
@@ -231,8 +232,15 @@ static uint64_t grlib_apb_pnp_read(void *opaque, hwaddr offset, unsigned size)
return apb_pnp->regs[offset >> 2];
}
+static void grlib_apb_pnp_write(void *opaque, hwaddr addr,
+ uint64_t val, unsigned size)
+{
+ qemu_log_mask(LOG_UNIMP, "%s not implemented\n", __func__);
+}
+
static const MemoryRegionOps grlib_apb_pnp_ops = {
.read = grlib_apb_pnp_read,
+ .write = grlib_apb_pnp_write,
.endianness = DEVICE_BIG_ENDIAN,
};
--
2.21.0
^ permalink raw reply related [flat|nested] 33+ messages in thread* [Xen-devel] [PULL 1/4] hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
@ 2019-11-05 14:42 ` Laurent Vivier
0 siblings, 0 replies; 33+ messages in thread
From: Laurent Vivier @ 2019-11-05 14:42 UTC (permalink / raw)
To: qemu-devel
Cc: Peter Maydell, Stefano Stabellini, Daniel P. Berrangé,
Eduardo Habkost, Paolo Bonzini, Paul Durrant, qemu-trivial,
Claudio Fontana, Cleber Rosa, Michael Tokarev, Laurent Vivier,
Fabien Chouteau, KONRAD Frederic, qemu-arm, qemu-ppc,
Cédric Le Goater, Marc-André Lureau, Anthony Perard,
xen-devel, Philippe Mathieu-Daudé, David Gibson
From: Philippe Mathieu-Daudé <philmd@redhat.com>
Guests can crash QEMU when writting to PnP registers:
$ echo 'writeb 0x800ff042 69' | qemu-system-sparc -M leon3_generic -S -bios /etc/magic -qtest stdio
[I 1571938309.932255] OPENED
[R +0.063474] writeb 0x800ff042 69
Segmentation fault (core dumped)
(gdb) bt
#0 0x0000000000000000 in ()
#1 0x0000555f4bcdf0bc in memory_region_write_with_attrs_accessor (mr=0x555f4d7be8c0, addr=66, value=0x7fff07d00f08, size=1, shift=0, mask=255, attrs=...) at memory.c:503
#2 0x0000555f4bcdf185 in access_with_adjusted_size (addr=66, value=0x7fff07d00f08, size=1, access_size_min=1, access_size_max=4, access_fn=0x555f4bcdeff4 <memory_region_write_with_attrs_accessor>, mr=0x555f4d7be8c0, attrs=...) at memory.c:539
#3 0x0000555f4bce2243 in memory_region_dispatch_write (mr=0x555f4d7be8c0, addr=66, data=69, op=MO_8, attrs=...) at memory.c:1489
#4 0x0000555f4bc80b20 in flatview_write_continue (fv=0x555f4d92c400, addr=2148528194, attrs=..., buf=0x7fff07d01120 "E", len=1, addr1=66, l=1, mr=0x555f4d7be8c0) at exec.c:3161
#5 0x0000555f4bc80c65 in flatview_write (fv=0x555f4d92c400, addr=2148528194, attrs=..., buf=0x7fff07d01120 "E", len=1) at exec.c:3201
#6 0x0000555f4bc80fb0 in address_space_write (as=0x555f4d7aa460, addr=2148528194, attrs=..., buf=0x7fff07d01120 "E", len=1) at exec.c:3291
#7 0x0000555f4bc8101d in address_space_rw (as=0x555f4d7aa460, addr=2148528194, attrs=..., buf=0x7fff07d01120 "E", len=1, is_write=true) at exec.c:3301
#8 0x0000555f4bcdb388 in qtest_process_command (chr=0x555f4c2ed7e0 <qtest_chr>, words=0x555f4db0c5d0) at qtest.c:432
Instead of crashing, log the access as unimplemented.
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: KONRAD Frederic <frederic.konrad@adacore.com>
Message-Id: <20191025110114.27091-2-philmd@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
hw/misc/grlib_ahb_apb_pnp.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/hw/misc/grlib_ahb_apb_pnp.c b/hw/misc/grlib_ahb_apb_pnp.c
index 7338461694c9..f3c015d2c35f 100644
--- a/hw/misc/grlib_ahb_apb_pnp.c
+++ b/hw/misc/grlib_ahb_apb_pnp.c
@@ -22,6 +22,7 @@
*/
#include "qemu/osdep.h"
+#include "qemu/log.h"
#include "hw/sysbus.h"
#include "hw/misc/grlib_ahb_apb_pnp.h"
@@ -231,8 +232,15 @@ static uint64_t grlib_apb_pnp_read(void *opaque, hwaddr offset, unsigned size)
return apb_pnp->regs[offset >> 2];
}
+static void grlib_apb_pnp_write(void *opaque, hwaddr addr,
+ uint64_t val, unsigned size)
+{
+ qemu_log_mask(LOG_UNIMP, "%s not implemented\n", __func__);
+}
+
static const MemoryRegionOps grlib_apb_pnp_ops = {
.read = grlib_apb_pnp_read,
+ .write = grlib_apb_pnp_write,
.endianness = DEVICE_BIG_ENDIAN,
};
--
2.21.0
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply related [flat|nested] 33+ messages in thread* [PULL 1/4] hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
@ 2019-11-05 14:42 ` Laurent Vivier
0 siblings, 0 replies; 33+ messages in thread
From: Laurent Vivier @ 2019-11-05 14:42 UTC (permalink / raw)
To: qemu-devel
Cc: Peter Maydell, Stefano Stabellini, Daniel P. Berrangé,
Eduardo Habkost, Paolo Bonzini, Paul Durrant, qemu-trivial,
Claudio Fontana, Cleber Rosa, Michael Tokarev, Laurent Vivier,
Fabien Chouteau, KONRAD Frederic, qemu-arm, qemu-ppc,
Cédric Le Goater, Marc-André Lureau, Anthony Perard,
xen-devel, Philippe Mathieu-Daudé, David Gibson
From: Philippe Mathieu-Daudé <philmd@redhat.com>
Guests can crash QEMU when writting to PnP registers:
$ echo 'writeb 0x800ff042 69' | qemu-system-sparc -M leon3_generic -S -bios /etc/magic -qtest stdio
[I 1571938309.932255] OPENED
[R +0.063474] writeb 0x800ff042 69
Segmentation fault (core dumped)
(gdb) bt
#0 0x0000000000000000 in ()
#1 0x0000555f4bcdf0bc in memory_region_write_with_attrs_accessor (mr=0x555f4d7be8c0, addr=66, value=0x7fff07d00f08, size=1, shift=0, mask=255, attrs=...) at memory.c:503
#2 0x0000555f4bcdf185 in access_with_adjusted_size (addr=66, value=0x7fff07d00f08, size=1, access_size_min=1, access_size_max=4, access_fn=0x555f4bcdeff4 <memory_region_write_with_attrs_accessor>, mr=0x555f4d7be8c0, attrs=...) at memory.c:539
#3 0x0000555f4bce2243 in memory_region_dispatch_write (mr=0x555f4d7be8c0, addr=66, data=69, op=MO_8, attrs=...) at memory.c:1489
#4 0x0000555f4bc80b20 in flatview_write_continue (fv=0x555f4d92c400, addr=2148528194, attrs=..., buf=0x7fff07d01120 "E", len=1, addr1=66, l=1, mr=0x555f4d7be8c0) at exec.c:3161
#5 0x0000555f4bc80c65 in flatview_write (fv=0x555f4d92c400, addr=2148528194, attrs=..., buf=0x7fff07d01120 "E", len=1) at exec.c:3201
#6 0x0000555f4bc80fb0 in address_space_write (as=0x555f4d7aa460, addr=2148528194, attrs=..., buf=0x7fff07d01120 "E", len=1) at exec.c:3291
#7 0x0000555f4bc8101d in address_space_rw (as=0x555f4d7aa460, addr=2148528194, attrs=..., buf=0x7fff07d01120 "E", len=1, is_write=true) at exec.c:3301
#8 0x0000555f4bcdb388 in qtest_process_command (chr=0x555f4c2ed7e0 <qtest_chr>, words=0x555f4db0c5d0) at qtest.c:432
Instead of crashing, log the access as unimplemented.
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: KONRAD Frederic <frederic.konrad@adacore.com>
Message-Id: <20191025110114.27091-2-philmd@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
hw/misc/grlib_ahb_apb_pnp.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/hw/misc/grlib_ahb_apb_pnp.c b/hw/misc/grlib_ahb_apb_pnp.c
index 7338461694c9..f3c015d2c35f 100644
--- a/hw/misc/grlib_ahb_apb_pnp.c
+++ b/hw/misc/grlib_ahb_apb_pnp.c
@@ -22,6 +22,7 @@
*/
#include "qemu/osdep.h"
+#include "qemu/log.h"
#include "hw/sysbus.h"
#include "hw/misc/grlib_ahb_apb_pnp.h"
@@ -231,8 +232,15 @@ static uint64_t grlib_apb_pnp_read(void *opaque, hwaddr offset, unsigned size)
return apb_pnp->regs[offset >> 2];
}
+static void grlib_apb_pnp_write(void *opaque, hwaddr addr,
+ uint64_t val, unsigned size)
+{
+ qemu_log_mask(LOG_UNIMP, "%s not implemented\n", __func__);
+}
+
static const MemoryRegionOps grlib_apb_pnp_ops = {
.read = grlib_apb_pnp_read,
+ .write = grlib_apb_pnp_write,
.endianness = DEVICE_BIG_ENDIAN,
};
--
2.21.0
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [PULL 2/4] hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
2019-11-05 14:42 ` Laurent Vivier
(?)
@ 2019-11-05 14:42 ` Laurent Vivier
-1 siblings, 0 replies; 33+ messages in thread
From: Laurent Vivier @ 2019-11-05 14:42 UTC (permalink / raw)
To: qemu-devel
Cc: qemu-trivial, Stefano Stabellini, xen-devel,
Cédric Le Goater, Laurent Vivier, Daniel P. Berrangé,
Eduardo Habkost, David Gibson, qemu-arm, Marc-André Lureau,
Fabien Chouteau, qemu-ppc, Paul Durrant, Michael Tokarev,
KONRAD Frederic, Cleber Rosa, Claudio Fontana, Anthony Perard,
Paolo Bonzini, Peter Maydell, Philippe Mathieu-Daudé,
Jiri Gaisler
From: Philippe Mathieu-Daudé <philmd@redhat.com>
The Plug & Play region of the AHB/APB bridge can be accessed
by various word size, however the implementation is clearly
restricted to 32-bit:
static uint64_t grlib_apb_pnp_read(void *opaque, hwaddr offset, unsigned size)
{
APBPnp *apb_pnp = GRLIB_APB_PNP(opaque);
return apb_pnp->regs[offset >> 2];
}
Set the MemoryRegionOps::impl min/max fields to 32-bit, so
memory.c::access_with_adjusted_size() can adjust when the
access is not 32-bit.
This is required to run RTEMS on leon3, the grlib scanning
functions do byte accesses.
Reported-by: Jiri Gaisler <jiri@gaisler.se>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: KONRAD Frederic <frederic.konrad@adacore.com>
Message-Id: <20191025110114.27091-3-philmd@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
hw/misc/grlib_ahb_apb_pnp.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/hw/misc/grlib_ahb_apb_pnp.c b/hw/misc/grlib_ahb_apb_pnp.c
index f3c015d2c35f..e230e2536361 100644
--- a/hw/misc/grlib_ahb_apb_pnp.c
+++ b/hw/misc/grlib_ahb_apb_pnp.c
@@ -242,6 +242,10 @@ static const MemoryRegionOps grlib_apb_pnp_ops = {
.read = grlib_apb_pnp_read,
.write = grlib_apb_pnp_write,
.endianness = DEVICE_BIG_ENDIAN,
+ .impl = {
+ .min_access_size = 4,
+ .max_access_size = 4,
+ },
};
static void grlib_apb_pnp_realize(DeviceState *dev, Error **errp)
--
2.21.0
^ permalink raw reply related [flat|nested] 33+ messages in thread* [Xen-devel] [PULL 2/4] hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
@ 2019-11-05 14:42 ` Laurent Vivier
0 siblings, 0 replies; 33+ messages in thread
From: Laurent Vivier @ 2019-11-05 14:42 UTC (permalink / raw)
To: qemu-devel
Cc: Peter Maydell, Paul Durrant, Michael Tokarev, KONRAD Frederic,
Stefano Stabellini, xen-devel, qemu-trivial, Anthony Perard,
Marc-André Lureau, Philippe Mathieu-Daudé, Jiri Gaisler,
Eduardo Habkost, Fabien Chouteau, qemu-arm, Cédric Le Goater,
Cleber Rosa, David Gibson, Daniel P. Berrangé,
Claudio Fontana, Laurent Vivier, qemu-ppc, Paolo Bonzini
From: Philippe Mathieu-Daudé <philmd@redhat.com>
The Plug & Play region of the AHB/APB bridge can be accessed
by various word size, however the implementation is clearly
restricted to 32-bit:
static uint64_t grlib_apb_pnp_read(void *opaque, hwaddr offset, unsigned size)
{
APBPnp *apb_pnp = GRLIB_APB_PNP(opaque);
return apb_pnp->regs[offset >> 2];
}
Set the MemoryRegionOps::impl min/max fields to 32-bit, so
memory.c::access_with_adjusted_size() can adjust when the
access is not 32-bit.
This is required to run RTEMS on leon3, the grlib scanning
functions do byte accesses.
Reported-by: Jiri Gaisler <jiri@gaisler.se>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: KONRAD Frederic <frederic.konrad@adacore.com>
Message-Id: <20191025110114.27091-3-philmd@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
hw/misc/grlib_ahb_apb_pnp.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/hw/misc/grlib_ahb_apb_pnp.c b/hw/misc/grlib_ahb_apb_pnp.c
index f3c015d2c35f..e230e2536361 100644
--- a/hw/misc/grlib_ahb_apb_pnp.c
+++ b/hw/misc/grlib_ahb_apb_pnp.c
@@ -242,6 +242,10 @@ static const MemoryRegionOps grlib_apb_pnp_ops = {
.read = grlib_apb_pnp_read,
.write = grlib_apb_pnp_write,
.endianness = DEVICE_BIG_ENDIAN,
+ .impl = {
+ .min_access_size = 4,
+ .max_access_size = 4,
+ },
};
static void grlib_apb_pnp_realize(DeviceState *dev, Error **errp)
--
2.21.0
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply related [flat|nested] 33+ messages in thread* [PULL 2/4] hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
@ 2019-11-05 14:42 ` Laurent Vivier
0 siblings, 0 replies; 33+ messages in thread
From: Laurent Vivier @ 2019-11-05 14:42 UTC (permalink / raw)
To: qemu-devel
Cc: Peter Maydell, Paul Durrant, Michael Tokarev, KONRAD Frederic,
Stefano Stabellini, xen-devel, qemu-trivial, Anthony Perard,
Marc-André Lureau, Philippe Mathieu-Daudé, Jiri Gaisler,
Eduardo Habkost, Fabien Chouteau, qemu-arm, Cédric Le Goater,
Cleber Rosa, David Gibson, Daniel P. Berrangé,
Claudio Fontana, Laurent Vivier, qemu-ppc, Paolo Bonzini
From: Philippe Mathieu-Daudé <philmd@redhat.com>
The Plug & Play region of the AHB/APB bridge can be accessed
by various word size, however the implementation is clearly
restricted to 32-bit:
static uint64_t grlib_apb_pnp_read(void *opaque, hwaddr offset, unsigned size)
{
APBPnp *apb_pnp = GRLIB_APB_PNP(opaque);
return apb_pnp->regs[offset >> 2];
}
Set the MemoryRegionOps::impl min/max fields to 32-bit, so
memory.c::access_with_adjusted_size() can adjust when the
access is not 32-bit.
This is required to run RTEMS on leon3, the grlib scanning
functions do byte accesses.
Reported-by: Jiri Gaisler <jiri@gaisler.se>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: KONRAD Frederic <frederic.konrad@adacore.com>
Message-Id: <20191025110114.27091-3-philmd@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
hw/misc/grlib_ahb_apb_pnp.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/hw/misc/grlib_ahb_apb_pnp.c b/hw/misc/grlib_ahb_apb_pnp.c
index f3c015d2c35f..e230e2536361 100644
--- a/hw/misc/grlib_ahb_apb_pnp.c
+++ b/hw/misc/grlib_ahb_apb_pnp.c
@@ -242,6 +242,10 @@ static const MemoryRegionOps grlib_apb_pnp_ops = {
.read = grlib_apb_pnp_read,
.write = grlib_apb_pnp_write,
.endianness = DEVICE_BIG_ENDIAN,
+ .impl = {
+ .min_access_size = 4,
+ .max_access_size = 4,
+ },
};
static void grlib_apb_pnp_realize(DeviceState *dev, Error **errp)
--
2.21.0
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [PULL 3/4] qom: Fix error message in object_class_property_add()
2019-11-05 14:42 ` Laurent Vivier
(?)
@ 2019-11-05 14:42 ` Laurent Vivier
-1 siblings, 0 replies; 33+ messages in thread
From: Laurent Vivier @ 2019-11-05 14:42 UTC (permalink / raw)
To: qemu-devel
Cc: qemu-trivial, Stefano Stabellini, xen-devel,
Cédric Le Goater, Laurent Vivier, Daniel P. Berrangé,
Eduardo Habkost, David Gibson, qemu-arm, Marc-André Lureau,
Fabien Chouteau, qemu-ppc, Paul Durrant, Michael Tokarev,
KONRAD Frederic, Cleber Rosa, Claudio Fontana, Anthony Perard,
Paolo Bonzini, Peter Maydell, Greg Kurz
From: Greg Kurz <groug@kaod.org>
The error message in object_class_property_add() was copied from
object_property_add() in commit 16bf7f522a2ff. Clarify that it is
about a class, not an object.
While here, have the format string in both functions to fit in a
single line for better grep-ability, despite the checkpatch warning.
Signed-off-by: Greg Kurz <groug@kaod.org>
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
Message-Id: <157287383591.234942.311840593519058490.stgit@bahia.tlslab.ibm.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
qom/object.c | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/qom/object.c b/qom/object.c
index 6fa9c619fac4..d51b57fba11e 100644
--- a/qom/object.c
+++ b/qom/object.c
@@ -1106,9 +1106,8 @@ object_property_add(Object *obj, const char *name, const char *type,
}
if (object_property_find(obj, name, NULL) != NULL) {
- error_setg(errp, "attempt to add duplicate property '%s'"
- " to object (type '%s')", name,
- object_get_typename(obj));
+ error_setg(errp, "attempt to add duplicate property '%s' to object (type '%s')",
+ name, object_get_typename(obj));
return NULL;
}
@@ -1139,9 +1138,8 @@ object_class_property_add(ObjectClass *klass,
ObjectProperty *prop;
if (object_class_property_find(klass, name, NULL) != NULL) {
- error_setg(errp, "attempt to add duplicate property '%s'"
- " to object (type '%s')", name,
- object_class_get_name(klass));
+ error_setg(errp, "attempt to add duplicate property '%s' to class (type '%s')",
+ name, object_class_get_name(klass));
return NULL;
}
--
2.21.0
^ permalink raw reply related [flat|nested] 33+ messages in thread* [Xen-devel] [PULL 3/4] qom: Fix error message in object_class_property_add()
@ 2019-11-05 14:42 ` Laurent Vivier
0 siblings, 0 replies; 33+ messages in thread
From: Laurent Vivier @ 2019-11-05 14:42 UTC (permalink / raw)
To: qemu-devel
Cc: Peter Maydell, Stefano Stabellini, Daniel P. Berrangé,
Eduardo Habkost, Paolo Bonzini, Paul Durrant, qemu-trivial,
Claudio Fontana, Cleber Rosa, Michael Tokarev, Laurent Vivier,
Fabien Chouteau, Greg Kurz, KONRAD Frederic, qemu-arm, qemu-ppc,
Cédric Le Goater, Marc-André Lureau, Anthony Perard,
xen-devel, David Gibson
From: Greg Kurz <groug@kaod.org>
The error message in object_class_property_add() was copied from
object_property_add() in commit 16bf7f522a2ff. Clarify that it is
about a class, not an object.
While here, have the format string in both functions to fit in a
single line for better grep-ability, despite the checkpatch warning.
Signed-off-by: Greg Kurz <groug@kaod.org>
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
Message-Id: <157287383591.234942.311840593519058490.stgit@bahia.tlslab.ibm.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
qom/object.c | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/qom/object.c b/qom/object.c
index 6fa9c619fac4..d51b57fba11e 100644
--- a/qom/object.c
+++ b/qom/object.c
@@ -1106,9 +1106,8 @@ object_property_add(Object *obj, const char *name, const char *type,
}
if (object_property_find(obj, name, NULL) != NULL) {
- error_setg(errp, "attempt to add duplicate property '%s'"
- " to object (type '%s')", name,
- object_get_typename(obj));
+ error_setg(errp, "attempt to add duplicate property '%s' to object (type '%s')",
+ name, object_get_typename(obj));
return NULL;
}
@@ -1139,9 +1138,8 @@ object_class_property_add(ObjectClass *klass,
ObjectProperty *prop;
if (object_class_property_find(klass, name, NULL) != NULL) {
- error_setg(errp, "attempt to add duplicate property '%s'"
- " to object (type '%s')", name,
- object_class_get_name(klass));
+ error_setg(errp, "attempt to add duplicate property '%s' to class (type '%s')",
+ name, object_class_get_name(klass));
return NULL;
}
--
2.21.0
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply related [flat|nested] 33+ messages in thread* [PULL 3/4] qom: Fix error message in object_class_property_add()
@ 2019-11-05 14:42 ` Laurent Vivier
0 siblings, 0 replies; 33+ messages in thread
From: Laurent Vivier @ 2019-11-05 14:42 UTC (permalink / raw)
To: qemu-devel
Cc: Peter Maydell, Stefano Stabellini, Daniel P. Berrangé,
Eduardo Habkost, Paolo Bonzini, Paul Durrant, qemu-trivial,
Claudio Fontana, Cleber Rosa, Michael Tokarev, Laurent Vivier,
Fabien Chouteau, Greg Kurz, KONRAD Frederic, qemu-arm, qemu-ppc,
Cédric Le Goater, Marc-André Lureau, Anthony Perard,
xen-devel, David Gibson
From: Greg Kurz <groug@kaod.org>
The error message in object_class_property_add() was copied from
object_property_add() in commit 16bf7f522a2ff. Clarify that it is
about a class, not an object.
While here, have the format string in both functions to fit in a
single line for better grep-ability, despite the checkpatch warning.
Signed-off-by: Greg Kurz <groug@kaod.org>
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
Message-Id: <157287383591.234942.311840593519058490.stgit@bahia.tlslab.ibm.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
qom/object.c | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/qom/object.c b/qom/object.c
index 6fa9c619fac4..d51b57fba11e 100644
--- a/qom/object.c
+++ b/qom/object.c
@@ -1106,9 +1106,8 @@ object_property_add(Object *obj, const char *name, const char *type,
}
if (object_property_find(obj, name, NULL) != NULL) {
- error_setg(errp, "attempt to add duplicate property '%s'"
- " to object (type '%s')", name,
- object_get_typename(obj));
+ error_setg(errp, "attempt to add duplicate property '%s' to object (type '%s')",
+ name, object_get_typename(obj));
return NULL;
}
@@ -1139,9 +1138,8 @@ object_class_property_add(ObjectClass *klass,
ObjectProperty *prop;
if (object_class_property_find(klass, name, NULL) != NULL) {
- error_setg(errp, "attempt to add duplicate property '%s'"
- " to object (type '%s')", name,
- object_class_get_name(klass));
+ error_setg(errp, "attempt to add duplicate property '%s' to class (type '%s')",
+ name, object_class_get_name(klass));
return NULL;
}
--
2.21.0
^ permalink raw reply related [flat|nested] 33+ messages in thread
* [PULL 4/4] global: Squash 'the the'
2019-11-05 14:42 ` Laurent Vivier
(?)
@ 2019-11-05 14:42 ` Laurent Vivier
-1 siblings, 0 replies; 33+ messages in thread
From: Laurent Vivier @ 2019-11-05 14:42 UTC (permalink / raw)
To: qemu-devel
Cc: qemu-trivial, Stefano Stabellini, xen-devel,
Cédric Le Goater, Laurent Vivier, Daniel P. Berrangé,
Eduardo Habkost, David Gibson, qemu-arm, Marc-André Lureau,
Fabien Chouteau, qemu-ppc, Paul Durrant, Michael Tokarev,
KONRAD Frederic, Cleber Rosa, Claudio Fontana, Anthony Perard,
Paolo Bonzini, Peter Maydell, Dr. David Alan Gilbert,
Alex Bennée, Philippe Mathieu-Daudé
From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
'the' has a tendency to double up; squash them back down.
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20191104185202.102504-1-dgilbert@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
disas/libvixl/vixl/invalset.h | 2 +-
docs/interop/pr-helper.rst | 2 +-
docs/specs/ppc-spapr-hotplug.txt | 2 +-
docs/specs/ppc-xive.rst | 2 +-
docs/specs/tpm.txt | 2 +-
include/hw/xen/interface/io/blkif.h | 2 +-
scripts/dump-guest-memory.py | 2 +-
7 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/disas/libvixl/vixl/invalset.h b/disas/libvixl/vixl/invalset.h
index ffdc0237b47c..ef5e49d6feb2 100644
--- a/disas/libvixl/vixl/invalset.h
+++ b/disas/libvixl/vixl/invalset.h
@@ -102,7 +102,7 @@ template<TEMPLATE_INVALSET_P_DECL> class InvalSet {
size_t size() const;
// Returns true if no elements are stored in the set.
- // Note that this does not mean the the backing storage is empty: it can still
+ // Note that this does not mean the backing storage is empty: it can still
// contain invalid elements.
bool empty() const;
diff --git a/docs/interop/pr-helper.rst b/docs/interop/pr-helper.rst
index 9f76d5bcf98f..e926f0a6c9cb 100644
--- a/docs/interop/pr-helper.rst
+++ b/docs/interop/pr-helper.rst
@@ -10,7 +10,7 @@ can delegate implementation of persistent reservations to an external
restricting access to block devices to specific initiators in a shared
storage setup.
-For a more detailed reference please refer the the SCSI Primary
+For a more detailed reference please refer to the SCSI Primary
Commands standard, specifically the section on Reservations and the
"PERSISTENT RESERVE IN" and "PERSISTENT RESERVE OUT" commands.
diff --git a/docs/specs/ppc-spapr-hotplug.txt b/docs/specs/ppc-spapr-hotplug.txt
index cc7833108e12..859d52cce6c8 100644
--- a/docs/specs/ppc-spapr-hotplug.txt
+++ b/docs/specs/ppc-spapr-hotplug.txt
@@ -385,7 +385,7 @@ Each LMB list entry consists of the following elements:
is used to retrieve the right associativity list to be used for this
LMB.
- A 32bit flags word. The bit at bit position 0x00000008 defines whether
- the LMB is assigned to the the partition as of boot time.
+ the LMB is assigned to the partition as of boot time.
ibm,dynamic-memory-v2
diff --git a/docs/specs/ppc-xive.rst b/docs/specs/ppc-xive.rst
index 148d57eb6ab2..83d43f658b90 100644
--- a/docs/specs/ppc-xive.rst
+++ b/docs/specs/ppc-xive.rst
@@ -163,7 +163,7 @@ Interrupt Priority Register (PIPR) is also updated using the IPB. This
register represent the priority of the most favored pending
notification.
-The PIPR is then compared to the the Current Processor Priority
+The PIPR is then compared to the Current Processor Priority
Register (CPPR). If it is more favored (numerically less than), the
CPU interrupt line is raised and the EO bit of the Notification Source
Register (NSR) is updated to notify the presence of an exception for
diff --git a/docs/specs/tpm.txt b/docs/specs/tpm.txt
index 5d8c26b1adba..9c8cca042da8 100644
--- a/docs/specs/tpm.txt
+++ b/docs/specs/tpm.txt
@@ -89,7 +89,7 @@ TPM upon reboot. The PPI specification defines the operation requests and the
actions the firmware has to take. The system administrator passes the operation
request number to the firmware through an ACPI interface which writes this
number to a memory location that the firmware knows. Upon reboot, the firmware
-finds the number and sends commands to the the TPM. The firmware writes the TPM
+finds the number and sends commands to the TPM. The firmware writes the TPM
result code and the operation request number to a memory location that ACPI can
read from and pass the result on to the administrator.
diff --git a/include/hw/xen/interface/io/blkif.h b/include/hw/xen/interface/io/blkif.h
index 8b1be50ce81e..d07fa1e07822 100644
--- a/include/hw/xen/interface/io/blkif.h
+++ b/include/hw/xen/interface/io/blkif.h
@@ -341,7 +341,7 @@
* access (even when it should be read-only). If the frontend hits the
* maximum number of allowed persistently mapped grants, it can fallback
* to non persistent mode. This will cause a performance degradation,
- * since the the backend driver will still try to map those grants
+ * since the backend driver will still try to map those grants
* persistently. Since the persistent grants protocol is compatible with
* the previous protocol, a frontend driver can choose to work in
* persistent mode even when the backend doesn't support it.
diff --git a/scripts/dump-guest-memory.py b/scripts/dump-guest-memory.py
index 2c587cbefc57..9371e4581308 100644
--- a/scripts/dump-guest-memory.py
+++ b/scripts/dump-guest-memory.py
@@ -170,7 +170,7 @@ class ELF(object):
self.ehdr.e_phnum += 1
def to_file(self, elf_file):
- """Writes all ELF structures to the the passed file.
+ """Writes all ELF structures to the passed file.
Structure:
Ehdr
--
2.21.0
^ permalink raw reply related [flat|nested] 33+ messages in thread* [PULL 4/4] global: Squash 'the the'
@ 2019-11-05 14:42 ` Laurent Vivier
0 siblings, 0 replies; 33+ messages in thread
From: Laurent Vivier @ 2019-11-05 14:42 UTC (permalink / raw)
To: qemu-devel
Cc: Peter Maydell, Paul Durrant, Michael Tokarev, KONRAD Frederic,
Stefano Stabellini, xen-devel, qemu-trivial, Anthony Perard,
Marc-André Lureau, Philippe Mathieu-Daudé,
Eduardo Habkost, Dr. David Alan Gilbert, Fabien Chouteau,
qemu-arm, Cédric Le Goater, Cleber Rosa, Alex Bennée,
David Gibson, Daniel P. Berrangé, Claudio Fontana,
Laurent Vivier, qemu-ppc, Paolo Bonzini
From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
'the' has a tendency to double up; squash them back down.
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20191104185202.102504-1-dgilbert@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
disas/libvixl/vixl/invalset.h | 2 +-
docs/interop/pr-helper.rst | 2 +-
docs/specs/ppc-spapr-hotplug.txt | 2 +-
docs/specs/ppc-xive.rst | 2 +-
docs/specs/tpm.txt | 2 +-
include/hw/xen/interface/io/blkif.h | 2 +-
scripts/dump-guest-memory.py | 2 +-
7 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/disas/libvixl/vixl/invalset.h b/disas/libvixl/vixl/invalset.h
index ffdc0237b47c..ef5e49d6feb2 100644
--- a/disas/libvixl/vixl/invalset.h
+++ b/disas/libvixl/vixl/invalset.h
@@ -102,7 +102,7 @@ template<TEMPLATE_INVALSET_P_DECL> class InvalSet {
size_t size() const;
// Returns true if no elements are stored in the set.
- // Note that this does not mean the the backing storage is empty: it can still
+ // Note that this does not mean the backing storage is empty: it can still
// contain invalid elements.
bool empty() const;
diff --git a/docs/interop/pr-helper.rst b/docs/interop/pr-helper.rst
index 9f76d5bcf98f..e926f0a6c9cb 100644
--- a/docs/interop/pr-helper.rst
+++ b/docs/interop/pr-helper.rst
@@ -10,7 +10,7 @@ can delegate implementation of persistent reservations to an external
restricting access to block devices to specific initiators in a shared
storage setup.
-For a more detailed reference please refer the the SCSI Primary
+For a more detailed reference please refer to the SCSI Primary
Commands standard, specifically the section on Reservations and the
"PERSISTENT RESERVE IN" and "PERSISTENT RESERVE OUT" commands.
diff --git a/docs/specs/ppc-spapr-hotplug.txt b/docs/specs/ppc-spapr-hotplug.txt
index cc7833108e12..859d52cce6c8 100644
--- a/docs/specs/ppc-spapr-hotplug.txt
+++ b/docs/specs/ppc-spapr-hotplug.txt
@@ -385,7 +385,7 @@ Each LMB list entry consists of the following elements:
is used to retrieve the right associativity list to be used for this
LMB.
- A 32bit flags word. The bit at bit position 0x00000008 defines whether
- the LMB is assigned to the the partition as of boot time.
+ the LMB is assigned to the partition as of boot time.
ibm,dynamic-memory-v2
diff --git a/docs/specs/ppc-xive.rst b/docs/specs/ppc-xive.rst
index 148d57eb6ab2..83d43f658b90 100644
--- a/docs/specs/ppc-xive.rst
+++ b/docs/specs/ppc-xive.rst
@@ -163,7 +163,7 @@ Interrupt Priority Register (PIPR) is also updated using the IPB. This
register represent the priority of the most favored pending
notification.
-The PIPR is then compared to the the Current Processor Priority
+The PIPR is then compared to the Current Processor Priority
Register (CPPR). If it is more favored (numerically less than), the
CPU interrupt line is raised and the EO bit of the Notification Source
Register (NSR) is updated to notify the presence of an exception for
diff --git a/docs/specs/tpm.txt b/docs/specs/tpm.txt
index 5d8c26b1adba..9c8cca042da8 100644
--- a/docs/specs/tpm.txt
+++ b/docs/specs/tpm.txt
@@ -89,7 +89,7 @@ TPM upon reboot. The PPI specification defines the operation requests and the
actions the firmware has to take. The system administrator passes the operation
request number to the firmware through an ACPI interface which writes this
number to a memory location that the firmware knows. Upon reboot, the firmware
-finds the number and sends commands to the the TPM. The firmware writes the TPM
+finds the number and sends commands to the TPM. The firmware writes the TPM
result code and the operation request number to a memory location that ACPI can
read from and pass the result on to the administrator.
diff --git a/include/hw/xen/interface/io/blkif.h b/include/hw/xen/interface/io/blkif.h
index 8b1be50ce81e..d07fa1e07822 100644
--- a/include/hw/xen/interface/io/blkif.h
+++ b/include/hw/xen/interface/io/blkif.h
@@ -341,7 +341,7 @@
* access (even when it should be read-only). If the frontend hits the
* maximum number of allowed persistently mapped grants, it can fallback
* to non persistent mode. This will cause a performance degradation,
- * since the the backend driver will still try to map those grants
+ * since the backend driver will still try to map those grants
* persistently. Since the persistent grants protocol is compatible with
* the previous protocol, a frontend driver can choose to work in
* persistent mode even when the backend doesn't support it.
diff --git a/scripts/dump-guest-memory.py b/scripts/dump-guest-memory.py
index 2c587cbefc57..9371e4581308 100644
--- a/scripts/dump-guest-memory.py
+++ b/scripts/dump-guest-memory.py
@@ -170,7 +170,7 @@ class ELF(object):
self.ehdr.e_phnum += 1
def to_file(self, elf_file):
- """Writes all ELF structures to the the passed file.
+ """Writes all ELF structures to the passed file.
Structure:
Ehdr
--
2.21.0
^ permalink raw reply related [flat|nested] 33+ messages in thread* [Xen-devel] [PULL 4/4] global: Squash 'the the'
@ 2019-11-05 14:42 ` Laurent Vivier
0 siblings, 0 replies; 33+ messages in thread
From: Laurent Vivier @ 2019-11-05 14:42 UTC (permalink / raw)
To: qemu-devel
Cc: Peter Maydell, Paul Durrant, Michael Tokarev, KONRAD Frederic,
Stefano Stabellini, xen-devel, qemu-trivial, Anthony Perard,
Marc-André Lureau, Philippe Mathieu-Daudé,
Eduardo Habkost, Dr. David Alan Gilbert, Fabien Chouteau,
qemu-arm, Cédric Le Goater, Cleber Rosa, Alex Bennée,
David Gibson, Daniel P. Berrangé, Claudio Fontana,
Laurent Vivier, qemu-ppc, Paolo Bonzini
From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
'the' has a tendency to double up; squash them back down.
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20191104185202.102504-1-dgilbert@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
disas/libvixl/vixl/invalset.h | 2 +-
docs/interop/pr-helper.rst | 2 +-
docs/specs/ppc-spapr-hotplug.txt | 2 +-
docs/specs/ppc-xive.rst | 2 +-
docs/specs/tpm.txt | 2 +-
include/hw/xen/interface/io/blkif.h | 2 +-
scripts/dump-guest-memory.py | 2 +-
7 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/disas/libvixl/vixl/invalset.h b/disas/libvixl/vixl/invalset.h
index ffdc0237b47c..ef5e49d6feb2 100644
--- a/disas/libvixl/vixl/invalset.h
+++ b/disas/libvixl/vixl/invalset.h
@@ -102,7 +102,7 @@ template<TEMPLATE_INVALSET_P_DECL> class InvalSet {
size_t size() const;
// Returns true if no elements are stored in the set.
- // Note that this does not mean the the backing storage is empty: it can still
+ // Note that this does not mean the backing storage is empty: it can still
// contain invalid elements.
bool empty() const;
diff --git a/docs/interop/pr-helper.rst b/docs/interop/pr-helper.rst
index 9f76d5bcf98f..e926f0a6c9cb 100644
--- a/docs/interop/pr-helper.rst
+++ b/docs/interop/pr-helper.rst
@@ -10,7 +10,7 @@ can delegate implementation of persistent reservations to an external
restricting access to block devices to specific initiators in a shared
storage setup.
-For a more detailed reference please refer the the SCSI Primary
+For a more detailed reference please refer to the SCSI Primary
Commands standard, specifically the section on Reservations and the
"PERSISTENT RESERVE IN" and "PERSISTENT RESERVE OUT" commands.
diff --git a/docs/specs/ppc-spapr-hotplug.txt b/docs/specs/ppc-spapr-hotplug.txt
index cc7833108e12..859d52cce6c8 100644
--- a/docs/specs/ppc-spapr-hotplug.txt
+++ b/docs/specs/ppc-spapr-hotplug.txt
@@ -385,7 +385,7 @@ Each LMB list entry consists of the following elements:
is used to retrieve the right associativity list to be used for this
LMB.
- A 32bit flags word. The bit at bit position 0x00000008 defines whether
- the LMB is assigned to the the partition as of boot time.
+ the LMB is assigned to the partition as of boot time.
ibm,dynamic-memory-v2
diff --git a/docs/specs/ppc-xive.rst b/docs/specs/ppc-xive.rst
index 148d57eb6ab2..83d43f658b90 100644
--- a/docs/specs/ppc-xive.rst
+++ b/docs/specs/ppc-xive.rst
@@ -163,7 +163,7 @@ Interrupt Priority Register (PIPR) is also updated using the IPB. This
register represent the priority of the most favored pending
notification.
-The PIPR is then compared to the the Current Processor Priority
+The PIPR is then compared to the Current Processor Priority
Register (CPPR). If it is more favored (numerically less than), the
CPU interrupt line is raised and the EO bit of the Notification Source
Register (NSR) is updated to notify the presence of an exception for
diff --git a/docs/specs/tpm.txt b/docs/specs/tpm.txt
index 5d8c26b1adba..9c8cca042da8 100644
--- a/docs/specs/tpm.txt
+++ b/docs/specs/tpm.txt
@@ -89,7 +89,7 @@ TPM upon reboot. The PPI specification defines the operation requests and the
actions the firmware has to take. The system administrator passes the operation
request number to the firmware through an ACPI interface which writes this
number to a memory location that the firmware knows. Upon reboot, the firmware
-finds the number and sends commands to the the TPM. The firmware writes the TPM
+finds the number and sends commands to the TPM. The firmware writes the TPM
result code and the operation request number to a memory location that ACPI can
read from and pass the result on to the administrator.
diff --git a/include/hw/xen/interface/io/blkif.h b/include/hw/xen/interface/io/blkif.h
index 8b1be50ce81e..d07fa1e07822 100644
--- a/include/hw/xen/interface/io/blkif.h
+++ b/include/hw/xen/interface/io/blkif.h
@@ -341,7 +341,7 @@
* access (even when it should be read-only). If the frontend hits the
* maximum number of allowed persistently mapped grants, it can fallback
* to non persistent mode. This will cause a performance degradation,
- * since the the backend driver will still try to map those grants
+ * since the backend driver will still try to map those grants
* persistently. Since the persistent grants protocol is compatible with
* the previous protocol, a frontend driver can choose to work in
* persistent mode even when the backend doesn't support it.
diff --git a/scripts/dump-guest-memory.py b/scripts/dump-guest-memory.py
index 2c587cbefc57..9371e4581308 100644
--- a/scripts/dump-guest-memory.py
+++ b/scripts/dump-guest-memory.py
@@ -170,7 +170,7 @@ class ELF(object):
self.ehdr.e_phnum += 1
def to_file(self, elf_file):
- """Writes all ELF structures to the the passed file.
+ """Writes all ELF structures to the passed file.
Structure:
Ehdr
--
2.21.0
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply related [flat|nested] 33+ messages in thread
* Re: [PULL 0/4] Trivial branch patches
2019-11-05 14:42 ` Laurent Vivier
(?)
@ 2019-11-05 15:48 ` no-reply
-1 siblings, 0 replies; 33+ messages in thread
From: no-reply @ 2019-11-05 15:48 UTC (permalink / raw)
To: laurent
Cc: qemu-devel, peter.maydell, sstabellini, berrange, ehabkost,
pbonzini, paul, qemu-trivial, claudio.fontana, crosa, mjt,
laurent, chouteau, frederic.konrad, qemu-arm, qemu-ppc, clg,
marcandre.lureau, anthony.perard, xen-devel, david
Patchew URL: https://patchew.org/QEMU/20191105144247.10301-1-laurent@vivier.eu/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Subject: [PULL 0/4] Trivial branch patches
Type: series
Message-id: 20191105144247.10301-1-laurent@vivier.eu
=== TEST SCRIPT BEGIN ===
#!/bin/bash
git rev-parse base > /dev/null || exit 0
git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram
./scripts/checkpatch.pl --mailback base..
=== TEST SCRIPT END ===
Switched to a new branch 'test'
85ac453 global: Squash 'the the'
9dd7da4 qom: Fix error message in object_class_property_add()
2b76b45 hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
bddcfd9 hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
=== OUTPUT BEGIN ===
1/4 Checking commit bddcfd9b6b24 (hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers)
2/4 Checking commit 2b76b451f9b7 (hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses)
3/4 Checking commit 9dd7da421bfb (qom: Fix error message in object_class_property_add())
WARNING: line over 80 characters
#31: FILE: qom/object.c:1109:
+ error_setg(errp, "attempt to add duplicate property '%s' to object (type '%s')",
WARNING: line over 80 characters
#43: FILE: qom/object.c:1141:
+ error_setg(errp, "attempt to add duplicate property '%s' to class (type '%s')",
total: 0 errors, 2 warnings, 22 lines checked
Patch 3/4 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
4/4 Checking commit 85ac453d1520 (global: Squash 'the the')
ERROR: do not use C99 // comments
#26: FILE: disas/libvixl/vixl/invalset.h:105:
+ // Note that this does not mean the backing storage is empty: it can still
total: 1 errors, 0 warnings, 56 lines checked
Patch 4/4 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
=== OUTPUT END ===
Test command exited with code: 1
The full log is available at
http://patchew.org/logs/20191105144247.10301-1-laurent@vivier.eu/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [Xen-devel] [PULL 0/4] Trivial branch patches
@ 2019-11-05 15:48 ` no-reply
0 siblings, 0 replies; 33+ messages in thread
From: no-reply @ 2019-11-05 15:48 UTC (permalink / raw)
To: laurent
Cc: peter.maydell, sstabellini, berrange, ehabkost, paul,
qemu-trivial, mjt, claudio.fontana, qemu-devel, chouteau, laurent,
marcandre.lureau, frederic.konrad, qemu-arm, qemu-ppc, clg, crosa,
anthony.perard, pbonzini, david, xen-devel
Patchew URL: https://patchew.org/QEMU/20191105144247.10301-1-laurent@vivier.eu/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Subject: [PULL 0/4] Trivial branch patches
Type: series
Message-id: 20191105144247.10301-1-laurent@vivier.eu
=== TEST SCRIPT BEGIN ===
#!/bin/bash
git rev-parse base > /dev/null || exit 0
git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram
./scripts/checkpatch.pl --mailback base..
=== TEST SCRIPT END ===
Switched to a new branch 'test'
85ac453 global: Squash 'the the'
9dd7da4 qom: Fix error message in object_class_property_add()
2b76b45 hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
bddcfd9 hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
=== OUTPUT BEGIN ===
1/4 Checking commit bddcfd9b6b24 (hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers)
2/4 Checking commit 2b76b451f9b7 (hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses)
3/4 Checking commit 9dd7da421bfb (qom: Fix error message in object_class_property_add())
WARNING: line over 80 characters
#31: FILE: qom/object.c:1109:
+ error_setg(errp, "attempt to add duplicate property '%s' to object (type '%s')",
WARNING: line over 80 characters
#43: FILE: qom/object.c:1141:
+ error_setg(errp, "attempt to add duplicate property '%s' to class (type '%s')",
total: 0 errors, 2 warnings, 22 lines checked
Patch 3/4 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
4/4 Checking commit 85ac453d1520 (global: Squash 'the the')
ERROR: do not use C99 // comments
#26: FILE: disas/libvixl/vixl/invalset.h:105:
+ // Note that this does not mean the backing storage is empty: it can still
total: 1 errors, 0 warnings, 56 lines checked
Patch 4/4 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
=== OUTPUT END ===
Test command exited with code: 1
The full log is available at
http://patchew.org/logs/20191105144247.10301-1-laurent@vivier.eu/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PULL 0/4] Trivial branch patches
@ 2019-11-05 15:48 ` no-reply
0 siblings, 0 replies; 33+ messages in thread
From: no-reply @ 2019-11-05 15:48 UTC (permalink / raw)
To: laurent
Cc: peter.maydell, sstabellini, berrange, ehabkost, paul,
qemu-trivial, mjt, claudio.fontana, qemu-devel, chouteau, laurent,
marcandre.lureau, frederic.konrad, qemu-arm, qemu-ppc, clg, crosa,
anthony.perard, pbonzini, david, xen-devel
Patchew URL: https://patchew.org/QEMU/20191105144247.10301-1-laurent@vivier.eu/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Subject: [PULL 0/4] Trivial branch patches
Type: series
Message-id: 20191105144247.10301-1-laurent@vivier.eu
=== TEST SCRIPT BEGIN ===
#!/bin/bash
git rev-parse base > /dev/null || exit 0
git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram
./scripts/checkpatch.pl --mailback base..
=== TEST SCRIPT END ===
Switched to a new branch 'test'
85ac453 global: Squash 'the the'
9dd7da4 qom: Fix error message in object_class_property_add()
2b76b45 hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
bddcfd9 hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
=== OUTPUT BEGIN ===
1/4 Checking commit bddcfd9b6b24 (hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers)
2/4 Checking commit 2b76b451f9b7 (hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses)
3/4 Checking commit 9dd7da421bfb (qom: Fix error message in object_class_property_add())
WARNING: line over 80 characters
#31: FILE: qom/object.c:1109:
+ error_setg(errp, "attempt to add duplicate property '%s' to object (type '%s')",
WARNING: line over 80 characters
#43: FILE: qom/object.c:1141:
+ error_setg(errp, "attempt to add duplicate property '%s' to class (type '%s')",
total: 0 errors, 2 warnings, 22 lines checked
Patch 3/4 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
4/4 Checking commit 85ac453d1520 (global: Squash 'the the')
ERROR: do not use C99 // comments
#26: FILE: disas/libvixl/vixl/invalset.h:105:
+ // Note that this does not mean the backing storage is empty: it can still
total: 1 errors, 0 warnings, 56 lines checked
Patch 4/4 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
=== OUTPUT END ===
Test command exited with code: 1
The full log is available at
http://patchew.org/logs/20191105144247.10301-1-laurent@vivier.eu/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PULL 0/4] Trivial branch patches
2019-11-05 15:48 ` no-reply
(?)
@ 2019-11-05 15:56 ` Laurent Vivier
-1 siblings, 0 replies; 33+ messages in thread
From: Laurent Vivier @ 2019-11-05 15:56 UTC (permalink / raw)
To: qemu-devel, Dr. David Alan Gilbert, Greg Kurz
Cc: peter.maydell, sstabellini, berrange, ehabkost, pbonzini, paul,
qemu-trivial, claudio.fontana, crosa, mjt, chouteau,
frederic.konrad, qemu-arm, qemu-ppc, clg, marcandre.lureau,
anthony.perard, xen-devel, david
Greg, Dave,
could you fix that?
Thanks,
Laurent
Le 05/11/2019 à 16:48, no-reply@patchew.org a écrit :
> Patchew URL: https://patchew.org/QEMU/20191105144247.10301-1-laurent@vivier.eu/
>
>
>
> Hi,
>
> This series seems to have some coding style problems. See output below for
> more information:
>
> Subject: [PULL 0/4] Trivial branch patches
> Type: series
> Message-id: 20191105144247.10301-1-laurent@vivier.eu
>
> === TEST SCRIPT BEGIN ===
> #!/bin/bash
> git rev-parse base > /dev/null || exit 0
> git config --local diff.renamelimit 0
> git config --local diff.renames True
> git config --local diff.algorithm histogram
> ./scripts/checkpatch.pl --mailback base..
> === TEST SCRIPT END ===
>
> Switched to a new branch 'test'
> 85ac453 global: Squash 'the the'
> 9dd7da4 qom: Fix error message in object_class_property_add()
> 2b76b45 hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
> bddcfd9 hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
>
> === OUTPUT BEGIN ===
> 1/4 Checking commit bddcfd9b6b24 (hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers)
> 2/4 Checking commit 2b76b451f9b7 (hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses)
> 3/4 Checking commit 9dd7da421bfb (qom: Fix error message in object_class_property_add())
> WARNING: line over 80 characters
> #31: FILE: qom/object.c:1109:
> + error_setg(errp, "attempt to add duplicate property '%s' to object (type '%s')",
>
> WARNING: line over 80 characters
> #43: FILE: qom/object.c:1141:
> + error_setg(errp, "attempt to add duplicate property '%s' to class (type '%s')",
>
> total: 0 errors, 2 warnings, 22 lines checked
>
> Patch 3/4 has style problems, please review. If any of these errors
> are false positives report them to the maintainer, see
> CHECKPATCH in MAINTAINERS.
> 4/4 Checking commit 85ac453d1520 (global: Squash 'the the')
> ERROR: do not use C99 // comments
> #26: FILE: disas/libvixl/vixl/invalset.h:105:
> + // Note that this does not mean the backing storage is empty: it can still
>
> total: 1 errors, 0 warnings, 56 lines checked
>
> Patch 4/4 has style problems, please review. If any of these errors
> are false positives report them to the maintainer, see
> CHECKPATCH in MAINTAINERS.
>
> === OUTPUT END ===
>
> Test command exited with code: 1
>
>
> The full log is available at
> http://patchew.org/logs/20191105144247.10301-1-laurent@vivier.eu/testing.checkpatch/?type=message.
> ---
> Email generated automatically by Patchew [https://patchew.org/].
> Please send your feedback to patchew-devel@redhat.com
>
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [Xen-devel] [PULL 0/4] Trivial branch patches
@ 2019-11-05 15:56 ` Laurent Vivier
0 siblings, 0 replies; 33+ messages in thread
From: Laurent Vivier @ 2019-11-05 15:56 UTC (permalink / raw)
To: qemu-devel, Dr. David Alan Gilbert, Greg Kurz
Cc: peter.maydell, sstabellini, berrange, ehabkost, paul,
qemu-trivial, mjt, claudio.fontana, chouteau, marcandre.lureau,
frederic.konrad, qemu-arm, qemu-ppc, clg, crosa, anthony.perard,
pbonzini, david, xen-devel
Greg, Dave,
could you fix that?
Thanks,
Laurent
Le 05/11/2019 à 16:48, no-reply@patchew.org a écrit :
> Patchew URL: https://patchew.org/QEMU/20191105144247.10301-1-laurent@vivier.eu/
>
>
>
> Hi,
>
> This series seems to have some coding style problems. See output below for
> more information:
>
> Subject: [PULL 0/4] Trivial branch patches
> Type: series
> Message-id: 20191105144247.10301-1-laurent@vivier.eu
>
> === TEST SCRIPT BEGIN ===
> #!/bin/bash
> git rev-parse base > /dev/null || exit 0
> git config --local diff.renamelimit 0
> git config --local diff.renames True
> git config --local diff.algorithm histogram
> ./scripts/checkpatch.pl --mailback base..
> === TEST SCRIPT END ===
>
> Switched to a new branch 'test'
> 85ac453 global: Squash 'the the'
> 9dd7da4 qom: Fix error message in object_class_property_add()
> 2b76b45 hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
> bddcfd9 hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
>
> === OUTPUT BEGIN ===
> 1/4 Checking commit bddcfd9b6b24 (hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers)
> 2/4 Checking commit 2b76b451f9b7 (hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses)
> 3/4 Checking commit 9dd7da421bfb (qom: Fix error message in object_class_property_add())
> WARNING: line over 80 characters
> #31: FILE: qom/object.c:1109:
> + error_setg(errp, "attempt to add duplicate property '%s' to object (type '%s')",
>
> WARNING: line over 80 characters
> #43: FILE: qom/object.c:1141:
> + error_setg(errp, "attempt to add duplicate property '%s' to class (type '%s')",
>
> total: 0 errors, 2 warnings, 22 lines checked
>
> Patch 3/4 has style problems, please review. If any of these errors
> are false positives report them to the maintainer, see
> CHECKPATCH in MAINTAINERS.
> 4/4 Checking commit 85ac453d1520 (global: Squash 'the the')
> ERROR: do not use C99 // comments
> #26: FILE: disas/libvixl/vixl/invalset.h:105:
> + // Note that this does not mean the backing storage is empty: it can still
>
> total: 1 errors, 0 warnings, 56 lines checked
>
> Patch 4/4 has style problems, please review. If any of these errors
> are false positives report them to the maintainer, see
> CHECKPATCH in MAINTAINERS.
>
> === OUTPUT END ===
>
> Test command exited with code: 1
>
>
> The full log is available at
> http://patchew.org/logs/20191105144247.10301-1-laurent@vivier.eu/testing.checkpatch/?type=message.
> ---
> Email generated automatically by Patchew [https://patchew.org/].
> Please send your feedback to patchew-devel@redhat.com
>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PULL 0/4] Trivial branch patches
@ 2019-11-05 15:56 ` Laurent Vivier
0 siblings, 0 replies; 33+ messages in thread
From: Laurent Vivier @ 2019-11-05 15:56 UTC (permalink / raw)
To: qemu-devel, Dr. David Alan Gilbert, Greg Kurz
Cc: peter.maydell, sstabellini, berrange, ehabkost, paul,
qemu-trivial, mjt, claudio.fontana, chouteau, marcandre.lureau,
frederic.konrad, qemu-arm, qemu-ppc, clg, crosa, anthony.perard,
pbonzini, david, xen-devel
Greg, Dave,
could you fix that?
Thanks,
Laurent
Le 05/11/2019 à 16:48, no-reply@patchew.org a écrit :
> Patchew URL: https://patchew.org/QEMU/20191105144247.10301-1-laurent@vivier.eu/
>
>
>
> Hi,
>
> This series seems to have some coding style problems. See output below for
> more information:
>
> Subject: [PULL 0/4] Trivial branch patches
> Type: series
> Message-id: 20191105144247.10301-1-laurent@vivier.eu
>
> === TEST SCRIPT BEGIN ===
> #!/bin/bash
> git rev-parse base > /dev/null || exit 0
> git config --local diff.renamelimit 0
> git config --local diff.renames True
> git config --local diff.algorithm histogram
> ./scripts/checkpatch.pl --mailback base..
> === TEST SCRIPT END ===
>
> Switched to a new branch 'test'
> 85ac453 global: Squash 'the the'
> 9dd7da4 qom: Fix error message in object_class_property_add()
> 2b76b45 hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
> bddcfd9 hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
>
> === OUTPUT BEGIN ===
> 1/4 Checking commit bddcfd9b6b24 (hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers)
> 2/4 Checking commit 2b76b451f9b7 (hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses)
> 3/4 Checking commit 9dd7da421bfb (qom: Fix error message in object_class_property_add())
> WARNING: line over 80 characters
> #31: FILE: qom/object.c:1109:
> + error_setg(errp, "attempt to add duplicate property '%s' to object (type '%s')",
>
> WARNING: line over 80 characters
> #43: FILE: qom/object.c:1141:
> + error_setg(errp, "attempt to add duplicate property '%s' to class (type '%s')",
>
> total: 0 errors, 2 warnings, 22 lines checked
>
> Patch 3/4 has style problems, please review. If any of these errors
> are false positives report them to the maintainer, see
> CHECKPATCH in MAINTAINERS.
> 4/4 Checking commit 85ac453d1520 (global: Squash 'the the')
> ERROR: do not use C99 // comments
> #26: FILE: disas/libvixl/vixl/invalset.h:105:
> + // Note that this does not mean the backing storage is empty: it can still
>
> total: 1 errors, 0 warnings, 56 lines checked
>
> Patch 4/4 has style problems, please review. If any of these errors
> are false positives report them to the maintainer, see
> CHECKPATCH in MAINTAINERS.
>
> === OUTPUT END ===
>
> Test command exited with code: 1
>
>
> The full log is available at
> http://patchew.org/logs/20191105144247.10301-1-laurent@vivier.eu/testing.checkpatch/?type=message.
> ---
> Email generated automatically by Patchew [https://patchew.org/].
> Please send your feedback to patchew-devel@redhat.com
>
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PULL 0/4] Trivial branch patches
2019-11-05 15:56 ` Laurent Vivier
(?)
@ 2019-11-05 16:03 ` Dr. David Alan Gilbert
-1 siblings, 0 replies; 33+ messages in thread
From: Dr. David Alan Gilbert @ 2019-11-05 16:03 UTC (permalink / raw)
To: Laurent Vivier
Cc: qemu-devel, Greg Kurz, peter.maydell, sstabellini, berrange,
ehabkost, pbonzini, paul, qemu-trivial, claudio.fontana, crosa,
mjt, chouteau, frederic.konrad, qemu-arm, qemu-ppc, clg,
marcandre.lureau, anthony.perard, xen-devel, david
* Laurent Vivier (laurent@vivier.eu) wrote:
> Greg, Dave,
>
> could you fix that?
>
> Thanks,
> Laurent
>
> Le 05/11/2019 à 16:48, no-reply@patchew.org a écrit :
> > Patchew URL: https://patchew.org/QEMU/20191105144247.10301-1-laurent@vivier.eu/
> >
> >
> >
> > Hi,
> >
> > This series seems to have some coding style problems. See output below for
> > more information:
> >
> > Subject: [PULL 0/4] Trivial branch patches
> > Type: series
> > Message-id: 20191105144247.10301-1-laurent@vivier.eu
> >
> > === TEST SCRIPT BEGIN ===
> > #!/bin/bash
> > git rev-parse base > /dev/null || exit 0
> > git config --local diff.renamelimit 0
> > git config --local diff.renames True
> > git config --local diff.algorithm histogram
> > ./scripts/checkpatch.pl --mailback base..
> > === TEST SCRIPT END ===
> >
> > Switched to a new branch 'test'
> > 85ac453 global: Squash 'the the'
> > 9dd7da4 qom: Fix error message in object_class_property_add()
> > 2b76b45 hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
> > bddcfd9 hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
> >
> > === OUTPUT BEGIN ===
> > 1/4 Checking commit bddcfd9b6b24 (hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers)
> > 2/4 Checking commit 2b76b451f9b7 (hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses)
> > 3/4 Checking commit 9dd7da421bfb (qom: Fix error message in object_class_property_add())
> > WARNING: line over 80 characters
> > #31: FILE: qom/object.c:1109:
> > + error_setg(errp, "attempt to add duplicate property '%s' to object (type '%s')",
> >
> > WARNING: line over 80 characters
> > #43: FILE: qom/object.c:1141:
> > + error_setg(errp, "attempt to add duplicate property '%s' to class (type '%s')",
> >
> > total: 0 errors, 2 warnings, 22 lines checked
> >
> > Patch 3/4 has style problems, please review. If any of these errors
> > are false positives report them to the maintainer, see
> > CHECKPATCH in MAINTAINERS.
> > 4/4 Checking commit 85ac453d1520 (global: Squash 'the the')
> > ERROR: do not use C99 // comments
> > #26: FILE: disas/libvixl/vixl/invalset.h:105:
> > + // Note that this does not mean the backing storage is empty: it can still
That one is a false positive; libvixl is written in C++ !
Dave
> > total: 1 errors, 0 warnings, 56 lines checked
> >
> > Patch 4/4 has style problems, please review. If any of these errors
> > are false positives report them to the maintainer, see
> > CHECKPATCH in MAINTAINERS.
> >
> > === OUTPUT END ===
> >
> > Test command exited with code: 1
> >
> >
> > The full log is available at
> > http://patchew.org/logs/20191105144247.10301-1-laurent@vivier.eu/testing.checkpatch/?type=message.
> > ---
> > Email generated automatically by Patchew [https://patchew.org/].
> > Please send your feedback to patchew-devel@redhat.com
> >
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [Xen-devel] [PULL 0/4] Trivial branch patches
@ 2019-11-05 16:03 ` Dr. David Alan Gilbert
0 siblings, 0 replies; 33+ messages in thread
From: Dr. David Alan Gilbert @ 2019-11-05 16:03 UTC (permalink / raw)
To: Laurent Vivier
Cc: peter.maydell, sstabellini, berrange, ehabkost, paul,
qemu-trivial, mjt, claudio.fontana, qemu-devel, chouteau,
Greg Kurz, marcandre.lureau, frederic.konrad, qemu-arm, qemu-ppc,
clg, crosa, anthony.perard, pbonzini, david, xen-devel
* Laurent Vivier (laurent@vivier.eu) wrote:
> Greg, Dave,
>
> could you fix that?
>
> Thanks,
> Laurent
>
> Le 05/11/2019 à 16:48, no-reply@patchew.org a écrit :
> > Patchew URL: https://patchew.org/QEMU/20191105144247.10301-1-laurent@vivier.eu/
> >
> >
> >
> > Hi,
> >
> > This series seems to have some coding style problems. See output below for
> > more information:
> >
> > Subject: [PULL 0/4] Trivial branch patches
> > Type: series
> > Message-id: 20191105144247.10301-1-laurent@vivier.eu
> >
> > === TEST SCRIPT BEGIN ===
> > #!/bin/bash
> > git rev-parse base > /dev/null || exit 0
> > git config --local diff.renamelimit 0
> > git config --local diff.renames True
> > git config --local diff.algorithm histogram
> > ./scripts/checkpatch.pl --mailback base..
> > === TEST SCRIPT END ===
> >
> > Switched to a new branch 'test'
> > 85ac453 global: Squash 'the the'
> > 9dd7da4 qom: Fix error message in object_class_property_add()
> > 2b76b45 hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
> > bddcfd9 hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
> >
> > === OUTPUT BEGIN ===
> > 1/4 Checking commit bddcfd9b6b24 (hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers)
> > 2/4 Checking commit 2b76b451f9b7 (hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses)
> > 3/4 Checking commit 9dd7da421bfb (qom: Fix error message in object_class_property_add())
> > WARNING: line over 80 characters
> > #31: FILE: qom/object.c:1109:
> > + error_setg(errp, "attempt to add duplicate property '%s' to object (type '%s')",
> >
> > WARNING: line over 80 characters
> > #43: FILE: qom/object.c:1141:
> > + error_setg(errp, "attempt to add duplicate property '%s' to class (type '%s')",
> >
> > total: 0 errors, 2 warnings, 22 lines checked
> >
> > Patch 3/4 has style problems, please review. If any of these errors
> > are false positives report them to the maintainer, see
> > CHECKPATCH in MAINTAINERS.
> > 4/4 Checking commit 85ac453d1520 (global: Squash 'the the')
> > ERROR: do not use C99 // comments
> > #26: FILE: disas/libvixl/vixl/invalset.h:105:
> > + // Note that this does not mean the backing storage is empty: it can still
That one is a false positive; libvixl is written in C++ !
Dave
> > total: 1 errors, 0 warnings, 56 lines checked
> >
> > Patch 4/4 has style problems, please review. If any of these errors
> > are false positives report them to the maintainer, see
> > CHECKPATCH in MAINTAINERS.
> >
> > === OUTPUT END ===
> >
> > Test command exited with code: 1
> >
> >
> > The full log is available at
> > http://patchew.org/logs/20191105144247.10301-1-laurent@vivier.eu/testing.checkpatch/?type=message.
> > ---
> > Email generated automatically by Patchew [https://patchew.org/].
> > Please send your feedback to patchew-devel@redhat.com
> >
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PULL 0/4] Trivial branch patches
@ 2019-11-05 16:03 ` Dr. David Alan Gilbert
0 siblings, 0 replies; 33+ messages in thread
From: Dr. David Alan Gilbert @ 2019-11-05 16:03 UTC (permalink / raw)
To: Laurent Vivier
Cc: peter.maydell, sstabellini, berrange, ehabkost, paul,
qemu-trivial, mjt, claudio.fontana, qemu-devel, chouteau,
Greg Kurz, marcandre.lureau, frederic.konrad, qemu-arm, qemu-ppc,
clg, crosa, anthony.perard, pbonzini, david, xen-devel
* Laurent Vivier (laurent@vivier.eu) wrote:
> Greg, Dave,
>
> could you fix that?
>
> Thanks,
> Laurent
>
> Le 05/11/2019 à 16:48, no-reply@patchew.org a écrit :
> > Patchew URL: https://patchew.org/QEMU/20191105144247.10301-1-laurent@vivier.eu/
> >
> >
> >
> > Hi,
> >
> > This series seems to have some coding style problems. See output below for
> > more information:
> >
> > Subject: [PULL 0/4] Trivial branch patches
> > Type: series
> > Message-id: 20191105144247.10301-1-laurent@vivier.eu
> >
> > === TEST SCRIPT BEGIN ===
> > #!/bin/bash
> > git rev-parse base > /dev/null || exit 0
> > git config --local diff.renamelimit 0
> > git config --local diff.renames True
> > git config --local diff.algorithm histogram
> > ./scripts/checkpatch.pl --mailback base..
> > === TEST SCRIPT END ===
> >
> > Switched to a new branch 'test'
> > 85ac453 global: Squash 'the the'
> > 9dd7da4 qom: Fix error message in object_class_property_add()
> > 2b76b45 hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
> > bddcfd9 hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
> >
> > === OUTPUT BEGIN ===
> > 1/4 Checking commit bddcfd9b6b24 (hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers)
> > 2/4 Checking commit 2b76b451f9b7 (hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses)
> > 3/4 Checking commit 9dd7da421bfb (qom: Fix error message in object_class_property_add())
> > WARNING: line over 80 characters
> > #31: FILE: qom/object.c:1109:
> > + error_setg(errp, "attempt to add duplicate property '%s' to object (type '%s')",
> >
> > WARNING: line over 80 characters
> > #43: FILE: qom/object.c:1141:
> > + error_setg(errp, "attempt to add duplicate property '%s' to class (type '%s')",
> >
> > total: 0 errors, 2 warnings, 22 lines checked
> >
> > Patch 3/4 has style problems, please review. If any of these errors
> > are false positives report them to the maintainer, see
> > CHECKPATCH in MAINTAINERS.
> > 4/4 Checking commit 85ac453d1520 (global: Squash 'the the')
> > ERROR: do not use C99 // comments
> > #26: FILE: disas/libvixl/vixl/invalset.h:105:
> > + // Note that this does not mean the backing storage is empty: it can still
That one is a false positive; libvixl is written in C++ !
Dave
> > total: 1 errors, 0 warnings, 56 lines checked
> >
> > Patch 4/4 has style problems, please review. If any of these errors
> > are false positives report them to the maintainer, see
> > CHECKPATCH in MAINTAINERS.
> >
> > === OUTPUT END ===
> >
> > Test command exited with code: 1
> >
> >
> > The full log is available at
> > http://patchew.org/logs/20191105144247.10301-1-laurent@vivier.eu/testing.checkpatch/?type=message.
> > ---
> > Email generated automatically by Patchew [https://patchew.org/].
> > Please send your feedback to patchew-devel@redhat.com
> >
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PULL 0/4] Trivial branch patches
2019-11-05 16:03 ` Dr. David Alan Gilbert
(?)
@ 2019-11-05 16:18 ` Laurent Vivier
-1 siblings, 0 replies; 33+ messages in thread
From: Laurent Vivier @ 2019-11-05 16:18 UTC (permalink / raw)
To: Dr. David Alan Gilbert
Cc: qemu-devel, Greg Kurz, peter.maydell, sstabellini, berrange,
ehabkost, pbonzini, paul, qemu-trivial, claudio.fontana, crosa,
mjt, chouteau, frederic.konrad, qemu-arm, qemu-ppc, clg,
marcandre.lureau, anthony.perard, xen-devel, david
Le 05/11/2019 à 17:03, Dr. David Alan Gilbert a écrit :
> * Laurent Vivier (laurent@vivier.eu) wrote:
>> Greg, Dave,
>>
>> could you fix that?
>>
>> Thanks,
>> Laurent
>>
>> Le 05/11/2019 à 16:48, no-reply@patchew.org a écrit :
>>> Patchew URL: https://patchew.org/QEMU/20191105144247.10301-1-laurent@vivier.eu/
>>>
>>>
>>>
>>> Hi,
>>>
>>> This series seems to have some coding style problems. See output below for
>>> more information:
>>>
>>> Subject: [PULL 0/4] Trivial branch patches
>>> Type: series
>>> Message-id: 20191105144247.10301-1-laurent@vivier.eu
>>>
>>> === TEST SCRIPT BEGIN ===
>>> #!/bin/bash
>>> git rev-parse base > /dev/null || exit 0
>>> git config --local diff.renamelimit 0
>>> git config --local diff.renames True
>>> git config --local diff.algorithm histogram
>>> ./scripts/checkpatch.pl --mailback base..
>>> === TEST SCRIPT END ===
>>>
>>> Switched to a new branch 'test'
>>> 85ac453 global: Squash 'the the'
>>> 9dd7da4 qom: Fix error message in object_class_property_add()
>>> 2b76b45 hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
>>> bddcfd9 hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
>>>
>>> === OUTPUT BEGIN ===
>>> 1/4 Checking commit bddcfd9b6b24 (hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers)
>>> 2/4 Checking commit 2b76b451f9b7 (hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses)
>>> 3/4 Checking commit 9dd7da421bfb (qom: Fix error message in object_class_property_add())
>>> WARNING: line over 80 characters
>>> #31: FILE: qom/object.c:1109:
>>> + error_setg(errp, "attempt to add duplicate property '%s' to object (type '%s')",
>>>
>>> WARNING: line over 80 characters
>>> #43: FILE: qom/object.c:1141:
>>> + error_setg(errp, "attempt to add duplicate property '%s' to class (type '%s')",
>>>
>>> total: 0 errors, 2 warnings, 22 lines checked
>>>
>>> Patch 3/4 has style problems, please review. If any of these errors
>>> are false positives report them to the maintainer, see
>>> CHECKPATCH in MAINTAINERS.
>>> 4/4 Checking commit 85ac453d1520 (global: Squash 'the the')
>>> ERROR: do not use C99 // comments
>>> #26: FILE: disas/libvixl/vixl/invalset.h:105:
>>> + // Note that this does not mean the backing storage is empty: it can still
>
> That one is a false positive; libvixl is written in C++ !
OK, thank you.
Laurent
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [Xen-devel] [PULL 0/4] Trivial branch patches
@ 2019-11-05 16:18 ` Laurent Vivier
0 siblings, 0 replies; 33+ messages in thread
From: Laurent Vivier @ 2019-11-05 16:18 UTC (permalink / raw)
To: Dr. David Alan Gilbert
Cc: peter.maydell, sstabellini, berrange, ehabkost, paul,
qemu-trivial, mjt, claudio.fontana, qemu-devel, chouteau,
Greg Kurz, marcandre.lureau, frederic.konrad, qemu-arm, qemu-ppc,
clg, crosa, anthony.perard, pbonzini, david, xen-devel
Le 05/11/2019 à 17:03, Dr. David Alan Gilbert a écrit :
> * Laurent Vivier (laurent@vivier.eu) wrote:
>> Greg, Dave,
>>
>> could you fix that?
>>
>> Thanks,
>> Laurent
>>
>> Le 05/11/2019 à 16:48, no-reply@patchew.org a écrit :
>>> Patchew URL: https://patchew.org/QEMU/20191105144247.10301-1-laurent@vivier.eu/
>>>
>>>
>>>
>>> Hi,
>>>
>>> This series seems to have some coding style problems. See output below for
>>> more information:
>>>
>>> Subject: [PULL 0/4] Trivial branch patches
>>> Type: series
>>> Message-id: 20191105144247.10301-1-laurent@vivier.eu
>>>
>>> === TEST SCRIPT BEGIN ===
>>> #!/bin/bash
>>> git rev-parse base > /dev/null || exit 0
>>> git config --local diff.renamelimit 0
>>> git config --local diff.renames True
>>> git config --local diff.algorithm histogram
>>> ./scripts/checkpatch.pl --mailback base..
>>> === TEST SCRIPT END ===
>>>
>>> Switched to a new branch 'test'
>>> 85ac453 global: Squash 'the the'
>>> 9dd7da4 qom: Fix error message in object_class_property_add()
>>> 2b76b45 hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
>>> bddcfd9 hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
>>>
>>> === OUTPUT BEGIN ===
>>> 1/4 Checking commit bddcfd9b6b24 (hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers)
>>> 2/4 Checking commit 2b76b451f9b7 (hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses)
>>> 3/4 Checking commit 9dd7da421bfb (qom: Fix error message in object_class_property_add())
>>> WARNING: line over 80 characters
>>> #31: FILE: qom/object.c:1109:
>>> + error_setg(errp, "attempt to add duplicate property '%s' to object (type '%s')",
>>>
>>> WARNING: line over 80 characters
>>> #43: FILE: qom/object.c:1141:
>>> + error_setg(errp, "attempt to add duplicate property '%s' to class (type '%s')",
>>>
>>> total: 0 errors, 2 warnings, 22 lines checked
>>>
>>> Patch 3/4 has style problems, please review. If any of these errors
>>> are false positives report them to the maintainer, see
>>> CHECKPATCH in MAINTAINERS.
>>> 4/4 Checking commit 85ac453d1520 (global: Squash 'the the')
>>> ERROR: do not use C99 // comments
>>> #26: FILE: disas/libvixl/vixl/invalset.h:105:
>>> + // Note that this does not mean the backing storage is empty: it can still
>
> That one is a false positive; libvixl is written in C++ !
OK, thank you.
Laurent
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PULL 0/4] Trivial branch patches
@ 2019-11-05 16:18 ` Laurent Vivier
0 siblings, 0 replies; 33+ messages in thread
From: Laurent Vivier @ 2019-11-05 16:18 UTC (permalink / raw)
To: Dr. David Alan Gilbert
Cc: peter.maydell, sstabellini, berrange, ehabkost, paul,
qemu-trivial, mjt, claudio.fontana, qemu-devel, chouteau,
Greg Kurz, marcandre.lureau, frederic.konrad, qemu-arm, qemu-ppc,
clg, crosa, anthony.perard, pbonzini, david, xen-devel
Le 05/11/2019 à 17:03, Dr. David Alan Gilbert a écrit :
> * Laurent Vivier (laurent@vivier.eu) wrote:
>> Greg, Dave,
>>
>> could you fix that?
>>
>> Thanks,
>> Laurent
>>
>> Le 05/11/2019 à 16:48, no-reply@patchew.org a écrit :
>>> Patchew URL: https://patchew.org/QEMU/20191105144247.10301-1-laurent@vivier.eu/
>>>
>>>
>>>
>>> Hi,
>>>
>>> This series seems to have some coding style problems. See output below for
>>> more information:
>>>
>>> Subject: [PULL 0/4] Trivial branch patches
>>> Type: series
>>> Message-id: 20191105144247.10301-1-laurent@vivier.eu
>>>
>>> === TEST SCRIPT BEGIN ===
>>> #!/bin/bash
>>> git rev-parse base > /dev/null || exit 0
>>> git config --local diff.renamelimit 0
>>> git config --local diff.renames True
>>> git config --local diff.algorithm histogram
>>> ./scripts/checkpatch.pl --mailback base..
>>> === TEST SCRIPT END ===
>>>
>>> Switched to a new branch 'test'
>>> 85ac453 global: Squash 'the the'
>>> 9dd7da4 qom: Fix error message in object_class_property_add()
>>> 2b76b45 hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
>>> bddcfd9 hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
>>>
>>> === OUTPUT BEGIN ===
>>> 1/4 Checking commit bddcfd9b6b24 (hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers)
>>> 2/4 Checking commit 2b76b451f9b7 (hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses)
>>> 3/4 Checking commit 9dd7da421bfb (qom: Fix error message in object_class_property_add())
>>> WARNING: line over 80 characters
>>> #31: FILE: qom/object.c:1109:
>>> + error_setg(errp, "attempt to add duplicate property '%s' to object (type '%s')",
>>>
>>> WARNING: line over 80 characters
>>> #43: FILE: qom/object.c:1141:
>>> + error_setg(errp, "attempt to add duplicate property '%s' to class (type '%s')",
>>>
>>> total: 0 errors, 2 warnings, 22 lines checked
>>>
>>> Patch 3/4 has style problems, please review. If any of these errors
>>> are false positives report them to the maintainer, see
>>> CHECKPATCH in MAINTAINERS.
>>> 4/4 Checking commit 85ac453d1520 (global: Squash 'the the')
>>> ERROR: do not use C99 // comments
>>> #26: FILE: disas/libvixl/vixl/invalset.h:105:
>>> + // Note that this does not mean the backing storage is empty: it can still
>
> That one is a false positive; libvixl is written in C++ !
OK, thank you.
Laurent
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PULL 0/4] Trivial branch patches
2019-11-05 15:56 ` Laurent Vivier
(?)
@ 2019-11-06 8:07 ` Greg Kurz
-1 siblings, 0 replies; 33+ messages in thread
From: Greg Kurz @ 2019-11-06 8:07 UTC (permalink / raw)
To: Laurent Vivier
Cc: qemu-devel, Dr. David Alan Gilbert, peter.maydell, sstabellini,
berrange, ehabkost, pbonzini, paul, qemu-trivial, claudio.fontana,
crosa, mjt, chouteau, frederic.konrad, qemu-arm, qemu-ppc, clg,
marcandre.lureau, anthony.perard, xen-devel, david
On Tue, 5 Nov 2019 16:56:11 +0100
Laurent Vivier <laurent@vivier.eu> wrote:
> Greg, Dave,
>
> could you fix that?
>
> Thanks,
> Laurent
>
> Le 05/11/2019 à 16:48, no-reply@patchew.org a écrit :
> > Patchew URL: https://patchew.org/QEMU/20191105144247.10301-1-laurent@vivier.eu/
> >
> >
> >
> > Hi,
> >
> > This series seems to have some coding style problems. See output below for
> > more information:
> >
> > Subject: [PULL 0/4] Trivial branch patches
> > Type: series
> > Message-id: 20191105144247.10301-1-laurent@vivier.eu
> >
> > === TEST SCRIPT BEGIN ===
> > #!/bin/bash
> > git rev-parse base > /dev/null || exit 0
> > git config --local diff.renamelimit 0
> > git config --local diff.renames True
> > git config --local diff.algorithm histogram
> > ./scripts/checkpatch.pl --mailback base..
> > === TEST SCRIPT END ===
> >
> > Switched to a new branch 'test'
> > 85ac453 global: Squash 'the the'
> > 9dd7da4 qom: Fix error message in object_class_property_add()
> > 2b76b45 hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
> > bddcfd9 hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
> >
> > === OUTPUT BEGIN ===
> > 1/4 Checking commit bddcfd9b6b24 (hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers)
> > 2/4 Checking commit 2b76b451f9b7 (hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses)
> > 3/4 Checking commit 9dd7da421bfb (qom: Fix error message in object_class_property_add())
> > WARNING: line over 80 characters
> > #31: FILE: qom/object.c:1109:
> > + error_setg(errp, "attempt to add duplicate property '%s' to object (type '%s')",
> >
> > WARNING: line over 80 characters
> > #43: FILE: qom/object.c:1141:
> > + error_setg(errp, "attempt to add duplicate property '%s' to class (type '%s')",
> >
As mentioned in the changelog, this is deliberate. AFAIK better grep-ability
has precedence over the 80 characters rule when it comes to error messages.
Maybe we should teach checkpatch about that ?
Cheers,
--
Greg
> > total: 0 errors, 2 warnings, 22 lines checked
> >
> > Patch 3/4 has style problems, please review. If any of these errors
> > are false positives report them to the maintainer, see
> > CHECKPATCH in MAINTAINERS.
> > 4/4 Checking commit 85ac453d1520 (global: Squash 'the the')
> > ERROR: do not use C99 // comments
> > #26: FILE: disas/libvixl/vixl/invalset.h:105:
> > + // Note that this does not mean the backing storage is empty: it can still
> >
> > total: 1 errors, 0 warnings, 56 lines checked
> >
> > Patch 4/4 has style problems, please review. If any of these errors
> > are false positives report them to the maintainer, see
> > CHECKPATCH in MAINTAINERS.
> >
> > === OUTPUT END ===
> >
> > Test command exited with code: 1
> >
> >
> > The full log is available at
> > http://patchew.org/logs/20191105144247.10301-1-laurent@vivier.eu/testing.checkpatch/?type=message.
> > ---
> > Email generated automatically by Patchew [https://patchew.org/].
> > Please send your feedback to patchew-devel@redhat.com
> >
>
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [Xen-devel] [PULL 0/4] Trivial branch patches
@ 2019-11-06 8:07 ` Greg Kurz
0 siblings, 0 replies; 33+ messages in thread
From: Greg Kurz @ 2019-11-06 8:07 UTC (permalink / raw)
To: Laurent Vivier
Cc: peter.maydell, sstabellini, berrange, ehabkost, paul,
qemu-trivial, mjt, claudio.fontana, qemu-devel, chouteau,
Dr. David Alan Gilbert, marcandre.lureau, frederic.konrad,
qemu-arm, qemu-ppc, clg, crosa, anthony.perard, pbonzini, david,
xen-devel
On Tue, 5 Nov 2019 16:56:11 +0100
Laurent Vivier <laurent@vivier.eu> wrote:
> Greg, Dave,
>
> could you fix that?
>
> Thanks,
> Laurent
>
> Le 05/11/2019 à 16:48, no-reply@patchew.org a écrit :
> > Patchew URL: https://patchew.org/QEMU/20191105144247.10301-1-laurent@vivier.eu/
> >
> >
> >
> > Hi,
> >
> > This series seems to have some coding style problems. See output below for
> > more information:
> >
> > Subject: [PULL 0/4] Trivial branch patches
> > Type: series
> > Message-id: 20191105144247.10301-1-laurent@vivier.eu
> >
> > === TEST SCRIPT BEGIN ===
> > #!/bin/bash
> > git rev-parse base > /dev/null || exit 0
> > git config --local diff.renamelimit 0
> > git config --local diff.renames True
> > git config --local diff.algorithm histogram
> > ./scripts/checkpatch.pl --mailback base..
> > === TEST SCRIPT END ===
> >
> > Switched to a new branch 'test'
> > 85ac453 global: Squash 'the the'
> > 9dd7da4 qom: Fix error message in object_class_property_add()
> > 2b76b45 hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
> > bddcfd9 hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
> >
> > === OUTPUT BEGIN ===
> > 1/4 Checking commit bddcfd9b6b24 (hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers)
> > 2/4 Checking commit 2b76b451f9b7 (hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses)
> > 3/4 Checking commit 9dd7da421bfb (qom: Fix error message in object_class_property_add())
> > WARNING: line over 80 characters
> > #31: FILE: qom/object.c:1109:
> > + error_setg(errp, "attempt to add duplicate property '%s' to object (type '%s')",
> >
> > WARNING: line over 80 characters
> > #43: FILE: qom/object.c:1141:
> > + error_setg(errp, "attempt to add duplicate property '%s' to class (type '%s')",
> >
As mentioned in the changelog, this is deliberate. AFAIK better grep-ability
has precedence over the 80 characters rule when it comes to error messages.
Maybe we should teach checkpatch about that ?
Cheers,
--
Greg
> > total: 0 errors, 2 warnings, 22 lines checked
> >
> > Patch 3/4 has style problems, please review. If any of these errors
> > are false positives report them to the maintainer, see
> > CHECKPATCH in MAINTAINERS.
> > 4/4 Checking commit 85ac453d1520 (global: Squash 'the the')
> > ERROR: do not use C99 // comments
> > #26: FILE: disas/libvixl/vixl/invalset.h:105:
> > + // Note that this does not mean the backing storage is empty: it can still
> >
> > total: 1 errors, 0 warnings, 56 lines checked
> >
> > Patch 4/4 has style problems, please review. If any of these errors
> > are false positives report them to the maintainer, see
> > CHECKPATCH in MAINTAINERS.
> >
> > === OUTPUT END ===
> >
> > Test command exited with code: 1
> >
> >
> > The full log is available at
> > http://patchew.org/logs/20191105144247.10301-1-laurent@vivier.eu/testing.checkpatch/?type=message.
> > ---
> > Email generated automatically by Patchew [https://patchew.org/].
> > Please send your feedback to patchew-devel@redhat.com
> >
>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PULL 0/4] Trivial branch patches
@ 2019-11-06 8:07 ` Greg Kurz
0 siblings, 0 replies; 33+ messages in thread
From: Greg Kurz @ 2019-11-06 8:07 UTC (permalink / raw)
To: Laurent Vivier
Cc: peter.maydell, sstabellini, berrange, ehabkost, paul,
qemu-trivial, mjt, claudio.fontana, qemu-devel, chouteau,
Dr. David Alan Gilbert, marcandre.lureau, frederic.konrad,
qemu-arm, qemu-ppc, clg, crosa, anthony.perard, pbonzini, david,
xen-devel
On Tue, 5 Nov 2019 16:56:11 +0100
Laurent Vivier <laurent@vivier.eu> wrote:
> Greg, Dave,
>
> could you fix that?
>
> Thanks,
> Laurent
>
> Le 05/11/2019 à 16:48, no-reply@patchew.org a écrit :
> > Patchew URL: https://patchew.org/QEMU/20191105144247.10301-1-laurent@vivier.eu/
> >
> >
> >
> > Hi,
> >
> > This series seems to have some coding style problems. See output below for
> > more information:
> >
> > Subject: [PULL 0/4] Trivial branch patches
> > Type: series
> > Message-id: 20191105144247.10301-1-laurent@vivier.eu
> >
> > === TEST SCRIPT BEGIN ===
> > #!/bin/bash
> > git rev-parse base > /dev/null || exit 0
> > git config --local diff.renamelimit 0
> > git config --local diff.renames True
> > git config --local diff.algorithm histogram
> > ./scripts/checkpatch.pl --mailback base..
> > === TEST SCRIPT END ===
> >
> > Switched to a new branch 'test'
> > 85ac453 global: Squash 'the the'
> > 9dd7da4 qom: Fix error message in object_class_property_add()
> > 2b76b45 hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
> > bddcfd9 hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
> >
> > === OUTPUT BEGIN ===
> > 1/4 Checking commit bddcfd9b6b24 (hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers)
> > 2/4 Checking commit 2b76b451f9b7 (hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses)
> > 3/4 Checking commit 9dd7da421bfb (qom: Fix error message in object_class_property_add())
> > WARNING: line over 80 characters
> > #31: FILE: qom/object.c:1109:
> > + error_setg(errp, "attempt to add duplicate property '%s' to object (type '%s')",
> >
> > WARNING: line over 80 characters
> > #43: FILE: qom/object.c:1141:
> > + error_setg(errp, "attempt to add duplicate property '%s' to class (type '%s')",
> >
As mentioned in the changelog, this is deliberate. AFAIK better grep-ability
has precedence over the 80 characters rule when it comes to error messages.
Maybe we should teach checkpatch about that ?
Cheers,
--
Greg
> > total: 0 errors, 2 warnings, 22 lines checked
> >
> > Patch 3/4 has style problems, please review. If any of these errors
> > are false positives report them to the maintainer, see
> > CHECKPATCH in MAINTAINERS.
> > 4/4 Checking commit 85ac453d1520 (global: Squash 'the the')
> > ERROR: do not use C99 // comments
> > #26: FILE: disas/libvixl/vixl/invalset.h:105:
> > + // Note that this does not mean the backing storage is empty: it can still
> >
> > total: 1 errors, 0 warnings, 56 lines checked
> >
> > Patch 4/4 has style problems, please review. If any of these errors
> > are false positives report them to the maintainer, see
> > CHECKPATCH in MAINTAINERS.
> >
> > === OUTPUT END ===
> >
> > Test command exited with code: 1
> >
> >
> > The full log is available at
> > http://patchew.org/logs/20191105144247.10301-1-laurent@vivier.eu/testing.checkpatch/?type=message.
> > ---
> > Email generated automatically by Patchew [https://patchew.org/].
> > Please send your feedback to patchew-devel@redhat.com
> >
>
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PULL 0/4] Trivial branch patches
2019-11-05 14:42 ` Laurent Vivier
(?)
@ 2019-11-05 17:52 ` no-reply
-1 siblings, 0 replies; 33+ messages in thread
From: no-reply @ 2019-11-05 17:52 UTC (permalink / raw)
To: laurent
Cc: qemu-devel, peter.maydell, sstabellini, berrange, ehabkost,
pbonzini, paul, qemu-trivial, claudio.fontana, crosa, mjt,
laurent, chouteau, frederic.konrad, qemu-arm, qemu-ppc, clg,
marcandre.lureau, anthony.perard, xen-devel, david
Patchew URL: https://patchew.org/QEMU/20191105144247.10301-1-laurent@vivier.eu/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Subject: [PULL 0/4] Trivial branch patches
Type: series
Message-id: 20191105144247.10301-1-laurent@vivier.eu
=== TEST SCRIPT BEGIN ===
#!/bin/bash
git rev-parse base > /dev/null || exit 0
git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram
./scripts/checkpatch.pl --mailback base..
=== TEST SCRIPT END ===
Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
Switched to a new branch 'test'
0102c79 global: Squash 'the the'
9c583b0 qom: Fix error message in object_class_property_add()
299eefd hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
c931231 hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
=== OUTPUT BEGIN ===
1/4 Checking commit c93123187df2 (hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers)
2/4 Checking commit 299eefd37522 (hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses)
3/4 Checking commit 9c583b04fdb1 (qom: Fix error message in object_class_property_add())
WARNING: line over 80 characters
#31: FILE: qom/object.c:1109:
+ error_setg(errp, "attempt to add duplicate property '%s' to object (type '%s')",
WARNING: line over 80 characters
#43: FILE: qom/object.c:1141:
+ error_setg(errp, "attempt to add duplicate property '%s' to class (type '%s')",
total: 0 errors, 2 warnings, 22 lines checked
Patch 3/4 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
4/4 Checking commit 0102c79a3068 (global: Squash 'the the')
ERROR: do not use C99 // comments
#26: FILE: disas/libvixl/vixl/invalset.h:105:
+ // Note that this does not mean the backing storage is empty: it can still
total: 1 errors, 0 warnings, 56 lines checked
Patch 4/4 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
=== OUTPUT END ===
Test command exited with code: 1
The full log is available at
http://patchew.org/logs/20191105144247.10301-1-laurent@vivier.eu/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [Xen-devel] [PULL 0/4] Trivial branch patches
@ 2019-11-05 17:52 ` no-reply
0 siblings, 0 replies; 33+ messages in thread
From: no-reply @ 2019-11-05 17:52 UTC (permalink / raw)
To: laurent
Cc: peter.maydell, sstabellini, berrange, ehabkost, paul,
qemu-trivial, mjt, claudio.fontana, qemu-devel, chouteau, laurent,
marcandre.lureau, frederic.konrad, qemu-arm, qemu-ppc, clg, crosa,
anthony.perard, pbonzini, david, xen-devel
Patchew URL: https://patchew.org/QEMU/20191105144247.10301-1-laurent@vivier.eu/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Subject: [PULL 0/4] Trivial branch patches
Type: series
Message-id: 20191105144247.10301-1-laurent@vivier.eu
=== TEST SCRIPT BEGIN ===
#!/bin/bash
git rev-parse base > /dev/null || exit 0
git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram
./scripts/checkpatch.pl --mailback base..
=== TEST SCRIPT END ===
Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
Switched to a new branch 'test'
0102c79 global: Squash 'the the'
9c583b0 qom: Fix error message in object_class_property_add()
299eefd hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
c931231 hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
=== OUTPUT BEGIN ===
1/4 Checking commit c93123187df2 (hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers)
2/4 Checking commit 299eefd37522 (hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses)
3/4 Checking commit 9c583b04fdb1 (qom: Fix error message in object_class_property_add())
WARNING: line over 80 characters
#31: FILE: qom/object.c:1109:
+ error_setg(errp, "attempt to add duplicate property '%s' to object (type '%s')",
WARNING: line over 80 characters
#43: FILE: qom/object.c:1141:
+ error_setg(errp, "attempt to add duplicate property '%s' to class (type '%s')",
total: 0 errors, 2 warnings, 22 lines checked
Patch 3/4 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
4/4 Checking commit 0102c79a3068 (global: Squash 'the the')
ERROR: do not use C99 // comments
#26: FILE: disas/libvixl/vixl/invalset.h:105:
+ // Note that this does not mean the backing storage is empty: it can still
total: 1 errors, 0 warnings, 56 lines checked
Patch 4/4 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
=== OUTPUT END ===
Test command exited with code: 1
The full log is available at
http://patchew.org/logs/20191105144247.10301-1-laurent@vivier.eu/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: [PULL 0/4] Trivial branch patches
@ 2019-11-05 17:52 ` no-reply
0 siblings, 0 replies; 33+ messages in thread
From: no-reply @ 2019-11-05 17:52 UTC (permalink / raw)
To: laurent
Cc: peter.maydell, sstabellini, berrange, ehabkost, paul,
qemu-trivial, mjt, claudio.fontana, qemu-devel, chouteau, laurent,
marcandre.lureau, frederic.konrad, qemu-arm, qemu-ppc, clg, crosa,
anthony.perard, pbonzini, david, xen-devel
Patchew URL: https://patchew.org/QEMU/20191105144247.10301-1-laurent@vivier.eu/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Subject: [PULL 0/4] Trivial branch patches
Type: series
Message-id: 20191105144247.10301-1-laurent@vivier.eu
=== TEST SCRIPT BEGIN ===
#!/bin/bash
git rev-parse base > /dev/null || exit 0
git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram
./scripts/checkpatch.pl --mailback base..
=== TEST SCRIPT END ===
Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
Switched to a new branch 'test'
0102c79 global: Squash 'the the'
9c583b0 qom: Fix error message in object_class_property_add()
299eefd hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses
c931231 hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers
=== OUTPUT BEGIN ===
1/4 Checking commit c93123187df2 (hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to PnP registers)
2/4 Checking commit 299eefd37522 (hw/misc/grlib_ahb_apb_pnp: Fix 8-bit accesses)
3/4 Checking commit 9c583b04fdb1 (qom: Fix error message in object_class_property_add())
WARNING: line over 80 characters
#31: FILE: qom/object.c:1109:
+ error_setg(errp, "attempt to add duplicate property '%s' to object (type '%s')",
WARNING: line over 80 characters
#43: FILE: qom/object.c:1141:
+ error_setg(errp, "attempt to add duplicate property '%s' to class (type '%s')",
total: 0 errors, 2 warnings, 22 lines checked
Patch 3/4 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
4/4 Checking commit 0102c79a3068 (global: Squash 'the the')
ERROR: do not use C99 // comments
#26: FILE: disas/libvixl/vixl/invalset.h:105:
+ // Note that this does not mean the backing storage is empty: it can still
total: 1 errors, 0 warnings, 56 lines checked
Patch 4/4 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
=== OUTPUT END ===
Test command exited with code: 1
The full log is available at
http://patchew.org/logs/20191105144247.10301-1-laurent@vivier.eu/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
^ permalink raw reply [flat|nested] 33+ messages in thread