>>> On 18.04.17 at 12:55, <yi.y....@linux.intel.com> wrote: > I made a test patch based on v10 and attached it in mail. Could you please > help to check it? Thanks!
This looks reasonable at the first glance, albeit I continue to be unconvinced that this is the only (reasonable) way of solving the problem. After all we don't have to go through similar hoops for any other of the register state associated with a vCPU. There are a number of cosmetic issues, but commenting on an attached (rather than inlined) patch is a little difficult. One remark regarding the locking though: Acquiring a lock in the context switch path should be made as low risk of long stalls as possible. Therefore you will want to consider using r/w locks instead of spin locks here, which would allow parallelism on all cores of a socket as long as COS IDs aren't being updated. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
