Re: [PATCH] video: hyperv_fb: Allow resolutions with size > 64 MB for Gen1

2022-05-19 Thread Helge Deller
On 5/18/22 20:48, Dexuan Cui wrote: >> From: Dexuan Cui >> Sent: Wednesday, May 4, 2022 10:05 AM >> To: Haiyang Zhang ; Wei Liu ; >>> ... >>> When I initially implemented this driver 10 years ago, I believe there >>> was smaller limit for the fb... But I think this patch is good for the >>> newer

RE: [PATCH] video: hyperv_fb: Allow resolutions with size > 64 MB for Gen1

2022-05-18 Thread Dexuan Cui
> From: Dexuan Cui > Sent: Wednesday, May 4, 2022 10:05 AM > To: Haiyang Zhang ; Wei Liu ; > > ... > > When I initially implemented this driver 10 years ago, I believe there > > was smaller limit for the fb... But I think this patch is good for the > > newer MMIO alloc scheme. I hope to see review

RE: [PATCH] video: hyperv_fb: Allow resolutions with size > 64 MB for Gen1

2022-05-04 Thread Dexuan Cui
> From: Haiyang Zhang > > > ... > > > - if (gen2vm) { > > > - pot_start = 0; > > > - pot_end = -1; > > > - } else { > > > - if (!(pci_resource_flags(pdev, 0) & IORESOURCE_MEM) || > > > - pci_resource_len(pdev, 0) < screen_fb_size) { > > > - pr_er

RE: [PATCH] video: hyperv_fb: Allow resolutions with size > 64 MB for Gen1

2022-05-04 Thread Haiyang Zhang
.kernel.org; linux- > fb...@vger.kernel.org; dri-devel@lists.freedesktop.org; linux- > ker...@vger.kernel.org > Subject: Re: [PATCH] video: hyperv_fb: Allow resolutions with size > 64 MB for > Gen1 > > On Wed, Apr 27, 2022 at 06:47:53AM -0700, Saurabh Sengar wrote: > >

Re: [PATCH] video: hyperv_fb: Allow resolutions with size > 64 MB for Gen1

2022-04-28 Thread Wei Liu
On Wed, Apr 27, 2022 at 06:47:53AM -0700, Saurabh Sengar wrote: > This patch fixes a bug where GEN1 VMs doesn't allow resolutions greater > than 64 MB size (eg 7680x4320). Unnecessary PCI check limits Gen1 VRAM > to legacy PCI BAR size only (ie 64MB). Thus any, resolution requesting > greater then

Re: [PATCH] video: hyperv_fb: Allow resolutions with size > 64 MB for Gen1

2022-04-27 Thread Helge Deller
On 4/27/22 15:47, Saurabh Sengar wrote: > This patch fixes a bug where GEN1 VMs doesn't allow resolutions greater > than 64 MB size (eg 7680x4320). Unnecessary PCI check limits Gen1 VRAM > to legacy PCI BAR size only (ie 64MB). Thus any, resolution requesting > greater then 64MB (eg 7680x4320) woul