On (24/10/10 23:21), Pintu Kumar wrote: [..] > @@ -1511,10 +1501,10 @@ static void zs_object_copy(struct size_class *class, unsigned long dst, > d_size -= size; > > /* > - * Calling kunmap_atomic(d_addr) is necessary. kunmap_atomic() > - * calls must occurs in reverse order of calls to kmap_atomic(). > - * So, to call kunmap_atomic(s_addr) we should first call > - * kunmap_atomic(d_addr). For more details see > + * Calling kunmap_local(d_addr) is necessary. kunmap_local() > + * calls must occurs in reverse order of calls to kmap_local_page(). > + * So, to call kunmap_local(s_addr) we should first call > + * kunmap_local(d_addr). For more details see > * Documentation/mm/highmem.rst. > */ I'd prefer this entire comment to be dropped.
