Re: [PATCH 1/2] qxl-wddm-dod: fix behavior on v5 device

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

 





On Mon, Jul 13, 2020 at 12:12 PM Frediano Ziglio <fziglio@xxxxxxxxxx> wrote:
>
> Hi,
>
> > @@ -4795,7 +4796,8 @@ NTSTATUS QxlDevice::HWClose(void)
> >  {
> >      PAGED_CODE();
> >      QxlClose();
> > -    if (m_bUefiMode)
> > +    /* QXL device rev 5+ requires explicit reset to switch to VGA mode */
> > +    if (m_bUefiMode || m_pQxlDod->Revision() > 4)
> >      {
> >          DbgPrint(TRACE_LEVEL_INFORMATION, ("%s: Resetting the device\n",
> >          __FUNCTION__));
> >          WRITE_PORT_UCHAR((PUCHAR)(m_IoBase + QXL_IO_RESET), 0);
>
> I'm wondering why this is conditional in the first place?
> Isn't it a good idea to reset the device on close no matter what?

The intention was to leave the entire procedure as is where possible.
For example, when we do not use the reset, user screen resize does not happen until the BDD starts programming the VGA.

>
> take care,
>   Gerd
>

Surely it's less willing to cause possible regressions.

Frediano

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]