CR3より先にContext::cr3に値を設定しておくべき？

[uchan-nos]

mikanos/kernel/terminal.cpp

Lines 135 to 136 in 88b352c

	SetCR3(cr3);
	current_task.Context().cr3 = cr3;

SetCR3 の直後にタスクスイッチしてしまうと、戻ってきたときに current_task.Context().cr3 = 0 となっていて、
先ほど生成した PML4 ではなく、デフォルトの PML4 が CR3 に設定されてしまうのではないか？

[alpha-netzilla]

mikanos/kernel/terminal.cpp

Lines 142 to 143 in 88b352c

	current_task.Context().cr3 = 0;
	ResetCR3();

同様の問題は上のコードでは発生しないでしょうか？

1. current_task.Context().cr3 = 0
2. ResetCR3()
3. タスクスイッチ
4. タスクスイッチ戻り

[uchan-nos]

@alpha-netzilla
問題無いと思います。
タスクスイッチ（SwitchContext）の中で、そのとき CR3 に設定されていた値が Context 構造体の cr3 フィールドに保存されるからです。

mikanos/kernel/asmfunc.asm

Lines 132 to 133 in d4048a3

	mov rax, cr3
	mov [rsi + 0x00], rax ; CR3

同様に、私が書いた 2 行の処理も問題無い気がします。

ご意見ください。

[alpha-netzilla]

@uchan-nos
ご指摘ありがとうございます。SwitchContext のコードを読み、私も問題ないと判断しました。
"その時の CR3 レジスタの値が"、"Context 構造体の cr3 フィールドに"、ここがポイントですね。