Page MenuHomeFreeBSD
Differential D49208

vmm: Emulate testb imm8,r/m8
ClosedPublic
Actions

Authored by jlduran on Mar 3 2025, 7:14 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, May 21, 2:53 PM
Unknown Object (File)
Sat, May 16, 7:26 PM
Unknown Object (File)
Sat, May 16, 7:26 PM
Unknown Object (File)
Sat, May 16, 7:26 PM
Unknown Object (File)
Sat, May 16, 7:23 PM
Unknown Object (File)
Thu, May 7, 7:43 AM
Unknown Object (File)
Mon, May 4, 5:28 AM
Unknown Object (File)
Sun, May 3, 12:32 AM
View All Files
Subscribers
bcran
driesm
imp
markj
rgrimes

Details

Reviewers
emaste
mp
scf
markj
Group Reviewers
bhyve
Commits
rG20083496866c: vmm: Emulate testb imm8,r/m8
rG3733546de363: vmm: Emulate testb imm8,r/m8
rG49a4838a0d94: vmm: Emulate testb imm8,r/m8
Summary

Add support for "testb imm8,r/m8" emulation.

PR: 261940
PR: 282852
Obtained from: Illumos (bug #14483)

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jlduran created this revision.Mar 3 2025, 7:14 AM
Herald added subscribers: bcran, rgrimes, imp. · View Herald TranscriptMar 3 2025, 7:14 AM
jlduran requested review of this revision.Mar 3 2025, 7:14 AM
Harbormaster completed remote builds in B62731: Diff 151767.Mar 3 2025, 7:14 AM
markj added a subscriber: markj.Mar 3 2025, 11:40 AM
markj added inline comments.
sys/amd64/vmm/vmm_instruction_emul.c
1312

Rather than duplicating the whole implementation, IMO it'd be neater to write:

case 0xF6:
    size = 1;
    /* FALLTHROUGH */
case 0xF7:
    ...
driesm added a subscriber: driesm.Mar 3 2025, 1:29 PM
jlduran updated this revision to Diff 151818.Mar 3 2025, 6:34 PM
jlduran marked an inline comment as done.

Address suggestions:

  • Keep things "DRY"
Harbormaster completed remote builds in B62744: Diff 151818.Mar 3 2025, 6:34 PM
emaste accepted this revision.Mar 3 2025, 6:47 PM
This revision is now accepted and ready to land.Mar 3 2025, 6:47 PM
jlduran edited the summary of this revision. (Show Details)Mar 3 2025, 7:02 PM
markj accepted this revision as: markj.Mar 3 2025, 11:19 PM
Closed by commit rG49a4838a0d94: vmm: Emulate testb imm8,r/m8 (authored by jlduran). · Explain WhyMar 4 2025, 1:54 PM
This revision was automatically updated to reflect the committed changes.
jlduran added a commit: rG49a4838a0d94: vmm: Emulate testb imm8,r/m8.
jlduran added a commit: rG3733546de363: vmm: Emulate testb imm8,r/m8.Mar 11 2025, 12:43 AM
jlduran added a commit: rG20083496866c: vmm: Emulate testb imm8,r/m8.

Revision Contents
Changeset List

PathSize
sys/
amd64/
vmm/
12 lines

Diff 151841

View Options

sys/amd64/vmm/vmm_instruction_emul.c

Loading...