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

OperatingSystem/seogeurim #38

Merged
merged 7 commits into from
May 25, 2021
Merged

OperatingSystem/seogeurim #38

merged 7 commits into from
May 25, 2021

Conversation

Seogeurim
Copy link
Owner

변경 사항

Issue 참고 : Note Status Check

  1. 프로세스 상태
  2. 스케줄링 알고리즘

Point of discussion

OS 관련 정리 보충해보았습니다 ! 앞으로도 계속 쭉쭉 할 예정인데 미리 올려봅니다 ㅎㅎ

Reference

공룡책 10판

@Seogeurim Seogeurim added the score NEED Score Board 업데이트 필요 label May 11, 2021
![image](materials/process-state-diagram.png)

- **New** : 프로세스가 생성됨
- **Running** : Instruction이 실행됨
Copy link
Collaborator

@Hee-Jae Hee-Jae May 16, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

다른 상태들은 '프로세스'로 설명됐는데 이 부분만 'Instruction'으로 설명이 되네요. '프로세스가 실행됨' 이라고 하는게 더 이해하기 쉬울 것 같아요.

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

공룡책에 이렇게 표시가 되어있는데, 그러면 '프로세스의 Instruction이 실행됨' 으로 하는 것이 더 이해하기 쉽고 정확하려나요? 아니면 상관 없이 그냥 '프로세스가 실행됨'으로 해도 될까요?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

다시 생각해보니 '프로세스가 실행됨' 이라고 하면 프로세스가 메모리에 적재된 이후 New 상태부터 Terminated 상태까지를 모두 통틀어 '프로세스가 실행되었다.' 라고 볼 수도 있겠네요. 공룡책에서는 정확히 프로세스 내의 Instruction들이 실행 됨을 알려주려고 저렇게 표기한 것 같아요. 독자분들에게 이해하기 쉽게 보여드리기 위해서라면 '프로세스의 Instruction이 실행됨' 으로 하는게 맞겠어요.

Copy link
Owner Author

@Seogeurim Seogeurim May 23, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

그르네요 ! 이렇게 풀어서 설명해보니까 확실히 개념이 잡히네요 ! 감사합니다. 프로세스의 Instruction이 실행됨으로 고쳐서 커밋하겠습니당 ㅎㅎ

Copy link
Collaborator

@ggjae ggjae left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

잘 읽었습니다 (__)

- Time quantum이 너무 크다면 : FCFS와 같아진다.
- Time quantum이 너무 작다면 : Context Switching이 너무 빈번하게 일어나 overhead가 발생한다.

위와 같이 RR 알고리즘의 성능은 time quantum의 크기에 좌우될 수 있으므로 적절히 선정해야 하며, 이는 context-switch time보다 큰 것이 좋으며, 또 너무 커서는 안 된다. (경험적으로 CPU burst의 80프로는 time quantum보다 짧은게 좋다고 함)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

그림씨의 쉼표 스타일이 아직 남아있네요 ㅎㅎㅎ 너무 잘 봤습니다~!
' 위와 같이 RR 알고리즘의 성능은 time quantum의 크기에 좌우될 수 있으므로 적절히 선정해야 하며 이는 context-switch time보다 큰 것이 좋지만 또 너무 커서는 안 된다. (경험적으로 CPU burst의 80프로는 time quantum보다 짧은게 좋다고 함)'

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아차 ! 또 쉼표를 ㅎㅎㅎ 전보다 훨씬 의식하며 쓰는데도 남아있네용 ㅎㅎ 고치겠습니당 ~!

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ㅎㅎㅎ 좋습니다!!!

Copy link
Collaborator

@KimKwon KimKwon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

운영체제 개념은 역시 확실히 하고 가는게 좋은 것 같습니다 👍

contents/operating-system/README.md Outdated Show resolved Hide resolved
@Seogeurim
Copy link
Owner Author

@ggjae 변경 요청 사항 반영했어요 ! 어푸룹 부탁드립니당 ㅎㅎ

@ggjae ggjae merged commit 0ea4ce6 into main May 25, 2021
@KimKwon KimKwon added score OK Score Board에 점수 반영 완료 and removed score NEED Score Board 업데이트 필요 labels Jun 5, 2021
@KimKwon
Copy link
Collaborator

KimKwon commented Jun 5, 2021

Seogeurim: + 15
Hee-Jae, ggjae, kimkwon: +5

@Seogeurim Seogeurim deleted the OS/Seogeurim branch September 4, 2021 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
score OK Score Board에 점수 반영 완료
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants