Re: [libav-devel] [PATCH 1/3] imgutils: add a function for copying image data from GPU mapped memory

2016-08-25 Thread Hendrik Leppkes
On Thu, Aug 25, 2016 at 9:47 AM, Vittorio Giovara wrote: > On Thu, Aug 25, 2016 at 4:07 AM, Anton Khirnov wrote: >> Quoting Vittorio Giovara (2016-08-24 16:58:12) >>> also, does it have to be a separate function? If so, what does the old >>> function do that the new one does not? >> >> The doxy s

Re: [libav-devel] [PATCH 1/3] imgutils: add a function for copying image data from GPU mapped memory

2016-08-25 Thread Vittorio Giovara
On Thu, Aug 25, 2016 at 4:07 AM, Anton Khirnov wrote: > Quoting Vittorio Giovara (2016-08-24 16:58:12) >> also, does it have to be a separate function? If so, what does the old >> function do that the new one does not? > > The doxy says what the difference is -- mainly that this function requires

Re: [libav-devel] [PATCH 1/3] imgutils: add a function for copying image data from GPU mapped memory

2016-08-25 Thread Anton Khirnov
Quoting Diego Biurrun (2016-08-25 10:43:03) > On Thu, Aug 25, 2016 at 10:40:43AM +0200, Anton Khirnov wrote: > > Quoting Diego Biurrun (2016-08-25 10:36:31) > > > On Thu, Aug 25, 2016 at 10:07:03AM +0200, Anton Khirnov wrote: > > > > Quoting Vittorio Giovara (2016-08-24 16:58:12) > > > > > On Wed,

Re: [libav-devel] [PATCH 1/3] imgutils: add a function for copying image data from GPU mapped memory

2016-08-25 Thread Diego Biurrun
On Thu, Aug 25, 2016 at 10:40:43AM +0200, Anton Khirnov wrote: > Quoting Diego Biurrun (2016-08-25 10:36:31) > > On Thu, Aug 25, 2016 at 10:07:03AM +0200, Anton Khirnov wrote: > > > Quoting Vittorio Giovara (2016-08-24 16:58:12) > > > > On Wed, Aug 24, 2016 at 6:06 AM, Diego Biurrun wrote: > > > >

Re: [libav-devel] [PATCH 1/3] imgutils: add a function for copying image data from GPU mapped memory

2016-08-25 Thread Anton Khirnov
Quoting Diego Biurrun (2016-08-25 10:36:31) > On Thu, Aug 25, 2016 at 10:07:03AM +0200, Anton Khirnov wrote: > > Quoting Vittorio Giovara (2016-08-24 16:58:12) > > > On Wed, Aug 24, 2016 at 6:06 AM, Diego Biurrun wrote: > > > > On Wed, Aug 24, 2016 at 11:24:01AM +0200, Anton Khirnov wrote: > > > >

Re: [libav-devel] [PATCH 1/3] imgutils: add a function for copying image data from GPU mapped memory

2016-08-25 Thread Diego Biurrun
On Thu, Aug 25, 2016 at 10:07:03AM +0200, Anton Khirnov wrote: > Quoting Vittorio Giovara (2016-08-24 16:58:12) > > On Wed, Aug 24, 2016 at 6:06 AM, Diego Biurrun wrote: > > > On Wed, Aug 24, 2016 at 11:24:01AM +0200, Anton Khirnov wrote: > > >> --- /dev/null > > >> +++ b/libavutil/x86/imgutils_in

Re: [libav-devel] [PATCH 1/3] imgutils: add a function for copying image data from GPU mapped memory

2016-08-25 Thread Anton Khirnov
Quoting Vittorio Giovara (2016-08-24 16:58:12) > On Wed, Aug 24, 2016 at 6:06 AM, Diego Biurrun wrote: > > On Wed, Aug 24, 2016 at 11:24:01AM +0200, Anton Khirnov wrote: > >> --- /dev/null > >> +++ b/libavutil/x86/imgutils_init.c > >> @@ -0,0 +1,49 @@ > >> + > >> +void ff_image_copy_plane_uc_from_

Re: [libav-devel] [PATCH 1/3] imgutils: add a function for copying image data from GPU mapped memory

2016-08-24 Thread Vittorio Giovara
On Wed, Aug 24, 2016 at 6:06 AM, Diego Biurrun wrote: > On Wed, Aug 24, 2016 at 11:24:01AM +0200, Anton Khirnov wrote: >> --- /dev/null >> +++ b/libavutil/x86/imgutils_init.c >> @@ -0,0 +1,49 @@ >> + >> +void ff_image_copy_plane_uc_from_sse4(uint8_t *dst, ptrdiff_t dst_linesize, >> +

Re: [libav-devel] [PATCH 1/3] imgutils: add a function for copying image data from GPU mapped memory

2016-08-24 Thread Diego Biurrun
On Wed, Aug 24, 2016 at 11:24:01AM +0200, Anton Khirnov wrote: > --- /dev/null > +++ b/libavutil/x86/imgutils_init.c > @@ -0,0 +1,49 @@ > + > +void ff_image_copy_plane_uc_from_sse4(uint8_t *dst, ptrdiff_t dst_linesize, > + const uint8_t *src, ptrdiff_t > src_li

[libav-devel] [PATCH 1/3] imgutils: add a function for copying image data from GPU mapped memory

2016-08-24 Thread Anton Khirnov
See https://software.intel.com/en-us/articles/copying-accelerated-video-decode-frame-buffers --- doc/APIchanges| 4 +++ libavutil/imgutils.c | 75 +++ libavutil/imgutils.h | 18 +++ libavutil/imgutils_internal.h |