Kexec Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Simon Horman <horms@kernel.org>
To: Dave Young <dyoung@redhat.com>
Cc: Song Shuai <songshuaishuai@tinylab.org>,
	mick@ics.forth.gr, lizhengyu3@huawei.com, liaochang1@huawei.com,
	xianting.tian@linux.alibaba.com, guoren@kernel.org,
	huanyi.xj@alibaba-inc.com, yixun.lan@gmail.com,
	kexec@lists.infradead.org, hursand@fedoraproject.org
Subject: Re: [PATCH 0/7] Kexec-tools: Improve RISC-V port
Date: Sat, 18 May 2024 09:33:04 +0100	[thread overview]
Message-ID: <20240518083304.GD475595@kernel.org> (raw)
In-Reply-To: <CALu+AoSr+an7Ps-jhcSbvrq4E4FnqkiNZVH-2BhV_3=B+oDcFQ@mail.gmail.com>

On Sat, May 18, 2024 at 11:39:18AM +0800, Dave Young wrote:
> On Wed, 11 Oct 2023 at 13:24, Song Shuai <songshuaishuai@tinylab.org> wrote:
> >
> >
> >
> > 在 2023/9/20 19:56, Simon Horman 写道:
> > > On Fri, Sep 15, 2023 at 11:50:06AM +0800, Song Shuai wrote:
> > >> Hi,
> > >>
> > >> This series is created to improve RISC-V port of kexec-tools,
> > >> and is based on the horms/kexec-tools:build-test-riscv-v2 branch.
> > >
> > > In my mind the big question is how to move RISC-V support
> > > from that branch, to being merged into main.
> > >
> > > IIRC there were some issues that needed to be addressed.
> > > Perhaps they are all addressed by this series, and with
> > > some appropriate squashing we can move forwards with a series
> > > based on main?
> >
> > Hi, Simon and Nick:
> >
> > I squashed the first four patches as a "RISC-V: Some fixes for riscv
> > port" patch and then took the horms/main as the base to collect the 2
> > patches from horms/build-test-riscv-v2 branch and this series togother.
> > These are the Github link and all commits for RISC-V.
> >
> > https://github.com/sugarfillet/kexec-tools/commits/main_rv
> >
> > 5dc133e RISC-V: Support loading Image binary file
> > b042f6d RISC-V: Separate elf_riscv_find_pbase out
> > 8f344c7 RISC-V: Enable kexec_file_load syscall
> > 7d4b982 RISC-V: Some fixes for riscv port
> > 3205c1c local: RISC-V: distribute purgatory/riscv/Makefile
> > 54f9daf RISC-V: Add support for riscv kexec/kdump on kexec-tools
> >
> > Since I didn't found the issues/fixes as Nick mentioned with these
> > commits, I prefer to merge them into horms/main and let more kexec/kdump
> > users to help improve/fixup RISC-V port.
> 
> Hi,  I noticed another pr for Fedora kexec-tools:
> https://src.fedoraproject.org/rpms/kexec-tools/pull-request/24
> 
> It is bad to take it as Fedora only,   I would suggest posting all the
> refreshed patches together here again for review.
> 
> If no enough reviewers can review them my another suggestion is to
> drop the kexec_load support code for the time being, and only enable
> the kexec_file_load support code in kexec-tools, and I assume below
> kernel commit make the kexec_file_load kernel piece of work done.
> Then it will be easier to review and make something working at least.
> commit 6261586e0c91db14c34f894f4bc48f2300cff1d4
> Author: Liao Chang <liaochang1@huawei.com>
> Date:   Fri Apr 8 18:09:11 2022 +0800
> 
>     RISC-V: Add kexec_file support

Hi,

I would gladly take a patchset for kexec-tools that adds RISC-V support in
a coherent manner.


_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec

  reply	other threads:[~2024-05-18  8:33 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-09-15  3:50 [PATCH 0/7] Kexec-tools: Improve RISC-V port Song Shuai
2023-09-15  3:50 ` [PATCH 1/7] RISC-V: Use linux,usable-memory-range for crash kernel Song Shuai
2023-09-15  3:50 ` [PATCH 2/7] RISC-V: Fix the undeclared ‘EM_RISCV’ build failure Song Shuai
2023-09-15  3:50 ` [PATCH 3/7] RISC-V: Get memory ranges from iomem Song Shuai
2023-09-15  3:50 ` [PATCH 4/7] RISC-V: Correct the usage of command line option Song Shuai
2023-09-15  3:50 ` [PATCH 5/7] RISC-V: Enable kexe_file_load Song Shuai
2023-09-15  3:50 ` [PATCH 6/7] RISC-V: Separate elf_riscv_find_pbase out Song Shuai
2023-09-15  3:50 ` [PATCH 7/7] RISC-V: Support loading Image binary file Song Shuai
2023-09-20 11:56 ` [PATCH 0/7] Kexec-tools: Improve RISC-V port Simon Horman
2023-10-11  5:22   ` Song Shuai
2024-05-18  3:39     ` Dave Young
2024-05-18  8:33       ` Simon Horman [this message]
2024-08-12 14:36         ` Petr Tesarik
2024-08-13 14:26           ` Simon Horman
2024-08-14  8:23           ` Dave Young
2025-01-10  9:22             ` Dave Young
2023-09-20 13:47 ` Nick Kossifidis

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240518083304.GD475595@kernel.org \
    --to=horms@kernel.org \
    --cc=dyoung@redhat.com \
    --cc=guoren@kernel.org \
    --cc=huanyi.xj@alibaba-inc.com \
    --cc=hursand@fedoraproject.org \
    --cc=kexec@lists.infradead.org \
    --cc=liaochang1@huawei.com \
    --cc=lizhengyu3@huawei.com \
    --cc=mick@ics.forth.gr \
    --cc=songshuaishuai@tinylab.org \
    --cc=xianting.tian@linux.alibaba.com \
    --cc=yixun.lan@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox