Message-ID: <9a01e87c-9c03-ea72-4115-0cdeb064d1e3@oracle.com>
Date:   Fri, 17 Dec 2021 14:23:02 -0500
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:91.0)
 Gecko/20100101 Thunderbird/91.4.0
Subject: Re: [PATCH V4 0/6] xen: Add support of extended regions (safe ranges)
 on Arm
Content-Language: en-US
To:     Oleksandr <olekstysh@gmail.com>
Cc:     Juergen Gross <jgross@suse.com>, xen-devel@lists.xenproject.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        Oleksandr Tyshchenko <oleksandr_tyshchenko@epam.com>,
        Stefano Stabellini <sstabellini@kernel.org>,
        Russell King <linux@armlinux.org.uk>,
        Julien Grall <julien@xen.org>,
        Bertrand Marquis <bertrand.marquis@arm.com>,
        Wei Chen <Wei.Chen@arm.com>, Henry Wang <Henry.Wang@arm.com>,
        Kaly Xin <Kaly.Xin@arm.com>, Jiamei Xie <Jiamei.Xie@arm.com>
References: <1639080336-26573-1-git-send-email-olekstysh@gmail.com>
 <1428a0be-b80c-f996-1f72-6545dd66c1bf@gmail.com>
 <81c97e6a-f5e3-3f5d-c9ae-a35933cf4e61@oracle.com>
 <4cc296cc-c7d5-1bce-4074-b708a18d9304@gmail.com>
From:   Boris Ostrovsky <boris.ostrovsky@oracle.com>
In-Reply-To: <4cc296cc-c7d5-1bce-4074-b708a18d9304@gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?RTRvY1B2cUllZTJiNkNqd210V1dkVjF0VWRERkVmajZoK1NKWnh1ZkJRcE1M?=
 =?utf-8?B?djlPVGkwY3FYK0p6MmxOVXNtVndJTWtWSmxQTjJldEliVUw0TzAwcnlOK1A0?=
 =?utf-8?B?UlRUTzVDWGkzMHEyUHdVUlIvU0hIZ3U3TEpXQlBFZzl0aU5VdGZKZzM3ZEVj?=
 =?utf-8?B?RTRSMGlLZGNMZnBVVmpoaWNRQUU0YnFIYm5ZcUFwdFk4dG9SbFJkRmpHWk1l?=
 =?utf-8?B?NFVBaU56eUhocXNzbDVza0ROOGNZRjZ0SURaajFEbjIybjZldjl2dUliamdG?=
 =?utf-8?B?dno4TlUwaTFBallGa3RSNUpORC9FMkZndXJKRzhucmVVZCtKRjNRNnhoT2FZ?=
 =?utf-8?B?aC9pQStMUVl3WXJwUFhNa0ovV0c2UHR1MGx5R28rdVhYNVM3TFdLMzErVmdt?=
 =?utf-8?B?RXgreGoyNGt4Y2h4L0VlcTY1cHM0UXhDdGdoTEtPRzRkbVdSZGh3ejUySmpS?=
 =?utf-8?B?bGhYS0hzOTZwOWUrM2QzVFF5S0FERCtsbzlocmhpZkpDazZGWTE3QVdDVnNJ?=
 =?utf-8?B?cnprcGEwTDVxVGtmV0hKVHpDVXpLaEI1Q1YvWXFVN0pLbkswNmVTa3I3Tmxv?=
 =?utf-8?B?TG44a2RPQ0JqbEJjcnRKUzhMNnhXM1FLTGxxWlpMNkhXSm02QkhaMGQyVU8x?=
 =?utf-8?B?R05uN1p5UWZKNFhTUjRieWMyMldVWis4OXd6b1YwanNSRkZTcXp2NnVkTWwz?=
 =?utf-8?B?WXFTMk9XZk9VTkwxNjRDTmVXbTBuUE5lYWlOV2hwVURoSmRENGpGL1NGQkJs?=
 =?utf-8?B?TFZnQjRSV3lxREZscnVqa1FFQkM3KzcrbUN6WktGOS9HNnltTURYbWlPM3Nl?=
 =?utf-8?B?cS85RTNzUXYvT2dmdElXaGZiQ0ZIK09pbWhyMTJWMDM5TE01c2RHa21MQnFX?=
 =?utf-8?B?d1d4a3F1RXJDOXBrZUhxajNJRHdIVTQ3UnduTjEzdS91cXVQV1RDK1BoTDJ0?=
 =?utf-8?B?UEhNenM0dllYVW9iQStMdzZkYkhVRGxNNkxpSzUyZE16bFRqQ2R5dlg0MUZm?=
 =?utf-8?B?TjdscnJ2RWdibXMrSmhnL1FQcmJaK29OcG9QSTFkVER3cWNQR3dyMzNSRUYy?=
 =?utf-8?B?aHhuV2lYQmNYQmdKVmt3MnBlNHZoKzFSUnBuM1RxRFNDLzZhbWdRWVRhNVdT?=
 =?utf-8?B?NWl4aDFUVWNwL0p5cXJYM0JPQ01XK1Y4SE50OC94N0diVUNJTjh4elAxbFVm?=
 =?utf-8?B?ZkR5Rm0vcXpXcmxWMEZGVXZWVlQvZllhMTVHVjNjOWpXNEZHcEZsRWxYVXox?=
 =?utf-8?B?c1k5WC9EMjFrcWJUKzBUeVNrUjBvODV2SzZ3MGZqdFVLYmRteng5c1hRQ1JJ?=
 =?utf-8?B?eDRrQXlPOE9iODNjdzRRZWwyMFpEY0ZoenM4S0xOTVV2RW8zaUJCdzNqdVFK?=
 =?utf-8?B?WFdTampESmtoa0h5bHFNZ2ovcEExZTFIMDJiOERLRHJrZVU4enV1U0RCUTlx?=
 =?utf-8?B?S0JOSGtEWDArdXgvSmNmWUlhdUhtaHBmSDhrLzRibUpCbjNGWFV5MFdaRGRa?=
 =?utf-8?B?K0JCR2tMY1lKZ0pYWWdHQTJDbE9xdUsrNCtKdi8vOHpFSGlDcmJOT2k5Nkcw?=
 =?utf-8?B?ZGlqS3ZaMWsvN0NkNXJvRFR0dzVWVldPTnRvMEVtMTRyZnNRNE9MVXRvOUo0?=
 =?utf-8?B?T1Jvam12QW5QTWhRY0ZsSlpkTjlDUHVhNzZwcGVFektQM1B5U05xQmFVWm9z?=
 =?utf-8?B?THJGZExxRWgzdG1YSG5xUEI3SGxtZVRSUDh3MzN1VWtzbG5iTm52WEQ4M2Jh?=
 =?utf-8?B?aVNxMXJoeVpRNHRnT0tvbklFUHZNdjkrMUprNHBSelQ0eXJlcW9nM3BickV5?=
 =?utf-8?B?ZkVLSU5OSEt4UUxVSUJyN3lNMjhvUnY1NFJFMXZoR2lFTG5KOE1uRUZFVmt6?=
 =?utf-8?B?UzBXTVA3VjQ0aFo3MFJIemd0ZTJudElZWVpFRlFmQlRDVXhKeW52NllKUlk4?=
 =?utf-8?B?ZnBCdVJoM3hIYzBaSkZtVFlqbFFjeDFFRWZqVHVaQ0dJcXNDSndKanJpQkZQ?=
 =?utf-8?B?am5sejVXQ1c3ZW1XV2ZGTlErcWFVMmZDU3hTYlQ0aTF4Z3FQZXVKcXpueVFW?=
 =?utf-8?B?SWtQbTVqV1RnRElGQzV0NkFXTW44VWdpa0hmY0graDV1blQ4bTFmZ0ppcUtp?=
 =?utf-8?B?LzdSdTNxQ0ExKzFPTG1Ya0NQVTBxRzdiQkZDekV5THlRRzd2NXFYZlRYZ0NR?=
 =?utf-8?B?UUM0eHU1MDZ1bkRjNUkrVDNST09rWTh2aHBMV1AyVDUyVHhYYXVuVFdPL1RF?=
 =?utf-8?B?Tm1halpBMUxYRjZJVTVwRkhETnZnPT0=?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 0a43832a-8294-4ed1-2abe-08d9c192a869
