Ticket #478 (closed defect: fixed)

Opened 3 years ago

Last modified 3 weeks ago

AcpiExecSingleCPU: Spinlock is not released

Reported by: erdmann Owned by: erdmann
Priority: major Milestone:
Component: ACPI PSD Version: 3.17
Keywords: Cc:

Description

src\acpi\psd\acpicpu.c, routine "AcpiExecSingleCPU":

around line: 802
if (ExeFlag? & SMPEXEC_IPITIME), a CPU spinlock is aquired, then the user function executed but the CPU spinlock is not released thereafter.
I believe the code should be corrected to read:

    if (ExeFlag & SMPEXEC_IPITIME)
    {
        while ( __lxchg((volatile int*)&CPULock, 1) );     // Wait if we are here
        CPUMask &= (~(1 << pPSDPLMA->procnum)); // Clear bit, this CPU was execute
        Status = Execute ( (void *)Context );                 // call function
        __lxchg((volatile int*)&CPULock, 0) ;                // give way to next CPU -> ADD THIS !
        return Status;
    }

Change History

comment:1 Changed 3 years ago by erdmann

  • Version changed from 3.16 to 3.17
  • Milestone changed from Release version 3.17 to eCS 2.x

comment:2 Changed 3 years ago by erdmann

  • Owner changed from eco to erdmann

comment:3 Changed 3 years ago by erdmann

Is now incorporated into trunk.

comment:4 Changed 3 years ago by erdmann

  • Status changed from new to closed
  • Resolution set to fixed

comment:5 Changed 3 weeks ago by dazarewicz

  • Milestone eCS 2.x deleted

Milestone eCS 2.x deleted

Note: See TracTickets for help on using tickets.