On 08/13/15 at 09:55am, Simon Horman wrote: > On Tue, Jul 28, 2015 at 12:46:42PM +0800, Minfei Huang wrote: > > Transforming PFN(Page Frame Number) to struct page is never failure, so > > we can simplify the code logic to do the image->control_page assignment > > directly in the loop, and remove the unnecessary conditional judgement. > > > > Signed-off-by: Minfei Huang <mnfhuang at gmail.com> > > Acked-by: Dave Young <dyoung at redhat.com> > > Acked-by: Vivek Goyal <vgoyal at redhat.com> > > Andrew, could you consider picking this up. > It seems to have been sufficiently reviewed, acked, etc... I saw this has been in linux-next tree. commit b90b6ef0f152ef42fe66ce5c9ccf2783ea84fa69 Author: Minfei Huang <mnfhuang at gmail.com> Date: Thu Jul 30 09:56:18 2015 +1000 kexec: remove unnecessary test in kimage_alloc_crash_control_pages() Thanks Baoquan