mbox series

[0/6] tests/acceptance: Test the Fuloong 2E machine

Message ID 20201021105035.2477784-1-f4bug@amsat.org
Headers show
Series tests/acceptance: Test the Fuloong 2E machine | expand

Message

Philippe Mathieu-Daudé Oct. 21, 2020, 10:50 a.m. UTC
This series add tests for the MIPS Fuloong 2E machine.

I have been using these tests for over 1 year now.
After recent discussions with Huacai and Mark regarding
how to contribute tests for machines using private blob,
I think it is time to publish this series, to see how
the final part will be discussed, as it tests a blob
which Lemote company doesn't publish the sources. The
binary is know to work and has been used for various
years on real hardware. Its MD5 is also know (I am not
using SHA-1 because it has been published with MD5).

The idea is to be able to share tests without infringing
licenses issues.

Please comment :)

Phil.

Philippe Mathieu-Daudé (6):
  tests/acceptance: Remove unused import
  tests/acceptance: Use .ppm extention for Portable PixMap files
  tests/acceptance: Extract tesseract_available() helper in new
    namespace
  tests/acceptance: Introduce tesseract_ocr() helper
  tests/acceptance: Test Linux on the Fuloong 2E machine
  tests/acceptance: Test PMON on the Fuloong 2E machine

 MAINTAINERS                                |   1 +
 tests/acceptance/machine_m68k_nextcube.py  |  51 ++---------
 tests/acceptance/machine_mips_fuloong2e.py | 101 +++++++++++++++++++++
 tests/acceptance/tesseract_utils.py        |  46 ++++++++++
 4 files changed, 158 insertions(+), 41 deletions(-)
 create mode 100644 tests/acceptance/machine_mips_fuloong2e.py
 create mode 100644 tests/acceptance/tesseract_utils.py

-- 
2.26.2

Comments

Mark Cave-Ayland Oct. 25, 2020, 11:18 a.m. UTC | #1
On 21/10/2020 11:50, Philippe Mathieu-Daudé wrote:

> This series add tests for the MIPS Fuloong 2E machine.

> 

> I have been using these tests for over 1 year now.

> After recent discussions with Huacai and Mark regarding

> how to contribute tests for machines using private blob,

> I think it is time to publish this series, to see how

> the final part will be discussed, as it tests a blob

> which Lemote company doesn't publish the sources. The

> binary is know to work and has been used for various

> years on real hardware. Its MD5 is also know (I am not

> using SHA-1 because it has been published with MD5).

> 

> The idea is to be able to share tests without infringing

> licenses issues.

> 

> Please comment :)

> 

> Phil.

> 

> Philippe Mathieu-Daudé (6):

>    tests/acceptance: Remove unused import

>    tests/acceptance: Use .ppm extention for Portable PixMap files

>    tests/acceptance: Extract tesseract_available() helper in new

>      namespace

>    tests/acceptance: Introduce tesseract_ocr() helper

>    tests/acceptance: Test Linux on the Fuloong 2E machine

>    tests/acceptance: Test PMON on the Fuloong 2E machine

> 

>   MAINTAINERS                                |   1 +

>   tests/acceptance/machine_m68k_nextcube.py  |  51 ++---------

>   tests/acceptance/machine_mips_fuloong2e.py | 101 +++++++++++++++++++++

>   tests/acceptance/tesseract_utils.py        |  46 ++++++++++

>   4 files changed, 158 insertions(+), 41 deletions(-)

>   create mode 100644 tests/acceptance/machine_mips_fuloong2e.py

>   create mode 100644 tests/acceptance/tesseract_utils.py


In general this looks good: certainly I don't see any reason why the tesseract_ocr 
helper changes shouldn't be merged, since I expect that this is something that will 
be used more in future.

Looking at the fuloong test it seems that it stops fairly early in the kernel boot: 
one of the problems that Zoltan's VIA southbridge patches were trying to solve 
earlier in the year was because of problems with PCI IRQ routing, so it would be nice 
to include a small OS image that can be used to boot to userspace to ensure that such 
problems can be detected in future.


ATB,

Mark.
Philippe Mathieu-Daudé Oct. 25, 2020, 12:22 p.m. UTC | #2
On 10/25/20 12:18 PM, Mark Cave-Ayland wrote:
> On 21/10/2020 11:50, Philippe Mathieu-Daudé wrote:

> 

>> This series add tests for the MIPS Fuloong 2E machine.

>>

>> I have been using these tests for over 1 year now.

>> After recent discussions with Huacai and Mark regarding

>> how to contribute tests for machines using private blob,

>> I think it is time to publish this series, to see how

>> the final part will be discussed, as it tests a blob

>> which Lemote company doesn't publish the sources. The

>> binary is know to work and has been used for various

>> years on real hardware. Its MD5 is also know (I am not

>> using SHA-1 because it has been published with MD5).

>>

>> The idea is to be able to share tests without infringing

>> licenses issues.

>>

>> Please comment :)

>>

>> Phil.

>>

>> Philippe Mathieu-Daudé (6):

>>    tests/acceptance: Remove unused import

>>    tests/acceptance: Use .ppm extention for Portable PixMap files

>>    tests/acceptance: Extract tesseract_available() helper in new

>>      namespace

>>    tests/acceptance: Introduce tesseract_ocr() helper

>>    tests/acceptance: Test Linux on the Fuloong 2E machine

>>    tests/acceptance: Test PMON on the Fuloong 2E machine

>>

>>   MAINTAINERS                                |   1 +

>>   tests/acceptance/machine_m68k_nextcube.py  |  51 ++---------

>>   tests/acceptance/machine_mips_fuloong2e.py | 101 +++++++++++++++++++++

>>   tests/acceptance/tesseract_utils.py        |  46 ++++++++++

>>   4 files changed, 158 insertions(+), 41 deletions(-)

>>   create mode 100644 tests/acceptance/machine_mips_fuloong2e.py

>>   create mode 100644 tests/acceptance/tesseract_utils.py

> 

> In general this looks good: certainly I don't see any reason why the 

> tesseract_ocr helper changes shouldn't be merged, since I expect that 

> this is something that will be used more in future.

> 

> Looking at the fuloong test it seems that it stops fairly early in the 

> kernel boot: one of the problems that Zoltan's VIA southbridge patches 

> were trying to solve earlier in the year was because of problems with 

> PCI IRQ routing, so it would be nice to include a small OS image that 

> can be used to boot to userspace to ensure that such problems can be 

> detected in future.


The Linux tests (ab)use of the rescue image prepared for the Yeeloong
machine which we don't model (Loongson 2F CPU, 1GB of DRAM). The kernel
triggers DATA ABORT while accessing >256MB because we do not model
mapping more than 256MB :/ (The Fuloong comes with 512MB extensible to 
1GB). I have a branch adding that but had the idea nobody was interested
in the Fuloong 2E so I didn't insist (I have more interest in embedded
32-bit CPUs). I might try to post it after the release, as it makes this
test complete.

Thanks for giving it a try!

Phil.
Xingtao Yao (Fujitsu)" via Oct. 25, 2020, 2:30 p.m. UTC | #3
On Sun, 25 Oct 2020, Philippe Mathieu-Daudé wrote:
> On 10/25/20 12:18 PM, Mark Cave-Ayland wrote:

>> On 21/10/2020 11:50, Philippe Mathieu-Daudé wrote:

>> 

>>> This series add tests for the MIPS Fuloong 2E machine.

>>> 

>>> I have been using these tests for over 1 year now.

>>> After recent discussions with Huacai and Mark regarding

>>> how to contribute tests for machines using private blob,

>>> I think it is time to publish this series, to see how

>>> the final part will be discussed, as it tests a blob

>>> which Lemote company doesn't publish the sources. The

>>> binary is know to work and has been used for various

>>> years on real hardware. Its MD5 is also know (I am not

>>> using SHA-1 because it has been published with MD5).

>>> 

>>> The idea is to be able to share tests without infringing

>>> licenses issues.

>>> 

>>> Please comment :)

>>> 

>>> Phil.

>>> 

>>> Philippe Mathieu-Daudé (6):

>>>    tests/acceptance: Remove unused import

>>>    tests/acceptance: Use .ppm extention for Portable PixMap files

>>>    tests/acceptance: Extract tesseract_available() helper in new

>>>      namespace

>>>    tests/acceptance: Introduce tesseract_ocr() helper

>>>    tests/acceptance: Test Linux on the Fuloong 2E machine

>>>    tests/acceptance: Test PMON on the Fuloong 2E machine

>>> 

>>>   MAINTAINERS                                |   1 +

>>>   tests/acceptance/machine_m68k_nextcube.py  |  51 ++---------

>>>   tests/acceptance/machine_mips_fuloong2e.py | 101 +++++++++++++++++++++

>>>   tests/acceptance/tesseract_utils.py        |  46 ++++++++++

>>>   4 files changed, 158 insertions(+), 41 deletions(-)

>>>   create mode 100644 tests/acceptance/machine_mips_fuloong2e.py

>>>   create mode 100644 tests/acceptance/tesseract_utils.py

>> 

>> In general this looks good: certainly I don't see any reason why the 

>> tesseract_ocr helper changes shouldn't be merged, since I expect that this 

>> is something that will be used more in future.

>> 

