Re: [FFmpeg-trac] #8750(swscale:new): Patch to fix #7124 break the build for powerpc

[FFmpeg]

#8750: Patch to fix #7124 break the build for powerpc
-+-
 Reporter:  marillat |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  swscale
  Version:  git-master   |   Resolution:
 Keywords:  ppc  |   Blocked By:
  regression |
 Blocking:   |  Reproduced by developer:  0
Analyzed by developer:  0|
-+-
Changes (by cehoyos):

 * keywords:   => ppc regression
 * priority:  normal => important
 * version:  unspecified => git-master
 * component:  undetermined => swscale


--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-trac

To unsubscribe, visit link above, or email
ffmpeg-trac-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-trac] #8750(swscale:new): Patch to fix #7124 break the build for powerpc

[FFmpeg]

#8750: Patch to fix #7124 break the build for powerpc
-+-
 Reporter:  marillat |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  swscale
  Version:  git-master   |   Resolution:
 Keywords:  ppc  |   Blocked By:
  regression |
 Blocking:   |  Reproduced by developer:  0
Analyzed by developer:  0|
-+-
Changes (by ChipKerchner):

 * Attachment "ffmpeg_altivec_yuv2rgb_novsx.patch" added.


--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-trac

To unsubscribe, visit link above, or email
ffmpeg-trac-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-trac] #8750(swscale:new): Patch to fix #7124 break the build for powerpc

[FFmpeg]

#8750: Patch to fix #7124 break the build for powerpc
-+-
 Reporter:  marillat |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  swscale
  Version:  git-master   |   Resolution:
 Keywords:  ppc  |   Blocked By:
  regression |
 Blocking:   |  Reproduced by developer:  0
Analyzed by developer:  0|
-+-

Comment (by ChipKerchner):

 I suggest adding an inline function for the vec_xl intrinsic in non-VSX
 environments - see above patch.

--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-trac

To unsubscribe, visit link above, or email
ffmpeg-trac-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-trac] #8750(swscale:new): Patch to fix #7124 break the build for powerpc

[FFmpeg]

#8750: Patch to fix #7124 break the build for powerpc
-+-
 Reporter:  marillat |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  swscale
  Version:  git-master   |   Resolution:
 Keywords:  ppc  |   Blocked By:
  regression |
 Blocking:   |  Reproduced by developer:  0
Analyzed by developer:  0|
-+-

