You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by ac...@apache.org on 2026/01/20 14:49:45 UTC

(nuttx) branch master updated: x64_64: it is not necessary to call restore_critical_section in irq

This is an automated email from the ASF dual-hosted git repository.

acassis pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/nuttx.git


The following commit(s) were added to refs/heads/master by this push:
     new 7ee3890e7a0 x64_64: it is not necessary to call restore_critical_section in irq
7ee3890e7a0 is described below

commit 7ee3890e7a09eb2e448af09606923b5ee904d65f
Author: hujun5 <hu...@xiaomi.com>
AuthorDate: Mon Jul 14 18:43:01 2025 +0800

    x64_64: it is not necessary to call restore_critical_section in irq
    
    Remove restore_critical_section() call from common_handler() in x86_64 interrupt
    processing as it is not necessary during interrupt handling. The critical section
    state is properly managed by the interrupt framework elsewhere.
    
    Signed-off-by: hujun5 <hu...@xiaomi.com>
---
 arch/x86_64/src/intel64/intel64_handlers.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/arch/x86_64/src/intel64/intel64_handlers.c b/arch/x86_64/src/intel64/intel64_handlers.c
index 93a536c8c0d..c224ad29c2d 100644
--- a/arch/x86_64/src/intel64/intel64_handlers.c
+++ b/arch/x86_64/src/intel64/intel64_handlers.c
@@ -115,10 +115,6 @@ static uint64_t *common_handler(int irq, uint64_t *regs)
        */
 
       *running_task = tcb;
-
-      /* Restore the cpu lock */
-
-      restore_critical_section(tcb, this_cpu());
     }
 
   /* Clear irq flag */