On Sun, Jun 26, 2022 at 7:51 AM Souptick Joarder <jrdr.linux@xxxxxxxxx> wrote:
>
> From: "Souptick Joarder (HPE)" <jrdr.linux@xxxxxxxxx>
>
> kernel test robot throws below warning ->
>
> arch/ia64/include/asm/mmu_context.h: In function 'reload_context':
> arch/ia64/include/asm/mmu_context.h:127:48: warning: variable
> 'old_rr4' set but not used [-Wunused-but-set-variable]
> 127 | unsigned long rr0, rr1, rr2, rr3, rr4, old_rr4;
>
> Added it under CONFIG_HUGETLB_PAGE
Any comments on this patch ?
>
> Reported-by: kernel test robot <lkp@xxxxxxxxx>
> Signed-off-by: Souptick Joarder (HPE) <jrdr.linux@xxxxxxxxx>
> ---
> arch/ia64/include/asm/mmu_context.h | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/arch/ia64/include/asm/mmu_context.h b/arch/ia64/include/asm/mmu_context.h
> index 87a0d5bc11ef..06257e355d00 100644
> --- a/arch/ia64/include/asm/mmu_context.h
> +++ b/arch/ia64/include/asm/mmu_context.h
> @@ -124,9 +124,12 @@ reload_context (nv_mm_context_t context)
> {
> unsigned long rid;
> unsigned long rid_incr = 0;
> - unsigned long rr0, rr1, rr2, rr3, rr4, old_rr4;
> + unsigned long rr0, rr1, rr2, rr3, rr4;
>
> +#ifdef CONFIG_HUGETLB_PAGE
> + unsigned long old_rr4;
> old_rr4 = ia64_get_rr(RGN_BASE(RGN_HPAGE));
> +#endif
> rid = context << 3; /* make space for encoding the region number */
> rid_incr = 1 << 8;
>
> --
> 2.25.1
>
|