-
Notifications
You must be signed in to change notification settings - Fork 522
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
nbd: unmap wait process exit #228
Conversation
nbd/src/NBDTool.cpp
Outdated
return 0; | ||
} | ||
|
||
// TODO(wuhanqing): make this to NBDConfig |
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.
这个TODO处理掉吧,没必要留着,考虑下需不需要加到配置里去,如果不需要,就单独一个常量定义就可以了,如果需要就增加到配置里吧
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.
Done
nbd/src/NBDTool.cpp
Outdated
return ret; | ||
} | ||
|
||
ret = WaitForTerminate(devpath, devpid, 5000); |
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.
这个5000也考虑下是否需要加到配置中,或者定义成常量,不要用裸的数字。
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.
Done
f4bc7c2
to
7baa919
Compare
CommunityBridge selected SMI project and mentee
What problem does this PR solve?
Issue Number: close #xxx
Problem Summary:
What is changed and how it works?
What's Changed:
How it Works:
Side effects(Breaking backward compatibility? Performance regression?):
Check List