Message-ID: <81c97e6a-f5e3-3f5d-c9ae-a35933cf4e61@oracle.com>
Date:   Fri, 17 Dec 2021 14:13:44 -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>, Juergen Gross <jgross@suse.com>
Cc:     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>
From:   Boris Ostrovsky <boris.ostrovsky@oracle.com>
In-Reply-To: <1428a0be-b80c-f996-1f72-6545dd66c1bf@gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?MmRDRUlzQzVCdnZOK1JDVFkwQ3BnaEZJWEpHcHJUa1lLNzVMdG4rUmhIMlJC?=
 =?utf-8?B?U2hrT0dBbVp6b0ltUkZ5b1RYcEZkSVlJbFkrbmtxZzZZOFJ0MzV3cW00WlVL?=
 =?utf-8?B?bEdHenFQc1VEdWQwNGhpV21nbmYzWWpwVEJGaEZ5bEZlV3ZwekVEU0p0K2dv?=
 =?utf-8?B?d0RYN1Vmb2NWZUxLMkRVMklRdmYrMFFTN2hGQU81ZisrK0xqRE8rWXBaelo0?=
 =?utf-8?B?MFBrZXV6dk9rVWhkVEN1NjZWdTlZSDV5VWhBcUV1TlRxY29uNGxKOGg2ZkJR?=
 =?utf-8?B?b1pHakZWUE8yNDRUMGE4VFpYbld0QXJTMFhBOUNJcUx1a0d6L1U4K3poTzdx?=
 =?utf-8?B?QkQ3VFpBd2pGV1l5azhNMkxBcFJ4bzdQYUlqTGhoOGdNWGpsK3Z6Qm1FZkQy?=
 =?utf-8?B?cnNlKzBVWVdNbDhJYlQ3d3lMRjROVHY1ZlNGUmkwb0F0a2MxK0RPeTMwV0JE?=
 =?utf-8?B?bDZVY2ttRmhpNXhub1VXdVpyVWo3RmQyKzBKU0U2YVdYYm5sYUhRRjlSb3NL?=
 =?utf-8?B?cC9JdWp6OUVmVUNmMnM1c05PWFhvUnU5a2M1RUc4T2ZGK242NVd2ZlRrdGdu?=
 =?utf-8?B?ZWFtdENJWElGOHA2NW4wRGFBV2h0UmdmRHlIQ1phWFVSZSthOVgzczZTTnNy?=
 =?utf-8?B?WTlydExEeVM3QzRHYlNuWDVGQjZNUW5rUGJhV2JsSERuL1I3S1oxU3IweDNB?=
 =?utf-8?B?RytQVng5N0tETWZCQWpIYm1HNzRMaFBCTUNQLzR4Z3VHN01rd0ovUjQxTkRk?=
 =?utf-8?B?RC8xWGRFTGJvZnB5NGsyRFRwbEt0U0orZlJYNThnNXJUZzdHZTZWOEVSbmRy?=
 =?utf-8?B?eVhyWW9RcXYrTUNGdk82MUJJL0d5ZWVNa3UvMlZuNXdrRnVadGZvNkE3NnB1?=
 =?utf-8?B?dC9KZ1dMZFo2VlhrMmdRV0FvaHQwVGRiK2cxMzcxZ0pHSVdnNDVRMUFCcFRa?=
 =?utf-8?B?T2Z2d01hNnBKU0MrTkRLQ25tVVFsd1o1czZxMk1KYm8vOExYVVErSDRGOGQw?=
 =?utf-8?B?V1BiYXdvaFE1d3ZLY0dqdjNLNEhKMnhpZEwxY2V3UWtqeHZ6STlvbnRaSjFp?=
 =?utf-8?B?d2dCQ0o0VDR2MWlWSGlITFBvSHRJUnNQdUdnRGJlZlpKZ21SVDZIclNGcEJO?=
 =?utf-8?B?cVdqa1FRWW4rOUExenB3aU93Q3ZabnhZSXRwTzgzckY5NnNLUHpyZ2Z4YWVE?=
 =?utf-8?B?VEJYRSsxcmRDaExwdnhldDgyOVdjYnlWVjhnbTVlQm54Z2JjdmxZbkxCZG9V?=
 =?utf-8?B?RXF6MUsyR1hiakJ5ZVZCMitFd1UvdE4zUTROSU1VclJmL1k1ZHBVSXpiK0Q5?=
 =?utf-8?B?N01WRVF1Q3N2b2Q1MXd2RWZEOEVOdVlLN2FwK0pGdE9NN0cwbmdSblc4QVJs?=
 =?utf-8?B?bi8vQjhqUDRVVG50QXo4U2Jhd24zUTlaMFlURFduRDJhREhKN0pPSHlMQnEr?=
 =?utf-8?B?SmdRaFZGS2lwbFA5SzBmMGorUlFseFFIREJrT1JZeGR6ZmZ1Y0Z2Q0lDZXpt?=
 =?utf-8?B?TjRPWTRDaW9JMHRQTVdOQjU2SXJQd3BGeUROSTdCZVQvbDRnN1praXJEL1h6?=
 =?utf-8?B?VTlLZEFrNW9NSHA2TXhyNU92RE9TalB2SGViUWR4VWlKemFlSGk1R21NakFJ?=
 =?utf-8?B?NGhLMytVbnR3NDNmOWd5Q1ZFRzVJWitXeEd4Nk5pSG9kSnpOTndOQ0s4QkJv?=
 =?utf-8?B?aWZPV0EzV0pxdDN4cjFWcDEyYnBpNFBmYjlCeHJTMHMwTWpNU2dEUG12bEpp?=
 =?utf-8?B?NzdCeURHRDcwYWcyTXl4TENTakJmVXJZcWRYN1hjNDFRMnFsQnRJQkV4RllM?=
 =?utf-8?B?VW4yQUhsbENZUW1BSThoSzJ0SkdXQ2NTNUtUZXUzV1diV25PbkU4QStXeGhD?=
 =?utf-8?B?RTB0TUw1NlZ1a3pINWlzTHFpK0t4Uk50Q3doclRaYm8yTmJnWlhia2J6a3NX?=
 =?utf-8?B?dW8wdXo4WHdESTNmSHpZNXNGMHNIYmxxOElXOVlwQXRCd04zblI2ditjODVX?=
 =?utf-8?B?bEd0dXdpWnBCaXRDMFRtTzh5bnl6cmEvcVdINFBwTmJSWEZoNXJJSjJEdm1W?=
 =?utf-8?B?OFdyeG5kWGFPREk3YlNPaG9SaUJlOXdOb29FMVdlY3JaUlFET0dlZm5VdnZ5?=
 =?utf-8?B?MmQwZ3BDckcvSXAzNnMwK1IyRnVwSnF5QzBxN2NwNXdkMTNabVdOVFFISUlt?=
 =?utf-8?B?SkZxT2dMOFdkSmI1bGp0S2p3MDQ1NGJ4TVhaK3BFRW5zcnNGWmw0bU5UTzlW?=
 =?utf-8?B?NUR6T3FuR1llVGVRUXRpakNBcEpnPT0=?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: d3915c82-3f03-472d-357d-08d9c1915dc9
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:13:54.0090
 (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: VPC4AxeABJ2JWoLw8v5yF3VmYs6vUy0OqTLtFqct6OKDU7Ds8EFLLUKSxQxShEgHWgLc1ynVDqgceTXZ5hH+/MymwiQgMcuTUSTZkyWnWh0=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL0PR10MB3043
X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10201 signatures=668683
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 suspectscore=0 adultscore=0
 malwarescore=0 bulkscore=0 spamscore=0 mlxlogscore=999 mlxscore=0
 phishscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2110150000 definitions=main-2112170109
X-Proofpoint-ORIG-GUID: 5B3jgs5mnv_TE7aw-EtQjKJ7RwoJULb9
X-Proofpoint-GUID: 5B3jgs5mnv_TE7aw-EtQjKJ7RwoJULb9
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


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) {


Other than that --- LGTM.


-boris