Comment (by marillat):

 Still doesn't build


 {{{
 libswscale/ppc/yuv2rgb_altivec.c: In function ‘altivec_yuv2_abgr’:
 libswscale/ppc/yuv2rgb_altivec.c:348:18: error: too many arguments to
 function ‘vec_xl’
   348 | y0 = vec_xl(0, y1i);
 \
   |  ^~
 libswscale/ppc/yuv2rgb_altivec.c:447:1: note: in expansion of macro
 ‘DEFCSP420_CVT’
   447 | DEFCSP420_CVT(yuv2_abgr,  out_abgr)
   | ^
 libswscale/ppc/yuv2rgb_altivec.c:287:36: note: declared here
   287 | static inline vector unsigned char vec_xl(const ubyte *addr)
   |^~
 libswscale/ppc/yuv2rgb_altivec.c:350:18: error: too many arguments to
 function ‘vec_xl’
   350 | y1 = vec_xl(0, y2i);
 \
   |  ^~
 libswscale/ppc/yuv2rgb_altivec.c:447:1: note: in expansion of macro
 ‘DEFCSP420_CVT’
   447 | DEFCSP420_CVT(yuv2_abgr,  out_abgr)
   | ^
 libswscale/ppc/yuv2rgb_altivec.c:287:36: note: declared here
   287 | static inline vector unsigned char vec_xl(const ubyte *addr)
   |^~
 libswscale/ppc/yuv2rgb_altivec.c:352:38: error: too many arguments to
 function ‘vec_xl’
   352 | u = (vector signed char) vec_xl(0, ui);
 \
   |  ^~
 libswscale/ppc/yuv2rgb_altivec.c:447:1: note: in expansion of macro
 ‘DEFCSP420_CVT’
   447 | DEFCSP420_CVT(yuv2_abgr,  out_abgr)
   | ^
 libswscale/ppc/yuv2rgb_altivec.c:287:36: note: declared here
   287 | static inline vector unsigned char vec_xl(const ubyte *addr)
   |^~
 libswscale/ppc/yuv2rgb_altivec.c:354:38: error: too many arguments to
 function ‘vec_xl’
   354 | v = (vector signed char) vec_xl(0, vi);
 \
   |  ^~
 libswscale/ppc/yuv2rgb_altivec.c:447:1: note: in expansion of macro
 ‘DEFCSP420_CVT’
   447 | DEFCSP420_CVT(yuv2_abgr,  out_abgr)
   | ^
 libswscale/ppc/yuv2rgb_altivec.c:287:36: note: declared here
   287 | static inline vector unsigned char vec_xl(const ubyte *addr)
   |^~
 libswscale/ppc/yuv2rgb_altivec.c: In function ‘altivec_yuv2_bgra’:
 libswscale/ppc/yuv2rgb_altivec.c:348:18: error: too many arguments to
 function ‘vec_xl’
   348 | y0 = vec_xl(0, y1i);
 \
   |  ^~
 libswscale/ppc/yuv2rgb_altivec.c:448:1: note: in expansion of macro
 ‘DEFCSP420_CVT’
   448 | DEFCSP420_CVT(yuv2_bgra,  out_bgra)
   | ^
 libswscale/ppc/yuv2rgb_altivec.c:287:36: note: declared here
   287 | static inline vector unsigned char vec_xl(const ubyte *addr)
   |^~
 libswscale/ppc/yuv2rgb_altivec.c:350:18: error: too many arguments to
 function ‘vec_xl’
   350 | y1 = vec_xl(0, y2i);
 \
   |  ^~
 libswscale/ppc/yuv2rgb_altivec.c:448:1: note: in expansion of macro
 ‘DEFCSP420_CVT’
   448 | DEFCSP420_CVT(yuv2_bgra,  out_bgra)
   | ^
 libswscale/ppc/yuv2rgb_altivec.c:287:36: note: declared here
   287 | static inline vector unsigned char vec_xl(const ubyte *addr)
   |^~
 libswscale/ppc/yuv2rgb_altivec.c:352:38: error: too many arguments to
 function ‘vec_xl’
   352 | u = (vector signed char) vec_xl(0, ui);
 \
   |  ^~
 libswscale/ppc/yuv2rgb_altivec.c:448:1: note: in expansion of macro
 ‘DEFCSP420_CVT’
   448 | DEFCSP420_CVT(yuv2_bgra,  out_bgra)
   | ^
 libswscale/ppc/yuv2rgb_altivec.c:287:36: note: declared here
   287 | static inline vector unsigned char vec_xl(const ubyte *addr)
   |^~
 libswscale/ppc/yuv2rgb_altivec.c:354:38: error: too many arguments to
 function ‘vec_xl’
   354 | v = (vector signed char) vec_xl(0, vi);
 \
   |  ^~
 libswscale/ppc/yuv2rgb_altivec.c:448:1: note: in expansion of macro
 ‘DEFCSP420_CVT’
   448 | DEFCSP420_CVT(yuv2_bgra,  out_bgra)
   | ^
 libswscale/ppc/yuv2rgb_altivec.c:287:36: note: declared here
   287 | static inline vector unsigned char 

Re: [FFmpeg-trac] #8750(swscale:new): Patch to fix #7124 break the build for powerpc

[FFmpeg]

#8750: Patch to fix #7124 break the build for powerpc
-+-
 Reporter:  marillat |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  swscale
  Version:  git-master   |   Resolution:
 Keywords:  ppc  |   Blocked By:
  regression |
 Blocking:   |  Reproduced by developer:  0
Analyzed by developer:  0|
-+-
Changes (by ChipKerchner):

 * Attachment "ffmpeg_altivec_yuv2rgb_novsx.patch" added.


--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-trac

To unsubscribe, visit link above, or email
ffmpeg-trac-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-trac] #8750(swscale:new): Patch to fix #7124 break the build for powerpc

[FFmpeg]

#8750: Patch to fix #7124 break the build for powerpc
-+-
 Reporter:  marillat |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  swscale
  Version:  git-master   |   Resolution:
 Keywords:  ppc  |   Blocked By:
  regression |
 Blocking:   |  Reproduced by developer:  0
Analyzed by developer:  0|
-+-

Comment (by marillat):

 The latest patch is the same than the first one.

--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-trac

To unsubscribe, visit link above, or email
ffmpeg-trac-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-trac] #8750(swscale:new): Patch to fix #7124 break the build for powerpc

[FFmpeg]

#8750: Patch to fix #7124 break the build for powerpc
-+-
 Reporter:  marillat |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  swscale
  Version:  git-master   |   Resolution:
 Keywords:  ppc  |   Blocked By:
  regression |
 Blocking:   |  Reproduced by developer:  0
Analyzed by developer:  0|
-+-

Comment (by ChipKerchner):

 Replying to [comment:5 marillat]:
 > The latest patch is the same than the first one.

 Changed the compiler directive to !HAVE_VSX (also I replaced the patch so
 you are NOT seeing the old one).

--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-trac

To unsubscribe, visit link above, or email
ffmpeg-trac-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-trac] #8750(swscale:new): Patch to fix #7124 break the build for powerpc

