> From: Dexuan Cui > Sent: Monday, September 16, 2019 2:46 PM > To: Michael Kelley <mikel...@microsoft.com>; Wei Hu <w...@microsoft.com>; > b.zolnier...@samsung.com; linux-hyp...@vger.kernel.org; > dri-devel@lists.freedesktop.org; linux-fb...@vger.kernel.org; > linux-ker...@vger.kernel.org; Stephen Hemminger > <sthem...@microsoft.com>; sas...@kernel.org; Haiyang Zhang > <haiya...@microsoft.com>; KY Srinivasan <k...@microsoft.com> > Cc: Iouri Tarassov <iou...@microsoft.com> > Subject: RE: [PATCH v4] video: hyperv: hyperv_fb: Obtain screen resolution > from Hyper-V host > > > From: linux-hyperv-ow...@vger.kernel.org > > <linux-hyperv-ow...@vger.kernel.org> On Behalf Of Dexuan Cui > > Sent: Thursday, September 12, 2019 11:39 PM > > To: Michael Kelley <mikel...@microsoft.com>; Wei Hu > <w...@microsoft.com>; > > b.zolnier...@samsung.com; linux-hyp...@vger.kernel.org; > > dri-devel@lists.freedesktop.org; linux-fb...@vger.kernel.org; > > linux-ker...@vger.kernel.org; Stephen Hemminger > > <sthem...@microsoft.com>; sas...@kernel.org; Haiyang Zhang > > <haiya...@microsoft.com>; KY Srinivasan <k...@microsoft.com> > > Cc: Iouri Tarassov <iou...@microsoft.com> > > Subject: RE: [PATCH v4] video: hyperv: hyperv_fb: Obtain screen resolution > > from Hyper-V host > > > > > From: Michael Kelley <mikel...@microsoft.com> > > > Sent: Thursday, September 5, 2019 7:06 AM > > > > > > From: Wei Hu <w...@microsoft.com> Sent: Thursday, September 5, 2019 > > 2:12 > > > AM > > > > > > > > Beginning from Windows 10 RS5+, VM screen resolution is obtained from > > > host. > > > > The "video=hyperv_fb" boot time option is not needed, but still can be > > > > used to overwrite what the host specifies. The VM resolution on the host > > > > could be set by executing the powershell "set-vmvideo" command. > > > > > > > > Signed-off-by: Iouri Tarassov <iou...@microsoft.com> > > > > Signed-off-by: Wei Hu <w...@microsoft.com> > > > > --- > > > > v2: > > > > - Implemented fallback when version negotiation failed. > > > > - Defined full size for supported_resolution array. > > > > > > > > v3: > > > > - Corrected the synthvid major and minor version comparison > > problem. > > > > > > > > v4: > > > > - Changed function name to synthvid_ver_ge(). > > > > > > > > drivers/video/fbdev/hyperv_fb.c | 159 > > > +++++++++++++++++++++++++++++--- > > > > 1 file changed, 147 insertions(+), 12 deletions(-) > > > > > > > > > > Reviewed-by: Michael Kelley <mikel...@microsoft.com> > > > > Looks good to me. > > > > Reviewed-by: Dexuan Cui <de...@microsoft.com> > > Hi Wei, > It turns out we need to make a further fix. :-) > > The patch forgets to take par->update into consideration. > > When the VM Connection window is closed (or minimized?), > the host sends a message to the guest, and the guest sets > par->update to false in synthvid_recv_sub(). > > If par->update is false, the guest doesn't need to call > synthvid_update(). > > Thanks, > -- Dexuan
Please ignore the last reply from me. It was meant to reply another mail: RE: [PATCH v5] video: hyperv: hyperv_fb: Support deferred IO for Hyper-V frame buffer driver Sorry for the confusion. Thanks, -- Dexuan
