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

Re: [Xen-devel] [PATCH 2/2] xen: merge temporary vcpu pinning scenarios


  • To: Juergen Gross <JGross@xxxxxxxx>
  • From: Jan Beulich <JBeulich@xxxxxxxx>
  • Date: Tue, 23 Jul 2019 14:14:42 +0000
  • Accept-language: en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1;spf=pass smtp.mailfrom=suse.com;dmarc=pass action=none header.from=suse.com;dkim=pass header.d=suse.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=qDIUdwSqu34N+tA/M/AWgaHPyt1BrvMxgOwOOH93iSM=; b=CXs42Z+5sbIG2SREKb23QS2yqm+6i2eMc9e+oxVkf2tHpnAK/ubF6AAoUU3ynNaMSmSKU6ZBwH5yB8oBJqSpZfJ/gWJbVrk1L4HIXUUGT8QOcbsWN5jvwI3nbkmBdLue6tHiVlslPF5HIvkEsc9e+YA90TPZHV7w2GJQdwUNVSpkCagERR9PnPIgP3nNxfIE+jW9GFizObJAMfbsWidp9TIa3Lj473rdZgro88HF0qCfwhMfpJarJD14IlOKkJapjTvJ2ttWGyqpnSDpf0sSum2D7AW7/DjlbYyZgbLX2Mf0C5rPv6vHb/rUIKQ8cuGyidbvCqF+eaasJqFpVL2D2A==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=a2FCODdcnBUbstw8RA0Ccbgrfqw5r210Fi+bvkYaeK+Uhe1TGCSHmxdWb9xzl6UvTsZOJ1TppxWC7Dzq6nrdpNfs9nHPkvyXEPyR1hsZI0NQmPdSPXkQw9qu5/jdK7iqmDI6x43e884a1hb3Gsp08DTJmJ11bKzNIFHGJ+0wA4VdeRSqr8ZvlIc0tXf/d3MUuFyHBnykeIZv5R9d62sqKfRaRAXxTHZyL28rk5otTdl7WA7txTIFgEm7zcT98oVRkdZV+TswBzm+DDACpxixuFq3ZLtckniHmhjKQ23BAXQN5wtBDAf7i8E1jFTmpnNwAr8bfoCPMHK75wBGVkCnDQ==
  • Authentication-results: spf=none (sender IP is ) smtp.mailfrom=JBeulich@xxxxxxxx;
  • Cc: Tim Deegan <tim@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>, George Dunlap <George.Dunlap@xxxxxxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, IanJackson <ian.jackson@xxxxxxxxxxxxx>, Dario Faggioli <dfaggioli@xxxxxxxx>, Julien Grall <julien.grall@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Delivery-date: Tue, 23 Jul 2019 14:15:42 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHVQTgCmOXaxQxPLUKxvpzXZTTLrabYJaWAgAARcbKAACa/gP//4aOA
  • Thread-topic: [Xen-devel] [PATCH 2/2] xen: merge temporary vcpu pinning scenarios

On 23.07.2019 16:03, Jan Beulich wrote:
> On 23.07.2019 15:44, Juergen Gross wrote:
>> On 23.07.19 14:42, Jan Beulich wrote:
>>> v->processor gets latched into st->processor before raising the softirq,
>>> but can't the vCPU be moved elsewhere by the time the softirq handler
>>> actually gains control? If that's not possible (and if it's not obvious
>>> why, and as you can see it's not obvious to me), then I think a code
>>> comment wants to be added there.
>>
>> You are right, it might be possible for the vcpu to move around.
>>
>> OTOH is it really important to run the target vcpu exactly on the cpu
>> it is executing (or has last executed) at the time the NMI/MCE is being
>> queued? This is in no way related to the cpu the MCE or NMI has been
>> happening on. It is just a random cpu, and so it would be if we'd do the
>> cpu selection when the softirq handler is running.
>>
>> One question to understand the idea nehind all that: _why_ is the vcpu
>> pinned until it does an iret? I could understand if it would be pinned
>> to the cpu where the NMI/MCE was happening, but this is not the case.
> 
> Then it was never finished or got broken, I would guess.

Oh, no. The #MC side use has gone away in 3a91769d6e, without cleaning
up other code. So there doesn't seem to be any such requirement anymore.

Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

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