Issue/2, 4, 31, 32 코드 리뷰

[YoonsungNam]

변경사항

• PendingQueue관련 변경(Issue PendingQueue가 add_bg()나 add_fg()시에 올바른 짝을 찾지 못함 #2)
• CoreIsolator로 변경(Issue SchedIsolator의 변경 제안 (CoreIsolator) #31)
• CoreIsolator로직 및 ResourceType.CPU 추가 (Issue CoreIsolator(구 ScheduIsolator)의 우선 호출이 가능한 Policy 구현 및 테스트 제안 #32)
  => 테스트는 더 필요 (적정한 _CPU_THRESHOLD값 결정부분)
• 하드코딩된 부분 제거 (Issue 하드코딩 된 부분들 #4) => core.py위주로 제거 (memory.py나 cache.py의 하드코드 미제거)

[YoonsungNam]

추가 커밋 사항:

• ResourceType.CPU를 진입하기 위한 조건 수정 (상대값인 MetricDiff 에서 현재절대값인 BasicMetric으로 변경)
  • BasicMetric의 l3_intensity(LLC util x LLC HitRatio)와 mem_intensity(LLC util x LLC Miss Ratio)의 사용
• fg의 strengthen/weaken 을 위한 조건 수정 (fg의 코어 할당시, 소켓범위내 유휴 코어가 있을 때에만 할당하도록 조건수정) - bugfix

Todo (Ongoing):

• fg의 strengthen/weaken은 현재 MetricDiff의 IPC값(즉, IPC의 diff)을 보고 결정하지만, IPC의 diff > 0 이어서 strengthen을 해줘도 ipc_diff이 줄어들지 않고 이는 무한정의 strengthen을 호출 BG가 FG의 코어를 모두 뺏아가는 문제가 생겨버림
  • 해결방안 : 메트릭(IPC)의 변경 or Threshold의 변경

[YoonsungNam]

Commit 974f940 의 변경사항 (Issue #18 관련)

• SwapIsolator추가 (enforcing부분을 제외한 나머지 코드)
• SwapIsolator는 Isolator를 상속하는 코드가 아닌 독립적인 코드 (controller.py에 FIXME자리에 호출될 예정)

[YoonsungNam]

SwapIsolator 구현 추가하여 완료
TODO (Ongoing):

• Swap후 후 처리가 필요함 (e.g., 기존 워크로드에 적용되던 isolation setting 재조정)
• 테스팅이 필요함

[isac322]

#40 에서 연장해서 개발 완료