mirror of
https://github.com/torvalds/linux.git
synced 2026-06-01 02:53:36 +02:00
KVM: x86: Bury emulator read/write ops in emulator_{read,write}_emulated()
Now that SEV-ES invokes vcpu_mmio_{read,write}() directly, bury the
read vs. write emulator ops in the dedicated emulator callbacks so that
they are colocated with their usage, and to make it harder for
non-emulator code to use hooks that are intended for the emulator.
Opportunistically rename the structures to get rid of the long-standing
"emultor" typo.
No functional change intended.
Tested-by: Tom Lendacky <thomas.lendacky@gmail.com>
Tested-by: Rick Edgecombe <rick.p.edgecombe@intel.com>
Link: https://patch.msgid.link/20260225012049.920665-11-seanjc@google.com
Signed-off-by: Sean Christopherson <seanjc@google.com>
This commit is contained in:
parent
326e810eaa
commit
3517193ef9
|
|
@ -8133,17 +8133,6 @@ static int write_emulate(struct kvm_vcpu *vcpu, gpa_t gpa,
|
|||
return emulator_write_phys(vcpu, gpa, val, bytes);
|
||||
}
|
||||
|
||||
static const struct read_write_emulator_ops read_emultor = {
|
||||
.read_write_emulate = read_emulate,
|
||||
.read_write_mmio = vcpu_mmio_read,
|
||||
};
|
||||
|
||||
static const struct read_write_emulator_ops write_emultor = {
|
||||
.read_write_emulate = write_emulate,
|
||||
.read_write_mmio = vcpu_mmio_write,
|
||||
.write = true,
|
||||
};
|
||||
|
||||
static int emulator_read_write_onepage(unsigned long addr, void *val,
|
||||
unsigned int bytes,
|
||||
struct x86_exception *exception,
|
||||
|
|
@ -8294,8 +8283,13 @@ static int emulator_read_emulated(struct x86_emulate_ctxt *ctxt,
|
|||
unsigned int bytes,
|
||||
struct x86_exception *exception)
|
||||
{
|
||||
return emulator_read_write(ctxt, addr, val, bytes,
|
||||
exception, &read_emultor);
|
||||
static const struct read_write_emulator_ops ops = {
|
||||
.read_write_emulate = read_emulate,
|
||||
.read_write_mmio = vcpu_mmio_read,
|
||||
.write = false,
|
||||
};
|
||||
|
||||
return emulator_read_write(ctxt, addr, val, bytes, exception, &ops);
|
||||
}
|
||||
|
||||
static int emulator_write_emulated(struct x86_emulate_ctxt *ctxt,
|
||||
|
|
@ -8304,8 +8298,13 @@ static int emulator_write_emulated(struct x86_emulate_ctxt *ctxt,
|
|||
unsigned int bytes,
|
||||
struct x86_exception *exception)
|
||||
{
|
||||
return emulator_read_write(ctxt, addr, (void *)val, bytes,
|
||||
exception, &write_emultor);
|
||||
static const struct read_write_emulator_ops ops = {
|
||||
.read_write_emulate = write_emulate,
|
||||
.read_write_mmio = vcpu_mmio_write,
|
||||
.write = true,
|
||||
};
|
||||
|
||||
return emulator_read_write(ctxt, addr, (void *)val, bytes, exception, &ops);
|
||||
}
|
||||
|
||||
#define emulator_try_cmpxchg_user(t, ptr, old, new) \
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user