On 08/12/2016 01:06 PM, Kees Cook wrote: > On Fri, Aug 12, 2016 at 1:12 AM, Jisheng Zhang <[email protected]> wrote: >> vdso_data_mapping is never modified, so mark it as const. >> >> vdso_total_pages, vdso_data_page, vdso_text_mapping and cntvct_ok are >> initialized by vdso_init(), thereafter are read only. >> >> The fact that they are read only after init makes them candidates for >> __ro_after_init declarations. >> >> Signed-off-by: Jisheng Zhang <[email protected]> > > Looks great; thanks for keeping __ro_after_init in mind. :) > > Reviewed-by: Kees Cook <[email protected]>
Looks fine and nothing goes wrong with my vdso tests. Please send it through Russell. Acked-by: Nathan Lynch <[email protected]>
