| Date | Fri, 1 May 2020 01:39:45 -0700 | From | Christoph Hellwig <> | Subject | Re: [PATCH V1 06/10] arch/kunmap_atomic: Consolidate duplicate code |
| |
On Thu, Apr 30, 2020 at 01:38:41PM -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>
|