All of 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.