From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from list by lists.gnu.org with archive (Exim 4.90_1) id 1mKAaq-0001Ur-6T for mharc-grub-devel@gnu.org; Sat, 28 Aug 2021 22:30:44 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:47422) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1mKAal-0001UP-RO for grub-devel@gnu.org; Sat, 28 Aug 2021 22:30:42 -0400 Received: from mail-qt1-x82f.google.com ([2607:f8b0:4864:20::82f]:38650) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1mKAaj-00030I-8M for grub-devel@gnu.org; Sat, 28 Aug 2021 22:30:39 -0400 Received: by mail-qt1-x82f.google.com with SMTP id g11so8744116qtk.5 for ; Sat, 28 Aug 2021 19:30:36 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=nZrlkUkFnDx0ooZIXyaeSQo0kU4/rmOiLnQPAn+Gu4c=; b=P8m6bbFAn/eeOgjVsEjMoHPJ3F2PgOctp4b8sZtdzUUgTeAeiyW1eos7+gdkcNbX50 h2dp82z4ooZdh8YxuAZpULdPLdYU3ouHuqTA7GJpzHGnPNVpscScaFrBn1C9XGjeCOBM YMNbt7H9CWgO8nL0HGcj63eieGjPL6IYae3XgdYqoD+i/67C5Z5Fp2reY5OfzCCy1Zil Qx9kiVX7bpSB4XYS5MC0Z0VcEctrDWySgz7yGawIF9fImKTRd0pjvK0PVfYqpfUf6Vv3 ggc3ycFc0t6y19sxWS4HbajHCfobFcjYvs2AihGHzildy4ANXbNytvXgGn2tjuas5GXm ss0A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=nZrlkUkFnDx0ooZIXyaeSQo0kU4/rmOiLnQPAn+Gu4c=; b=qoQbAlJ7zJA23kCs55633ASNzfKaU10FCL2uyhdopY/q7uSTHEx9H9qp7jhUFtmYdo l5FVmU2y1DaY/h6qK9X4utsK2Y9Ccy3MphoyiymqwsbK1JBOOUvlDThTiRbctMnL7apc giWHr9kmF+r6+XmhFEGxQX8wk+kirp5DSYmX4URzN+2+kmGVe44HtNUVhkkNUeCyCWLA mBOYLd+41ZDdRThPDn/ZpsKZQ0oc7iOOtQv4NRfVX9ITbR+12/SD7vkc68ZjKSN025sV 9w5Ox9lIurNeSJH3CWruV35FTwOUyYGaIa0Xt7A6Ph9HXOHE5B/GqiDI9UMhGsKvrsF8 8+Yg== X-Gm-Message-State: AOAM530f5npzT5R1+FxA2I6efQ6WWjNHe40xBVOkRdN6CKruHPru8aGL KNruQHjxQm20aCLLmHZ7/SX2+rPr0QA+XkK/5DQ= X-Google-Smtp-Source: ABdhPJwf7qlVVaCsKqWpu+6l8H1CoA61TUJFpe2uKuUSYYV28Js0naunJ74TPc65EZ/yIO4oHRilP2aXOZtTIP2uFxw= X-Received: by 2002:ac8:4c89:: with SMTP id j9mr14978646qtv.78.1630204235629; Sat, 28 Aug 2021 19:30:35 -0700 (PDT) MIME-Version: 1.0 References: <20210602211235.377366-1-arei@altlinux.org> In-Reply-To: From: Fu Wei Date: Sun, 29 Aug 2021 10:30:24 +0800 Message-ID: Subject: Re: [PATCH v2 0/7] Add LoadFile2 and riscv Linux loader To: Nikita Ermakov Cc: Atish Patra , The development of GNU GRUB , Ard Biesheuvel , Atish Patra , Daniel Kiper , Heinrich Schuchardt , Leif Lindholm Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Received-SPF: pass client-ip=2607:f8b0:4864:20::82f; envelope-from=tekkamanninja@gmail.com; helo=mail-qt1-x82f.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: The development of GNU GRUB List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 29 Aug 2021 02:30:42 -0000 Hi Nikita, Nikita Ermakov =E4=BA=8E2021=E5=B9=B48=E6=9C=8828=E6=97= =A5=E5=91=A8=E5=85=AD =E4=B8=8B=E5=8D=888:22=E5=86=99=E9=81=93=EF=BC=9A > > Hi Atish, Fu, > > On Fri, 27 Aug 2021 at 19:29, Fu Wei wrote: > > > > Hi Atish, > > > > Great thanks for adding me the loop! > > > > I have tested this these whole patchset on Starlight board and Allwinne= r D1 , > > > > https://github.com/tekkamanninja/grub/tree/riscv_devel_Nikita_V2 > > > > I would be nice If you can add my Tested-by for this patchset! > > > > Tested-by: Wei Fu > > > Fu, thank you for the test and the patch regarding boot-hartid. > I can send V3 with "Tested-by: Wei Fu " > appended to the patchset and add the 8'th patch to set boot-hartid. > Is it ok? Sure , NP :-) . Please add my Tested-by :-) > > > Atish Patra =E4=BA=8E2021=E5=B9=B48=E6=9C=8828= =E6=97=A5=E5=91=A8=E5=85=AD =E4=B8=8A=E5=8D=8812:22=E5=86=99=E9=81=93=EF=BC= =9A > > > > > > On Wed, Jun 2, 2021 at 2:13 PM Nikita Ermakov wro= te: > > > > > > > > Hello, > > > > > > > > This is the second version of the patch series. The patch series is= almost the > > > > same as the previous one except that the Heinrich's suggestions wer= e > > > > implemented. > > > > > > > > This series contains patches to add support for LoadFile2 protocol = to load > > > > initrd on EFI systems. Also it contains patches to load Linux kerne= l with EFI > > > > stub on riscv platforms and unites arm and riscv codes together int= o common > > > > loader code for EFI systems. > > > > > > + Wei Fu who tested this series on Beagleboard. > > > > > > According to Wei we do another patch to add a boot-hartid entry in th= e > > > chosen node if DTB is > > > loaded from grub instead of U-Boot [1]. > > > > > > [1] https://github.com/tekkamanninja/grub/commit/99c5dc1ba897e506f185= eb19d4dadc6d41937214 > > > > > > > > Ard Biesheuvel (3): > > > > efi: add definition of LoadFile2 protocol > > > > efi: implemented LoadFile2 initrd loading protocol for Linux > > > > linux: ignore FDT unless we need to modify it > > > > > > > > Atish Patra (3): > > > > loader: Move arm64 linux loader to common code > > > > RISC-V: Update image header > > > > RISC-V: Use common linux loader > > > > > > > > Nikita Ermakov (1): > > > > loader: drop argv[] argument in grub_initrd_load() > > > > > > > > grub-core/Makefile.core.def | 8 +- > > > > grub-core/commands/efi/lsefi.c | 1 + > > > > grub-core/loader/arm/linux.c | 2 +- > > > > grub-core/loader/efi/fdt.c | 7 +- > > > > grub-core/loader/{arm64 =3D> efi}/linux.c | 143 ++++++++++++++++= +++--- > > > > grub-core/loader/i386/linux.c | 2 +- > > > > grub-core/loader/i386/pc/linux.c | 2 +- > > > > grub-core/loader/i386/xen.c | 3 +- > > > > grub-core/loader/ia64/efi/linux.c | 2 +- > > > > grub-core/loader/linux.c | 4 +- > > > > grub-core/loader/mips/linux.c | 2 +- > > > > grub-core/loader/powerpc/ieee1275/linux.c | 2 +- > > > > grub-core/loader/riscv/linux.c | 59 --------- > > > > grub-core/loader/sparc64/ieee1275/linux.c | 2 +- > > > > include/grub/arm/linux.h | 2 +- > > > > include/grub/arm64/linux.h | 2 +- > > > > include/grub/efi/api.h | 15 +++ > > > > include/grub/linux.h | 2 +- > > > > include/grub/riscv32/linux.h | 16 +-- > > > > include/grub/riscv64/linux.h | 16 +-- > > > > 20 files changed, 185 insertions(+), 107 deletions(-) > > > > rename grub-core/loader/{arm64 =3D> efi}/linux.c (72%) > > > > delete mode 100644 grub-core/loader/riscv/linux.c > > > > > > > > -- > > > > 2.29.3 > > > > > > > > > > > > _______________________________________________ > > > > Grub-devel mailing list > > > > Grub-devel@gnu.org > > > > https://lists.gnu.org/mailman/listinfo/grub-devel > > > > > > > > > > > > -- > > > Regards, > > > Atish > > -- > Thanks, > Nikita > B8 00 4C CD 21