>>> On 16.05.19 at 15:52, <roger....@citrix.com> wrote:
> On Wed, May 08, 2019 at 06:48:16AM -0600, Jan Beulich wrote:
>> @@ -1114,19 +1114,18 @@ static void irq_guest_eoi_timer_fn(void
>>
>> action = (irq_guest_action_t *)desc->action;
>>
>> + ASSERT(action->ack_type != ACKTYPE_NONE);
>> +
>> if ( !action->in_flight || timer_is_active(&action->eoi_timer) )
>> goto out;
>>
>> - if ( action->ack_type != ACKTYPE_NONE )
>> + for ( i = 0; i < action->nr_guests; i++ )
>> {
>> - unsigned int i;
>> - for ( i = 0; i < action->nr_guests; i++ )
>> - {
>> - struct domain *d = action->guest[i];
>
> I think you could constify d here.
Now that I've tried I recall that I did so already when originally
putting together the patch. It doesn't work, because
radix_tree_lookup() requires a non-const pointer.
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
