On Thu, 21 Dec 2023 14:12:58 -0800, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> wrote: > On Thu, 21 Dec 2023 12:23:08 +0800 Yuntao Wang <ytcoode@xxxxxxxxx> wrote: > > > image->control_page represents the starting address for allocating the next > > control page, while hole_end represents the address of the last valid byte > > of the currently allocated control page. > > > > Therefore, after successfully allocating a control page, image->control_page > > should be updated to `hole_end + 1`, rather than hole_end. > > Thanks. Again, please include a description of the userspace-visible effects of the bug. This bug actually does not affect the correctness of allocating control pages, because image->control_page is currently only used in kimage_alloc_crash_control_pages(), and this function, when allocating control pages, will first align image->control_page up to the nearest `(1 << order) << PAGE_SHIFT` boundary, then use this value as the starting address of the next control page. This ensures that the newly allocated control page will use the correct starting address and not overlap with previously allocated control pages. Although it does not affect the correctness of the final result, it is better for us to set image->control_page to the correct value, in case it might be used elsewhere in the future, potentially causing errors. _______________________________________________ kexec mailing list kexec@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/kexec
