> to a crash when we access vm_mm(its offset is 0x40) later in Would the text variant “… vm_mm (its …” be a bit nicer? … > +++ b/mm/ksm.c > @@ -2112,8 +2112,15 @@ static void cmp_and_merge_page(struct page *page, struct rmap_item *rmap_item) … > + if (vma) > + err = try_to_merge_one_page(vma, page, > + ZERO_PAGE(rmap_item->address)); Can the parameter alignment trigger further software development considerations for such a function call? Regards, Markus