Re: [Qemu-devel] [PATCH 3/3] disas/arm: avoid clang shifting negative signed warning

2015-11-10 Thread Steven Noonan
On Tue, Nov 10, 2015 at 12:06 PM, Markus Armbruster wrote: > Steven Noonan writes: > >> On Tue, Nov 10, 2015 at 10:52 AM, Markus Armbruster >> wrote: >>> Peter Maydell writes: >>> >>>> On 10 November 2015 at 17:33, Paolo Bonzini wrote: >&g

Re: [Qemu-devel] [PATCH 3/3] disas/arm: avoid clang shifting negative signed warning

2015-11-10 Thread Steven Noonan
On Tue, Nov 10, 2015 at 10:52 AM, Markus Armbruster wrote: > Peter Maydell writes: > >> On 10 November 2015 at 17:33, Paolo Bonzini wrote: >>> >>> >>> On 10/11/2015 16:57, Stefan Hajnoczi wrote: clang 3.7.0 on x86_64 warns about the following: disas/arm.c:1782:17: warning: shift

[Qemu-devel] [PATCH v3] configure: add option to disable -fstack-protector flags

2014-01-13 Thread Steven Noonan
option to disable the stack protector entirely, and conditional stack protector flag selection (in order, based on availability): -fstack-protector-strong, -fstack-protector, no stack protector. Signed-off-by: Steven Noonan Cc: Anthony Liguori Reviewed-by: Stefan Weil --- configure | 18

[Qemu-devel] [PATCH] configure: add option to disable -fstack-protector flags

2014-01-13 Thread Steven Noonan
option to disable the stack protector entirely, and conditional stack protector flag selection (in order, based on availability): -fstack-protector-strong, -fstack-protector, no stack protector. Signed-off-by: Steven Noonan --- configure | 18 +++--- 1 file changed, 15 insertions

[Qemu-devel] [PATCH 1/3] configure: add option to disable -fstack-protector flags

2014-01-09 Thread Steven Noonan
From: Steven Noonan The -fstack-protector flag family is useful for ensuring safety and for debugging, but has a performance impact. Here's a boot time comparison between a QEMU build of qemu-system-arm with and without the -fstack-protector-all flag: # WITHOUT -fstack-protecto

[Qemu-devel] [PATCH 2/3] virtfs-proxy-helper.c: fix compile error

2014-01-09 Thread Steven Noonan
From: Steven Noonan This is caused by a linux/xattr.h and sys/xattr.h incompatibility: In file included from /home/snoonan/Development/qemu/include/qemu/xattr.h:27:0, from fsdev/virtfs-proxy-helper.c:25: /usr/include/sys/xattr.h:31:3: error: expected identifier

[Qemu-devel] [PATCH 3/3] Makefile: add bios-256k.bin to BLOBS

2014-01-09 Thread Steven Noonan
From: Steven Noonan Signed-off-by: Steven Noonan Cc: Anthony Liguori --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index bdff4e4..807054b 100644 --- a/Makefile +++ b/Makefile @@ -290,7 +290,7 @@ common de-ch es fo fr-ca hu ja