Re: [Pixman] pixman_blt on aarch64

[Heiko Lewin]

    On 2/19/23 00:38, BALATON Zoltan wrote:
> On Sun,
>       5 Feb 2023, BALATON Zoltan wrote:
>       
> > On Sun, 5 Feb 2023, Richard Henderson
> >         wrote:
> >         
> > > On 2/4/23 06:57, BALATON Zoltan wrote:
> > >           
> > > > This has just bounced, I hoped to
> > > >             still be able to post after moderation but now I'm resending
> > > >             it after subscribing to the pixman list. Meanwhile I've
> > > >             found this ticket as well:
> > > >             https://gitlab.freedesktop.org/pixman/pixman/-/merge_requests/71
> > > >             
> > > >             See the rest of the message below. Looks like this is being
> > > >             worked on but I'm not sure how far is it from getting
> > > >             resolved. Any info on that?
> > > >             
> > > >           
> > >           
> > >           Please try this:
> > >           
> > >           
> > >           https://gitlab.freedesktop.org/rth7680/pixman/-/tree/general
> > >           
> > >           
> > >           It provides a pure C version for ultimate fallback.
> > >           
> > >           Unfortunately, there are no test cases for this, nor
> > >           documentation.
> > >           
> > >         
> >         
> >         Thanks, I don't have hardware to test this but maybe Akihiko or
> >         somebody else here cam try. Do you think pixman_fill won't have
> >         the same problem? It seems to have at least a fast_path
> >         implementation but I'm not sure how pixman selects these.
> >         
> >       
>       
>       We have tried the branch above and while it does make it better
>       and usable in 16bit mode, 8bit is still missing (no wonder as it
>       does not have that implemented but the AmigaOS driver only allows
>       16bit and 8bit with the sm501 device this is used in but probably
>       that's not important when 16 bit works). Even in 16bit more there
>       were some graphics problems seen but that may need some more
>       checking to make sure it's not some other change as the report was
>       not clear if that worked before. I've asked to check the changes
>       in
>       https://gitlab.freedesktop.org/pixman/pixman/-/merge_requests/71
>       next to see if that would work better.
>       
>       
>       Regards,
>       
>       BALATON Zoltan
>       
>     
    
    Hello!
    I just wanted to say that I have just made the last few changes to
    https://gitlab.freedesktop.org/pixman/pixman/-/merge_requests/71
    that were obviously missing in that patch.
    However, I have no easy way to test the lib on Apple hardware -
    neither if building the lib works with Xcode et al
    nor if the resulting binaries are good. I used clang to
    cross-compile for Apple arm64 from a linux system which
    seemed to work. Feel free to test the PR and report problems.
    
    Regards,
    Heiko