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
F134727475: D49208.id151818.diff
Tue, Nov 4, 5:32 AM
Unknown Object (File)
Thu, Oct 30, 9:25 AM
Unknown Object (File)
Thu, Oct 30, 9:25 AM
Unknown Object (File)
Thu, Oct 30, 12:49 AM
Unknown Object (File)
Wed, Oct 29, 9:02 PM
Unknown Object (File)
Wed, Oct 29, 1:13 PM
Unknown Object (File)
Fri, Oct 24, 4:45 AM
Unknown Object (File)
Fri, Oct 24, 3:48 AM
View All 81 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 Skipped
Unit
Tests Skipped
Build Status
Buildable 62731
Build 59615: arc lint + arc unit

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/
26 lines

Diff 151767

View Options

sys/amd64/vmm/vmm_instruction_emul.c

Loading...