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

Re: [PATCH 3/7] xen/arm: head: Add missing isb after writing to SCTLR_EL2/HSCTLR


  • To: Julien Grall <julien@xxxxxxx>
  • From: Bertrand Marquis <Bertrand.Marquis@xxxxxxx>
  • Date: Mon, 27 Jun 2022 14:00:40 +0000
  • Accept-language: en-GB, en-US
  • Arc-authentication-results: i=2; mx.microsoft.com 1; spf=pass (sender ip is 63.35.35.123) smtp.rcpttodomain=lists.xenproject.org smtp.mailfrom=arm.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=arm.com; dkim=pass (signature was verified) header.d=armh.onmicrosoft.com; arc=pass (0 oda=1 ltdi=1 spf=[1,1,smtp.mailfrom=arm.com] dkim=[1,1,header.d=arm.com] dmarc=[1,1,header.from=arm.com])
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none
  • Arc-message-signature: i=2; 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=nw/a06kpE9QmGdv1iXE6qe/tbKTvHBbhcSF5ltAIUnY=; b=VsuwpigZGgZ0OmX+2LhUlujvdHIbfBFZPCogcDIGmasGk2Vm+uCwbJKVEUQKrOVCapdMUmjLqR3VblAAEOfwD4Xycvx4+JPq0TxsqcAwTUqQZT+IywDiXxTBlpvSd6FkXjFI5l6M4PvVRSmXz1ENubIlKVzKeAnVRSgbqqjceeQidbdrSrUTp59/ylSL7YZDDy/5xJwTWIarIJFMn47aG2rCYWKYAQkbgjYmq3bPqZCBFlEkxgsujXjlV/tyhk05xh1WiYUscivAAFg+HDG5wIP01+IgcI1aTZM3pYs3saowKiqjP2pkdEQtWdUJn8BXSH0nC3Pz9k7e6EmLPKMaxw==
  • 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=nw/a06kpE9QmGdv1iXE6qe/tbKTvHBbhcSF5ltAIUnY=; b=K3xgx/i/GdNkwPr7qYQZD27NejfstKIfPL7grHLeCe1qSqAx3mwyFCXEuWQ1Y1k4OR1XgS/dCdWFuAsard/Uaz+4o2ZMlY9On1zIgJbfuS0Fn970NqXJaeY0Zygzq6TOKdwtbeT5UT+fVg1NdFvpc/jzThacaJNgz6KeViSu7ubSQ/HPkSH6mJAg3cCkYx4w4PlfiStDbaTRq8uezPnSRA2LamjNbQecTf+GVsbxf2hYA/pJCVkVtzmgotdNLnJrJI2NTwwFux+YTDI6XLpkHzRdgLqEFTNB91xc6s5e9HDuJfSX7MZ8CU34XX9BjbFD77lj87zWbku+M4I5BvMiIw==
  • Arc-seal: i=2; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=pass; b=P7RqHL4/mo6aC47eFDLm29W/KlJ81zKHE4L6GqGccx+3+i34RPwckwCYqY7zBku4z1VnaoVxlWviEUI9u8RNtoVO3CLVmO2pOATHfjLxfkQ6sXgHv3SxyDrPssiAyWKXKYHyTw8HOzkYgSsgoyA95vjBsohbqlOhEytIbWt5fh76bD5LM0z7ZmUGqNf252TBjun8lHgSfdCYL20jqNJlr7rVuajr13kBHVrTDRYfkjdimwC4KZ2E3OLvitdhKp/PJTRX8YCMW5rXDXVwO3eH4E+jgPY1I8OFyBM442lu9nfgsBXpqS/3iozx2xZY1jWDMd3EercKiuFhZCS+9VXqkg==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=k109H30vt1Xtlhw+BXY9DIyoM7p6A+lVCBQkBoc8DXEyCwlvLbajZvDM9W0Phzx+54zoZCaQK6RvteN7q6rbWqFgoDHUdrhRwDNpLsJ1GuFz2O/hRf74QXEOaDJephB1JvyAGSbeODdG4A31KOoceq/mr1hDWHMyNkDUuJU6RWbWgoOl3wAGaWk1xmFASMZ5W1/9PtFjGSc0Ra7jtqYyfbuu9kdq6zQtk7MRLsuH8bCXtHNvkJ153I8gBjTcWwc/72A+4qntXrMro2qR84mFZvaTI6gtPdSDywg20pX6CpsG3CDQuHdV7cLzHH0S42C/lxfXARdSpknkjkLoWfqqXA==
  • Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Cc: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Julien Grall <jgrall@xxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>
  • Delivery-date: Mon, 27 Jun 2022 14:00:57 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Nodisclaimer: true
  • Original-authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Thread-index: AQHYh6p3WB7MoRj1SkKsSuzOQQLxTq1jTTwA
  • Thread-topic: [PATCH 3/7] xen/arm: head: Add missing isb after writing to SCTLR_EL2/HSCTLR

Hi Julien,

> On 24 Jun 2022, at 10:11, Julien Grall <julien@xxxxxxx> wrote:
> 
> From: Julien Grall <jgrall@xxxxxxxxxx>
> 
> Write to SCTLR_EL2/HSCTLR may not be visible until the next context
> synchronization. When initializing the CPU, we want the update to take
> effect right now. So add an isb afterwards.
> 
> Spec references:
>    - AArch64: D13.1.2 ARM DDI 0406C.d
>    - AArch32 v8: G8.1.2 ARM DDI 0406C.d
>    - AArch32 v7: B5.6.3 ARM DDI 0406C.d
> 
> Signed-off-by: Julien Grall <jgrall@xxxxxxxxxx>
Reviewed-by: Bertrand Marquis <bertrand.marquis@xxxxxxx>

Cheers
Bertrand




 


Rackspace

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