Re: [PATCH v3 3/4] fbdev/deferred-io: Support contiguous kernel memory framebuffers

2025-06-12 Thread Thomas Zimmermann
Hi Am 12.06.25 um 01:18 schrieb Michael Kelley: From: Michael Kelley Sent: Thursday, June 5, 2025 10:39 AM From: Thomas Zimmermann Sent: Thursday, June 5, 2025 8:36 AM Hi Am 04.06.25 um 23:43 schrieb Michael Kelley: [...] Nonetheless, there's an underlying issue. A main cause of the differe

RE: [PATCH v3 3/4] fbdev/deferred-io: Support contiguous kernel memory framebuffers

2025-06-11 Thread Michael Kelley
From: Michael Kelley Sent: Thursday, June 5, 2025 10:39 AM > > From: Thomas Zimmermann Sent: Thursday, June 5, 2025 > 8:36 AM > > > > Hi > > > > Am 04.06.25 um 23:43 schrieb Michael Kelley: > > [...] > > > Nonetheless, there's an underlying issue. A main cause of the difference > > > is the numbe

Re: [PATCH v3 3/4] fbdev/deferred-io: Support contiguous kernel memory framebuffers

2025-06-06 Thread Thomas Zimmermann
Hi Am 05.06.25 um 19:38 schrieb Michael Kelley: [...] I try to address the problem with the patches at https://lore.kernel.org/dri-devel/20250605152637.98493-1-tzimmerm...@suse.de/ Testing and feedback is much appreciated. Nice! I ran the same test case with your patches, and everything wor

RE: [PATCH v3 3/4] fbdev/deferred-io: Support contiguous kernel memory framebuffers

2025-06-05 Thread Michael Kelley
From: Thomas Zimmermann Sent: Thursday, June 5, 2025 8:36 AM > > Hi > > Am 04.06.25 um 23:43 schrieb Michael Kelley: > [...] > > Nonetheless, there's an underlying issue. A main cause of the difference > > is the number of messages to Hyper-V to update dirty regions. With > > hyperv_fb using de

Re: [PATCH v3 3/4] fbdev/deferred-io: Support contiguous kernel memory framebuffers

2025-06-05 Thread Thomas Zimmermann
Hi Am 04.06.25 um 23:43 schrieb Michael Kelley: [...] Nonetheless, there's an underlying issue. A main cause of the difference is the number of messages to Hyper-V to update dirty regions. With hyperv_fb using deferred I/O, the messages are limited 20/second, so the total number of messages to H

Re: [PATCH v3 3/4] fbdev/deferred-io: Support contiguous kernel memory framebuffers

2025-06-05 Thread David Hildenbrand
On 04.06.25 23:58, Michael Kelley wrote: From: Michael Kelley Sent: Tuesday, June 3, 2025 10:25 AM From: David Hildenbrand Sent: Tuesday, June 3, 2025 12:55 AM On 03.06.25 03:49, Michael Kelley wrote: From: David Hildenbrand Sent: Monday, June 2, 2025 2:48 AM [snip] @@ -182,20 +221,

Re: [PATCH v3 3/4] fbdev/deferred-io: Support contiguous kernel memory framebuffers

2025-06-05 Thread Thomas Zimmermann
Hi Am 04.06.25 um 23:43 schrieb Michael Kelley: From: Simona Vetter Sent: Wednesday, June 4, 2025 7:46 AM On Wed, Jun 04, 2025 at 10:12:45AM +0200, Thomas Zimmermann wrote: Hi Am 03.06.25 um 19:50 schrieb Michael Kelley: From: Thomas Zimmermann Sent: Monday, June 2, 2025 11:25 PM Hi Am

RE: [PATCH v3 3/4] fbdev/deferred-io: Support contiguous kernel memory framebuffers

2025-06-04 Thread Michael Kelley
From: Michael Kelley Sent: Tuesday, June 3, 2025 10:25 AM > > From: David Hildenbrand Sent: Tuesday, June 3, 2025 12:55 > AM > > > > On 03.06.25 03:49, Michael Kelley wrote: > > > From: David Hildenbrand Sent: Monday, June 2, 2025 > > > 2:48 AM > > >> [snip] > > >>> @@ -182,20 +221,34 @@ st

RE: [PATCH v3 3/4] fbdev/deferred-io: Support contiguous kernel memory framebuffers

2025-06-04 Thread Michael Kelley
From: Simona Vetter Sent: Wednesday, June 4, 2025 7:46 AM > > On Wed, Jun 04, 2025 at 10:12:45AM +0200, Thomas Zimmermann wrote: > > Hi > > > > Am 03.06.25 um 19:50 schrieb Michael Kelley: > > > From: Thomas Zimmermann Sent: Monday, June 2, 2025 > > > 11:25 PM > > > > Hi > > > > > > > > Am 03.

Re: [PATCH v3 3/4] fbdev/deferred-io: Support contiguous kernel memory framebuffers

2025-06-04 Thread Simona Vetter
On Wed, Jun 04, 2025 at 10:12:45AM +0200, Thomas Zimmermann wrote: > Hi > > Am 03.06.25 um 19:50 schrieb Michael Kelley: > > From: Thomas Zimmermann Sent: Monday, June 2, 2025 > > 11:25 PM > > > Hi > > > > > > Am 03.06.25 um 03:49 schrieb Michael Kelley: > > > [...] > > > > > Will the VMA have