>> Looking at the fuloong test it seems that it stops fairly early in the 

>> kernel boot: one of the problems that Zoltan's VIA southbridge patches were 

>> trying to solve earlier in the year was because of problems with PCI IRQ 

>> routing, so it would be nice to include a small OS image that can be used 

>> to boot to userspace to ensure that such problems can be detected in 

>> future.

>

> The Linux tests (ab)use of the rescue image prepared for the Yeeloong

> machine which we don't model (Loongson 2F CPU, 1GB of DRAM). The kernel

> triggers DATA ABORT while accessing >256MB because we do not model

> mapping more than 256MB :/ (The Fuloong comes with 512MB extensible to 1GB). 

> I have a branch adding that but had the idea nobody was interested

> in the Fuloong 2E so I didn't insist (I have more interest in embedded

> 32-bit CPUs). I might try to post it after the release, as it makes this

> test complete.


I've noticed that missing high RAM when cleaning up SPD EEPROMs and addded 
a FIXME comment but did't know MIPS enough to attemt to fix it. Anyway I'm 
also only interested in fuloong2e as a test case for the devices also used 
by Pegasos2 (that I'll need to update and clean up to submit eventually) 
but not beyond that.

I haven't seen these patches before, so I've spotted only now that your 
test case uses -vga std. Shouldn't it work with ati-vga,model=rv100 which 
I think is the default for this machine and that's what the pmon firmware 
expects and has a driver for?

Regards,
BALATON Zoltan
Philippe Mathieu-Daudé Oct. 26, 2020, 8:50 a.m. UTC | #4
On 10/21/20 12:50 PM, Philippe Mathieu-Daudé wrote:
> This series add tests for the MIPS Fuloong 2E machine.

> 

> I have been using these tests for over 1 year now.

> After recent discussions with Huacai and Mark regarding

> how to contribute tests for machines using private blob,

> I think it is time to publish this series, to see how

> the final part will be discussed, as it tests a blob

> which Lemote company doesn't publish the sources. The

> binary is know to work and has been used for various

> years on real hardware. Its MD5 is also know (I am not

> using SHA-1 because it has been published with MD5).

> 

> The idea is to be able to share tests without infringing

> licenses issues.

> 

> Please comment :)

> 

> Phil.

> 

> Philippe Mathieu-Daudé (6):

>    tests/acceptance: Remove unused import

>    tests/acceptance: Use .ppm extention for Portable PixMap files

>    tests/acceptance: Extract tesseract_available() helper in new

>      namespace

>    tests/acceptance: Introduce tesseract_ocr() helper

>    tests/acceptance: Test Linux on the Fuloong 2E machine

>    tests/acceptance: Test PMON on the Fuloong 2E machine

> 

>   MAINTAINERS                                |   1 +

>   tests/acceptance/machine_m68k_nextcube.py  |  51 ++---------

>   tests/acceptance/machine_mips_fuloong2e.py | 101 +++++++++++++++++++++

>   tests/acceptance/tesseract_utils.py        |  46 ++++++++++

>   4 files changed, 158 insertions(+), 41 deletions(-)

>   create mode 100644 tests/acceptance/machine_mips_fuloong2e.py

>   create mode 100644 tests/acceptance/tesseract_utils.py


Thanks, patches #1 and #2 applied to my acceptance-testing tree.
Philippe Mathieu-Daudé Oct. 26, 2020, 5:20 p.m. UTC | #5
On 10/26/20 9:50 AM, Philippe Mathieu-Daudé wrote:
> On 10/21/20 12:50 PM, Philippe Mathieu-Daudé wrote:

>> Philippe Mathieu-Daudé (6):

>>    tests/acceptance: Remove unused import

>>    tests/acceptance: Use .ppm extention for Portable PixMap files

>>    tests/acceptance: Extract tesseract_available() helper in new

>>      namespace

>>    tests/acceptance: Introduce tesseract_ocr() helper

>>    tests/acceptance: Test Linux on the Fuloong 2E machine

>>    tests/acceptance: Test PMON on the Fuloong 2E machine

>>

>>   MAINTAINERS                                |   1 +

>>   tests/acceptance/machine_m68k_nextcube.py  |  51 ++---------

>>   tests/acceptance/machine_mips_fuloong2e.py | 101 +++++++++++++++++++++

>>   tests/acceptance/tesseract_utils.py        |  46 ++++++++++

>>   4 files changed, 158 insertions(+), 41 deletions(-)

>>   create mode 100644 tests/acceptance/machine_mips_fuloong2e.py

>>   create mode 100644 tests/acceptance/tesseract_utils.py

> 

> Thanks, patches #1 and #2 applied to my acceptance-testing tree.


Dequeuing as Thomas already merged them.