search

bigsys-gnu / mvcc-os

KhronOS, a scalable operating systems based on sv6 (MIT) with MV-RLU (multi-version concurrency control mechanism)
Other
1 stars 0 forks source link

bug-fix: reads should count contains #79

Closed HanMinGyun closed 2 years ago

MadPlayer commented 2 years ago

읽기 시도만 완료하면 전부 읽은걸로 쳐야된다는 거지?

bigsys-gnu commented 2 years ago

읽기 시도만 완료하면 전부 읽은걸로 쳐야된다는 거지?

찾고자 하는 값이 없을뿐이지 실제로 읽기 연산을 완료했기 때문에 카운트해요

bigsys-gnu commented 2 years ago

읽기 시도만 완료하면 전부 읽은걸로 쳐야된다는 거지?

찾고자 하는 값이 없을뿐이지 실제로 읽기 연산을 완료했기 때문에 카운트해요

커널벤치에서도 확인하고 수정해주세요.

MadPlayer commented 2 years ago

읽기 시도만 완료하면 전부 읽은걸로 쳐야된다는 거지?

찾고자 하는 값이 없을뿐이지 실제로 읽기 연산을 완료했기 때문에 카운트해요

커널벤치에서도 확인하고 수정해주세요.

https://github.com/bigsys-gnu/mvcc-os/blob/43c7ea57a23d912c17dd6eca56d1dfae621cf691/os/sv6/scalefs/kernel/sysbench.cc#L870-L880

커널벤치에서는 수정이 이미 되어있는 것으로 보입니다.

bigsys-gnu commented 2 years ago

읽기 시도만 완료하면 전부 읽은걸로 쳐야된다는 거지?

찾고자 하는 값이 없을뿐이지 실제로 읽기 연산을 완료했기 때문에 카운트해요

커널벤치에서도 확인하고 수정해주세요.

https://github.com/bigsys-gnu/mvcc-os/blob/43c7ea57a23d912c17dd6eca56d1dfae621cf691/os/sv6/scalefs/kernel/sysbench.cc#L870-L880

커널벤치에서는 수정이 이미 되어있는 것으로 보입니다.

877 라인에 found를 contains로 수정해야 되요