From: "Alex Bennée" <alex.bennee@linaro.org> To: Sylvain Pelissier <sylvain.pelissier@gmail.com> Cc: "open list:RISC-V" <qemu-riscv@nongnu.org>, Sagar Karandikar <sagark@eecs.berkeley.edu>, Bastian Koppelmann <kbastian@mail.uni-paderborn.de>, qemu-devel@nongnu.org, Alistair Francis <Alistair.Francis@wdc.com>, Alistair Francis <alistair23@gmail.com>, Bin Meng <bmeng.cn@gmail.com>, Palmer Dabbelt <palmer@dabbelt.com> Subject: Re: [PATCH v2] gdb: riscv: Add target description Date: Wed, 06 Jan 2021 11:35:08 +0000 [thread overview] Message-ID: <87czyi2t0z.fsf@linaro.org> (raw) In-Reply-To: <CAOkUe-AqC4UXOPZeX+uyXqucF12AaW_76oDqgn6EE-PomckYjA@mail.gmail.com> Sylvain Pelissier <sylvain.pelissier@gmail.com> writes: > Hello, > > I may have made an error by copy pasting the comment into the file. I sent > a new v3 with git send-email. I hope it is fine now. Your v3 doesn't include the review tags you got for v2 which makes it look un-reviewed. See: https://wiki.qemu.org/Contribute/SubmitAPatch#Proper_use_of_Reviewed-by:_tags_can_aid_review You can either apply them manually by copy and paste when you reword the commit message or use a tool to apply the old version and collect tags from the mailing list archive. > Regards > > Sylvain > > On Tue, 5 Jan 2021 at 22:03, Alistair Francis <alistair23@gmail.com> wrote: > >> On Wed, Dec 30, 2020 at 12:26 AM Sylvain Pelissier >> <sylvain.pelissier@gmail.com> wrote: >> > >> > Target description is not currently implemented in RISC-V architecture. >> Thus GDB won't set it properly when attached. The patch implements the >> target description response. >> > >> > Signed-off-by: Sylvain Pelissier <sylvain.pelissier@gmail.com> >> >> Hello, >> >> This patch fails to apply. How did you send the email? >> >> Alistair >> >> > --- >> > target/riscv/cpu.c | 13 +++++++++++++ >> > 1 file changed, 13 insertions(+) >> > >> > diff --git a/target/riscv/cpu.c b/target/riscv/cpu.c >> > index 254cd83f8b..ed4971978b 100644 >> > --- a/target/riscv/cpu.c >> > +++ b/target/riscv/cpu.c >> > @@ -556,6 +556,18 @@ static Property riscv_cpu_properties[] = { >> > DEFINE_PROP_END_OF_LIST(), >> > }; >> > >> > +static gchar *riscv_gdb_arch_name(CPUState *cs) >> > +{ >> > + RISCVCPU *cpu = RISCV_CPU(cs); >> > + CPURISCVState *env = &cpu->env; >> > + >> > + if (riscv_cpu_is_32bit(env)) { >> > + return g_strdup("riscv:rv32"); >> > + } else { >> > + return g_strdup("riscv:rv64"); >> > + } >> > +} >> > + >> > static void riscv_cpu_class_init(ObjectClass *c, void *data) >> > { >> > RISCVCPUClass *mcc = RISCV_CPU_CLASS(c); >> > @@ -591,6 +603,7 @@ static void riscv_cpu_class_init(ObjectClass *c, >> void *data) >> > /* For now, mark unmigratable: */ >> > cc->vmsd = &vmstate_riscv_cpu; >> > #endif >> > + cc->gdb_arch_name = riscv_gdb_arch_name; >> > #ifdef CONFIG_TCG >> > cc->tcg_initialize = riscv_translate_init; >> > cc->tlb_fill = riscv_cpu_tlb_fill; >> > -- >> > 2.25.1 >> -- Alex Bennée
WARNING: multiple messages have this Message-ID (diff)
From: "Alex Bennée" <alex.bennee@linaro.org> To: Sylvain Pelissier <sylvain.pelissier@gmail.com> Cc: Alistair Francis <alistair23@gmail.com>, "open list:RISC-V" <qemu-riscv@nongnu.org>, Sagar Karandikar <sagark@eecs.berkeley.edu>, Bastian Koppelmann <kbastian@mail.uni-paderborn.de>, Alistair Francis <Alistair.Francis@wdc.com>, Palmer Dabbelt <palmer@dabbelt.com>, Bin Meng <bmeng.cn@gmail.com>, qemu-devel@nongnu.org Subject: Re: [PATCH v2] gdb: riscv: Add target description Date: Wed, 06 Jan 2021 11:35:08 +0000 [thread overview] Message-ID: <87czyi2t0z.fsf@linaro.org> (raw) In-Reply-To: <CAOkUe-AqC4UXOPZeX+uyXqucF12AaW_76oDqgn6EE-PomckYjA@mail.gmail.com> Sylvain Pelissier <sylvain.pelissier@gmail.com> writes: > Hello, > > I may have made an error by copy pasting the comment into the file. I sent > a new v3 with git send-email. I hope it is fine now. Your v3 doesn't include the review tags you got for v2 which makes it look un-reviewed. See: https://wiki.qemu.org/Contribute/SubmitAPatch#Proper_use_of_Reviewed-by:_tags_can_aid_review You can either apply them manually by copy and paste when you reword the commit message or use a tool to apply the old version and collect tags from the mailing list archive. > Regards > > Sylvain > > On Tue, 5 Jan 2021 at 22:03, Alistair Francis <alistair23@gmail.com> wrote: > >> On Wed, Dec 30, 2020 at 12:26 AM Sylvain Pelissier >> <sylvain.pelissier@gmail.com> wrote: >> > >> > Target description is not currently implemented in RISC-V architecture. >> Thus GDB won't set it properly when attached. The patch implements the >> target description response. >> > >> > Signed-off-by: Sylvain Pelissier <sylvain.pelissier@gmail.com> >> >> Hello, >> >> This patch fails to apply. How did you send the email? >> >> Alistair >> >> > --- >> > target/riscv/cpu.c | 13 +++++++++++++ >> > 1 file changed, 13 insertions(+) >> > >> > diff --git a/target/riscv/cpu.c b/target/riscv/cpu.c >> > index 254cd83f8b..ed4971978b 100644 >> > --- a/target/riscv/cpu.c >> > +++ b/target/riscv/cpu.c >> > @@ -556,6 +556,18 @@ static Property riscv_cpu_properties[] = { >> > DEFINE_PROP_END_OF_LIST(), >> > }; >> > >> > +static gchar *riscv_gdb_arch_name(CPUState *cs) >> > +{ >> > + RISCVCPU *cpu = RISCV_CPU(cs); >> > + CPURISCVState *env = &cpu->env; >> > + >> > + if (riscv_cpu_is_32bit(env)) { >> > + return g_strdup("riscv:rv32"); >> > + } else { >> > + return g_strdup("riscv:rv64"); >> > + } >> > +} >> > + >> > static void riscv_cpu_class_init(ObjectClass *c, void *data) >> > { >> > RISCVCPUClass *mcc = RISCV_CPU_CLASS(c); >> > @@ -591,6 +603,7 @@ static void riscv_cpu_class_init(ObjectClass *c, >> void *data) >> > /* For now, mark unmigratable: */ >> > cc->vmsd = &vmstate_riscv_cpu; >> > #endif >> > + cc->gdb_arch_name = riscv_gdb_arch_name; >> > #ifdef CONFIG_TCG >> > cc->tcg_initialize = riscv_translate_init; >> > cc->tlb_fill = riscv_cpu_tlb_fill; >> > -- >> > 2.25.1 >> -- Alex Bennée
next prev parent reply other threads:[~2021-01-06 11:41 UTC|newest] Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-12-30 8:25 [PATCH v2] gdb: riscv: Add target description Sylvain Pelissier 2020-12-30 8:32 ` Bin Meng 2020-12-30 8:32 ` Bin Meng 2021-01-05 20:33 ` Alistair Francis 2021-01-05 20:33 ` Alistair Francis 2021-01-05 21:03 ` Alistair Francis 2021-01-05 21:03 ` Alistair Francis 2021-01-06 10:48 ` Sylvain Pelissier 2021-01-06 10:48 ` Sylvain Pelissier 2021-01-06 11:35 ` Alex Bennée [this message] 2021-01-06 11:35 ` Alex Bennée
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=87czyi2t0z.fsf@linaro.org \ --to=alex.bennee@linaro.org \ --cc=Alistair.Francis@wdc.com \ --cc=alistair23@gmail.com \ --cc=bmeng.cn@gmail.com \ --cc=kbastian@mail.uni-paderborn.de \ --cc=palmer@dabbelt.com \ --cc=qemu-devel@nongnu.org \ --cc=qemu-riscv@nongnu.org \ --cc=sagark@eecs.berkeley.edu \ --cc=sylvain.pelissier@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: linkBe 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.