Re: [PATCH] KVM: Veirfy memory slot only for readability

Sasha Levin
2011-12-02T05:46:33Z

On Fri, 2011-12-02 at 12:16 +0900, Takuya Yoshikawa wrote:
> (2011/12/02 10:15), Takuya Yoshikawa wrote:
> > (2011/12/02 4:42), Sasha Levin wrote:
> >> It's enough for memory slot to be readable, as the comment above the check
> >> states.
> >>
> >> A user should be able to create read-only memory slot.
>
> Note: at that time, it looked to me that the API did not allow me to know
> which type was being registered.

Which code exactly assumes that all memory slots are writable?

> Do you want to create read only memory slots for kvm tool?

What KVM tool currently does is copy the kernel into guest memory and
run it from there. An idea raised recently was instead of copying it we
should mmap it into the memory to reduce footprint.

This is why I'm looking into adding a read only memory slot. The KVM
code doesn't have to know it's read only.

--

Sasha.