Yes, thank you. I created another patch where I am suppressing it. (void)
err. That should be better i hope

Thank you.



On Fri, Jun 20, 2025 at 10:07 PM Damien Zammit <dam...@zamaudio.com> wrote:

> As discussed, this patch is not correct.
>
> Thanks,
> Damien
>
> On 6/19/25 2:56 PM, Milos Nikic wrote:
> > The variable  was assigned but never used, and was marked volatile
> > unnecessarily. This patch removes the dead assignment and declaration,
> and one compiler warning.
> > ---
> >   i386/i386/smp.c | 2 --
> >   1 file changed, 2 deletions(-)
> >
> > diff --git a/i386/i386/smp.c b/i386/i386/smp.c
> > index e3e4cc82..cf390ace 100644
> > --- a/i386/i386/smp.c
> > +++ b/i386/i386/smp.c
> > @@ -129,11 +129,9 @@ static int
> >   smp_send_ipi_startup_twice(int bsp_apic_id, int vector)
> >   {
> >       int i, accept_err, send_err;
> > -    volatile int err;
> >
> >       for (i = 0; i < 2; i++) {
> >           lapic->error_status.r = 0;
> > -        err = lapic->error_status.r;
> >
> >           /* StartUp IPI:
> >            *
> > --
> > 2.40.1
> >
> >
>
>

Reply via email to