CA-395093: sync between tapback slave process and xenopsd#435
Open
chunjiez wants to merge 1 commit intoxapi-project:masterfrom
Open
CA-395093: sync between tapback slave process and xenopsd#435chunjiez wants to merge 1 commit intoxapi-project:masterfrom
chunjiez wants to merge 1 commit intoxapi-project:masterfrom
Conversation
Signed-off-by: Chunjie Zhu <chunjie.zhu@cloud.com>
Contributor
Author
|
The xenopsd side code update, xapi-project/xen-api#6825 |
Contributor
Author
|
add @MarkSymsCtx @TimSmithCtx @LunfanZhang to review |
MarkSymsCtx
requested changes
Jan 9, 2026
Contributor
MarkSymsCtx
left a comment
MarkSymsCtx
left a comment
There was a problem hiding this comment.
This all feels like it is adding complexity and risk to an already fragile part of the VM start process. I can easily see the tapback process getting stuck waiting for the response. I think we need to find a better way of achieving the aims of this pair of PRs.
| err = errno; | ||
| goto fail; | ||
| } | ||
| err = fprintf(fp, "ping"); |
Contributor
There was a problem hiding this comment.
This should follow the precendent set by the PV ring handshake and use integer state values.
|
Independent of the correct solution, the explanation that is part of this PR should be part of the commit message. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
In some corner case, physical-device-path xenstore watch event is fired before slave tapback process ready to process xenstore watch event, thus, slave tapback process would miss xenstore watch event, then blktap io datapath fails to establish.
In xenopsd side, the vbd-script waits for tapback slave process ready by checking /var/run/tapback..statefile, if the file is present and file contains "ping" string, then vbd-script updates the file, writes "pong" to the file and continues to update xenstore, otherwise, just wait.
In tapback slave process side, once it get prepared to process xenstore watch event, it writes "ping" string to /var/run/tapback..statefile, then waits for acknowledge by checking if the file contains "pong" string, after seeing "pong" string, it removes /var/run/tapback..statefile and continues to work.