Thread (1 message) 1 message, 1 author, 2021-09-07

Re: [PATCH] fuse: Use kmap_local_page()

From: Miklos Szeredi <miklos@szeredi.hu>
Date: 2021-09-07 15:04:20

On Wed, 18 Aug 2021 at 09:48, Hao Peng [off-list ref] wrote:
quoted
On Tue, Aug 17, 2021 at 11:42 PM Miklos Szeredi [off-list ref] wrote:
quoted
On Tue, 17 Aug 2021 at 05:17, Hao Peng [off-list ref] wrote:
quoted
kmap_local_page() is enough.
quoted
This explanation is not enough for me to understand the patch.  Please
describe in more detail.
   Due to the introduction of kmap_local_*, the storage of slots used for short-term mapping has
   changed from per-CPU to per-thread. kmap_atomic() disable preemption, while kmap_local_*()
   only disable migration.
   There is no need to disable preemption in several kamp_atomic places used in fuse.
   The detailed introduction of kmap_local_*d can be found here:
   https://lwn.net/Articles/836144/

Can you please resubmit the patch with this explanation and without
tab/space corruption?

Thanks,
Miklos
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help