Received-SPF: pass (google.com: domain of linux-kernel+bounces-122063-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
From: Volodymyr Babchuk <Volodymyr_Babchuk@epam.com>
To: Konrad Dybcio <konrad.dybcio@linaro.org>
CC: Caleb Connolly <caleb.connolly@linaro.org>,
        Bjorn Andersson
	<andersson@kernel.org>,
        "linux-arm-msm@vger.kernel.org"
	<linux-arm-msm@vger.kernel.org>,
        "linux-kernel@vger.kernel.org"
	<linux-kernel@vger.kernel.org>,
        Douglas Anderson <dianders@chromium.org>,
        Rob
 Clark <robdclark@gmail.com>
Subject: Re: [PATCH] soc: qcom: cmd-db: map shared memory as WT, not WB
Thread-Topic: [PATCH] soc: qcom: cmd-db: map shared memory as WT, not WB
Thread-Index: AQHagIKviCrPH112HEOpsjr5BCtYWLFMDkOAgAAEYgA=
Date: Wed, 27 Mar 2024 21:04:13 +0000
Message-ID: <87a5mjz8s3.fsf@epam.com>
References: <20240327200917.2576034-1-volodymyr_babchuk@epam.com>
 <e0586d43-284c-4bef-a8be-4ffbc12bf787@linaro.org>
In-Reply-To: <e0586d43-284c-4bef-a8be-4ffbc12bf787@linaro.org>
Accept-Language: en-US
Content-Language: en-US
user-agent: mu4e 1.10.7; emacs 29.1
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: 
 =?iso-8859-1?Q?y/MNfGEpMZGTLDayiKWRS8JElzfJTSmfE2C/bjrUm4cz5OlGEZ3umhOvLY?=
 =?iso-8859-1?Q?ybWIpGsRj3aGFWN0Wxds5wEELNDF95yWbleiROf8ZQQ5gLffnDdAXhHABG?=
 =?iso-8859-1?Q?+u/1+Rm30rT/AjpKLu98Yf7hEUeV3ihvr2Hq963XBKlYSXpwQRwBux3j5Q?=
 =?iso-8859-1?Q?sao9H2xlHU7spfo2SWSnKUdMQwzEny1eZJIDWmqlreIRRKnMYhjfVMizUy?=
 =?iso-8859-1?Q?QnsVwAbCbAxblaYySIKHi0LyK3Lr1VhGbwUJi07lnq+XNTUUhEwi9URsxc?=
 =?iso-8859-1?Q?VjK6qcWoLBhD15J1/Y1LtWzY+Lrj6+/LvglGm/4ksdkT2kQRWZMWct+CCU?=
 =?iso-8859-1?Q?geCZ0x5Lx1k4qp+F1TKf9q6Fh51sRRG1l8V020DDpVqiQHE7Enj/dritem?=
 =?iso-8859-1?Q?pHyHvS2rH7Bt7cDwtc+QBYKcXNDPOvhADuwbIZo4EEOmtsJKMVfg13QBlP?=
 =?iso-8859-1?Q?oWz2wSVh+qMBUbGV3qAdpfgA9rgejxnENR5LYl9xyC4jM9DjKPBb5FgHcu?=
 =?iso-8859-1?Q?gLsciy/5qozvRAZBP6V1iphSTyQNFwa14JjTBnLZd1piStL3V6xHV5xvkx?=
 =?iso-8859-1?Q?rtPKzSDpCaeLyTjQCTtM5zB5vKC5QB30Bb37k1smb1x9NjuEGQATn7hLDh?=
 =?iso-8859-1?Q?07oY3XaGEp2U7Zrh98J9SXdk8Udm4rBoW8EXsBCJ/o/nvDXo7KQZINpCWr?=
 =?iso-8859-1?Q?bybHTOYwfEK6Yf4r93RNpzzCkA0KFkpAGX36jDi+y7fTrE5gnbgdMFsPrZ?=
 =?iso-8859-1?Q?WTF2K5iNFV+MysdjkFr8E2SKDVaA/QcHIGagRfRF9h8LmJ6P6R1pouWh2v?=
 =?iso-8859-1?Q?V05ljPD2WWvsyse3JVGiyn5WmIl/obsNJjydREO+QGBy8jzWZkasPjR9kW?=
 =?iso-8859-1?Q?ryXfeypeMGIDUsgKYn/v9KpfleVnMiEHcl+5Z9y2qNay5ljB4j/BPMonzy?=
 =?iso-8859-1?Q?DDfdWt/F+i7dJ5nqvDpL/yGiP1fDEiDtaF9ILvHOfl3h37otChmDNQFmgb?=
 =?iso-8859-1?Q?rUM55T3npb/nH09Mj1Kpp2zlKGLYqjKdhx1DkZ2SMuCaqC0WfqopIs/60S?=
 =?iso-8859-1?Q?GNtubu9Syh5uRWm9LWsAj7BkjZShy/WgvOHBl2W8NLhHHVtnxykFa86sDO?=
 =?iso-8859-1?Q?ReqTWqYhLeE140t0NAodzXOJsbcUU9D3YKYoDTNj6X0TV+2hyX9p6C7yIh?=
 =?iso-8859-1?Q?lHPrwOL27oTsGjiMTTPNPnX1UJfT3mh8yctKo4IFALyNi6Dlfbvfs9FzAs?=
 =?iso-8859-1?Q?3nTIBSibapZvqLB/F9NPJp+lbQcYgcUcminYCh+vKPqpD+vUmVHjwgDH9n?=
 =?iso-8859-1?Q?Bx7RhNnMXMMSS27fPGVwwSJwNXmEyzY/wo8V3qTSGB8vSvf4SykFsSJDXT?=
 =?iso-8859-1?Q?RqCbMznmCth2hLYsOZHv8fba3ahWIZNUSpiijsP5WCaLm18GS0RM+KYsb9?=
 =?iso-8859-1?Q?AAAB6hR+SVq8W6vZfldEPFXP0d5yumOILjuFS0AQZpgXfeAts2/+Luitpp?=
 =?iso-8859-1?Q?b2Ls8zKvBb/9QhbIzkYuaWZeNfABVoq3fpzdCgi9IHx1tebdq51dYy3EoO?=
 =?iso-8859-1?Q?7SUUut/2iVDCFXkWl6owTFa0D0FfiYJU+lKPOGhgD+IJ0k2+T9B/99NMkg?=
 =?iso-8859-1?Q?MGFcqyToe9+pRxeMEe/9JLto7/zu5UOpXCF+LrnFGBNBQJUOtvZVGFZA?=
 =?iso-8859-1?Q?=3D=3D?=
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-OriginatorOrg: epam.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: GV1PR03MB10456.eurprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 5b06d9b5-db31-4bc4-b5af-08dc4ea174a6
X-MS-Exchange-CrossTenant-originalarrivaltime: 27 Mar 2024 21:04:13.3141
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: b41b72d0-4e9f-4c26-8a69-f949f367c91d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: Ieyzz0Py4ChcHca82pe8Bmbw2/TOEpBF6lDNmyt+11XgUXpBK4PAH2k2JnPrMvJvxBUJFT4X/iN8x0xFabkYJY++TClVCTPE6ilVLl2AMlA=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR03MB7227
X-Proofpoint-GUID: D87HMTI_2x9DmDAUEDPMEOc7Oqksf5aC
X-Proofpoint-ORIG-GUID: D87HMTI_2x9DmDAUEDPMEOc7Oqksf5aC
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.272,Aquarius:18.0.1011,Hydra:6.0.619,FMLib:17.11.176.26
 definitions=2024-03-27_18,2024-03-27_01,2023-05-22_02
X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 mlxscore=0 adultscore=0
 clxscore=1011 spamscore=0 impostorscore=0 lowpriorityscore=0 bulkscore=0
 malwarescore=0 phishscore=0 priorityscore=1501 suspectscore=0
 mlxlogscore=847 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.19.0-2403210001 definitions=main-2403270149


Hi Konrad,

Konrad Dybcio <konrad.dybcio@linaro.org> writes:

> On 27.03.2024 9:09 PM, Volodymyr Babchuk wrote:
>> It appears that hardware does not like cacheable accesses to this
>> region. Trying to access this shared memory region as Normal Memory
>> leads to secure interrupt which causes an endless loop somewhere in
>> Trust Zone.
>>=20
>> The only reason it is working right now is because Qualcomm Hypervisor
>> maps the same region as Non-Cacheable memory in Stage 2 translation
>> tables. The issue manifests if we want to use another hypervisor (like
>> Xen or KVM), which does not know anything about those specific
>> mappings. This patch fixes the issue by mapping the shared memory as
>> Write-Through. This removes dependency on correct mappings in Stage 2
>> tables.
>>=20
>> I tested this on SA8155P with Xen.
>>=20
>> Signed-off-by: Volodymyr Babchuk <volodymyr_babchuk@epam.com>
>> ---
>
> Interesting..
>
> +Doug, Rob have you ever seen this on Chrome? (FYI, Volodymyr, chromebook=
s
> ship with no qcom hypervisor)

Well, maybe I was wrong when called this thing "hypervisor". All I know
that it sits in hyp.mbn partition and all what it does is setup EL2
before switching to EL1 and running UEFI.

In my experiments I replaced contents of hyp.mbn with U-Boot, which gave
me access to EL2 and I was able to boot Xen and then Linux as Dom0.

--=20
WBR, Volodymyr=