[Qemu-devel] [PATCH 0/4] xenfb: Add vkbd-only option

Owen Smith posted 4 patches 8 years, 4 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1496927734-29174-1-git-send-email-owen.smith@citrix.com
Test FreeBSD passed
Test checkpatch passed
Test docker passed
Test s390x passed
hw/display/xenfb.c   | 44 ++++++++++++++++++++++++++++++++------------
include/ui/console.h |  2 ++
ui/input-legacy.c    | 12 ++++++++++++
3 files changed, 46 insertions(+), 12 deletions(-)
[Qemu-devel] [PATCH 0/4] xenfb: Add vkbd-only option
Posted by Owen Smith 8 years, 4 months ago
Adds the ability for a vkbd device to connect without the
QemuConsole, in order to support a standalone PV mouse and
keyboard frontend.
This series adds a new feature flag, which will need adding
to the xen's include/public/io/kbdif.h
"feature-vkbd-standalone" is set to 1 by backends that allow 
the vkbd device model to connect without requiring a vfb device
connected. The vkbd device will only bypass the check for
the vfb device if the frontend sets "request-vkbd-standalone"
to 1.
The last 2 patches add a couple of missing input handler
functions, and uses these to remove a leak in the vkbd device
model.

Owen Smith (4):
  xenfb: Add feature-vkbd-standalone
  xenfb: Activate mouse handler
  ui/input: Add activate/remove for keyboard handlers
  xenfb: Fix leak by adding/removing keyboard handler

 hw/display/xenfb.c   | 44 ++++++++++++++++++++++++++++++++------------
 include/ui/console.h |  2 ++
 ui/input-legacy.c    | 12 ++++++++++++
 3 files changed, 46 insertions(+), 12 deletions(-)

-- 
2.1.4


Re: [Qemu-devel] [PATCH 0/4] xenfb: Add vkbd-only option
Posted by Stefano Stabellini 8 years, 4 months ago
On Thu, 8 Jun 2017, Owen Smith wrote:
> Adds the ability for a vkbd device to connect without the
> QemuConsole, in order to support a standalone PV mouse and
> keyboard frontend.
> This series adds a new feature flag, which will need adding
> to the xen's include/public/io/kbdif.h

Please do so, I would like that change to be applied to xen before this
series is applied to QEMU.


> "feature-vkbd-standalone" is set to 1 by backends that allow 
> the vkbd device model to connect without requiring a vfb device
> connected. The vkbd device will only bypass the check for
> the vfb device if the frontend sets "request-vkbd-standalone"
> to 1.
> The last 2 patches add a couple of missing input handler
> functions, and uses these to remove a leak in the vkbd device
> model.
> 
> Owen Smith (4):
>   xenfb: Add feature-vkbd-standalone
>   xenfb: Activate mouse handler
>   ui/input: Add activate/remove for keyboard handlers
>   xenfb: Fix leak by adding/removing keyboard handler
> 
>  hw/display/xenfb.c   | 44 ++++++++++++++++++++++++++++++++------------
>  include/ui/console.h |  2 ++
>  ui/input-legacy.c    | 12 ++++++++++++
>  3 files changed, 46 insertions(+), 12 deletions(-)
> 
> -- 
> 2.1.4
>