>>> On 19.11.14 at 23:21, <konrad.w...@oracle.com> wrote: > @@ -97,13 +97,15 @@ bool_t pt_pirq_softirq_active(struct hvm_pirq_dpci > *pirq_dpci) > } > > /* > - * Reset the pirq_dpci->dom parameter to NULL. > + * Cancels an outstanding pirq_dpci (if scheduled). Also if clear is set, > + * reset pirq_dpci->dom parameter to NULL (used for teardown). > * > * This function checks the different states to make sure it can do it > * at the right time. If it unschedules the 'hvm_dirq_assist' from running > * it also refcounts (which is what the softirq would have done) properly. > */ > -static void pt_pirq_softirq_reset(struct hvm_pirq_dpci *pirq_dpci) > +static void pt_pirq_softirq_cancel(struct hvm_pirq_dpci *pirq_dpci, > + unsigned int clear)
Looks reasonable apart from this wanting to be bool_t, and apart from the fact that iiuc it still doesn't eliminate the observed crashes (in which case the fix for that may better be folded into here). I'm not really convinced that this fix is what you currently have as patch 2 (as the previously mentioned problem of _reset() [now _cancel()] clearing STATE_SCHED without removing the list entry from the list is now becoming even more obvious), but I'll also comment there. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
