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

[Xen-changelog] [xen staging] x86/hvm: Adjust hvmemul_rep_stos() to compile with -Wvla



commit e70934f44ad63e5433756f785ab4bcb316f860f2
Author:     Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
AuthorDate: Mon Sep 17 16:32:32 2018 +0100
Commit:     Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
CommitDate: Tue Sep 25 11:59:32 2018 +0100

    x86/hvm: Adjust hvmemul_rep_stos() to compile with -Wvla
    
    When using -Wvla, the typecast of buf triggers a Variable Length Array
    warning.  This is less than ideal, as this typecast doesn't occupy any stack
    space, but we don't have a finer grain option to use.
    
    Alter the asm expression to avoid the typecast, which necessitates the
    introduction of a memory clobber as the compiler can no longer identify
    the total quantity of written memory.
    
    Despite the memory clobber, there is no change to the generated asm.
    
    Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
    Reviewed-by: Paul Durrant <paul.durrant@xxxxxxxxxx>
    Reviewed-by: Wei Liu <wei.liu2@xxxxxxxxxx>
---
 xen/arch/x86/hvm/emulate.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/xen/arch/x86/hvm/emulate.c b/xen/arch/x86/hvm/emulate.c
index a577685dc6..eab66eab77 100644
--- a/xen/arch/x86/hvm/emulate.c
+++ b/xen/arch/x86/hvm/emulate.c
@@ -1859,7 +1859,7 @@ static int hvmemul_rep_stos(
     switch ( p2mt )
     {
         unsigned long bytes;
-        void *buf;
+        char *buf;
 
     default:
         /* Allocate temporary buffer. */
@@ -1882,10 +1882,10 @@ static int hvmemul_rep_stos(
 #define CASE(bits, suffix)                                     \
             case (bits) / 8:                                   \
                 asm ( "rep stos" #suffix                       \
-                      : "=m" (*(char (*)[bytes])buf),          \
+                      : "=m" (*buf),                           \
                         "=D" (dummy), "=c" (dummy)             \
                       : "a" (*(const uint##bits##_t *)p_data), \
-                         "1" (buf), "2" (*reps) );             \
+                        "1" (buf), "2" (*reps) : "memory" );   \
                 break
             CASE(8, b);
             CASE(16, w);
--
generated by git-patchbot for /home/xen/git/xen.git#staging

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/xen-changelog

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.