Re: [PATCH v7 00/35] Hexagon patch series

[Philippe Mathieu-Daudé]

Hi Taylor,

On 1/20/21 4:28 AM, Taylor Simpson wrote:
> This series adds support for the Hexagon processor with Linux user support
> 
> See patch 02/33 Hexagon README for detailed information.
> 
> This series assumes int128_or() is implemented.
> https://lists.nongnu.org/archive/html/qemu-devel/2020-10/msg06004.html
> 
> The series is also available at https://github.com/quic/qemu on branch
> small_series_v7.
> 
> 
> Once the series is applied, the Hexagon port will pass "make check-tcg".
> The series also includes Hexagon-specific tests in tcg/tests/hexagon.
> 
> The final patch in the series add docker support.  Thanks to Alessandro
> Di Federico <ale@rev.ng> and Brian Cain <bcain@quicinc.com> for making this
> happen.  The default container (debian-hexagon-cross) uses a toolchain built
> by rev.ng.  Alternatively, there is a container that will build the toolchain
> locally (debian-hexagon-cross-build-local).
> 
> Here are the commands to verify the code:
>     mkdir build
>     cd build
>     ../configure --target-list=hexagon-linux-user
>     make
>     make check-tcg

I'm looking at stressing a bit more your work.

Are your Hexagon Linux user-land binaries available on some repository?
Eventually busybox binary (known to work on real hardware) is a good
candidate.

Also I noticed these sample apps:
https://developer.qualcomm.com/software/hexagon-dsp-sdk/sample-apps

These could be nice to have as integration testing:

- appi_fir
  APPI-based module that applies a FIR filter to audio data
- image downscale
  Example showing deeply optimized 2D image downscaling function
- cornerApp
  Example that detects corners in an image, using aDSP FastCV primitives

Any other real-world binary you could share?

Thanks,

Phil.