Re: [PATCH 2/3] mm: provide mapping_wrprotect_page() function

2025-02-04 Thread Simona Vetter
On Mon, Feb 03, 2025 at 04:30:04PM +, Lorenzo Stoakes wrote: > On Mon, Feb 03, 2025 at 04:49:34PM +0100, Simona Vetter wrote: > > On Fri, Jan 31, 2025 at 06:28:57PM +, Lorenzo Stoakes wrote: > > > in the fb_defio video driver, page dirty state is used to determine when > > > frame buffer pa

Re: [PATCH 2/3] mm: provide mapping_wrprotect_page() function

2025-02-04 Thread Thomas Zimmermann
Hi Am 04.02.25 um 06:36 schrieb Christoph Hellwig: Hi Simona, On Mon, Feb 03, 2025 at 04:49:34PM +0100, Simona Vetter wrote: Could we go one step further and entirely drop the struct page? Similar to unmap_mapping_range for VM_SPECIAL mappings, except it only updates the write protection. Th

Re: [PATCH 2/3] mm: provide mapping_wrprotect_page() function

2025-02-03 Thread Lorenzo Stoakes
On Mon, Feb 03, 2025 at 04:49:34PM +0100, Simona Vetter wrote: > On Fri, Jan 31, 2025 at 06:28:57PM +, Lorenzo Stoakes wrote: > > in the fb_defio video driver, page dirty state is used to determine when > > frame buffer pages have been changed, allowing for batched, deferred I/O to > > be perfo

Re: [PATCH 2/3] mm: provide mapping_wrprotect_page() function

2025-02-03 Thread Simona Vetter
On Fri, Jan 31, 2025 at 06:28:57PM +, Lorenzo Stoakes wrote: > in the fb_defio video driver, page dirty state is used to determine when > frame buffer pages have been changed, allowing for batched, deferred I/O to > be performed for efficiency. > > This implementation had only one means of doi

[PATCH 2/3] mm: provide mapping_wrprotect_page() function

2025-01-31 Thread Lorenzo Stoakes
in the fb_defio video driver, page dirty state is used to determine when frame buffer pages have been changed, allowing for batched, deferred I/O to be performed for efficiency. This implementation had only one means of doing so effectively - the use of the folio_mkclean() function. However, this