| Date | Tue, 5 May 2020 23:12:31 -0700 | From | Christoph Hellwig <> | Subject | Re: [PATCH V2 07/11] arch/kunmap_atomic: Consolidate duplicate code |
| |
On Sun, May 03, 2020 at 06:09:08PM -0700, ira.weiny@intel.com wrote: > From: Ira Weiny <ira.weiny@intel.com> > > Every single architecture (including !CONFIG_HIGHMEM) calls... > > pagefault_enable(); > preempt_enable(); > > ... before returning from __kunmap_atomic(). Lift this code into the > kunmap_atomic() macro. > > While we are at it rename __kunmap_atomic() to kunmap_atomic_high() to > be consistent. > > Signed-off-by: Ira Weiny <ira.weiny@intel.com>
Looks good,
Reviewed-by: Christoph Hellwig <hch@lst.de>
|