Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

armv7-a gic related update #13466

Merged
merged 6 commits into from
Sep 16, 2024
Merged

armv7-a gic related update #13466

merged 6 commits into from
Sep 16, 2024

Conversation

GUIDINGLI
Copy link
Contributor

Summary

armv7-a/r gic related update

    arm_gicv2: cpu 0 wait other cpu gic init done
    
    After move the SGI irq to group1, other cpu can't response the
    sgi request from cpu0 when its gic not initialized.
    So let cpu0 wait until all other cpus gic initialize done.
    armv7-a/r: check gic init wait done when using sgi
    
    In SMP mode, qemu/goldfish platform, cpu0 use up_cpu_start()
    to start others cpus.
    
    But in previous patch(mathion ahead), arm_gic_initialize() will
    wait others cpus start, so deadlocked!
    
    Resolve:
    Move the wait logic when use using sgi

Impact

armv7a gic

Testing

bes board & qemu

GUIDINGLI and others added 6 commits September 14, 2024 20:59
cause some platform will have their special way

Signed-off-by: ligd <liguiding1@xiaomi.com>
Signed-off-by: ligd <liguiding1@xiaomi.com>
After move the SGI irq to group1, other cpu can't response the
sgi request from cpu0 when its gic not initialized.
So let cpu0 wait until all other cpus gic initialize done.

Signed-off-by: Bowen Wang <wangbowen6@xiaomi.com>
Signed-off-by: Bowen Wang <wangbowen6@xiaomi.com>
In SMP mode, qemu/goldfish platform, cpu0 use up_cpu_start()
to start others cpus.

But in previous patch(mathion ahead), arm_gic_initialize() will
wait others cpus start, so deadlocked!

Resolve:
Move the wait logic when use using sgi

Signed-off-by: ligd <liguiding1@xiaomi.com>
Signed-off-by: hujun5 <hujun5@xiaomi.com>
@xiaoxiang781216 xiaoxiang781216 merged commit c16fd8c into apache:master Sep 16, 2024
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants