Hi Deng-Cheng,
On 24/06/14 18:31, Deng-Cheng Zhu wrote:
> @@ -2213,8 +2209,8 @@ enum emulation_result kvm_mips_check_privilege(unsigned long cause,
> * address error exception to the guest
> */
> if (badvaddr >= (unsigned long) KVM_GUEST_KSEG0) {
> - printk("%s: LD MISS @ %#lx\n", __func__,
> - badvaddr);
> + kvm_err("%s: LD MISS @ %#lx\n", __func__,
> + badvaddr);
This should probably be kvm_debug since it isn't fatal to the whole VM
(the exception gets passed on to the guest kernel to handle), otherwise
guest userland could maliciously spam the host log by repeatedly trying
to access beyond the T&E useg.
Same goes for the other printks in this function
It probably was only useful to sanity check that userland wasn't trying
to access memory that would be accessible on a normal MIPS core but
isn't with the T&E segment layout.
Otherwise this patch looks okay to me.
Cheers
James