X-MS-Exchange-CrossTenant-AuthSource: BLAPR10MB5009.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Dec 2021 19:23:08.6417
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: RAkp6soNi4anjumKYtlfY1UUtH6v1Woe1k+HEeSlI7zX0QA0mAUAoj8mU4dXiG/pah8I8pU0/+z6Qr8RgwFl1QpEs+7A4pPA8M/cDJIlnfQ=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BLAPR10MB5252
X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10201 signatures=668683
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxscore=0 bulkscore=0 spamscore=0
 mlxlogscore=999 suspectscore=0 adultscore=0 malwarescore=0 phishscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2110150000
 definitions=main-2112170110
X-Proofpoint-ORIG-GUID: HAN_Fd2cJPnzTnC7mERx-z50_X9d0FKR
X-Proofpoint-GUID: HAN_Fd2cJPnzTnC7mERx-z50_X9d0FKR
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 12/17/21 2:19 PM, Oleksandr wrote:
>
> On 17.12.21 21:13, Boris Ostrovsky wrote:
>
> Hi Boris
>
>>
>> On 12/16/21 5:02 PM, Oleksandr wrote:
>>>
>>> On 09.12.21 22:05, Oleksandr Tyshchenko wrote:
>>>
>>>
>>> Hello Juergen, Boris
>>>
>>>
>>> May I please ask, are you happy (or otherwise) with current patch series (I assume, especially with commits #3-4)?
>>
>>
>> I think I mentioned last time, in patch 4:
>>
>>
>> +    if (target_resource != &iomem_resource) {
>> +        tmp_res = kzalloc(sizeof(*tmp_res), GFP_KERNEL);
>> +        if (!res) {
>
> Yes, this is already fixed in V4 of patch [1]


So this email did escape. Yes, I realized I was looking at V3 but apparently didn't hit ^C quickly enough ;-)


-boris