early_smp_init和process_init的顺序问题

yuyi2439 · 2024 年11 月 13 日 15:15

我在实现#1035时遇到如题的问题

我尝试在ProcessSchedulerInfo.inner_locked中加入cpu_mask，并设置初始值为smp_cpu_manager().possible_cpus()

但这里遇到了个问题：
在early_smp_init中，需要调用ProcessManager::idle_pcb()
在process_init中，需要调用ProcessSchedulerInfo::new(None)
由于我修改的代码，这两项操作互相依赖，我已有了一些改法，但想听听别人的意见

val213 · 2024 年11 月 15 日 08:10

或许先默认为None，SmpCpuManager::new()之后再初始化？

yuyi2439 · 2024 年11 月 15 日 14:34

现在的做法是
ProcessSchedulerInfo::new的参数on_cpu为None时，将cpu_mask设为空CpuMask

yuyi2439 · 2024 年11 月 18 日 15:30

github.com

DragonOS-Community/DragonOS/blob/b5b571e02693d91eb6918d3b7561e088c3e7ee81/kernel/src/mm/percpu.rs#L16


      
              libs::lazy_init::Lazy,
              smp::{
                  core::smp_get_processor_id,
                  cpu::{smp_cpu_manager, ProcessorId},
              },
          };
          
          /// 系统中的CPU数量
          ///
          /// todo: 待smp模块重构后，从smp模块获取CPU数量。
          /// 目前由于smp模块初始化时机较晚，导致大部分内核模块无法在早期初始化PerCpu变量。
          static CPU_NUM_ATOMIC: AtomicU32 = AtomicU32::new(PerCpu::MAX_CPU_NUM);
          
          #[derive(Debug)]
          pub struct PerCpu;
          
          impl PerCpu {
              #[cfg(target_arch = "x86_64")]
              pub const MAX_CPU_NUM: u32 = 128;
              #[cfg(target_arch = "riscv64")]
              pub const MAX_CPU_NUM: u32 = 64;

这里的意思应该就是说需要把smp初始化提前
我去研究研究