From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 4DAF7C433F5 for ; Thu, 10 Mar 2022 17:07:02 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 090D783ABF; Thu, 10 Mar 2022 18:07:00 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=reject dis=none) header.from=seco.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=seco.com header.i=@seco.com header.b="rHhJBAHq"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 9A85D83AC0; Thu, 10 Mar 2022 18:06:58 +0100 (CET) Received: from EUR01-VE1-obe.outbound.protection.outlook.com (mail-ve1eur01on0602.outbound.protection.outlook.com [IPv6:2a01:111:f400:fe1f::602]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 187A483ABD for ; Thu, 10 Mar 2022 18:06:55 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=reject dis=none) header.from=seco.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=sean.anderson@seco.com ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=LwkWQ2yG4qSFGugB96Hfc1aq5o7WH8xweBAO0j96InSnsSoi3SCdJWdCpT8OFPGsDx0MtY18fFDgCgqSA2pAMasoo1aIZ++GZmwowrydSP/E/5VKfLSox8Ip1qijXeDr7+2+805H4JFtXhpz+o6uNGBoVuWx3KgLWWqwj75pcHZW0zmfTDf8RyAlItXS1noeJFKAiu7chj9dyJwgkMFk9PP7X7cBLDwY6NLNL4JExpdl7KZ6oqvRhnvSS9LS1tmg90PPHoaAL8Vbk4hW1059d4D1mdzZg2n/Q3MuJi1rJ15NxSrz0KbjgGQNyHBHBZW2iESfG06XfMHBHOu5m0m/+g== 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=2qlExhmES5Jcp6APtnPPHg8kKSLet5BfOGuX4nqYAxk=; b=gWE82PAaN6IefcNu0NtUTJIRp8Yh5kXRXS9AkDvsRCx0gHtIZ+VhLdDhUHeSIwLfEuLenLx+mxmuDECgQhDu7TYpH/hvoVHEWUelXOA1PmWNQlbKfAirQ6sT71UL4bl5VIEItAfGZTgrDVn7RiaX1m+c0/lhYEO8K0h/HvhzqQ0WaS9Wj/Atnikrj1W6h7RHQiurUK4x7RwoFJ2i8V1vBCNUpjawBGr1QXGSZcJBSlwpFyge4p/AQS7SIXBuMaeqkY4TjWV5HRHm/TbFtaq3AmUXYvWL/99r2PJ6FxWtAZl0bFrk9jtdlixo1nKxkrD/PeXbgUDW2dufUDVzQnzKUw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=seco.com; dmarc=pass action=none header.from=seco.com; dkim=pass header.d=seco.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=seco.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=2qlExhmES5Jcp6APtnPPHg8kKSLet5BfOGuX4nqYAxk=; b=rHhJBAHqnrWFgTMsHG7RaIaCbkxBi3P899Wg8G63PErJbfBRTddBZUS/YP9F1LU+BXfX4IQ9ggcUNdufJ50bUYXY99TZJZ8+AP0EhPIT1V93FahsQtFoMkgquvMvmLtLk2jA2Hk8ph9iTzQyQVIfgdaYuEhydN/vq3zwIln1St35qIBcHHBKgxQhRYpmP7fMmBAYLjYmXOIiuQjWQ4hcs/vyHrlMSDnxAGi/ap6ftu2vVgzzf60O4/vp5oockM1P84Jz7Ku4FJRlQ3QLgAp4pjffkVZYFGw0RrRgtmNZ9MWZEYIWVj0f4V3eolfBio+GdLYjYk4wQLJR3R/b4VjMbg== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=seco.com; Received: from DB7PR03MB4523.eurprd03.prod.outlook.com (2603:10a6:10:19::27) by DU2PR03MB8121.eurprd03.prod.outlook.com (2603:10a6:10:2f0::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5038.15; Thu, 10 Mar 2022 17:06:53 +0000 Received: from DB7PR03MB4523.eurprd03.prod.outlook.com ([fe80::9860:b15b:f166:5896]) by DB7PR03MB4523.eurprd03.prod.outlook.com ([fe80::9860:b15b:f166:5896%3]) with mapi id 15.20.5061.022; Thu, 10 Mar 2022 17:06:53 +0000 Subject: Re: [PATCH 00/17] arm: semihosting: Cleanups and new features To: Andre Przywara Cc: Tom Rini , Linus Walleij , Simon Glass , u-boot@lists.denx.de, Mingkai Hu , Priyanka Jain , Rajesh Bhagat References: <20220303204400.2787389-1-sean.anderson@seco.com> <20220304114753.23bc4666@donnerap.cambridge.arm.com> <120ddbc5-f452-422e-cba5-6ffd0faf4ed1@seco.com> <20220304184637.GZ5020@bill-the-cat> <95c128ae-652a-0e00-695b-47b6a2d150df@seco.com> <20220310170150.01cd713f@donnerap.cambridge.arm.com> From: Sean Anderson Message-ID: <7f89b90e-2ea8-2c84-e77e-aa0942116538@seco.com> Date: Thu, 10 Mar 2022 12:06:48 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 In-Reply-To: <20220310170150.01cd713f@donnerap.cambridge.arm.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-ClientProxiedBy: MN2PR14CA0004.namprd14.prod.outlook.com (2603:10b6:208:23e::9) To DB7PR03MB4523.eurprd03.prod.outlook.com (2603:10a6:10:19::27) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 43c2ab65-1ca1-4006-1357-08da02b86000 X-MS-TrafficTypeDiagnostic: DU2PR03MB8121:EE_ X-Microsoft-Antispam-PRVS: X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: jr2ABVeZnEtxrt07IbEtQfV0xD8617pvnhweJSmYYfWe9IqIYjVXF/kKTHthisoq10eXKf9P/S3m7ff+RP0FDtZ3Z0/W3bXzibsrvKGhSCuB4DJvelPERdzhEWw1ckIshzKJWDI6fllkdxCJsaIEfKVccfe3g1QVkHoNlGwU9LDJ+IdI8xqBI5/8x6y4Xa/4CrpvKaUFb9EuJxjrak9cXT0fhTj0oualmlcuzJRifsPGW2GZQJfnDrasgLT3FHkGLV2Zw7Ev1d7M9xQixhJE3uyGt2ScC2pCCUAXl+aQcdDrUrVPJnhfqbXg7F2LmP2vcjLT35Aj0zlCGMB74XMu5hhLscJ9hKF5tcyq6SyBa7E9YkNHU/uU2yngUvp3tRM6USNmbnu0iAV0pfssptv120YwmSvMEGZzsMct8zjAh5zh3RDTWx4Ar2oibgnsY7Ak3+9euxXTymkzw2Wph6AFL+scCE6hmwgWMGj6WCpHUMrwRt+bCFnETaIIp5COpScFlD95VK8/L60zdFIE5ZaB04+1qFoHWFeXMl0JavbP29Bt/IS+EWJ28rvhUs8ETI0PeFnEPeZrp8e+NsW7W1wOcS+JT/vqDZO3vhet4aNKQeLKduyDGGbNvTQtxC4k/M5WxO+/oGSwS5A+rjbPiFWOC4Wws6Sx5khFKdHtogoRMUHTogMM6F+LbNMbcHvPqxFPcBJwaQoIMWldJVEcmTxAlI77gFyESABkBwZYP3lvkkxYQCew/GpAVXwimPRbN2jnT2eJoRNhwH1jHuemwBHlRw== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:DB7PR03MB4523.eurprd03.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230001)(366004)(31696002)(53546011)(6512007)(508600001)(6666004)(2616005)(6486002)(186003)(52116002)(26005)(316002)(38350700002)(38100700002)(83380400001)(86362001)(66476007)(8936002)(31686004)(5660300002)(8676002)(4326008)(66946007)(66556008)(36756003)(44832011)(2906002)(54906003)(6916009)(6506007)(45980500001)(43740500002); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?V0JBb0UvUVJyTVZaUVc5ZEppN3lhVGo3VXJDdU91NGtiREcwYmpnL2NEdzJM?= =?utf-8?B?aDZsQnl5SlVtS1VBMUtoT3pWZHZzVFJod3hEaXZVMXNaWk4xRS84bElhczFy?= =?utf-8?B?WmZnc1VvM1FkR2NxSktrbEZVLzhBZDRsT2x5RC9UNlRDNDdjN1RFYjNXL1Qx?= =?utf-8?B?dnRmYUk2dDV2ZTJUL203NUxXdmNQS2ozYmttNUttbWpTem9kUElrV3NwS28y?= =?utf-8?B?RGc5M3NDanJtdzNyTld5bFE4c2p6MVdiajJ6dzltengvUVhmUTcyOUxvNG1q?= =?utf-8?B?SU5tdE9SMW4yZjlseGp6TnNvRGN5SkY5YmwwdXpoUEtla1BWc2p6dnVlaTUx?= =?utf-8?B?bGZCa0ZnMGh2N0ZYdFB6YVVPWWRzRk83aUhicWt5VUtLWGN5eDZNdWsyUE5C?= =?utf-8?B?UUpPQWxnUURTUmw2ZFNCSzRKL3VsSFFqS1lnZzVBNmNQc1ZzdzQ3NTVnMEYx?= =?utf-8?B?UlJPdWg1WFhTZit0RnZEMFdFSmxGckY4MGRrVFNNS1RSeGhLWkltaU1UVXNx?= =?utf-8?B?TVdrYXM0eHRrTWsvcEJTWi85dkJtUEsxMzgxaUZDRklMNzFDbmVIM0lDREhU?= =?utf-8?B?U1lOVmpyNHBraDU1TG9YNXlkZXEzdU1za0hobUdjWVZoeStabTExY01QSEhS?= =?utf-8?B?R2M3TE5wamlIbWtnSm9QY3QxQ3RXcllTbm05WlVESUJJUjY5NmJ3RGlJa050?= =?utf-8?B?V2FWaXJmZG4zc3dOTTBXSis5a2tZaFhIWWZBb0UwZ1g5c05tK0FHNlJsQzh5?= =?utf-8?B?RDVlVjJEekgwak8rOGpqZDNYK0s0Y243bUlic1RPU3NCdEtKd1lMWHF5Q1JT?= =?utf-8?B?Vi92OWdVc2tIVDdwNmw1STBCelZiU21lRUNSano4akd5YkhYeFcraHZqNmRF?= =?utf-8?B?MWt3dXQ3OHg4emF3WHVaV25vVFJKVE9wNGsweVcybE1veVBXaTNIUVFMajlk?= =?utf-8?B?NGI3OU5qQXpDdUZ1TGJEWGUvRWgwY3gwZGY3bjJncWRqYktDcTk1Rnp3Rjlt?= =?utf-8?B?QUsxa1FzWUNKcHpYekR2eWU2NUkxeERua29zSkZ5aU9ZSmFmS0hMbVM5RGc0?= =?utf-8?B?elpERitKbk1ydzJTRHcxeUNVNUVKSnkyU3pBWjhpN0MvUGc5c2plZVFQWjJK?= =?utf-8?B?dkcreGVvSUwxdWdXWG96ZSs5WjJmMjBrdW4rTldRL3IyQ2lYOEhwcVA5Q0dO?= =?utf-8?B?STJSSVlUSGljcXZQYWRuTEtTMnEvSWNSblRyTjdacU1QN0lGVlNtRnVuc3dX?= =?utf-8?B?dWR1cFJaVTB1WlAwaDh6WUtvaXVVdWZNOUxueEhhSDk5MHJpUmpQOGJQOFA3?= =?utf-8?B?bm1DcTl1S2U5OXE0dFhmTU5hdmpiMGt5QlBWWDlUeW1Lc0YvNUJ1RVRncWY2?= =?utf-8?B?N3BjL1JVR1dsSlNvVnRKaVMxTjVXSFBqUkI2NWsrN3QrNDA1U2U2T2JaT0xk?= =?utf-8?B?MWVHb1hrcXRGQzR0elZCN041VFdGNDhGUC92bDNGSmhScFFnTWxtazhVcndG?= =?utf-8?B?d2lLNlFTOG9Obk1TUmtsTThmNDQrNXZ4MzdyKy9tZXZOUFJ2R0YrL2lpQkdL?= =?utf-8?B?Y2FDbTNyaGhtZWNPSGJpUlprNkpScS9rN0krdWlQdmk0ZGYwbU1QeDlBTWxr?= =?utf-8?B?bThBT3NCRGVWWEdwVHNqSkFTZ1hQckFhZWxrZUQrdkc3QUh6RklXVFRoRmpP?= =?utf-8?B?V2J0clN2S0UxMWoxdkg3NTFJUDE5U2haMS9DTDJUUVZJWWU3NDdDcGV3VHhs?= =?utf-8?B?RVRwUURtYytmNkJldTAxTk9KazZJaUtHYlZSN0Zqd1hVQlozUEsrMTRleTYy?= =?utf-8?B?ZG5kdTNoS1dMREQwbTRsSFRHazRHQ1QrRS9DWnFXZ1ZUekJEaGVIMElja1lD?= =?utf-8?B?UlRuNEJnY3NnM0NWaWZXcXQwV1pPWWQrSWJvTVYxdWJOTkQvcG9lVXJLMWVr?= =?utf-8?B?S0RxTm5BOUs0YmtPK3FoaWdkR2wveXVCS2F2TUEzQU9PODNsN3BsTVp1V1Rt?= =?utf-8?B?b1VNN2VWV0RoL2lmOXhZdTVaSVRDVzZaa3BRS09jMXpUdUFJOWJqQVN6NXNn?= =?utf-8?B?TWF6bDZRWnRTaFc0a2l2UEo3UncyZUhXa01ma3JnamFwak1mL21VTG5pRFdI?= =?utf-8?B?MmtLVnhmSUxpQjh1OVh0VHNSY285WEZKbFdVdVFIM3lFTHM1Nm5PWFdIaERX?= =?utf-8?Q?ltp6wWyPn2VN2LWbI/AZI00=3D?= X-OriginatorOrg: seco.com X-MS-Exchange-CrossTenant-Network-Message-Id: 43c2ab65-1ca1-4006-1357-08da02b86000 X-MS-Exchange-CrossTenant-AuthSource: DB7PR03MB4523.eurprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Mar 2022 17:06:53.4983 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: bebe97c3-6438-442e-ade3-ff17aa50e733 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: b7JvOzeedOMvtCtlYw0dRHr78CRM3gnsi+Dtn6OSg4IgGHc4FOBgzetO0fTEdEq0cROtTo2qKyIRcs6c9aYt4g== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU2PR03MB8121 X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.5 at phobos.denx.de X-Virus-Status: Clean On 3/10/22 12:01 PM, Andre Przywara wrote: > On Thu, 10 Mar 2022 11:48:50 -0500 > Sean Anderson wrote: > > Hi, > >> On 3/4/22 1:46 PM, Tom Rini wrote: >> > On Fri, Mar 04, 2022 at 12:19:27PM -0500, Sean Anderson wrote: >> >> Hi Andre, >> >> >> >> On 3/4/22 6:47 AM, Andre Przywara wrote: >> >> > On Fri, 4 Mar 2022 02:06:03 +0100 >> >> > Linus Walleij wrote: >> >> > >> >> > Hi, >> >> > >> >> > Linus, thanks for the heads up, much appreciated! >> >> > >> >> >> On Thu, Mar 3, 2022 at 9:44 PM Sean Anderson wrote: >> >> >> >> >> >> > The main device affected by these changes is vexpress64, so I'd appreciate >> >> >> > if Linus (or anyone else) could try booting. >> >> > >> >> > As it happens, I am sitting on a series updating vexpress_aemv8a_semi, and >> >> > letting it use OF_CONTROL. I will send this ASAP. >> >> > As it turned out, the automatic semihosting boot command is actually broken >> >> > in v2022.01, the fix is the first patch in my series. This clashes with >> >> > this series, but it's easy to resolve. >> >> >> >> I, for one, appreciate that the boot command is no longer all one line after >> >> your series. >> >> >> >> Simon/Tom, which series would you like to be based off the other? >> > >> > I think it's however it's easiest for you two to collaborate. >> >> I think it will be cleaner for me to base my series on his, but I just want to >> avoid having to rebase a lot :) > > Yeah, it should just be this one line, though, right? > Tom, would you consider taking patch 01/11 now, as this fixes a > regression? The Sean could at least rebase on top of this. Rebasing on top > of the rest should be more straight-forward, because it's not one > super-long line anymore. The patch which makes the boot command multi-line is [PATCH 07/11] vexpress64: fvp: add distro_boot support. I'm not sure how much that patch depends on the others, but presumably you would need to apply 5 and 6 as well. --Sean >> >> >> > From skimming over this series here, it looks quite nice, I especially love >> >> > promoting the semihosting load to a proper filesystem. The only drawback >> >> > seems to be that this will break existing scripts used by people in their >> >> > deployments, which rely on the smhload command. I don't really know if >> >> > there are actually any users doing so, but I guess we will find out. >> >> >> >> Yeah, I'm not sure what the stance on shell compatibility is. As I understand >> >> it, for the C API only in-tree users matter. However, shell commands are more >> >> of an "external" API, used by distro boot scripts. Simon/Tom, what's the >> >> process for this? >> > >> > Well, in this case we can be a bit more cavalier than typical because of >> > the very limited user base. So long as we have good documentation > > Yes, I agree. I don't think there are many users. The fact that it's > broken in v2022.01 and nobody complained so far tells you something. And I > can certainly handle a good part of the users internally here. I guess the > other half is in Linaro ;-) > > Cheers, > Andre > >> > around it including something around how to transition to the new >> > commands, it should be fine to do so, when the code is otherwise ready. >> >> OK, I will add some more documentation about transitioning to this series. >> >> --Sean >