[FFmpeg]

#8750: Patch to fix #7124 break the build for powerpc
-+-
 Reporter:  marillat |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  swscale
  Version:  git-master   |   Resolution:
 Keywords:  ppc  |   Blocked By:
  regression |
 Blocking:   |  Reproduced by developer:  0
Analyzed by developer:  0|
-+-
Changes (by ChipKerchner):

 * Attachment "ffmpeg_altivec_yuv2rgb_novsx.patch" added.


--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-trac

To unsubscribe, visit link above, or email
ffmpeg-trac-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-trac] #8750(swscale:new): Patch to fix #7124 break the build for powerpc

[FFmpeg]

#8750: Patch to fix #7124 break the build for powerpc
-+-
 Reporter:  marillat |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  swscale
  Version:  git-master   |   Resolution:
 Keywords:  ppc  |   Blocked By:
  regression |
 Blocking:   |  Reproduced by developer:  0
Analyzed by developer:  0|
-+-

Comment (by marillat):

 Build fine now :)

 Thanks for the patch.

--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-trac

To unsubscribe, visit link above, or email
ffmpeg-trac-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-trac] #8750(swscale:new): Patch to fix #7124 break the build for powerpc

[FFmpeg]

#8750: Patch to fix #7124 break the build for powerpc
-+-
 Reporter:  marillat |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  swscale
  Version:  git-master   |   Resolution:
 Keywords:  ppc  |   Blocked By:
  regression |
 Blocking:   |  Reproduced by developer:  0
Analyzed by developer:  0|
-+-
Changes (by glaubitz):

 * cc: glaubitz@… (added)


--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-trac

To unsubscribe, visit link above, or email
ffmpeg-trac-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-trac] #8750(swscale:new): Patch to fix #7124 break the build for powerpc

[FFmpeg]

#8750: Patch to fix #7124 break the build for powerpc
-+-
 Reporter:  marillat |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  swscale
  Version:  git-master   |   Resolution:
 Keywords:  ppc  |   Blocked By:
  regression |
 Blocking:   |  Reproduced by developer:  0
Analyzed by developer:  0|
-+-

Comment (by glaubitz):

 Hello!

 I just ran into this issue as well. Any chance it can be fixed?

 See: https://trac.ffmpeg.org/ticket/8855

--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-trac

To unsubscribe, visit link above, or email
ffmpeg-trac-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-trac] #8750(swscale:new): Patch to fix #7124 break the build for powerpc

[FFmpeg]

#8750: Patch to fix #7124 break the build for powerpc
-+-
 Reporter:  marillat |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  swscale
  Version:  git-master   |   Resolution:
 Keywords:  ppc  |   Blocked By:
  regression |
 Blocking:   |  Reproduced by developer:  0
Analyzed by developer:  0|
-+-

Comment (by marillat):

 Replying to [comment:8 glaubitz]:
 > Hello!
 >
 > I just ran into this issue as well. Any chance it can be fixed?

 This bug has been fixed in the patch ffmpeg_altivec_yuv2rgb_novsx.patch​

--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-trac

To unsubscribe, visit link above, or email
ffmpeg-trac-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-trac] #8750(swscale:new): Patch to fix #7124 break the build for powerpc

[FFmpeg]

