Skip to content

perftest: enable CQ events support for WRITE_IMM operations #348

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

perftest: enable CQ events support for WRITE_IMM operations #348

wants to merge 1 commit into from

Conversation

@mellowcandle
Copy link
Contributor

@mellowcandle mellowcandle commented Sep 11, 2025

Remove the restriction that prevented WRITE_IMM operations from using
completion queue events. As WRITE_IMM posts to the reciever CQ, there's
no reason not to support the event parameter (-e) as in WRITE.

Changes include:

  • Remove WRITE_IMM exclusion from events parameter parsing
  • Add CQ notification setup for write_bw and write_lat tools
  • Implement event notification in run_iter_lat_write_imm()
  • Update polling logic to respect use_event flag
@mellowcandle
perftest: enable CQ events support for WRITE_IMM operations
354163a 
Remove the restriction that prevented WRITE_IMM operations from using completion queue events. As WRITE_IMM posts to the reciever CQ, there's no reason not to support the event parameter (-e) as in WRITE. Changes include: - Remove WRITE_IMM exclusion from events parameter parsing - Add CQ notification setup for write_bw and write_lat tools - Implement event notification in run_iter_lat_write_imm() - Update polling logic to respect use_event flag Reviewed-by: Michael Margolin <mrgolin@amazon.com> Reviewed-by: Yonatan Nachum <ynachum@amazon.com> Signed-off-by: Ramon Fried <friedram@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant

@mellowcandle