Re: [PATCH v3 3/4] fbdev/deferred-io: Support contiguous kernel memory framebuffers

2025-06-04 Thread Thomas Zimmermann
Hi Am 03.06.25 um 19:50 schrieb Michael Kelley: From: Thomas Zimmermann Sent: Monday, June 2, 2025 11:25 PM Hi Am 03.06.25 um 03:49 schrieb Michael Kelley: [...] Will the VMA have VM_PFNMAP or VM_MIXEDMAP set? PFN_SPECIAL is a horrible hack. In another thread, you mention that you use PFN_

RE: [PATCH v3 3/4] fbdev/deferred-io: Support contiguous kernel memory framebuffers

2025-06-03 Thread Michael Kelley
From: Thomas Zimmermann Sent: Monday, June 2, 2025 11:25 PM > > Hi > > Am 03.06.25 um 03:49 schrieb Michael Kelley: > [...] > >> Will the VMA have VM_PFNMAP or VM_MIXEDMAP set? PFN_SPECIAL is a > >> horrible hack. > >> > >> In another thread, you mention that you use PFN_SPECIAL to bypass the >

RE: [PATCH v3 3/4] fbdev/deferred-io: Support contiguous kernel memory framebuffers

2025-06-03 Thread Michael Kelley
From: David Hildenbrand Sent: Tuesday, June 3, 2025 12:55 AM > > On 03.06.25 03:49, Michael Kelley wrote: > > From: David Hildenbrand Sent: Monday, June 2, 2025 2:48 > > AM > >> > >> On 23.05.25 18:15, mhkelle...@gmail.com wrote: > >>> From: Michael Kelley > >>> > >>> Current defio code works

Re: [PATCH v3 3/4] fbdev/deferred-io: Support contiguous kernel memory framebuffers

2025-06-03 Thread David Hildenbrand
On 03.06.25 03:49, Michael Kelley wrote: From: David Hildenbrand Sent: Monday, June 2, 2025 2:48 AM On 23.05.25 18:15, mhkelle...@gmail.com wrote: From: Michael Kelley Current defio code works only for framebuffer memory that is allocated with vmalloc(). The code assumes that the underlying

Re: [PATCH v3 3/4] fbdev/deferred-io: Support contiguous kernel memory framebuffers

2025-06-02 Thread Thomas Zimmermann
Hi Am 03.06.25 um 03:49 schrieb Michael Kelley: [...] Will the VMA have VM_PFNMAP or VM_MIXEDMAP set? PFN_SPECIAL is a horrible hack. In another thread, you mention that you use PFN_SPECIAL to bypass the check in vm_mixed_ok(), so VM_MIXEDMAP is likely not set? The VMA has VM_PFNMAP set, not V

RE: [PATCH v3 3/4] fbdev/deferred-io: Support contiguous kernel memory framebuffers

2025-06-02 Thread Michael Kelley
From: David Hildenbrand Sent: Monday, June 2, 2025 2:48 AM > > On 23.05.25 18:15, mhkelle...@gmail.com wrote: > > From: Michael Kelley > > > > Current defio code works only for framebuffer memory that is allocated > > with vmalloc(). The code assumes that the underlying page refcount can > > be

Re: [PATCH v3 3/4] fbdev/deferred-io: Support contiguous kernel memory framebuffers

2025-06-02 Thread David Hildenbrand
On 23.05.25 18:15, mhkelle...@gmail.com wrote: From: Michael Kelley Current defio code works only for framebuffer memory that is allocated with vmalloc(). The code assumes that the underlying page refcount can be used by the mm subsystem to manage each framebuffer page's lifecycle, including fr

Re: [PATCH v3 3/4] fbdev/deferred-io: Support contiguous kernel memory framebuffers

2025-05-26 Thread Christoph Hellwig
On Fri, May 23, 2025 at 09:15:21AM -0700, mhkelle...@gmail.com wrote: > Commit 37b4837959cb ("video: deferred io with physically contiguous > memory") from the year 2008 purported to add support for contiguous > kernel memory framebuffers. The motivating device, sh_mobile_lcdcfb, uses > dma_alloc_c

Re: [PATCH v3 3/4] fbdev/deferred-io: Support contiguous kernel memory framebuffers

2025-05-24 Thread kernel test robot
everything patch link: https://lore.kernel.org/r/20250523161522.409504-4-mhklinux%40outlook.com patch subject: [PATCH v3 3/4] fbdev/deferred-io: Support contiguous kernel memory framebuffers config: arm-randconfig-001-20250524 (https://download.01.org/0day-ci/archive/20250524/20250524155

[PATCH v3 3/4] fbdev/deferred-io: Support contiguous kernel memory framebuffers

2025-05-23 Thread mhkelley58
From: Michael Kelley Current defio code works only for framebuffer memory that is allocated with vmalloc(). The code assumes that the underlying page refcount can be used by the mm subsystem to manage each framebuffer page's lifecycle, including freeing the page if the refcount goes to 0. This ap