On Wed, 18 Aug 2021 at 09:48, Hao Peng <flyingpenghao@xxxxxxxxx> wrote: > > >On Tue, Aug 17, 2021 at 11:42 PM Miklos Szeredi <miklos@xxxxxxxxxx> wrote: >> >> >On Tue, 17 Aug 2021 at 05:17, Hao Peng <flyingpenghao@xxxxxxxxx> wrote: >> >> >> >> kmap_local_page() is enough. >> >> >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
