On Mon, Nov 22, 2021 at 9:24 AM Antoine Tenart <atenart@xxxxxxxxxx> wrote: > > When checking an address is located in a global data section also check > for the .bss section as global variables initialized to 0 can be in > there (-fzero-initialized-in-bss). > > This was found when looking at ensure_safe_net_sysctl which was failing > to detect non-init sysctl pointing to a global data section when the > data was in the .bss section. > > Signed-off-by: Antoine Tenart <atenart@xxxxxxxxxx> > Acked-by: Steven Rostedt (VMware) <rostedt@xxxxxxxxxxx> Co-Developed-by: Jonathon Reinhart <jonathon.reinhart@xxxxxxxxx> Signed-off-by: Jonathon Reinhart <jonathon.reinhart@xxxxxxxxx> > --- > > A few remarks: > > - This still targets net-next but I added Arnd if he prefers to take it > through the 'asm-generic' tree, now that is_kernel_core_data is in > include/asm-generic/. > > - I kept the Acked-by tag as the change is the same really, the > difference is the core_kernel_data function was renamed to > is_kernel_core_data and moved since then. > > - @Jonathon: with your analysis and suggestion I think you should be > listed as a co-developer. If that's fine please say so, and reply > with both a Co-developed-by and a Signed-off-by tags. Added, thanks. Although it appears I may have missed the boat.