[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] xen/events: reset active flag for lateeoi events later


  • To: Juergen Gross <jgross@xxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
  • From: Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx>
  • Date: Wed, 23 Jun 2021 19:18:36 -0400
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oracle.com; dmarc=pass action=none header.from=oracle.com; dkim=pass header.d=oracle.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=NXaHNbeqOPtKkQw9Dha8CY4CHp/tDnQ+pswMmyzLSBE=; b=hw/t5+sDyfonvPOL476MLtABlWL+bbyMrvJDCT7JLoQm0K5AwhemBI+6Ly1vqtXTQuvDSgWnt26FY69Sw2rMZ09CDSNBkUG6klNx+GJ15Oao6vZvigiDMCxqMjFCa6C3J3Rihpzytzu53bP7yz4LLhS+NKxJbkaJJiLj1AKXRdtPQNk+5Vzuj7I+hig7+KrzO0/w2WpnaNkrmOM3lWG5Ul202wM1+rh2a7rplHx0QPQFzOo5ERlMM+pzZwhs3bTJFpa0JSD1BYwO3FiTqN8E7sW9BX6qUVXjnJ5oigSXjRUzYvdUpvZNvePhiFklFzjFK++7j17mXvLey6Ces5tbgw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Ss2M0SQwbLE+mlwY1AYzlTSJyF675t05Zx9kicGoOAU7jKlGy3V3Kkj3aUmQ9DvLzY9xDO9uA4v8zD+4miZM0uAs20n9+vf2Y7Ov2KEG7jLUtk+1sWMYhQji6lm2UpsMqF4CZGeX0cyV1E9soZO+gruNEY+Xqj5aFwIkhOaMSrhoqM1Q/2iEaCDSVYrsnhKBEmse8RrcZV2nQMLfEaSyRlERIXRvkaT5druzL8PUq8Wii1//dKo4rrB39WbTjQcV0n7xc64utbq2vEmeqPhvPF3l3xATOXi0gYANb97gsqTSL8Z2LPxERYhtmVcvXN0RZMgO+3XzNWoomdLDnn5Cag==
  • Authentication-results: xen.org; dkim=none (message not signed) header.d=none;xen.org; dmarc=none action=none header.from=oracle.com;
  • Cc: Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>
  • Delivery-date: Wed, 23 Jun 2021 23:19:11 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 6/23/21 9:09 AM, Juergen Gross wrote:
> In order to avoid a race condition for user events when changing
> cpu affinity reset the active flag only when EOI-ing the event.
>
> This is working fine as all user events are lateeoi events. Note that
> lateeoi_ack_mask_dynirq() is not modified as there is no explicit call
> to xen_irq_lateeoi() expected later.
>
> Reported-by: Julien Grall <julien@xxxxxxx>
> Fixes: b6622798bc50b62 ("xen/events: avoid handling the same event on two 
> cpus at the same time")
> Tested-by: Julien Grall <julien@xxxxxxx>
> Signed-off-by: Juergen Gross <jgross@xxxxxxxx>


Reviewed-by: Boris Ostrovsky <boris.ostrvsky@xxxxxxxxxx>





 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.