#8750: Patch to fix #7124 break the build for powerpc
-+-
 Reporter:  marillat |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  swscale
  Version:  git-master   |   Resolution:
 Keywords:  ppc  |   Blocked By:
  regression |
 Blocking:   |  Reproduced by developer:  0
Analyzed by developer:  0|
-+-

Comment (by marillat):

 Replying to [comment:8 glaubitz]:
 > Hello!
 >
 > I just ran into this issue as well. Any chance it can be fixed?

 This bug has been fixed in the patch ffmpeg_altivec_yuv2rgb_novsx.patch​

--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-trac

To unsubscribe, visit link above, or email
ffmpeg-trac-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-trac] #8750(swscale:new): Patch to fix #7124 break the build for powerpc

[FFmpeg]

#8750: Patch to fix #7124 break the build for powerpc
-+-
 Reporter:  marillat |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  swscale
  Version:  git-master   |   Resolution:
 Keywords:  ppc  |   Blocked By:
  regression |
 Blocking:   |  Reproduced by developer:  0
Analyzed by developer:  0|
-+-

Comment (by glaubitz):

 Yes, but has the patch has not been merged yet.

 See: https://github.com/FFmpeg/FFmpeg/commits/master/libswscale/ppc

--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-trac

To unsubscribe, visit link above, or email
ffmpeg-trac-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-trac] #8750(swscale:new): Patch to fix #7124 break the build for powerpc

[FFmpeg]

#8750: Patch to fix #7124 break the build for powerpc
-+-
 Reporter:  marillat |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  swscale
  Version:  git-master   |   Resolution:
 Keywords:  ppc  |   Blocked By:
  regression |
 Blocking:   |  Reproduced by developer:  0
Analyzed by developer:  0|
-+-

Comment (by glaubitz):

 Hi!

 Is there a chance this fix could get merged into ffmpeg master?

 The build is still broken on older POWER systems.

 Thanks,
 Adrian

--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-trac

To unsubscribe, visit link above, or email
ffmpeg-trac-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-trac] #8750(swscale:new): Patch to fix #7124 break the build for powerpc

[FFmpeg]

#8750: Patch to fix #7124 break the build for powerpc
-+-
 Reporter:  marillat |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  swscale
  Version:  git-master   |   Resolution:
 Keywords:  ppc  |   Blocked By:
  regression |
 Blocking:   |  Reproduced by developer:  0
Analyzed by developer:  0|
-+-

Comment (by cus):

 Please send the patch to the ffmpeg-devel mailing list.

--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-trac

To unsubscribe, visit link above, or email
ffmpeg-trac-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-trac] #8750(swscale:new): Patch to fix #7124 break the build for powerpc

[FFmpeg]

#8750: Patch to fix #7124 break the build for powerpc
-+-
 Reporter:  marillat |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  swscale
  Version:  git-master   |   Resolution:
 Keywords:  ppc  |   Blocked By:
  regression |
 Blocking:   |  Reproduced by developer:  0
Analyzed by developer:  0|
-+-

Comment (by taliho):

 @cus, the patch is on ML

 
https://patchwork.ffmpeg.org/project/ffmpeg/patch/OF3956EA65.6FB918F0-ON0025859D
 .0045c8de-0025859d.0045d...@notes.na.collabserv.com/

 http://ffmpeg.org/pipermail/ffmpeg-devel/2020-October/271784.html

--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-trac

To unsubscribe, visit link above, or email
ffmpeg-trac-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-trac] #8750(swscale:new): Patch to fix #7124 break the build for powerpc

[FFmpeg]

#8750: Patch to fix #7124 break the build for powerpc
-+-
 Reporter:  marillat |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  swscale
  Version:  git-master   |   Resolution:
 Keywords:  ppc  |   Blocked By:
  regression |
 Blocking:   |  Reproduced by developer:  0
Analyzed by developer:  0|
-+-

Comment (by glaubitz):

 Hi!

 Any news on this? The ffmpeg package is still broken on powerpc and ppc64
 in Debian unstable:

 > https://buildd.debian.org/status/package.php?p=ffmpeg&suite=sid

 Would be great if this could be fixed before the next point release. I
 think the fix is trival enough to be added without further testing besides
 the tests from Debian.

 Adrian

--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-trac

To unsubscribe, visit link above, or email
ffmpeg-trac-requ...@ffmpeg.org with subject "unsubscribe".