hw/misc/grlib_ahb_apb_pnp.c | 24 ++++++++++++++++++++++-- target/sparc/int32_helper.c | 23 ++++++++++++----------- hw/misc/trace-events | 4 ++++ hw/timer/trace-events | 2 +- tests/acceptance/machine_sparc_leon3.py | 4 ++++ 5 files changed, 43 insertions(+), 14 deletions(-)
The Leon3Machine.test_leon3_helenos_uimage has been running erratically since some time now. Time to disable it (at least until we make it reliable). Few other patches added while tracking the issue. Philippe Mathieu-Daudé (7): tests/acceptance/machine_sparc_leon3: Disable HelenOS test hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to AHB PnP registers hw/misc/grlib_ahb_apb_pnp: Fix AHB PnP 8-bit accesses hw/misc/grlib_ahb_apb_pnp: Add trace events on read accesses hw/timer/grlib_gptimer: Display frequency in decimal target/sparc/int32_helper: Remove DEBUG_PCALL definition target/sparc/int32_helper: Extract and use excp_name_str() hw/misc/grlib_ahb_apb_pnp.c | 24 ++++++++++++++++++++++-- target/sparc/int32_helper.c | 23 ++++++++++++----------- hw/misc/trace-events | 4 ++++ hw/timer/trace-events | 2 +- tests/acceptance/machine_sparc_leon3.py | 4 ++++ 5 files changed, 43 insertions(+), 14 deletions(-) -- 2.21.1
On 3/31/20 12:50 PM, Philippe Mathieu-Daudé wrote: > Philippe Mathieu-Daudé (7): > hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to AHB PnP > registers > hw/misc/grlib_ahb_apb_pnp: Fix AHB PnP 8-bit accesses Ping ^^^ for 5.0? > hw/misc/grlib_ahb_apb_pnp: Add trace events on read accesses > hw/timer/grlib_gptimer: Display frequency in decimal > target/sparc/int32_helper: Remove DEBUG_PCALL definition > target/sparc/int32_helper: Extract and use excp_name_str() > > hw/misc/grlib_ahb_apb_pnp.c | 24 ++++++++++++++++++++++-- > target/sparc/int32_helper.c | 23 ++++++++++++----------- > hw/misc/trace-events | 4 ++++ > hw/timer/trace-events | 2 +- > tests/acceptance/machine_sparc_leon3.py | 4 ++++ > 5 files changed, 43 insertions(+), 14 deletions(-) >
Le 4/11/20 à 7:30 PM, Philippe Mathieu-Daudé a écrit : > On 3/31/20 12:50 PM, Philippe Mathieu-Daudé wrote: >> Philippe Mathieu-Daudé (7): >> hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to AHB PnP >> registers >> hw/misc/grlib_ahb_apb_pnp: Fix AHB PnP 8-bit accesses > > Ping ^^^ for 5.0? Hi Philippe, You already have my rb tag for those one, and IMHO they should be good candidate for 5.0 (if it's not too late). Cheers, Fred > >> hw/misc/grlib_ahb_apb_pnp: Add trace events on read accesses >> hw/timer/grlib_gptimer: Display frequency in decimal >> target/sparc/int32_helper: Remove DEBUG_PCALL definition >> target/sparc/int32_helper: Extract and use excp_name_str() >> >> hw/misc/grlib_ahb_apb_pnp.c | 24 ++++++++++++++++++++++-- >> target/sparc/int32_helper.c | 23 ++++++++++++----------- >> hw/misc/trace-events | 4 ++++ >> hw/timer/trace-events | 2 +- >> tests/acceptance/machine_sparc_leon3.py | 4 ++++ >> 5 files changed, 43 insertions(+), 14 deletions(-) >>
[Cc'ing Peter] On 4/13/20 12:12 PM, KONRAD Frederic wrote: > Le 4/11/20 à 7:30 PM, Philippe Mathieu-Daudé a écrit : >> On 3/31/20 12:50 PM, Philippe Mathieu-Daudé wrote: >>> Philippe Mathieu-Daudé (7): >>> hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to AHB PnP >>> registers >>> hw/misc/grlib_ahb_apb_pnp: Fix AHB PnP 8-bit accesses >> >> Ping ^^^ for 5.0? > > Hi Philippe, > > You already have my rb tag for those one, and IMHO they should be good > candidate > for 5.0 (if it's not too late). Yes, thanks for the reviews. I see Mark Cave-Ayland merged this file first, but you are listed as maintainer :) I was hoping you could send a pull request. $ scripts/get_maintainer.pl -f hw/misc/grlib_ahb_apb_pnp.c Fabien Chouteau <chouteau@adacore.com> (maintainer:Leon3) KONRAD Frederic <frederic.konrad@adacore.com> (maintainer:Leon3) qemu-devel@nongnu.org (open list:All patches CC here) This is a bug but not 'security critical', so it might wait 5.1 and go via qemu-trivial tree. Regards, Phil. > > Cheers, > Fred > >> >>> hw/misc/grlib_ahb_apb_pnp: Add trace events on read accesses >>> hw/timer/grlib_gptimer: Display frequency in decimal >>> target/sparc/int32_helper: Remove DEBUG_PCALL definition >>> target/sparc/int32_helper: Extract and use excp_name_str() >>> >>> hw/misc/grlib_ahb_apb_pnp.c | 24 ++++++++++++++++++++++-- >>> target/sparc/int32_helper.c | 23 ++++++++++++----------- >>> hw/misc/trace-events | 4 ++++ >>> hw/timer/trace-events | 2 +- >>> tests/acceptance/machine_sparc_leon3.py | 4 ++++ >>> 5 files changed, 43 insertions(+), 14 deletions(-) >>> >
Le 4/13/20 à 11:07 PM, Philippe Mathieu-Daudé a écrit : > [Cc'ing Peter] > > On 4/13/20 12:12 PM, KONRAD Frederic wrote: >> Le 4/11/20 à 7:30 PM, Philippe Mathieu-Daudé a écrit : >>> On 3/31/20 12:50 PM, Philippe Mathieu-Daudé wrote: >>>> Philippe Mathieu-Daudé (7): >>>> hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to AHB PnP >>>> registers >>>> hw/misc/grlib_ahb_apb_pnp: Fix AHB PnP 8-bit accesses >>> >>> Ping ^^^ for 5.0? >> >> Hi Philippe, >> >> You already have my rb tag for those one, and IMHO they should be good >> candidate >> for 5.0 (if it's not too late). > > Yes, thanks for the reviews. I see Mark Cave-Ayland merged this file > first, but you are listed as maintainer :) I was hoping you could send a > pull request. Yes that's usually Mark who take the patches, sorry I didn't get it. > > $ scripts/get_maintainer.pl -f hw/misc/grlib_ahb_apb_pnp.c > Fabien Chouteau <chouteau@adacore.com> (maintainer:Leon3) > KONRAD Frederic <frederic.konrad@adacore.com> (maintainer:Leon3) > qemu-devel@nongnu.org (open list:All patches CC here) > > This is a bug but not 'security critical', so it might wait 5.1 and go > via qemu-trivial tree. Well let's do that then if you're ok. Best Regards, Fred > > Regards, > > Phil. > >> >> Cheers, >> Fred >> >>> >>>> hw/misc/grlib_ahb_apb_pnp: Add trace events on read accesses >>>> hw/timer/grlib_gptimer: Display frequency in decimal >>>> target/sparc/int32_helper: Remove DEBUG_PCALL definition >>>> target/sparc/int32_helper: Extract and use excp_name_str() >>>> >>>> hw/misc/grlib_ahb_apb_pnp.c | 24 ++++++++++++++++++++++-- >>>> target/sparc/int32_helper.c | 23 ++++++++++++----------- >>>> hw/misc/trace-events | 4 ++++ >>>> hw/timer/trace-events | 2 +- >>>> tests/acceptance/machine_sparc_leon3.py | 4 ++++ >>>> 5 files changed, 43 insertions(+), 14 deletions(-) >>>> >>
On 4/14/20 12:00 PM, KONRAD Frederic wrote: > Le 4/13/20 à 11:07 PM, Philippe Mathieu-Daudé a écrit : >> [Cc'ing Peter] >> >> On 4/13/20 12:12 PM, KONRAD Frederic wrote: >>> Le 4/11/20 à 7:30 PM, Philippe Mathieu-Daudé a écrit : >>>> On 3/31/20 12:50 PM, Philippe Mathieu-Daudé wrote: >>>>> Philippe Mathieu-Daudé (7): >>>>> hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to AHB PnP >>>>> registers >>>>> hw/misc/grlib_ahb_apb_pnp: Fix AHB PnP 8-bit accesses >>>> >>>> Ping ^^^ for 5.0? >>> >>> Hi Philippe, >>> >>> You already have my rb tag for those one, and IMHO they should be good >>> candidate >>> for 5.0 (if it's not too late). >> >> Yes, thanks for the reviews. I see Mark Cave-Ayland merged this file >> first, but you are listed as maintainer :) I was hoping you could send a >> pull request. > > Yes that's usually Mark who take the patches, sorry I didn't get it. No worries. As there are other sparc64 patches on the list, maybe Mark will prepare a pull request now. > >> >> $ scripts/get_maintainer.pl -f hw/misc/grlib_ahb_apb_pnp.c >> Fabien Chouteau <chouteau@adacore.com> (maintainer:Leon3) >> KONRAD Frederic <frederic.konrad@adacore.com> (maintainer:Leon3) >> qemu-devel@nongnu.org (open list:All patches CC here) >> > >> This is a bug but not 'security critical', so it might wait 5.1 and go >> via qemu-trivial tree. > > Well let's do that then if you're ok. OK, then ping? :) > > Best Regards, > Fred > >> >> Regards, >> >> Phil. >> >>> >>> Cheers, >>> Fred >>> >>>> >>>>> hw/misc/grlib_ahb_apb_pnp: Add trace events on read accesses >>>>> hw/timer/grlib_gptimer: Display frequency in decimal >>>>> target/sparc/int32_helper: Remove DEBUG_PCALL definition >>>>> target/sparc/int32_helper: Extract and use excp_name_str() >>>>> >>>>> hw/misc/grlib_ahb_apb_pnp.c | 24 >>>>> ++++++++++++++++++++++-- >>>>> target/sparc/int32_helper.c | 23 >>>>> ++++++++++++----------- >>>>> hw/misc/trace-events | 4 ++++ >>>>> hw/timer/trace-events | 2 +- >>>>> tests/acceptance/machine_sparc_leon3.py | 4 ++++ >>>>> 5 files changed, 43 insertions(+), 14 deletions(-) >>>>> >>> >
ping? On 5/11/20 9:03 AM, Philippe Mathieu-Daudé wrote: > On 4/14/20 12:00 PM, KONRAD Frederic wrote: >> Le 4/13/20 à 11:07 PM, Philippe Mathieu-Daudé a écrit : >>> [Cc'ing Peter] >>> >>> On 4/13/20 12:12 PM, KONRAD Frederic wrote: >>>> Le 4/11/20 à 7:30 PM, Philippe Mathieu-Daudé a écrit : >>>>> On 3/31/20 12:50 PM, Philippe Mathieu-Daudé wrote: >>>>>> Philippe Mathieu-Daudé (7): >>>>>> hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to AHB PnP >>>>>> registers >>>>>> hw/misc/grlib_ahb_apb_pnp: Fix AHB PnP 8-bit accesses >>>>> >>>>> Ping ^^^ for 5.0? >>>> >>>> Hi Philippe, >>>> >>>> You already have my rb tag for those one, and IMHO they should be good >>>> candidate >>>> for 5.0 (if it's not too late). >>> >>> Yes, thanks for the reviews. I see Mark Cave-Ayland merged this file >>> first, but you are listed as maintainer :) I was hoping you could send a >>> pull request. >> >> Yes that's usually Mark who take the patches, sorry I didn't get it. > > No worries. As there are other sparc64 patches on the list, maybe Mark > will prepare a pull request now. > >> >>> >>> $ scripts/get_maintainer.pl -f hw/misc/grlib_ahb_apb_pnp.c >>> Fabien Chouteau <chouteau@adacore.com> (maintainer:Leon3) >>> KONRAD Frederic <frederic.konrad@adacore.com> (maintainer:Leon3) >>> qemu-devel@nongnu.org (open list:All patches CC here) >>> >> >>> This is a bug but not 'security critical', so it might wait 5.1 and go >>> via qemu-trivial tree. >> >> Well let's do that then if you're ok. > > OK, then ping? :) > >> >> Best Regards, >> Fred >> >>> >>> Regards, >>> >>> Phil. >>> >>>> >>>> Cheers, >>>> Fred >>>> >>>>> >>>>>> hw/misc/grlib_ahb_apb_pnp: Add trace events on read accesses >>>>>> hw/timer/grlib_gptimer: Display frequency in decimal >>>>>> target/sparc/int32_helper: Remove DEBUG_PCALL definition >>>>>> target/sparc/int32_helper: Extract and use excp_name_str() >>>>>> >>>>>> hw/misc/grlib_ahb_apb_pnp.c | 24 >>>>>> ++++++++++++++++++++++-- >>>>>> target/sparc/int32_helper.c | 23 >>>>>> ++++++++++++----------- >>>>>> hw/misc/trace-events | 4 ++++ >>>>>> hw/timer/trace-events | 2 +- >>>>>> tests/acceptance/machine_sparc_leon3.py | 4 ++++ >>>>>> 5 files changed, 43 insertions(+), 14 deletions(-) >>>>>> >>>> >> >
Sorry Philippe I missed that. Would be happy to do a PR if needed but: * I never did that. * Looking at https://wiki.qemu.org/Contribute/SubmitAPullRequest, I don't have the signed GPG key either. Cheers, Fred Le 5/23/20 à 7:26 PM, Philippe Mathieu-Daudé a écrit : > ping? > > On 5/11/20 9:03 AM, Philippe Mathieu-Daudé wrote: >> On 4/14/20 12:00 PM, KONRAD Frederic wrote: >>> Le 4/13/20 à 11:07 PM, Philippe Mathieu-Daudé a écrit : >>>> [Cc'ing Peter] >>>> >>>> On 4/13/20 12:12 PM, KONRAD Frederic wrote: >>>>> Le 4/11/20 à 7:30 PM, Philippe Mathieu-Daudé a écrit : >>>>>> On 3/31/20 12:50 PM, Philippe Mathieu-Daudé wrote: >>>>>>> Philippe Mathieu-Daudé (7): >>>>>>> hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to AHB PnP >>>>>>> registers >>>>>>> hw/misc/grlib_ahb_apb_pnp: Fix AHB PnP 8-bit accesses >>>>>> >>>>>> Ping ^^^ for 5.0? >>>>> >>>>> Hi Philippe, >>>>> >>>>> You already have my rb tag for those one, and IMHO they should be good >>>>> candidate >>>>> for 5.0 (if it's not too late). >>>> >>>> Yes, thanks for the reviews. I see Mark Cave-Ayland merged this file >>>> first, but you are listed as maintainer :) I was hoping you could send a >>>> pull request. >>> >>> Yes that's usually Mark who take the patches, sorry I didn't get it. >> >> No worries. As there are other sparc64 patches on the list, maybe Mark >> will prepare a pull request now. >> >>> >>>> >>>> $ scripts/get_maintainer.pl -f hw/misc/grlib_ahb_apb_pnp.c >>>> Fabien Chouteau <chouteau@adacore.com> (maintainer:Leon3) >>>> KONRAD Frederic <frederic.konrad@adacore.com> (maintainer:Leon3) >>>> qemu-devel@nongnu.org (open list:All patches CC here) >>>> >>> >>>> This is a bug but not 'security critical', so it might wait 5.1 and go >>>> via qemu-trivial tree. >>> >>> Well let's do that then if you're ok. >> >> OK, then ping? :) >> >>> >>> Best Regards, >>> Fred >>> >>>> >>>> Regards, >>>> >>>> Phil. >>>> >>>>> >>>>> Cheers, >>>>> Fred >>>>> >>>>>> >>>>>>> hw/misc/grlib_ahb_apb_pnp: Add trace events on read accesses >>>>>>> hw/timer/grlib_gptimer: Display frequency in decimal >>>>>>> target/sparc/int32_helper: Remove DEBUG_PCALL definition >>>>>>> target/sparc/int32_helper: Extract and use excp_name_str() >>>>>>> >>>>>>> hw/misc/grlib_ahb_apb_pnp.c | 24 >>>>>>> ++++++++++++++++++++++-- >>>>>>> target/sparc/int32_helper.c | 23 >>>>>>> ++++++++++++----------- >>>>>>> hw/misc/trace-events | 4 ++++ >>>>>>> hw/timer/trace-events | 2 +- >>>>>>> tests/acceptance/machine_sparc_leon3.py | 4 ++++ >>>>>>> 5 files changed, 43 insertions(+), 14 deletions(-) >>>>>>> >>>>> >>> >>
On 5/25/20 1:02 PM, Fred Konrad wrote: > Sorry Philippe I missed that. > > Would be happy to do a PR if needed but: > * I never did that. > * Looking at https://wiki.qemu.org/Contribute/SubmitAPullRequest, I > don't have > the signed GPG key either. Thanks Fred for following this series. I am not insisting for you to do the pull request, I was waiting for Artyom (sparc32) and Mark (sparc64) who usually handle the pull requests. Artyom seems busy lately. Mark, do you plan to send a hw/sparc* pull request soon? Do you prefer I ask to trivial@ for this series? FYI there is another sparc32 candidate patch: https://www.mail-archive.com/qemu-devel@nongnu.org/msg701850.html Regards, Phil. > > Cheers, > Fred > > Le 5/23/20 à 7:26 PM, Philippe Mathieu-Daudé a écrit : >> ping? >> >> On 5/11/20 9:03 AM, Philippe Mathieu-Daudé wrote: >>> On 4/14/20 12:00 PM, KONRAD Frederic wrote: >>>> Le 4/13/20 à 11:07 PM, Philippe Mathieu-Daudé a écrit : >>>>> [Cc'ing Peter] >>>>> >>>>> On 4/13/20 12:12 PM, KONRAD Frederic wrote: >>>>>> Le 4/11/20 à 7:30 PM, Philippe Mathieu-Daudé a écrit : >>>>>>> On 3/31/20 12:50 PM, Philippe Mathieu-Daudé wrote: >>>>>>>> Philippe Mathieu-Daudé (7): >>>>>>>> hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to AHB PnP >>>>>>>> registers >>>>>>>> hw/misc/grlib_ahb_apb_pnp: Fix AHB PnP 8-bit accesses >>>>>>> >>>>>>> Ping ^^^ for 5.0? >>>>>> >>>>>> Hi Philippe, >>>>>> >>>>>> You already have my rb tag for those one, and IMHO they should be >>>>>> good >>>>>> candidate >>>>>> for 5.0 (if it's not too late). >>>>> >>>>> Yes, thanks for the reviews. I see Mark Cave-Ayland merged this file >>>>> first, but you are listed as maintainer :) I was hoping you could >>>>> send a >>>>> pull request. >>>> >>>> Yes that's usually Mark who take the patches, sorry I didn't get it. >>> >>> No worries. As there are other sparc64 patches on the list, maybe Mark >>> will prepare a pull request now. >>> >>>> >>>>> >>>>> $ scripts/get_maintainer.pl -f hw/misc/grlib_ahb_apb_pnp.c >>>>> Fabien Chouteau <chouteau@adacore.com> (maintainer:Leon3) >>>>> KONRAD Frederic <frederic.konrad@adacore.com> (maintainer:Leon3) >>>>> qemu-devel@nongnu.org (open list:All patches CC here) >>>>> >>>> >>>>> This is a bug but not 'security critical', so it might wait 5.1 and go >>>>> via qemu-trivial tree. >>>> >>>> Well let's do that then if you're ok. >>> >>> OK, then ping? :) >>> >>>> >>>> Best Regards, >>>> Fred >>>> >>>>> >>>>> Regards, >>>>> >>>>> Phil. >>>>> >>>>>> >>>>>> Cheers, >>>>>> Fred >>>>>> >>>>>>> >>>>>>>> hw/misc/grlib_ahb_apb_pnp: Add trace events on read accesses >>>>>>>> hw/timer/grlib_gptimer: Display frequency in decimal >>>>>>>> target/sparc/int32_helper: Remove DEBUG_PCALL definition >>>>>>>> target/sparc/int32_helper: Extract and use excp_name_str() >>>>>>>> >>>>>>>> hw/misc/grlib_ahb_apb_pnp.c | 24 >>>>>>>> ++++++++++++++++++++++-- >>>>>>>> target/sparc/int32_helper.c | 23 >>>>>>>> ++++++++++++----------- >>>>>>>> hw/misc/trace-events | 4 ++++ >>>>>>>> hw/timer/trace-events | 2 +- >>>>>>>> tests/acceptance/machine_sparc_leon3.py | 4 ++++ >>>>>>>> 5 files changed, 43 insertions(+), 14 deletions(-) >>>>>>>> >>>>>> >>>> >>> >
On 5/26/20 11:50 AM, Philippe Mathieu-Daudé wrote: > On 5/25/20 1:02 PM, Fred Konrad wrote: >> Sorry Philippe I missed that. >> >> Would be happy to do a PR if needed but: >> * I never did that. >> * Looking at https://wiki.qemu.org/Contribute/SubmitAPullRequest, I >> don't have >> the signed GPG key either. > > Thanks Fred for following this series. > > I am not insisting for you to do the pull request, I was waiting for > Artyom (sparc32) and Mark (sparc64) who usually handle the pull requests. > > Artyom seems busy lately. Artyom, if you agree I can send a pullreq with this series and the empty_slot one. > > Mark, do you plan to send a hw/sparc* pull request soon? Do you prefer I > ask to trivial@ for this series? > > FYI there is another sparc32 candidate patch: > https://www.mail-archive.com/qemu-devel@nongnu.org/msg701850.html > > Regards, > > Phil. > >> >> Cheers, >> Fred >> >> Le 5/23/20 à 7:26 PM, Philippe Mathieu-Daudé a écrit : >>> ping? >>> >>> On 5/11/20 9:03 AM, Philippe Mathieu-Daudé wrote: >>>> On 4/14/20 12:00 PM, KONRAD Frederic wrote: >>>>> Le 4/13/20 à 11:07 PM, Philippe Mathieu-Daudé a écrit : >>>>>> [Cc'ing Peter] >>>>>> >>>>>> On 4/13/20 12:12 PM, KONRAD Frederic wrote: >>>>>>> Le 4/11/20 à 7:30 PM, Philippe Mathieu-Daudé a écrit : >>>>>>>> On 3/31/20 12:50 PM, Philippe Mathieu-Daudé wrote: >>>>>>>>> Philippe Mathieu-Daudé (7): >>>>>>>>> hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to AHB PnP >>>>>>>>> registers >>>>>>>>> hw/misc/grlib_ahb_apb_pnp: Fix AHB PnP 8-bit accesses >>>>>>>> >>>>>>>> Ping ^^^ for 5.0? >>>>>>> >>>>>>> Hi Philippe, >>>>>>> >>>>>>> You already have my rb tag for those one, and IMHO they should be >>>>>>> good >>>>>>> candidate >>>>>>> for 5.0 (if it's not too late). >>>>>> >>>>>> Yes, thanks for the reviews. I see Mark Cave-Ayland merged this file >>>>>> first, but you are listed as maintainer :) I was hoping you could >>>>>> send a >>>>>> pull request. >>>>> >>>>> Yes that's usually Mark who take the patches, sorry I didn't get it. >>>> >>>> No worries. As there are other sparc64 patches on the list, maybe Mark >>>> will prepare a pull request now. >>>> >>>>> >>>>>> >>>>>> $ scripts/get_maintainer.pl -f hw/misc/grlib_ahb_apb_pnp.c >>>>>> Fabien Chouteau <chouteau@adacore.com> (maintainer:Leon3) >>>>>> KONRAD Frederic <frederic.konrad@adacore.com> (maintainer:Leon3) >>>>>> qemu-devel@nongnu.org (open list:All patches CC here) >>>>>> >>>>> >>>>>> This is a bug but not 'security critical', so it might wait 5.1 and go >>>>>> via qemu-trivial tree. >>>>> >>>>> Well let's do that then if you're ok. >>>> >>>> OK, then ping? :) >>>> >>>>> >>>>> Best Regards, >>>>> Fred >>>>> >>>>>> >>>>>> Regards, >>>>>> >>>>>> Phil. >>>>>> >>>>>>> >>>>>>> Cheers, >>>>>>> Fred >>>>>>> >>>>>>>> >>>>>>>>> hw/misc/grlib_ahb_apb_pnp: Add trace events on read accesses >>>>>>>>> hw/timer/grlib_gptimer: Display frequency in decimal >>>>>>>>> target/sparc/int32_helper: Remove DEBUG_PCALL definition >>>>>>>>> target/sparc/int32_helper: Extract and use excp_name_str() >>>>>>>>> >>>>>>>>> hw/misc/grlib_ahb_apb_pnp.c | 24 >>>>>>>>> ++++++++++++++++++++++-- >>>>>>>>> target/sparc/int32_helper.c | 23 >>>>>>>>> ++++++++++++----------- >>>>>>>>> hw/misc/trace-events | 4 ++++ >>>>>>>>> hw/timer/trace-events | 2 +- >>>>>>>>> tests/acceptance/machine_sparc_leon3.py | 4 ++++ >>>>>>>>> 5 files changed, 43 insertions(+), 14 deletions(-) >>>>>>>>> >>>>>>> >>>>> >>>> >> >
On 08/06/2020 16:20, Philippe Mathieu-Daudé wrote: > On 5/26/20 11:50 AM, Philippe Mathieu-Daudé wrote: >> On 5/25/20 1:02 PM, Fred Konrad wrote: >>> Sorry Philippe I missed that. >>> >>> Would be happy to do a PR if needed but: >>> * I never did that. >>> * Looking at https://wiki.qemu.org/Contribute/SubmitAPullRequest, I >>> don't have >>> the signed GPG key either. >> >> Thanks Fred for following this series. >> >> I am not insisting for you to do the pull request, I was waiting for >> Artyom (sparc32) and Mark (sparc64) who usually handle the pull requests. >> >> Artyom seems busy lately. > > Artyom, if you agree I can send a pullreq with this series and the > empty_slot one. > >> >> Mark, do you plan to send a hw/sparc* pull request soon? Do you prefer I >> ask to trivial@ for this series? >> >> FYI there is another sparc32 candidate patch: >> https://www.mail-archive.com/qemu-devel@nongnu.org/msg701850.html Apologies, it looks as if this one slipped under my radar. I don't have any SPARC patches queued at the moment, so if you have a series prepared with both these patches then I'm happy for you to include them. The bits of time I have had to work on QEMU this cycle have been spent looking at the QOM patches and some ADB fixes I hope to finish soon... ATB, Mark.
On Mon, Jun 8, 2020 at 5:20 PM Philippe Mathieu-Daudé <f4bug@amsat.org> wrote: > > On 5/26/20 11:50 AM, Philippe Mathieu-Daudé wrote: > > On 5/25/20 1:02 PM, Fred Konrad wrote: > >> Sorry Philippe I missed that. > >> > >> Would be happy to do a PR if needed but: > >> * I never did that. > >> * Looking at https://wiki.qemu.org/Contribute/SubmitAPullRequest, I > >> don't have > >> the signed GPG key either. > > > > Thanks Fred for following this series. > > > > I am not insisting for you to do the pull request, I was waiting for > > Artyom (sparc32) and Mark (sparc64) who usually handle the pull requests. > > > > Artyom seems busy lately. > > Artyom, if you agree I can send a pullreq with this series and the > empty_slot one. > Yes, please. Artyom > > > > Mark, do you plan to send a hw/sparc* pull request soon? Do you prefer I > > ask to trivial@ for this series? > > > > FYI there is another sparc32 candidate patch: > > https://www.mail-archive.com/qemu-devel@nongnu.org/msg701850.html > > > > Regards, > > > > Phil. > > > >> > >> Cheers, > >> Fred > >> > >> Le 5/23/20 à 7:26 PM, Philippe Mathieu-Daudé a écrit : > >>> ping? > >>> > >>> On 5/11/20 9:03 AM, Philippe Mathieu-Daudé wrote: > >>>> On 4/14/20 12:00 PM, KONRAD Frederic wrote: > >>>>> Le 4/13/20 à 11:07 PM, Philippe Mathieu-Daudé a écrit : > >>>>>> [Cc'ing Peter] > >>>>>> > >>>>>> On 4/13/20 12:12 PM, KONRAD Frederic wrote: > >>>>>>> Le 4/11/20 à 7:30 PM, Philippe Mathieu-Daudé a écrit : > >>>>>>>> On 3/31/20 12:50 PM, Philippe Mathieu-Daudé wrote: > >>>>>>>>> Philippe Mathieu-Daudé (7): > >>>>>>>>> hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to AHB PnP > >>>>>>>>> registers > >>>>>>>>> hw/misc/grlib_ahb_apb_pnp: Fix AHB PnP 8-bit accesses > >>>>>>>> > >>>>>>>> Ping ^^^ for 5.0? > >>>>>>> > >>>>>>> Hi Philippe, > >>>>>>> > >>>>>>> You already have my rb tag for those one, and IMHO they should be > >>>>>>> good > >>>>>>> candidate > >>>>>>> for 5.0 (if it's not too late). > >>>>>> > >>>>>> Yes, thanks for the reviews. I see Mark Cave-Ayland merged this file > >>>>>> first, but you are listed as maintainer :) I was hoping you could > >>>>>> send a > >>>>>> pull request. > >>>>> > >>>>> Yes that's usually Mark who take the patches, sorry I didn't get it. > >>>> > >>>> No worries. As there are other sparc64 patches on the list, maybe Mark > >>>> will prepare a pull request now. > >>>> > >>>>> > >>>>>> > >>>>>> $ scripts/get_maintainer.pl -f hw/misc/grlib_ahb_apb_pnp.c > >>>>>> Fabien Chouteau <chouteau@adacore.com> (maintainer:Leon3) > >>>>>> KONRAD Frederic <frederic.konrad@adacore.com> (maintainer:Leon3) > >>>>>> qemu-devel@nongnu.org (open list:All patches CC here) > >>>>>> > >>>>> > >>>>>> This is a bug but not 'security critical', so it might wait 5.1 and go > >>>>>> via qemu-trivial tree. > >>>>> > >>>>> Well let's do that then if you're ok. > >>>> > >>>> OK, then ping? :) > >>>> > >>>>> > >>>>> Best Regards, > >>>>> Fred > >>>>> > >>>>>> > >>>>>> Regards, > >>>>>> > >>>>>> Phil. > >>>>>> > >>>>>>> > >>>>>>> Cheers, > >>>>>>> Fred > >>>>>>> > >>>>>>>> > >>>>>>>>> hw/misc/grlib_ahb_apb_pnp: Add trace events on read accesses > >>>>>>>>> hw/timer/grlib_gptimer: Display frequency in decimal > >>>>>>>>> target/sparc/int32_helper: Remove DEBUG_PCALL definition > >>>>>>>>> target/sparc/int32_helper: Extract and use excp_name_str() > >>>>>>>>> > >>>>>>>>> hw/misc/grlib_ahb_apb_pnp.c | 24 > >>>>>>>>> ++++++++++++++++++++++-- > >>>>>>>>> target/sparc/int32_helper.c | 23 > >>>>>>>>> ++++++++++++----------- > >>>>>>>>> hw/misc/trace-events | 4 ++++ > >>>>>>>>> hw/timer/trace-events | 2 +- > >>>>>>>>> tests/acceptance/machine_sparc_leon3.py | 4 ++++ > >>>>>>>>> 5 files changed, 43 insertions(+), 14 deletions(-) > >>>>>>>>> > >>>>>>> > >>>>> > >>>> > >> > > -- Regards, Artyom Tarasenko SPARC and PPC PReP under qemu blog: http://tyom.blogspot.com/search/label/qemu
On 3/31/20 12:50 PM, Philippe Mathieu-Daudé wrote: > The Leon3Machine.test_leon3_helenos_uimage has been running > erratically since some time now. Time to disable it (at least > until we make it reliable). > Few other patches added while tracking the issue. > > Philippe Mathieu-Daudé (7): > tests/acceptance/machine_sparc_leon3: Disable HelenOS test > hw/misc/grlib_ahb_apb_pnp: Avoid crash when writing to AHB PnP > registers > hw/misc/grlib_ahb_apb_pnp: Fix AHB PnP 8-bit accesses > hw/misc/grlib_ahb_apb_pnp: Add trace events on read accesses > hw/timer/grlib_gptimer: Display frequency in decimal > target/sparc/int32_helper: Remove DEBUG_PCALL definition > target/sparc/int32_helper: Extract and use excp_name_str() > > hw/misc/grlib_ahb_apb_pnp.c | 24 ++++++++++++++++++++++-- > target/sparc/int32_helper.c | 23 ++++++++++++----------- > hw/misc/trace-events | 4 ++++ > hw/timer/trace-events | 2 +- > tests/acceptance/machine_sparc_leon3.py | 4 ++++ > 5 files changed, 43 insertions(+), 14 deletions(-) > Thanks, series applied to for the next (temporary) sparc-next pull request.
© 2016 - 2024 Red Hat, Inc.