-
Notifications
You must be signed in to change notification settings - Fork 653
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
tests: Add tests for various error flows #1117
Conversation
@@ -31,6 +31,23 @@ def create_players(self, send_ops_flags=0, qp_count=8): | |||
self.client.pre_run(self.server.psns, self.server.qps_num) | |||
self.server.pre_run(self.client.psns, self.client.qps_num) | |||
|
|||
def full_sq_bad_flow(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason why you chose to skip UD in this case?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
UD is implemented by different providers. I don't think all of them return in an error in this manner and I don't want to break them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Address Edward's remarks
PyverbsRDMAErrno is using errno as the error code, this is a problem where the error code is meaningful and can be checked. Moreover, errno is not meaningful in these situations. Reviewed-by: Chen Brasch <cbrasch@amazon.com> Reviewed-by: Yossi Leybovich <sleybo@amazon.com> Signed-off-by: Yonatan Nachum <ynachum@amazon.com> Signed-off-by: Firas Jahjah <firasj@amazon.com>
…et state Add generic error flow tests to check post receive/send while QP is in reset state, which is an invalid state for traffic. Add instances of the tests in UD and SRD. Reviewed-by: Chen Brasch <cbrasch@amazon.com> Signed-off-by: Yonatan Nachum <ynachum@amazon.com> Signed-off-by: Firas Jahjah <firasj@amazon.com>
Add generic error flow tests to check traffic in full RQ in QP. Add instances of the tests in UD and SRD. Reviewed-by: Chen Brasch <cbrasch@amazon.com> Signed-off-by: Yonatan Nachum <ynachum@amazon.com> Signed-off-by: Firas Jahjah <firasj@amazon.com>
Add SRD error flow test to check traffic in full SQ in QP. Reviewed-by: Chen Brasch <cbrasch@amazon.com> Signed-off-by: Yonatan Nachum <ynachum@amazon.com> Signed-off-by: Firas Jahjah <firasj@amazon.com>
Add generic error flow test to check post receive with RWR that have more than allowed SG entries. Add instances of the test in UD and SRD. Reviewed-by: Chen Brasch <cbrasch@amazon.com> Signed-off-by: Yonatan Nachum <ynachum@amazon.com> Signed-off-by: Firas Jahjah <firasj@amazon.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the delay.
I have addressed all of your comments and updated the commits.
@@ -31,6 +31,23 @@ def create_players(self, send_ops_flags=0, qp_count=8): | |||
self.client.pre_run(self.server.psns, self.server.qps_num) | |||
self.server.pre_run(self.client.psns, self.client.qps_num) | |||
|
|||
def full_sq_bad_flow(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
UD is implemented by different providers. I don't think all of them return in an error in this manner and I don't want to break them.
Kind reminder. |
Thanks for the reminder |
This series adds various error flow tests on post send/receive operations, and adds instances of these tests for UD and SRD.