From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9210123AD for ; Thu, 22 Sep 2022 07:22:18 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 394C0C43470 for ; Thu, 22 Sep 2022 07:22:18 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1663831338; bh=ECswKkioxqUYJvDjL4amLlR/VyDBspn+RwnVrk9djqQ=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=A9yoNAbTVGtjEQqZg/x1c+935RfCNXgG/1O1+Ifu2znV8rKVXUWfYsdmz4zo+k0Gq 4Bv3IrwfeBREyDN953Z0cbSWrV/0WvmhD2nN69U3+WABFzVKf37yiExE+NNDuUDjb8 /GHEpkW6zCBeNiAKvn4iWElq+dLPOUwN2hbMYLqsDAvtqQNDjKCLepSP8bbAm+0ZtX GV9AOEppt1eUtyuOkmRbqil3A2fqdvmFPtGCxMEvWW4KShbi8SYsqEFQdWwoTzfklQ arF+Jes33FBus0Qzqds0g6Pc++PPUPYIVcuB6PAaNn8E0SW3Ob5oKXWJW+WBl3HCK3 DJ3CTsnjYyqZQ== Received: by mail-vs1-f53.google.com with SMTP id u189so9344465vsb.4 for ; Thu, 22 Sep 2022 00:22:18 -0700 (PDT) X-Gm-Message-State: ACrzQf0AtU9R34H95UlU/hPJ/MRmx7kOtffV1b9zbJ1rXXQ2xctr5JP8 qlgDaX5HLqkpxctZSpKD5qBamebLf5wPcT6Thd4= X-Google-Smtp-Source: AMsMyM4Eg74+i/jU4lJi1FgtdcwGnDPhkPELFMznXR2qE2jKyOdz8dFPlGue8k0dBbIikQMyd/PZR69nH+90u4yZ9zw= X-Received: by 2002:a05:6102:3d05:b0:398:7dce:8492 with SMTP id i5-20020a0561023d0500b003987dce8492mr692585vsv.43.1663831337107; Thu, 22 Sep 2022 00:22:17 -0700 (PDT) Precedence: bulk X-Mailing-List: loongarch@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 References: <20220920183554.3870247-1-ardb@kernel.org> <20220920183554.3870247-9-ardb@kernel.org> In-Reply-To: From: Huacai Chen Date: Thu, 22 Sep 2022 15:21:05 +0800 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v2 8/8] efi/loongarch: libstub: remove dependency on flattened DT To: Ard Biesheuvel Cc: linux-efi , loongarch@lists.linux.dev, Arnd Bergmann , Ilias Apalodimas , Huacai Chen , Xi Ruoyao Content-Type: text/plain; charset="UTF-8" On Thu, Sep 22, 2022 at 3:13 PM Ard Biesheuvel wrote: > > On Thu, 22 Sept 2022 at 04:15, Huacai Chen wrote: > > > > On Wed, Sep 21, 2022 at 4:15 PM Ard Biesheuvel wrote: > > > > > > On Wed, 21 Sept 2022 at 06:02, Huacai Chen wrote: > > > > > > > > Acked-by: Huacai Chen > > > > > > > > > > Thank you Huacai > > > > > > I have created a separate tag with these patches applied onto the > > > first loongarch efi enablement patch (see below) > > > > > > I would prefer to keep the loongarch trees separate for the next merge > > > window, but if needed to avoid massive conflicts, you could merge the > > > tag into the loongarch tree and rebase your changes on top. But please > > > take care not to rebase those patches themselves: the commit SHAs need > > > to remain the same. > > > > > > However, if the conflicts are only minor, you can just explain in your > > > PR to Linus what the resolution should look like. > > > > > > The tag can be found here: > > > https://git.kernel.org/pub/scm/linux/kernel/git/efi/efi.git/tag/?h=efi-loongarch-for-v6.1-2 > > > > > > I have merged this into the EFI tree as well and pushed it out; it > > > should appear in -next tomorrow. > > > > > > > > > > > > b093dc55ba9a efi/loongarch: libstub: remove dependency on flattened DT > > > 102faf0747c2 efi: libstub: install boot-time memory map as config table > > > 7318926e7bc9 efi: libstub: remove DT dependency from generic stub > > > 7cb6671e3caa efi: libstub: unify initrd loading between architectures > > I will have to respin the initrd patch, unfortunately, as it breaks > x86. So the signed tag will be updated, and the SHAs above will change > as well. Emm, initrd is also broken on LoongArch, the core kernel gets an EFI_INVALID_TABLE_ADDR address. > > > > 1add08ceb15b efi: libstub: remove pointless goto kludge > > > d2130a9cf0f7 efi: libstub: simplify efi_get_memory_map() and struct > > > efi_boot_memmap > > > bc5853aa56de efi: libstub: avoid efi_get_memory_map() for allocating > > > the virt map > > > d80ca810f096 efi: libstub: drop pointless get_memory_map() call > > > ead384d95634 (tag: efi-loongarch-for-v6.1) efi/loongarch: Add efistub > > > booting support > > > 568035b01cfb (tag: v6.0-rc1) Linux 6.0-rc1 > > Hmmm, I want to also merge zboot support in the loongarch-next's base, > > is that possible? > > > > No, let's keep that separate. LoongArch boots fine without it, so we > can wait until it arrives via the EFI tree, no? Both OK but I prefer to use a compressed kernel in daily work. Huacai