Skip to content

IB/RC/TEST: Add more detailed logging.#135

Merged
yosefe merged 1 commit intoopenucx:masterfrom
yosefe:topic/rc-add-packet-logging
Feb 23, 2015
Merged

IB/RC/TEST: Add more detailed logging.#135
yosefe merged 1 commit intoopenucx:masterfrom
yosefe:topic/rc-add-packet-logging

Conversation

@yosefe
Copy link
Copy Markdown
Contributor

@yosefe yosefe commented Feb 22, 2015

No description provided.

@yosefe
IB/RC/TEST: Add more detailed logging.
3f36d8f 
+ Add macro to test log level.
+ Improve mpool log message.
+ Improve QP create/connect log message.
+ Add IB packet-level logging, which logs every posted send operation
  and every received message.
+ Print details of every posted WQE.
+ Disable prints from data transfer tests, in case logging is enabled.
+ Logging should be disabled from fast path in release mode.
+ In p2p tests, make sure log messages are generated.
@mellanox-github
Copy link
Copy Markdown
Contributor

mellanox-github commented Feb 22, 2015

Refer to this link for build results (access rights to CI server needed):
http://bgate.mellanox.com:8000/jenkins-secure/job/gh-ucx-pr/281/
Test PASSed.

@shamisp
Copy link
Copy Markdown
Contributor

shamisp commented Feb 22, 2015

does it replace the debug_trace in UCS ?

@yosefe
Copy link
Copy Markdown
Contributor Author

yosefe commented Feb 22, 2015

basically it uses ucs_trace_data()

@shamisp
Copy link
Copy Markdown
Contributor

shamisp commented Feb 22, 2015

So it is fancy wrapper ?

From: Yossi Itigin [mailto:notifications@github.com]
Sent: Sunday, February 22, 2015 2:06 PM
To: uccs/ucx
Cc: Shamis, Pavel
Subject: Re: [ucx] IB/RC/TEST: Add more detailed logging. (#135)

basically it uses ucs_trace_data()

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/135#issuecomment-75451487.

@yosefe
Copy link
Copy Markdown
Contributor Author

yosefe commented Feb 22, 2015

wrapper that dumps full wqe / send_wr contents in common format

@shamisp
Copy link
Copy Markdown
Contributor

shamisp commented Feb 22, 2015

Sweet. Go for it.

From: Yossi Itigin [mailto:notifications@github.com]
Sent: Sunday, February 22, 2015 2:08 PM
To: uccs/ucx
Cc: Shamis, Pavel
Subject: Re: [ucx] IB/RC/TEST: Add more detailed logging. (#135)

wrapper that dumps full wqe / send_wr contents in common format

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/135#issuecomment-75451580.

yosefe added a commit that referenced this pull request Feb 23, 2015
@yosefe
Merge pull request #135 from yosefe/topic/rc-add-packet-logging
0317dc4 
IB/RC/TEST: Add more detailed logging.
@yosefe yosefe merged commit 0317dc4 into openucx:master Feb 23, 2015
@yosefe yosefe deleted the topic/rc-add-packet-logging branch February 23, 2015 09:45
dmitrygx pushed a commit to dmitrygx/ucx that referenced this pull request Dec 1, 2021
@quasiben
Merge pull request openucx#135 from rapidsai/use-sizet-for-length
bf85723 
Use size_t/ssize_t to support large count transfers
@cserranobr cserranobr mentioned this pull request Nov 25, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@yosefe @mellanox-github @shamisp