Hi Baoquan, I am using a list here to store all the mapped addresses, and unmap them out of iounmap. About the reason, please check the old mails. I cannot remember the detailed reasons. Thanks Zhenhua On 05/13/2015 05:00 PM, Baoquan He wrote: > On 05/11/15 at 05:52pm, Li, Zhen-Hua wrote: >> Add some functions to copy the data from old kernel. >> These functions are used to copy context tables and page tables. >> >> To avoid calling iounmap between spin_lock_irqsave and spin_unlock_irqrestore, >> use a link here, store the pointers , and then use iounmap to free them in >> another place. > > Hi Zhenhua, > > I remember you mentioned iounmap will cause error inside > spin_lock_irqsave. Do you know why it happened now? And could you also > describe why avoid calling iounmap between > spin_lock_irqsave/unlock_irqsave is needed here and what's the status > now? > > I think other reviewer may want to know it too. > > Thanks > Baoquan >