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

Re: [PATCH v4 2/3] x86/time: yield to hyperthreads after updating TSC during rendezvous


  • To: Jan Beulich <jbeulich@xxxxxxxx>
  • From: Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Date: Tue, 20 Apr 2021 17:59:00 +0200
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.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=iTqAJs4NqeUp60N4LWJpLG3HoOl+mpLjdy/SFXBOj8k=; b=eVWmBWIed4ouLhkuvyUbfHLZBd+etbP/QF1eiCGcXNFWBU6s1fKAW9PNE7h9FdqK9OiTHPFuRdE2Y9OVwUZJg4eU9TBYJTuLsrto8qRxqu7EEl0gt0I4/go78hb3beyHfpcsIj+eO07/TdE9vKdle92bwc4S08NQoWxEqfsB84uD3q23ZHJY9/21hRq0qXnIJCj8K10GsfWCv3mWa0Z7U/nRkwGrThFvxeb4ixYEdX9vZ3JB5LHSY8zJorCSUUfWyjxj2uLJZSxpA7SlpdEIyuoUIQYjS/5+tUjH07y0R4+lpx6DOyJt2m4DDa22gpgLLZj6/8zDe2GKa8X9DQSc3g==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=R7bCqVNnzg3xZZhMKJnQz0Gv5hfKUSzQbmyeuQIq6M4H1jyyjmBVUt5c4GCOkCbyKpS1OBi6vMxLU3HzsVxydSkEvInkZv4AjKpBcQ0HxAOKVAKTDe7K6WnikrngOf2z0w+O85wAgTb9W9tDs25AZVUvWGS1gZjYBoBUV8CkpjlLWyY5qUFoBLg49JwXenDGwBfWNRepTfv+ZYArNutn0HKAzAf3Ru9AZcF3wtsFqFqTn47JHTS00mJE6yZUv3A4Roln0c2rpi+MSCxO3USngCZ9c+s8+SZ09goewYYOTdlt1/8dWv0TN4uIUI0DdKtlz3OWvBoXYWkNnbPZPT1HxQ==
  • Authentication-results: esa6.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, "Andrew Cooper" <andrew.cooper3@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>
  • Delivery-date: Tue, 20 Apr 2021 15:59:17 +0000
  • Ironport-hdrordr: A9a23:8kxcFahnWFbGN1XDDYT1fYRzKHBQXwB13DAbvn1ZSRFFG/Gwv9 yynfgdyB//gCsQXnZlotybJKycWxrnmKJdy48XILukQU3aqHKlRbsSiLfK7h/BP2nF9uBb3b p9aKQWMrfNJHVzkMqS2maFOvk6xt3vys6VrMP/61socg1wcaFn6G5Ce2SmO2l7XhNPC5Z8NL f03Kp6jgGtc3gWcci3b0NtN4T+jubGiY78Zlo+DwMngTPksRqT9LX4HxKEty1uMA9n/LFKyw n4uj283IqPmbWRyhjQ12jchq4m4ufJ+594K+GnzuQQIjXooA60aIpmQK3qhkFJnMifrGwEvf OJjxA8P9liy365RBDInTLdnzPO/Rxry3j+xUSWiXHuyPaJOw4SOo56qq9yNj76gnBQ2O1U4e Zw8E+y86dzN1fmmh/w4tDZPisa7nackD4ZvsM4y0BEXZB2Us43kaUvuHl7Pb0nByzA5IUuAI BVfbrhzccTS1+cYnzD11MfpuCEbzA2FheCdEAIptaY5ThQhGx41EsV3qUk7w49yK4=
  • Ironport-sdr: 1fc8gM0yeQDoiyC+F8mVaFSz2KHW5yoLUtNGSdI9YEuzPw6cJVMEnWWJVHheuPWvoy0wwHCDFA ouOWrjKYK/71S8dORKef4CsugSwy3xMidMTpN4LVsDQHsXQ3p9o1eM50cDUTAsnB0OQjrZXA8d 2o+0ZSBTrxeOAktlhyOfQxnB9fG1e8tLXS1eyb+Snrhv9vV4rexkyN8MVzcEvERphSLW9oKMIu PfNSy6wOsvdoa9KCDGmZk80D0bzYDQ+I6shrf/mAMijnnPHtXdeOttHen6CHdfsoqULeFmw8qe TTw=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Thu, Apr 01, 2021 at 11:54:27AM +0200, Jan Beulich wrote:
> Since we'd like the updates to be done as synchronously as possible,
> make an attempt at yielding immediately after the TSC write.
> 
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>

Reviewed-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>

Did you observe any difference with the pause inserted?

I wonder whether that's enough to give a chance the hyperthread to
also perform the TSC write. In any case there's no harm from it
certainly.

Thanks, Roger.



 


Rackspace

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