Hi,
> > I can see that it simplifies the logic in spice-server if we have a
> > single function call instead of two. So we could deprecate
> > spice_qxl_set_max_monitors() in favour of your
> > spice_qxl_set_device_info() variant.
> >
> > spice_qxl_set_max_monitors() would then basically do
>
> On Mon, 2018-11-05 at 14:08 +0100, Gerd Hoffmann wrote:
> > On Mon, Nov 05, 2018 at 01:18:57PM +0100, Lukáš Hrázký wrote:
> > > On Mon, 2018-11-05 at 07:52 +0100, Gerd Hoffmann wrote:
> > > > > 2. Have a single function as follows:
> > > > >
> > > > > void
On Mon, 2018-11-05 at 14:08 +0100, Gerd Hoffmann wrote:
> On Mon, Nov 05, 2018 at 01:18:57PM +0100, Lukáš Hrázký wrote:
> > On Mon, 2018-11-05 at 07:52 +0100, Gerd Hoffmann wrote:
> > > > 2. Have a single function as follows:
> > > >
> > > > void spice_qxl_set_device_info(QXLInstance *instance,
>
On Mon, Nov 05, 2018 at 01:18:57PM +0100, Lukáš Hrázký wrote:
> On Mon, 2018-11-05 at 07:52 +0100, Gerd Hoffmann wrote:
> > > 2. Have a single function as follows:
> > >
> > > void spice_qxl_set_device_info(QXLInstance *instance,
> > >const char *device_address,
>
> On Mon, 2018-11-05 at 07:52 +0100, Gerd Hoffmann wrote:
> > > 2. Have a single function as follows:
> > >
> > > void spice_qxl_set_device_info(QXLInstance *instance,
> > >const char *device_address,
> > >uint32_t
On Mon, 2018-11-05 at 07:52 +0100, Gerd Hoffmann wrote:
> > 2. Have a single function as follows:
> >
> > void spice_qxl_set_device_info(QXLInstance *instance,
> >const char *device_address,
> >uint32_t device_display_id_start,
> >
On Mon, Nov 05, 2018 at 03:46:37AM -0500, Frediano Ziglio wrote:
> >
> > > 2. Have a single function as follows:
> > >
> > > void spice_qxl_set_device_info(QXLInstance *instance,
> > >const char *device_address,
> > >uint32_t
On Mon, 2018-11-05 at 03:46 -0500, Frediano Ziglio wrote:
> >
> > > 2. Have a single function as follows:
> > >
> > > void spice_qxl_set_device_info(QXLInstance *instance,
> > >const char *device_address,
> > >uint32_t
>
> > 2. Have a single function as follows:
> >
> > void spice_qxl_set_device_info(QXLInstance *instance,
> >const char *device_address,
> >uint32_t device_display_id_start,
> >uint32_t
> 2. Have a single function as follows:
>
> void spice_qxl_set_device_info(QXLInstance *instance,
>const char *device_address,
>uint32_t device_display_id_start,
>uint32_t device_display_id_count);
Hello,
On Mon, 2018-10-22 at 13:46 +0200, Lukáš Hrázký wrote:
> Hello,
>
> On Thu, 2018-10-18 at 03:16 -0400, Frediano Ziglio wrote:
> > >
> > > Adds two functions to let QEMU provide information to identify graphics
> > > devices and their monitors in the guest:
> > >
> > > * device address -
Hello,
On Thu, 2018-10-18 at 03:16 -0400, Frediano Ziglio wrote:
> >
> > Adds two functions to let QEMU provide information to identify graphics
> > devices and their monitors in the guest:
> >
> > * device address - The path identifying the device on the system (e.g. PCI
> > path):
> >
> > + * supported address at the moment, other identifiers can be introduced
> > later.
> > + * is the PCI domain, followed by . of any PCI
> > bridges
> > + * in the chain leading to the device. The last . is the
> > + * graphics device.
>
> Maybe better to specify also the encoding, like
>
> Adds two functions to let QEMU provide information to identify graphics
> devices and their monitors in the guest:
>
> * device address - The path identifying the device on the system (e.g. PCI
> path):
> spice_qxl_set_device_address(...)
>
> * device display ID - The index of the
14 matches
Mail list logo