Applied.
Are you planning to send another patches
which should be included in the 3.3 release?
(of course it would depend on test resuts...)
thanks,
On Wed, Aug 13, 2008 at 01:05:47PM +0900, Kouya Shimura wrote:
Content-Description: message body text
> Isaku Yamahata writes:
> > But, is the hunk in vtlb.c really necessary?
> > vmx_vcpu_flush_vtlb_all() is called only by domain_flush_vtlb_all()
> > which checks is_initialized member.
>
> Hmm, it is not needed.
> new patch is attached.
>
> Thanks,
> Kouya
>
> Signed-off-by: Kouya Shimura <kouya@xxxxxxxxxxxxxx>
>
> diff -r d24bc5f549ba xen/arch/ia64/xen/vhpt.c
> --- a/xen/arch/ia64/xen/vhpt.c Mon Aug 11 11:04:52 2008 +0900
> +++ b/xen/arch/ia64/xen/vhpt.c Wed Aug 13 10:21:45 2008 +0900
> @@ -526,6 +526,8 @@ void flush_tlb_for_log_dirty(struct doma
> /* NB. There is no race because all vcpus are paused. */
> if (is_hvm_domain(d)) {
> for_each_vcpu (d, v) {
> + if (!v->is_initialised)
> + continue;
> /* XXX: local_flush_tlb_all is called redundantly */
> thash_purge_all(v);
> }
> @@ -533,6 +535,8 @@ void flush_tlb_for_log_dirty(struct doma
> NULL, 1, 1);
> } else if (HAS_PERVCPU_VHPT(d)) {
> for_each_vcpu (d, v) {
> + if (!v->is_initialised)
> + continue;
> vcpu_purge_tr_entry(&PSCBX(v,dtlb));
> vcpu_purge_tr_entry(&PSCBX(v,itlb));
> vcpu_vhpt_flush(v);
> _______________________________________________
> Xen-ia64-devel mailing list
> Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-ia64-devel
--
yamahata
_______________________________________________
Xen-ia64-devel mailing list
Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-ia64-devel
|