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

Proper definition of IOPS #18

Open
vitalyisaev2 opened this issue Feb 5, 2019 · 0 comments
Open

Proper definition of IOPS #18

vitalyisaev2 opened this issue Feb 5, 2019 · 0 comments

Comments

@vitalyisaev2
Copy link

vitalyisaev2 commented Feb 5, 2019

Hello, could you please check the definition of IOPS. In this article it is said that

r/s & w/s- Read/Write requests (after merges) per-second. Their sum is the IOPS!

But if you have a look at iostat source code, you'll see that r/s is defined as a difference between two rd_ios:

			/* r/s */
			cprintf_f(NO_UNIT, 1, 7, 2,
				  S_VALUE(ioj->rd_ios, ioi->rd_ios, itv));

whereas rd_ios is defined as

	/* # of read operations issued to the device */
	unsigned long rd_ios		__attribute__ ((packed));

so rd_ios has nothing to do with merges.

Therefore, r/s reflects the rate of requests issued, but yet not merged. This runs counter to your definition of IOPS.

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

1 participant