include/exec/poison: Mark TARGET_FMT_lu as poisoned, too

[Qemu-devel] [PATCH] include/exec/poison: Mark TARGET_FMT_lu as poisoned, too

Thomas Huth posted 1 patch 6 years, 6 months ago

Download mbox

Test docker-mingw@fedora passed

Test asan passed

Test docker-clang@ubuntu passed

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190508150608.3311-1-thuth@redhat.com

include/exec/poison.h | 1 +
1 file changed, 1 insertion(+)

Expand all Fold all

[Qemu-devel] [PATCH] include/exec/poison: Mark TARGET_FMT_lu as poisoned, too

Posted by Thomas Huth 6 years, 6 months ago

We already poison TARGET_FMT_lx and TARGET_FMT_ld, but apparently
forgot to poison TARGET_FMT_lu, too. Do it now.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 include/exec/poison.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/include/exec/poison.h b/include/exec/poison.h
index 1a7a57baae..b862320fa6 100644
--- a/include/exec/poison.h
+++ b/include/exec/poison.h
@@ -44,6 +44,7 @@
 #pragma GCC poison TARGET_LONG_BITS
 #pragma GCC poison TARGET_FMT_lx
 #pragma GCC poison TARGET_FMT_ld
+#pragma GCC poison TARGET_FMT_lu
 
 #pragma GCC poison TARGET_PAGE_SIZE
 #pragma GCC poison TARGET_PAGE_MASK
-- 
2.21.0

Re: [Qemu-devel] [PATCH] include/exec/poison: Mark TARGET_FMT_lu as poisoned, too

Posted by Richard Henderson 6 years, 6 months ago

On 5/8/19 8:06 AM, Thomas Huth wrote:
> We already poison TARGET_FMT_lx and TARGET_FMT_ld, but apparently
> forgot to poison TARGET_FMT_lu, too. Do it now.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  include/exec/poison.h | 1 +
>  1 file changed, 1 insertion(+)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>


r~

Re: [Qemu-devel] [PATCH] include/exec/poison: Mark TARGET_FMT_lu as poisoned, too

Posted by Paolo Bonzini 6 years, 6 months ago

On 08/05/19 10:06, Thomas Huth wrote:
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  include/exec/poison.h | 1 +
>  1 file changed, 1 insertion(+)

Queued, thanks.

Paolo

Re: [Qemu-devel] [PATCH] include/exec/poison: Mark TARGET_FMT_lu as poisoned, too

Posted by Philippe Mathieu-Daudé 6 years, 6 months ago

On 5/9/19 11:10 AM, Paolo Bonzini wrote:
> On 08/05/19 10:06, Thomas Huth wrote:
>>
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> ---
>>  include/exec/poison.h | 1 +
>>  1 file changed, 1 insertion(+)
> 
> Queued, thanks.

Thomas already included it in his "qtests, copyright statement
clarifications and misc patches" pull request.