CMU-SAFARI / MQSim

MQSim is a fast and accurate simulator modeling the performance of modern multi-queue (MQ) SSDs as well as traditional SATA based SSDs. MQSim faithfully models new high-bandwidth protocol implementations, steady-state SSD conditions, and the full end-to-end latency of requests in modern SSDs. It is described in detail in the FAST 2018 paper by Arash Tavakkol et al., "MQSim: A Framework for Enabling Realistic Studies of Modern Multi-Queue SSD Devices" (https://people.inf.ethz.ch/omutlu/pub/MQSim-SSD-simulation-framework_fast18.pdf)
https://people.inf.ethz.ch/omutlu/pub/MQSim-SSD-simulation-framework_fast18.pdf
MIT License
273 stars 149 forks source link

FLIN scheduler #36

Open sac2019 opened 4 years ago

sac2019 commented 4 years ago

hi

does the MQSim support the FLIN scheduler? why the .c and .h file of FLIN are commented in the source code of simulator?

Regards

sac2019 commented 4 years ago

@arashta

HrithikSampson commented 4 years ago

Maybe because flin reduces performance

arashta commented 4 years ago

Hello @sac2019 and @HrithikSampson, The FLIN source code was prepared before the official release of MQSim. The FLIN source should be refactored to be able to work with the current version of MQSim which has been majorly refactored and boosted with new features after the release.

HrithikSampson commented 4 years ago

Sir , Though,I am just an undergraduate student who read the code but I think if in nvm_transaction_flash.cpp if you change the constructor parameter corresponding to bool FLIN_barrier to true and carefully uncomment the code in tsu flin .h and .CPP it might work

On Tue, 28 Apr, 2020, 11:08 PM Arash Tavakkol, notifications@github.com wrote:

Hello @sac2019 https://github.com/sac2019 and @HrithikSampson https://github.com/HrithikSampson, The FLIN source code was prepared before the official release of MQSim. The FLIN source should be refactored to be able to work with the current version of MQSim which has been majorly refactored and boosted with new features after the release.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/CMU-SAFARI/MQSim/issues/36#issuecomment-620752773, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANLNWAZ5BDMN4CRS6R23TJLRO4IAJANCNFSM4J4M6HKA .

HrithikSampson commented 4 years ago

I may be wrong

On Wed, 29 Apr, 2020, 1:25 AM Hrithik Sampson, < hrithikedwardsampson@gmail.com> wrote:

Sir , Though,I am just an undergraduate student who read the code but I think if in nvm_transaction_flash.cpp if you change the constructor parameter corresponding to bool FLIN_barrier to true and carefully uncomment the code in tsu flin .h and .CPP it might work

On Tue, 28 Apr, 2020, 11:08 PM Arash Tavakkol, notifications@github.com wrote:

Hello @sac2019 https://github.com/sac2019 and @HrithikSampson https://github.com/HrithikSampson, The FLIN source code was prepared before the official release of MQSim. The FLIN source should be refactored to be able to work with the current version of MQSim which has been majorly refactored and boosted with new features after the release.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/CMU-SAFARI/MQSim/issues/36#issuecomment-620752773, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANLNWAZ5BDMN4CRS6R23TJLRO4IAJANCNFSM4J4M6HKA .

sac2019 commented 4 years ago

Sir , Though,I am just an undergraduate student who read the code but I think if in nvm_transaction_flash.cpp if you change the constructor parameter corresponding to bool FLIN_barrier to true and carefully uncomment the code in tsu flin .h and .CPP it might work On Tue, 28 Apr, 2020, 11:08 PM Arash Tavakkol, @.***> wrote: Hello @sac2019 https://github.com/sac2019 and @HrithikSampson https://github.com/HrithikSampson, The FLIN source code was prepared before the official release of MQSim. The FLIN source should be refactored to be able to work with the current version of MQSim which has been majorly refactored and boosted with new features after the release. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#36 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANLNWAZ5BDMN4CRS6R23TJLRO4IAJANCNFSM4J4M6HKA .

it doesn't work....