Akio Takebe wrote:
> Hi, Xiantao
>
>>> [3/17] Add firmware virtualization support.
>
> +void kvm_sal_emul(struct kvm_vcpu *vcpu)
> +{
> +
> + struct sal_ret_values result;
> + u64 index, in1, in2, in3, in4, in5, in6, in7;
> +
> + kvm_get_sal_call_data(vcpu, &index, &in1, &in2,
> + &in3, &in4, &in5, &in6, &in7);
> + if (index) {
> + result = sal_emulator(index, in1, in2, in3,
> + in4, in5, in6, in7);
> + set_sal_result(vcpu, result);
> + } else
> + printk(KERN_WARNING"kvm:Unsupported sal call called,"
> + " index:%d!\n", index);
> +}
> The printk always show index:0.
> And set_sal_result() doesn't work at the time of index=0.
> So I think the following code is better.
Good catch! Originally, I want to use it for kvm_get_sal_call debug, but
forget to remove it.
Thank you! :)
> +void kvm_sal_emul(struct kvm_vcpu *vcpu)
> +{
> +
> + struct sal_ret_values result;
> + u64 index, in1, in2, in3, in4, in5, in6, in7;
> +
> + kvm_get_sal_call_data(vcpu, &index, &in1, &in2,
> + &in3, &in4, &in5, &in6, &in7);
> + result = sal_emulator(index, in1, in2, in3,
> + in4, in5, in6, in7);
> + set_sal_result(vcpu, result);
> +}
>
> Best Regards,
>
> Akio Takebe
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
kvm-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/kvm-devel
