qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] libvixl: Fix format strings for several int64_t


From: Peter Maydell
Subject: Re: [Qemu-devel] [PATCH] libvixl: Fix format strings for several int64_t values
Date: Mon, 10 Mar 2014 14:20:55 +0000

On 7 March 2014 22:01, Andreas Färber <address@hidden> wrote:
> Am 07.03.2014 21:01, schrieb Stefan Weil:
>> Am 07.03.2014 20:42, schrieb Andreas Färber:
>>> Am 07.03.2014 20:15, schrieb Stefan Weil:
>>>> "%d" or "%x" won't work on hosts where int values are smaller than 64 bit.
>>>>
>>>> Signed-off-by: Stefan Weil <address@hidden>
>>>> ---
>>>>  disas/libvixl/a64/disasm-a64.cc |   20 ++++++++++----------
>>>>  1 file changed, 10 insertions(+), 10 deletions(-)
>>>
>>> Patch looks correct, but I see no indication that this is a backport
>>> from upstream libvixl. Have you submitted it there too?
>>
>> If I knew how to submit it to libvixl, I'd have done that already. I did
>> not find any hint (e-mail address, keyword contrib) in the sources from
>> github. Can you help me?
>
> Google led me to the following:
> https://github.com/armvixl/vixl
>
> address@hidden

This is the right email address and git repo, yes.
(our disas/libvixl/README gives the github URL, at least.)

I would encourage you to email them patches, it will
act as a prod for them to get themselves set up to
handle incoming contributions properly :-)

>> It would also be fine if someone just takes my
>> patch and applies it there.
>
> Hopefully Peter can help; I just remembered someone asking not to apply
> spelling fixes to not complicate functional backports.

I will apply this patch to target-arm.next. I'm happy
that we carry genuine bugfixes to libvixl (hopefully
in anticipation of them going away on a future update),
it's only non-functional changes I'd rather we don't
carry in our tree.

thanks
-- PMM



reply via email to

[Prev in Thread] Current Thread [Next in Thread]