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

Re: [PATCH v3 1/4] x86/shadow: replace sh_reset_l3_up_pointers()


  • To: Jan Beulich <jbeulich@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  • Date: Thu, 9 Feb 2023 17:28:34 +0000
  • 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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=sXD9az4p+c3IoeHYNHwgBsFRy4V8+JXh4R/P6E1A7SQ=; b=oWruzryN7mCTCNg+yCPt0zS0B35G2fZtgceD3Jzr6g1QBCDjtOEtKpjCfIhYg9KZptreCwqD6CDkzZEK0aM7cujoHuaqhU8mMdtMXIHzp78IexOVbxSZUrZo/stjLJb5BU0ja9Ym3G3RZI7iBzsTRMqd2WBwkFt7O3G1h5NMZM34pf4GRCSFNSqEMVOfFukOakigRvNOZpHrXuEkhEFkmxzlqDY278ALk2bQBEzQ/rmCuW131cZM2o2oE5eNxmgklTPVLpw4FmkaeViZpSlWjVhWg/SqlKRReNOWOKPyQ+MY706GPUg595U2rpMX5NermZWfLzz2+DNfngmyQ0XjKA==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Iqjoqmv8rXVgaGnjBGjR+Ed9tsMlK1hS1uN7FK8w6sp0Q5eWKcZrKV6z8mEEJMj/jr9nYJ/+Fqa8broGWU6rlGAaJ3IqcJLxmMnK2umYmtaSCy9TtUl+OuLlAoyKHwfvas0DuPFGHTULMm1cj9sKjN+0ze23b6dZh1yw6Ib8ppYnWVYlCv5Wf8EEugVyIXVne40TY7RrsIKXCV8fuJ49+3L7gkEP2HuTkYtKgT414EsAGFcOKTPar0JLvzU06mW4PF8T5BZpZKUc7e4RSPfJrV31WrF74MxPFUMI/1ytB5mo1/tqoxkI+GEHZZ03KI5wXQgYrfNCnxOL5N4UXtBXgg==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=citrix.com;
  • Cc: Wei Liu <wl@xxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Tim Deegan <tim@xxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>
  • Delivery-date: Thu, 09 Feb 2023 17:28:50 +0000
  • Ironport-data: A9a23:cKJxQ6n0FOfOPE24KQgTb9To5gywJ0RdPkR7XQ2eYbSJt1+Wr1Gzt xIZDGvSMvuLMGTzeo9/Otiy8k8GsZaGzddmSAs9/3gwEiMWpZLJC+rCIxarNUt+DCFhoGFPt JxCN4aafKjYaleG+39B55C49SEUOZmgH+a6U6icfHgqH2eIcQ954Tp7gek1n4V0ttawBgKJq LvartbWfVSowFaYCEpNg064gE4p7auaVA8w5ARkPqgR5A6GzBH5MbpETU2PByqgKmVrNrbSq 9brlNmR4m7f9hExPdKp+p6TnpoiG+O60aCm0xK6aoD66vRwjnVaPpUTbZLwXXx/mTSR9+2d/ f0W3XCGpaXFCYWX8AgVe0Ew/yiTpsSq8pefSZS0mZT7I0Er7xIAahihZa07FdRwxwp5PY1B3 a05IgsKRDzcvtrs4qOyEco2ne16dPC+aevzulk4pd3YJdAPZMmZBonvu5pf1jp2gd1SF/HDY cZfcSBocBnLfxxIPBEQFY46m+CrwHL4dlW0qnrM/fZxvzeVkV03iea8WDbWUoXiqcF9t0CUv G/ZuU/+BQkXLoe3wjuZ6HO8wOTImEsXXapDSuHgrKQ32jV/wEQZJEVLFmmJvMO/k2PkZN5da EMF/xsX+P1aGEuDC4OVsweDiHyOswMYWtFQO/Yn8wzLwa3Riy6JC25BQjNfZdgOsM4tWSdsx lKPh8nuBzFkrPuSU3313qiQhSO/P24SN2BqTTMFSCMV7t+lp5s85i8jVf5mGa+xy9HwRzf5x mnQqDBk3utCy8kWy6+84FbLxSq2oYTERRI04QORWX+56gR+Z8iuYInABUXn0Mus5b2xFjGp1 EXoUeDHhAzSJflhTBCwfdg=
  • Ironport-hdrordr: A9a23:ZtLFdaMCMSAXLsBcTh2jsMiBIKoaSvp037By7TEJdfU0SKalf8 DHpoV56faGskdoZJhAo6H9BEDkexjhHPFOkO4s1NuZPTUO21HYU72Kj7GSpQEIcheWnoIzu5 uMa5ISNDSZNykFsS+Q2njbLz9U+qjizEnev5a6854Cd2FXgupbnnxE406gYzdLrSd9dOcEKK Y=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 08/02/2023 2:37 pm, Jan Beulich wrote:
> Rather than doing a separate hash walk (and then even using the vCPU
> variant, which is to go away), do the up-pointer-clearing right in
> sh_unpin(), as an alternative to the (now further limited) enlisting on
> a "free floating" list fragment. This utilizes the fact that such list
> fragments are traversed only for multi-page shadows (in shadow_free()
> and sh_next_page()). Furthermore sh_terminate_list() is a safe guard
> only anyway, which isn't in use in the common case (it actually does
> something only for BIGMEM configurations).
>
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>

Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>



 


Rackspace

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