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 > > > > > >