Re: [PATCH v9 5/7] vfio: Define vfio based dma-buf operations

From	Gerd Hoffmann <kraxel@redhat.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v9 5/7] vfio: Define vfio based dma-buf operations
Date	2017-06-21 09:30 +0200
Message-ID	<tUIfn-34h-13@gated-at.bofh.it> (permalink)
References	(6 earlier) <tT2BA-2DV-11@gated-at.bofh.it> <tTYvT-7uN-3@gated-at.bofh.it> <tU6jO-3YD-61@gated-at.bofh.it> <tUmRA-6dE-9@gated-at.bofh.it> <tUrRg-Tl-13@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

  Hi,

> We don't support cursor for console vnc. Ideally console vnc should
> be
> used by admin for configuration or during maintenance, which refresh
> primary surface at low refresh rate, 10 fps.

But you surely want a mouse pointer for the admin?
You render it directly to the primary surface then I guess?

> Right we need to know this at device initialization time for both
> cases
> to initialize VGACommonState structure for that device

Why do you need a VGACommonState?

> and also need
> NONE to decide whether to init console vnc or not. We have a
> mechanism
> to disable console vnc path and we recommend to disable it for better
> performance.

Hmm, maybe we should have a ioctl to configure the refresh rate, or a
ioctl to allow qemu ask for a refresh when needed?

qemu can throttle the display update rate, which for example happens in
case no vnc client is connected.  qemu updates the display only once
every few seconds then.

cheers,
  Gerd

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH v9 5/7] vfio: Define vfio based dma-buf operations Gerd Hoffmann <kraxel@redhat.com> - 2017-06-20 10:40 +0200
  Re: [PATCH v9 5/7] vfio: Define vfio based dma-buf operations Kirti Wankhede <kwankhede@nvidia.com> - 2017-06-20 16:00 +0200
    Re: [PATCH v9 5/7] vfio: Define vfio based dma-buf operations Gerd Hoffmann <kraxel@redhat.com> - 2017-06-21 09:30 +0200

csiph-web