-
Notifications
You must be signed in to change notification settings - Fork 71
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
update qemu version to v8.0.4 #120
Conversation
8103a93
to
71097f4
Compare
71097f4
to
db32480
Compare
db32480
to
35f7557
Compare
e4d02bf
to
928fc72
Compare
ef63c28
to
7488927
Compare
35b0049
to
19ad4ea
Compare
test/test.bats
Outdated
@@ -83,7 +83,7 @@ execdirect() { | |||
@test "shebang-direct" { | |||
execdirect ./shebang.sh foo bar1 | |||
assert_success | |||
assert_output "./printargs ./shebang.sh foo bar1" | |||
assert_output "./printargs /work/shebang.sh foo bar1" |
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.
@tonistiigi About this change: https://github.com/tonistiigi/binfmt/actions/runs/5305194882/jobs/9602044053#step:4:385
#43 15.22 not ok 11 shebang-direct
#43 15.22 # (from function `assert_output' in file bats-assert/src/assert.bash, line 239,
#43 15.22 # in test file test.bats, line 86)
#43 15.23 # `assert_output "./printargs ./shebang.sh foo bar1"' failed
#43 15.23 #
#43 15.23 # -- output differs --
#43 15.23 # expected : ./printargs ./shebang.sh foo bar1
#43 15.23 # actual : ./printargs /work/shebang.sh foo bar1
#43 15.23 # --
#43 15.23 #
linux-user now resolves the full path name of the started process at startup. See qemu/qemu@258bec3
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.
Isn't this a breaking change (at least for the direct mode without -P
)? What if the program reads its args[0]
, it wouldn't expect that the value gets modified.
If this is how it behaves with QEMU_PRESERVE_ARGV0
in kernel then it just looks plain wrong. If it is just a problem with our direct exec patches then the patches may need updates.
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.
@hdeller are we missing some context in here?
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.
My change just makes the emulated binary work the same way as a native program would. This means /proc/self/exe will return the currently running executable. I don't think my patch modified args[] values.
I think if you would run your testsuite in the ./work directory you wouldn't see that changed.
.../buildkit-direct-execve-v8.0/0001-linux-user-have-execve-call-qemu-via-proc-self-exe-t.patch
Outdated
Show resolved
Hide resolved
Hello there, |
My suggestion is to go ahead with this and follow up with 8.1 BUT not update the |
19ad4ea
to
f39c431
Compare
f39c431
to
84ab07d
Compare
We can skip latest: binfmt/.github/workflows/release.yml Lines 19 to 22 in 82a943b
|
84ab07d
to
92308da
Compare
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
linux-user now resolves the full path name of the started process: qemu/qemu@258bec3. we should fix our patches in a follow-up. Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
92308da
to
a1603cd
Compare
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
a1603cd
to
0f51065
Compare
Can be tested with |
changelog: https://wiki.qemu.org/ChangeLog/8.0
notable changes:
changelog: https://wiki.qemu.org/ChangeLog/7.2
notable changes:
max
by default on x86: qemu/qemu@d135f78/proc/self/exe
withexecve()
syscall: qemu/qemu@f07eb1c