Re: [Xen-devel] [PATCH 1/2 v3] xenfb: Use Input Handlers directly

[Stefano Stabellini <sstabellini@xxxxxxxxxx>]

Ping?

On Thu, 7 Sep 2017, Anthony PERARD wrote:
> On Mon, Aug 21, 2017 at 04:12:27PM -0700, Stefano Stabellini wrote:
> > Anthony,
> > 
> > The code looks good. I tested this patch with Linux guests and seems to
> > work OK, can you also confirm?
> 
> I've tested with Linux as well, an HVM guess, I did not spot any issue.
> 
> But, the code compiles with warnings...
> 
> In file included from 
> /root/build/qemu/include/standard-headers/linux/input.h:16:0,
>                  from hw/display/xenfb.c:41:
> /root/build/qemu/include/standard-headers/linux/input-event-codes.h:88:0: 
> error: "KEY_BACKSPACE" redefined [-Werror]
>  #define KEY_BACKSPACE  14
>  
> In file included from /root/build/qemu/include/ui/console.h:342:0,
>                  from hw/display/xenfb.c:31:
> /usr/include/curses.h:1494:0: note: this is the location of the previous 
> definition
>  #define KEY_BACKSPACE 0407  /* backspace key */
>  
> In file included from 
> /root/build/qemu/include/standard-headers/linux/input.h:16:0,
>                  from hw/display/xenfb.c:41:
> /root/build/qemu/include/standard-headers/linux/input-event-codes.h:102:0: 
> error: "KEY_ENTER" redefined [-Werror]
>  #define KEY_ENTER  28
>  
> In file included from /root/build/qemu/include/ui/console.h:342:0,
>                  from hw/display/xenfb.c:31:
> /usr/include/curses.h:1512:0: note: this is the location of the previous 
> definition
>  #define KEY_ENTER 0527  /* enter/send key */
>  
> In file included from 
> /root/build/qemu/include/standard-headers/linux/input.h:16:0,
>                  from hw/display/xenfb.c:41:
> /root/build/qemu/include/standard-headers/linux/input-event-codes.h:107:0: 
> error: "KEY_F" redefined [-Werror]
>  #define KEY_F   33
>  
> In file included from /root/build/qemu/include/ui/console.h:342:0,
>                  from hw/display/xenfb.c:31:
> /usr/include/curses.h:1496:0: note: this is the location of the previous 
> definition
>  #define KEY_F(n) (KEY_F0+(n)) /* Value of function key n */
> [...]
> 
> And a lot more...
> 
> -- 
> Anthony PERARD
> 

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel