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
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
