On Tue, Jun 20, 2017 at 10:21:27AM -0600, Jan Beulich wrote:
> >>> On 08.06.17 at 19:11, <wei.l...@citrix.com> wrote:
> > @@ -1053,8 +982,8 @@ void do_invalid_op(struct cpu_user_regs *regs)
> >
> > if ( likely(guest_mode(regs)) )
> > {
> > - if ( !emulate_invalid_rdtscp(regs) &&
> > - !emulate_forced_invalid_op(regs) )
> > + if ( !pv_emulate_invalid_rdtscp(regs) &&
> > + !pv_emulate_forced_invalid_op(regs) )
>
> I wonder if the first couldn't be called by the second, making it
> unnecessary to export both. Or maybe have a wrapper
> pv_emulate_invalid_op() around both.
>
Do you want me to refactor and move code in the same patch? Wouldn't
that make it hard for you to review?
I can submit a follow-up patch to do what you ask for.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
