Avi Kivity wrote: > Zhang, Xiantao wrote: >> Avi Kivity wrote: >> >>> Carsten Otte wrote: >>> >>>> I still do strongly agree with the general idea of this patch, and >>>> most of the split comes out just right now. However, there is one >>>> thing I'd like to pick on: >>>> >>>> decache_vcpus_on_cpu should be an arch callback, and rather than >>>> kvm_arch_vcpu_decache. There's no reason for s390 to grab locks and >>>> do the arch callback in a loop. The whole thing is nop for us, thus >>>> the whole thing should be a callback. >>>> >>>> I'd really like to see this changed before the patch gets merged. >>>> >>> Xiantao, can you address this? >>> >> >> Yes, adressed in attached patch. :) >> > > Just to be sure: this is a replacement for your previous #3 patch, > all the rest remain unchanged?
Also work. But the last two patches will have some warnings about hunk offset. Had better use the attached patches which have been rebased. Thanks -Xiantao
0005-Move-some-x86-specific-part-from-kvm_init-to-kvm_arc.patch
Description: 0005-Move-some-x86-specific-part-from-kvm_init-to-kvm_arc.patch
0001-Move-some-includes-to-x86.c-from-kvm_main.c-since-t.patch
Description: 0001-Move-some-includes-to-x86.c-from-kvm_main.c-since-t.patch
0002-move-kvm_x86_ops-to-x86.c.patch
Description: 0002-move-kvm_x86_ops-to-x86.c.patch
0003-Using-kvm-arch-support-to-replace-kvm_x86_ops-and-f.patch
Description: 0003-Using-kvm-arch-support-to-replace-kvm_x86_ops-and-f.patch
0004-Combine-kvm_init-and-kvm_init_x86-and-will-be-calle.patch
Description: 0004-Combine-kvm_init-and-kvm_init_x86-and-will-be-calle.patch
------------------------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configuration files using AJAX and a browser. Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________ kvm-devel mailing list kvm-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/kvm-devel
