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 84986C433F5 for ; Mon, 10 Oct 2022 10:16:10 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id BC85084B06; Mon, 10 Oct 2022 12:16:07 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=reject dis=none) header.from=prevas.dk Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (1024-bit key; unprotected) header.d=prevas.dk header.i=@prevas.dk header.b="IiIwBHps"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id E2EE184E2E; Mon, 10 Oct 2022 12:16:05 +0200 (CEST) Received: from EUR01-HE1-obe.outbound.protection.outlook.com (mail-eopbgr130110.outbound.protection.outlook.com [40.107.13.110]) (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 83E8F84ADF for ; Mon, 10 Oct 2022 12:16:02 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=reject dis=none) header.from=prevas.dk Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=rasmus.villemoes@prevas.dk ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=gQz+RFvZy/WoeGr9N5MfoP7ttQ4YUkhaHWs41zf2zQo+qTEL3nJkR0HsIBc/FynCdsmw6R2p+dg6+EmzXefGi/ronaFsD/FdO+66GFrCpLcULZfPwA23Y0UB7hYepRlyq4siceQJQlxHBUArThn5Rdy2nYqde3gITtg/ZVPI6GRJn8ZTkVNgTG3OwGiWHf+9+ozz43p/gX4zKCy8GimGDx6K66mxHAazaOA3SXAiR6ep3/+ZWJifvwZvpgLmWbZNkXVq3Qb6yuX7eypR8HBkZ4ynOtF5bTUA2QbW7yosxUfuTtnMEcEBThobvqYb5bjndRKzQWVRVdemfh8SKPPLIA== 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=VTn66qSXMiXtMWxj+d2mxDCuUImQlhLLeDXDDRpAZ0s=; b=mcV0TmArK7U8+gxdIm8OgVQGOnWTwZBuL6bqsEgyg7aVhH4pCP0nbWHXwCX5kU1ce5kRragD0Y0asr2zu8KBq6pVFb1c8qObpzdtjpkIrbCL57iuIPlpwOlACGxIIKXIyaDa6jxloZYoSC3AUvnMLc/FH4wjmjAXe5PtoVLdMBQl3gPke5048YMSkGRDK/FpVoE6PJotffveUGmuFRUwJXHWgLrkRxIZTFN0jO59rWcy/WArnGs4MUUox9Pruc6UbojMuvkd5foRDuGWjbNoQQ9zCr8/5AYFI0eA8fi0wpEDRhDmAA/uK7EaSTlxJYtmOiS/srL3uOs9EZIGLZNnsA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=prevas.dk; dmarc=pass action=none header.from=prevas.dk; dkim=pass header.d=prevas.dk; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=prevas.dk; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=VTn66qSXMiXtMWxj+d2mxDCuUImQlhLLeDXDDRpAZ0s=; b=IiIwBHpsDzHT1X5cpJLgUpcAQVGGacnEWi2/+aGtTkrsEPouNztykU88n085V33HLKgu/dpjvbpmBljBsORrp8EJYlHrX6BIaz6w9LgXcsOP1rd/xvz5SltpIeVpey0bh15Si1MyHvrXiMao5NyAgVP4MmQWZ1NzXggKIu+OJPA= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=prevas.dk; Received: from DU0PR10MB5266.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:10:34a::22) by PAXPR10MB5734.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:102:248::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5676.28; Mon, 10 Oct 2022 10:16:00 +0000 Received: from DU0PR10MB5266.EURPRD10.PROD.OUTLOOK.COM ([fe80::dc91:193:719e:5fb0]) by DU0PR10MB5266.EURPRD10.PROD.OUTLOOK.COM ([fe80::dc91:193:719e:5fb0%6]) with mapi id 15.20.5676.034; Mon, 10 Oct 2022 10:16:00 +0000 Message-ID: <618ffc64-3880-393b-e13c-0908d8b577ad@prevas.dk> Date: Mon, 10 Oct 2022 12:15:58 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.11.0 Subject: Re: [PATCH v2 01/38] Makefile: Fake external blobs by default with binman Content-Language: en-US To: Simon Glass , U-Boot Mailing List Cc: Masahiro Yamada , Tom Rini References: <20220110031413.1970836-1-sjg@chromium.org> <20220110031413.1970836-2-sjg@chromium.org> From: Rasmus Villemoes In-Reply-To: <20220110031413.1970836-2-sjg@chromium.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-ClientProxiedBy: MM0P280CA0108.SWEP280.PROD.OUTLOOK.COM (2603:10a6:190:9::9) To DU0PR10MB5266.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:10:34a::22) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DU0PR10MB5266:EE_|PAXPR10MB5734:EE_ X-MS-Office365-Filtering-Correlation-Id: 980c2d39-de5f-4c00-1ebe-08daaaa86e2f X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: GhKyyuYsdg2AmIsZbmg0A835xClsUioIY2dy1CeZ0RLmSjSClyuC/B/XM6OwAtjEyWRCejKZrNSyBGWS51NOaKRDCryrLH3ef51NC6trEKCgY5ZLLgzNEUmmwZLN3TQKX5KuGVjl3+kr5fsHi/wXkPASf4o/A3MQ+1TDzV6qN4Lc/P6s7PadJ+nBsE5pha8DuO6Sfj44p4LzKTXCHypGmHGihXiau7cwqpTgpW2Irn2by08qzqhlRsoHxQb0XwV6iykEw4/nBF2mhKU4Gto+Y4kNl0n81+AJhzy+tOMTbGJAamQlt74XeglneH9nwtt186sh//hpaBJ0zx3wsj7viDCweUpfE5PudtO4H0EHtb5RQOaR+RD/dMwa7nJ3GMKjsaqYTpercRHGK/GAslUhHPq0Ui4ejFzmVbRaYJrJICeqeoNC93f0CNjGuYzPUprug0KE3KiOO05PVo3f2vE9Yqg5PHmkMifMS9uh6QRXwAagj3NDENUpe1+CNyOoqtj2Y6L3mnru2UCzqQMhTpO4JhUwSmf0ISRokthi6Moo2+NxsPCoUpA8rdP2N1SfmeKaovsBSnxwxOttFu4wmTzCzyA0GmG/0y4s7r/TZTRbmbgz1H4pirad8BXpGde9KQ/SjrlDx0lxz0GbwiWK5et3/pT9KJ0uHScsq0pfSIGYHomLg2ms2b41UQlk+lcazCjt9uiU30/9W9jYt9GNaXjdosdjBamVUnHXok5T08LCcqKDN35b4m5fx7TkDs7U6ieQXh3qla/3EXlVWR7m+ExXOWfVmo+V2pFNX5YDZLaih4zxqypDksjYEb/jklAvG30e X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:DU0PR10MB5266.EURPRD10.PROD.OUTLOOK.COM; PTR:; CAT:NONE; SFS:(13230022)(396003)(39850400004)(346002)(366004)(376002)(136003)(451199015)(5660300002)(2906002)(6506007)(41300700001)(26005)(86362001)(38350700002)(31696002)(38100700002)(31686004)(66946007)(66476007)(83380400001)(66556008)(8976002)(186003)(44832011)(6486002)(316002)(2616005)(52116002)(110136005)(8936002)(54906003)(4326008)(36756003)(478600001)(8676002)(6512007)(43740500002)(45980500001); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?SFFzeGVXL2JoeVB5aE9iYzVOVnJlWTIzWmFNNTBFTXNodmdFL0JsVUxPOVpR?= =?utf-8?B?ajlteFAvZVRReHMrTzR1Q01UVnVsbkV6ZXNuUFRYTTR2aUVBd3NzNGlVaUxT?= =?utf-8?B?NDlDYUk1Q2J5cWtOUVc1SWNVci9NVjY5NytBeGw1a0Rzd25jREowTFd2QjVJ?= =?utf-8?B?SkNQREk0OFhBQjVUTUhEREErd2pQamtYY1BieDc3b0pvOVQzNi90djJucVhV?= =?utf-8?B?RG9JWmJGOVN1bkt1YWxGYWdnTnpUekRXcDZSRURsZkdoa0FQemRSaEZyQkhz?= =?utf-8?B?cWtiV05rQlVyQTFyYmVMS3NiL3pnZlBNeDVzcm1ySjNDVnFXZW9uTVRtSFpm?= =?utf-8?B?SVhVVUM0bDFGbVhtZzdBaExNeDk2MVczNWMzbVg1MG1xR2lIa2dGTDRGNkVF?= =?utf-8?B?SHY1aUFMVG9VekltOWJLV01oRGRZMGFjUW9xYk04dTVRQ1dzVDV0QkU5aUcr?= =?utf-8?B?ZnltMHp6SUpzcjVhNmJHOTdDMUhhVXliQVZ2YWNhYkFnaFJ1SlZ5WW04b0k4?= =?utf-8?B?QmtubHhGUE5ORmpoSmdtUWxid2M2ZktmeUpFemNCUHV3ck9GWXdkeFY1WDFy?= =?utf-8?B?aW9aVE5BcTVYZHlBQ3JWUG5JWEIwQU90MW5XWXZpOVZ5dTVOd2JJZjRJTDl4?= =?utf-8?B?dHUwa1ZMdmxCK2tiUVhTVlpUWEpQUEdvRzQzajRjQlJRWTdEOC9jeWxmYXdY?= =?utf-8?B?Ynk2UnJhMVlaTXhsejhZVEVmbDdYREpwVXg5N1IzcEh3VFh3aEkwSXJXQk03?= =?utf-8?B?YzViMExWY1UvaGxkaTFXN3NMSXBhaHdoTmp2MUJJbWNXODdHSGJROXhlRXF2?= =?utf-8?B?aEoxOGdmRXZ4RXJTUnBCdmx1b0JxY2cyQmJlVHBmMHp3c1NrT3p1eUpPbHNo?= =?utf-8?B?S2lzRmgvVUFzNUt1S0xuYjE1elp2Vm9OaVhPMlhrMndGaVNpUEhzM04zOENX?= =?utf-8?B?N0ZVbGkvWW9ucFoxczg5NG04WTdUaS9lT0pBaFZrOGUrdHlBejUzb2s4WjVO?= =?utf-8?B?VXBpMnZqMy9TdzhPNFR6RElHS2NJK3ZMRFM4NGZyTG42YnJTaGtaSml5aHRl?= =?utf-8?B?S2xna1crRXV2eGgxdXhGblhIQ2U2OUlRVVFaQWQwdzZuNE5zN3FGSGZhM25X?= =?utf-8?B?MmtvWTFOdHBLblRrMklLWWxQMytpSGV5bExiYm8xTGlwb3lBVVNZM0FHUGNO?= =?utf-8?B?SG0zQ0RuSkRaRmFtU2tSaFhiVVVHRE1QN0hJSmxYS2I1MWhRNzB1czZoWE1x?= =?utf-8?B?Z0JPRHA5TXVhWUlDMmlYekNHSHV2Wk8reml2Q0Q0Ukx5ZmorTnhENkY5dE5I?= =?utf-8?B?alpVT3ZURUZ5cE8vcGpqcVRBYi9ZSU0rWWNoK3M3STNZd0x3Z1BBa1FQRHNq?= =?utf-8?B?ZzJZL0Y1ZUl1aGxYZ0NJMFBpV2JKdGV0RDdhQndDMjhqVnV6bnhoeDZ4dk9a?= =?utf-8?B?Zzc0T0RaeGpEWDhhN3pHdXhxY2hCZUtoM2dtOXRCbW43RUhuaEIzcHFkMnpD?= =?utf-8?B?ZWVWNTY2TUdmOFpBTkdqQlJuVCt1NHduMkYxWHdKSXRaUW0xMHBvVXREc1lu?= =?utf-8?B?Uk1FT3JKS1Jpa01ZL2ZvWURzQ2NrNFVPZmhuQTdvNityU0g3VXdVSVRlK2w0?= =?utf-8?B?VkxPTjZrMk81VkpLVjBBOFgzQ2F1KzBtSmZ6Z2wzSkd3bzdrWUl0Y1FBNnp6?= =?utf-8?B?NG9BRnpYNFVwMDN1MkVNcVB2MzFpMnJJMTNzREd1M2N0dzcwZStQMmJOTEs0?= =?utf-8?B?b2Q1SmhybUcvcjR0UnRFbXNoSkR3cUkzUWpwQklTdDhFQVpzUmJVWFRlQnJT?= =?utf-8?B?Rk9sTmpiTFRaZmFSRUxEK0hSV1duRTZqVDU3Vlo0R1lid1k3NHYxNzdjRUsx?= =?utf-8?B?ZXFKZGVSQUVkK0RaUVR4WS9zVDBCY0YwT242SWtPK3paRWFPOSt6ZGx0am5R?= =?utf-8?B?T2x0MWNmZnZhbnZtV1JvOWNzSjE3K3NGdmFiSkcvT2R1ZDhSNGVZZTRvQVNK?= =?utf-8?B?OVpDUW04OU9sRTlBQ0E1STZvdjBpTWNmTm00OWtQL2NtVUdYMTdMWlp5Wld6?= =?utf-8?B?Zy9WZW9Wc3FMSW1nQVNLUWZxTUFyUEZUcGY3Y3Y2NHhLdXVKMVJWazZ2M3dB?= =?utf-8?B?aDhvZlg0SG1QcjdEMnA3a3hkVCtSSlVnYVE1MXM4Vk56cklkUDRJRHhkWWhJ?= =?utf-8?B?SUE9PQ==?= X-OriginatorOrg: prevas.dk X-MS-Exchange-CrossTenant-Network-Message-Id: 980c2d39-de5f-4c00-1ebe-08daaaa86e2f X-MS-Exchange-CrossTenant-AuthSource: DU0PR10MB5266.EURPRD10.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Oct 2022 10:16:00.8343 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: d350cf71-778d-4780-88f5-071a4cb1ed61 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 8xYjObG+6KojFxQ5M4T6SErn3Jcnv9AplKS5c180NDJaOeYGLZ2/BPkDqeEwhkIi8dZJ8o2LugSm+Tf7fOjoHmMMOQt5bTy2wmNmjI1bOKs= X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXPR10MB5734 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.6 at phobos.denx.de X-Virus-Status: Clean On 10/01/2022 04.13, Simon Glass wrote: > This behaviour is necessary with boards where the binman description > requires processing external blobs, since these may be missing. > > Enable it by default, so that CI is happy. Warnings indicate that a valid > image is not produced, as with the --allow-missing option. I know I have talked and warned about this before, and now I was actually bitten by it IRL. Can we _please_ stop doing this by default. I understand why upstream U-Boot's CI system needs this, but it should be possible for that CI system to set an environment variable or pass a make parameter to opt-in to generating these fake blobs. Generating potentially non-functional binaries by default, and currently without even a way to opt out of that (that I have found), is frankly not sane. Yeah, a warning is printed, but that easily scrolls away or in the case of automated builds is hidden away in some log file people would only ever look at if the job failed. I want my/our CI to _fail hard_ when I have failed to update the Yocto metadata to stage the necessary blobs before do_compile. And if upstream U-Boot want to continue to have this by default, can we at the very least get an ergonomic way to opt out (preferably a CONFIG_ option I can set or clear in my out-of-tree defconfig files). Rasmus