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

Add an option to set BlockSize #18

Open
Zero3K opened this issue Dec 8, 2023 · 2 comments
Open

Add an option to set BlockSize #18

Zero3K opened this issue Dec 8, 2023 · 2 comments

Comments

@Zero3K
Copy link

Zero3K commented Dec 8, 2023

It would be nice if that could be added in order to see if the current size being used is causing errors while testing a UDF drive with an open source driver being used to mount it.

@Zero3K Zero3K changed the title Add option to set BlockSize Add an option to set BlockSize Dec 8, 2023
@ncw
Copy link
Owner

ncw commented Dec 8, 2023

The block size is set here

BlockSize = 2 * MB

It would be quite easy to make it controllable by a flag if you wanted to have a go at that, eg put this just below

blockSize      = flag.Int("s", 2*MB, "Size of the read / write buffer")

And replace all occurrences of BlockSize with *blockSize

@Zero3K
Copy link
Author

Zero3K commented Dec 8, 2023

Actually, this is not needed since I was able to figure out the reason why the driver is causing errors when using StressDisk. It has a bug(s) with the use of DirectIO.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants