Re: [Qemu-devel] [PATCH 2/2] thunk.h uses TARGET_ABI_BITS without including abitypes.h

2017-09-25 Thread Daniel Loffgren
> On Sep 25, 2017, at 9:10 AM, Eric Blake <ebl...@redhat.com> wrote: > > On 09/24/2017 08:02 PM, Daniel Loffgren wrote: >> >> Signed-off-by: Daniel Loffgren <dan...@loffgren.org >> <mailto:dan...@loffgren.org>> >> --- >> inclu

Re: [Qemu-devel] [PATCH 2/2] thunk.h uses TARGET_ABI_BITS without including abitypes.h

2017-09-25 Thread Daniel Loffgren
uth <th...@redhat.com> wrote: > > > Did this cause any trouble? ... one of two sentences in the patch > description would be nice, I think. > > Thomas > > > On 25.09.2017 03:02, Daniel Loffgren wrote: >> >> Signed-off-by: Daniel Loffgren <dan...@lo

Re: [Qemu-devel] [PATCH 1/2] fprintf-fn.h uses GCC_FMT, but doesn't include the definition of it

2017-09-25 Thread Daniel Loffgren
this change (I was fixing things in order of compiler failures). I dropped this commit from my branch and it didn’t break. So, feel free to disregard this. > On Sep 25, 2017, at 9:18 AM, Eric Blake <ebl...@redhat.com> wrote: > > On 09/24/2017 08:02 PM, Daniel Loffgren wrote: &g

[Qemu-devel] [PATCH 1/2] fprintf-fn.h uses GCC_FMT, but doesn't include the definition of it

2017-09-25 Thread Daniel Loffgren
Signed-off-by: Daniel Loffgren <dan...@loffgren.org> --- include/qemu/fprintf-fn.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/qemu/fprintf-fn.h b/include/qemu/fprintf-fn.h index 9068a960b3..80361d87bf 100644 --- a/include/qemu/fprintf-fn.h +++ b/include/qemu/fprint

[Qemu-devel] [PATCH 2/2] thunk.h uses TARGET_ABI_BITS without including abitypes.h

2017-09-25 Thread Daniel Loffgren
Signed-off-by: Daniel Loffgren <dan...@loffgren.org> --- include/exec/user/thunk.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/exec/user/thunk.h b/include/exec/user/thunk.h index f19ef4b230..12b5449d8c 100644 --- a/include/exec/user/thunk.h +++ b/include/exec/user/t