mbox

[PULL,0/4] target-hppa fixes

Message ID 20200728193550.3578-1-deller@gmx.de
State New
Headers show

Pull-request

https://github.com/hdeller/qemu-hppa.git target-hppa

Message

Helge Deller July 28, 2020, 7:35 p.m. UTC
Please pull to fix those two bugs in target-hppa:

* Fix the SeaBIOS-hppa firmware build with gcc-10 on Debian

* Fix the following runtime warning with artist framebuffer:
  "write outside bounds: wants 1256x1023, max size 1280x1024"

in addition the SeaBIOS-hppa firmware now includes a version check to prevent
starting when it's incompatible to the emulated qemu hardware.

Helge

----------------------------------------------------------------

The following changes since commit 3461487523b897d324e8d91f3fd20ed55f849544:

  Merge remote-tracking branch 'remotes/pmaydell/tags/pull-target-arm-20200727' into staging (2020-07-28 18:43:48 +0100)

are available in the Git repository at:

  https://github.com/hdeller/qemu-hppa.git target-hppa

for you to fetch changes up to 9aa10e5543566facf328e76d3b5a4aa9d2b79756:

  hw/display/artist.c: fix out of bounds check (2020-07-28 21:17:44 +0200)

----------------------------------------------------------------

Helge Deller (3):
      hw/hppa: Sync hppa_hardware.h file with SeaBIOS sources
      seabios-hppa: Update to SeaBIOS hppa version 1
      hw/hppa: Implement proper SeaBIOS version check

Sven Schnelle (1):
      hw/display/artist.c: fix out of bounds check

 hw/display/artist.c       |  18 ++++++------------
 hw/hppa/hppa_hardware.h   |   6 ++++++
 hw/hppa/lasi.c            |   2 --
 hw/hppa/machine.c         |  22 ++++++++++++++++++++++
 pc-bios/hppa-firmware.img | Bin 766136 -> 783144 bytes
 roms/seabios-hppa         |   2 +-
 6 files changed, 35 insertions(+), 15 deletions(-)
--
2.21.3

Helge Deller (3):
  hw/hppa: Sync hppa_hardware.h file with SeaBIOS sources
  seabios-hppa: Update to SeaBIOS hppa version 1
  hw/hppa: Implement proper SeaBIOS version check

Sven Schnelle (1):
  hw/display/artist.c: fix out of bounds check

 hw/display/artist.c       |  18 ++++++------------
 hw/hppa/hppa_hardware.h   |   6 ++++++
 hw/hppa/lasi.c            |   2 --
 hw/hppa/machine.c         |  22 ++++++++++++++++++++++
 pc-bios/hppa-firmware.img | Bin 766136 -> 783144 bytes
 roms/seabios-hppa         |   2 +-
 6 files changed, 35 insertions(+), 15 deletions(-)

--
2.21.3

Comments

Peter Maydell July 29, 2020, 3:06 p.m. UTC | #1
On Tue, 28 Jul 2020 at 20:35, Helge Deller <deller@gmx.de> wrote:
>
> Please pull to fix those two bugs in target-hppa:
>
> * Fix the SeaBIOS-hppa firmware build with gcc-10 on Debian
>
> * Fix the following runtime warning with artist framebuffer:
>   "write outside bounds: wants 1256x1023, max size 1280x1024"
>
> in addition the SeaBIOS-hppa firmware now includes a version check to prevent
> starting when it's incompatible to the emulated qemu hardware.

Hi. I'd rather not take a pull request from somebody who's
never submitted one before at rc2 with four patches none
of which have been reviewed, I'm afraid.

Richard, you're listed in MAINTAINERS for most of these
files, would you mind giving them a look over?

thanks
-- PMM
Helge Deller July 29, 2020, 3:49 p.m. UTC | #2
On 29.07.20 17:06, Peter Maydell wrote:
> On Tue, 28 Jul 2020 at 20:35, Helge Deller <deller@gmx.de> wrote:
>>
>> Please pull to fix those two bugs in target-hppa:
>>
>> * Fix the SeaBIOS-hppa firmware build with gcc-10 on Debian
>>
>> * Fix the following runtime warning with artist framebuffer:
>>   "write outside bounds: wants 1256x1023, max size 1280x1024"
>>
>> in addition the SeaBIOS-hppa firmware now includes a version check to prevent
>> starting when it's incompatible to the emulated qemu hardware.
>
> Hi. I'd rather not take a pull request from somebody who's
> never submitted one before at rc2 with four patches none
> of which have been reviewed, I'm afraid.

Sure, your reluctance is understood.

> Richard, you're listed in MAINTAINERS for most of these
> files, would you mind giving them a look over?

If Richard gives his Ok, it would be really nice if you could
apply those.

Thanks!
Helge