Wrap mutable states of VCpu with SpinLock (partially done)

[efenniht]

• Closes Use SpinLock instead of RawSpinLock #20.
  • VCpu의 interrupts 에 대한 락을 명시적으로 타입으로 나타냅니다.
  • 그 이외의 내부 상태 fields 를 묶어서 inner 로 만들고 기존의 execution_lock의 락을 가지도록 합니다.
  • Cpu의 내부 상태를 묶어서 기존의 락을 가지도록 합니다.
• Global unsafe functions into safe methods (API) #17 의 일부입니다.
  • Interrupts 관련 코드를 Interrupts에 넣습니다.
  • api_switch_to_primary 를 바꾸어서 더 정리할 수 있을 듯 합니다.
  • Mailbox 관련 함수도 따로 또 정리해야 합니다..
• Closes A bug in SpinLockGuard::into_raw #37.
  • 제 생각으로는 concurrent_save_store 가 느려졌던 문제(A test got slower after api_vcpu_prepare_run was ported #10) 의 실제 원인이 이것인 걸로 보입니다.
• Closes enable_and_pending_count is not locked but accessed #26.
• 테스트를 모두 통과합니다.

이 PR은 아직 완성된 것은 아닙니다. 개인적인 생각으로는,

• Cpu의 is_on 그리고 lock이 굉장히 수상합니다. is_on을 수정할 때만 락을 거는 게 아니라 실제로 PSCI에 명령을 주고 완료될 때까지 걸려있어야 할 것 같아요. 별로 좋은 생각이 아니었습니다.
• VCpuLockedPair 라는 이름과 그 방식이 너무 구려요. 더 좋은 방법을 생각해야 합니다.
• VCpuInner는 락이 걸리면서 그 내부가 invalidate됩니다. 타입으로 나타내면 좋을 듯...? AtomicOption 따위가 없을까요?
• VCpuInner의 락 (execution_lock)은 다른 하프늄의 락과 달리 하프늄을 벗어날 때 락이 걸려 있습니다. 지금은 문제가 없지만 한 VCPU가 Hafnium을 통해 다른 VCPU를 접근할 때는 매우 조심해야 합니다.
  • 다른 VCPU의 락을 걸지 못합니다. 기본적으로 락을 거는 행위가 실행 중을 나타내는 시멘틱이기도 하고, 락을 걸다가 deadlock이 터질 수도 있습니다 (lock ordering 규칙을 만족하기 어려워요)
  • 그래서 접근하려는 다른 VcpuInner에 대한 락이 안 걸려 있는 지 수동으로 체크해야 합니다.

[jeehoonkang]

• 일단 코멘트 남깁니다.
• 이 PR은 제가 마무리할게요. 애써주셔서 감사합니다!

[efenniht]

@jeehoonkang 제 생각이 따로 있는 것들에 대해서는 모두 댓글을 달았습니다. 감사합니다 (__)