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
msg/async: execute on core specified by core_id not its index #20659
msg/async: execute on core specified by core_id not its index #20659
Conversation
dafe3e8
to
6ae143b
Compare
otherwise the dpdk plugin does not compile. * remove workarounds for C++11. for example, std::void_t is available in C++17 now. * drop seastar::shared_ptr<>, as it's not used in async/dpdk now. * update the editor variable to match seastar's coding style Signed-off-by: Kefu Chai <kchai@redhat.com>
Signed-off-by: Kefu Chai <kchai@redhat.com>
Reported-by: shangfufei <shangfufei@inspur.com> Signed-off-by: Kefu Chai <kchai@redhat.com>
6ae143b
to
494bf4b
Compare
if (i-- == 0) { | ||
break; | ||
} | ||
} |
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.
what's the meaning?
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.
I think the idea of #20638 would be better
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.
@yuyuyu101 it's to get the core id from the its index. for example, if i is 0, the first non master core id is used.
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.
@shangfufei, i'd be happy to learn the rationale behind your assertion.
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.
Leave a comment
@yuyuyu101 ping? |
@@ -1,4 +1,4 @@ | |||
// -*- mode:C++; tab-width:8; c-basic-offset:2; indent-tabs-mode:t -*- |
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.
just verify, is this update from seastar upstream file?
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.
no, it's not. see the commit message of d11a824 , none of the items listed in that commit message is from seastar upstream.
Reported-by: shangfufei shangfufei@inspur.com
Signed-off-by: Kefu Chai kchai@redhat.com