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

read-mshot.t: test with differing buffer size #981

Merged
merged 1 commit into from
Jan 2, 2024
Merged

Conversation

DylanZA
Copy link
Contributor

@DylanZA DylanZA commented Nov 5, 2023

This patch adds a check for differing buffer sizes, and amkes sure that the returned size is correct.

This fails currently, but should work with https://lore.kernel.org/io-uring/20231105223008.125563-1-dyudaken@gmail.com/T/#t


git request-pull output:


The following changes since commit 78eef455c9538cbc145d7c658367f7662aeba39a:

  liburing.spec: move to minor 6 (2023-11-04 15:15:11 -0600)

are available in the Git repository at:

  https://github.com/dylanZA/liburing fix1

for you to fetch changes up to 3df9b683d12e1c6bb68e48bfccaf9befddc86be9:

  read-mshot.t: test with differing buffer size (2023-11-05 22:33:16 +0000)

----------------------------------------------------------------
Dylan Yudaken (1):
      read-mshot.t: test with differing buffer size

 test/read-mshot.c | 24 ++++++++++++++----------
 1 file changed, 14 insertions(+), 10 deletions(-)


Also this checks that the returned size is correct

Signed-off-by: Dylan Yudaken <dyudaken@gmail.com>
@axboe axboe merged commit 01723bf into axboe:master Jan 2, 2024
13 checks passed
@axboe
Copy link
Owner

axboe commented Jan 2, 2024

For some reason I had missed this one, merged now, thanks!

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

Successfully merging this pull request may close these issues.

None yet

2 participants