On 01.08.2023 10:58, Nicola Vetrini wrote: > Rule 5.3 has the following headline: > "An identifier declared in an inner scope shall not hide an > identifier declared in an outer scope" > > The renames done by this patch avoid shadowing from happening. > They are as follows: > - s/socket_info/sock_info/ in 'do_write_psr_msrs' > > The parameter 'cpu_khz' that causes a violation in 'pit_init' > is unused, and hence can be removed. > > Parameter 'debug_stack_lines' in 'compat_show_guest_stack' is removed, > since the shadowed static variable has the same purpose. > > The declaration of 'init_e820' is renamed to be consistent with its > definition. > > Signed-off-by: Nicola Vetrini <nicola.vetr...@bugseng.com>
Acked-by: Jan Beulich <jbeul...@suse.com> > This patch is the non-controversial part of patch 1/4 from series > https://lore.kernel.org/xen-devel/cover.1690788513.git.nicola.vetr...@bugseng.com/ > The other part has been dealt with with a patch from Jan Beulich > (not yet committed at the time of writing) > https://lore.kernel.org/xen-devel/e9035197-b329-af2e-65ed-af31cd037...@suse.com/ (Just) naming this v2 (or whichever version would have come next) would have been nice. Jan