Re: [FFmpeg-devel] [PATCH] vdpau_mpeg4: Do not fail on unknown profile.
On Sun, Nov 02, 2014 at 10:02:50AM +, Carl Eugen Hoyos wrote: > Reimar Döffinger gmx.de> writes: > > > Are there still objections? > > I just wanted to know if we shouldn't always set ASP? > Or at least for everything != SP? I am unsure. It seemed to me this way would be more reliable and ensure we review it if we were to ever add a new profile. But if anyone has a real opinion on it I'll happily follow it. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] vdpau_mpeg4: Do not fail on unknown profile.
Reimar Döffinger gmx.de> writes: > Are there still objections? I just wanted to know if we shouldn't always set ASP? Or at least for everything != SP? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] vdpau_mpeg4: Do not fail on unknown profile.
On 01.11.2014, at 13:45, Reimar Döffinger wrote: > This is a regression, previous code did not require > users to set the profile to use VDPAU. > In addition, ASP is a superset of SP, so there is not > even a good reason to require it. Are there still objections? As said, to my knowledge every ASP decoder is required to be a fully working SP decoder, so I think this can't break anything with compliant implementations, which is my main justification (besides the regression part). Does anyone disagree? ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] vdpau_mpeg4: Do not fail on unknown profile.
On Sat, 1 Nov 2014 15:13:17 +0100 Reimar Döffinger wrote: > On Sat, Nov 01, 2014 at 01:52:17PM +0100, wm4 wrote: > > On Sat, 1 Nov 2014 13:45:46 +0100 > > Reimar Döffinger wrote: > > > > > This is a regression, previous code did not require > > > users to set the profile to use VDPAU. > > > > It still doesn't...? > > It does if you want the hwaccel API. > > > There are basically 3 ffmpeg vdpau hwaccel APIs now. > > (possibly 4 depending on how you could, but) > Adding the newest one because of this code had the > side-effect that you now need to set the profile > even for the older ones (not for the oldest one > using mpeg4_vdpau though it seems?). > Obviously it would be possible to just skip this in the > case of the older API, but as said I see no good reason. That seems indeed like a bug, then. It would mean every API user got broken. Nice job, Libav. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] vdpau_mpeg4: Do not fail on unknown profile.
On Sat, Nov 01, 2014 at 01:52:17PM +0100, wm4 wrote: > On Sat, 1 Nov 2014 13:45:46 +0100 > Reimar Döffinger wrote: > > > This is a regression, previous code did not require > > users to set the profile to use VDPAU. > > It still doesn't...? It does if you want the hwaccel API. > There are basically 3 ffmpeg vdpau hwaccel APIs now. (possibly 4 depending on how you could, but) Adding the newest one because of this code had the side-effect that you now need to set the profile even for the older ones (not for the oldest one using mpeg4_vdpau though it seems?). Obviously it would be possible to just skip this in the case of the older API, but as said I see no good reason. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] vdpau_mpeg4: Do not fail on unknown profile.
On Sat, Nov 01, 2014 at 02:18:06PM +0100, Jean-Baptiste Kempf wrote: > On 01 Nov, Reimar Döffinger wrote : > > This is a regression, previous code did not require > > users to set the profile to use VDPAU. > > If you don't set the profile, how can you know that the hardware > supports it? Firstly: As I said, SP is a subset of ASP, so if the hardware supports ASP it supports SP. Secondly: The code actually interacting with the hardware can handle that. There is no need for the FFmpeg decoder to require that information. Obviously I can just hack around this by setting it to ASP if it's unknown, but I'd like a good reason why that is a better solution than graceful degradation here. Only slightly related: I generally dislike that part of the design as it mixes up the whole VDPAU structure/bitstream generation with the actual hardware decoding, which is a pain if you want to use security feature like process separation of decoding (which is a higher security risk) from actual display. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] vdpau_mpeg4: Do not fail on unknown profile.
Jean-Baptiste Kempf videolan.org> writes: > On 01 Nov, Reimar Döffinger wrote : > > This is a regression, previous code did not require > > users to set the profile to use VDPAU. > > If you don't set the profile, how can you know that > the hardware supports it? Does this have any relevance for ASP? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] vdpau_mpeg4: Do not fail on unknown profile.
On 01 Nov, Reimar Döffinger wrote : > This is a regression, previous code did not require > users to set the profile to use VDPAU. If you don't set the profile, how can you know that the hardware supports it? -- Jean-Baptiste Kempf http://www.jbkempf.com/ - +33 672 704 734 Sent from my Electronic Device ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] vdpau_mpeg4: Do not fail on unknown profile.
Reimar Döffinger gmx.de> writes: > +case FF_PROFILE_UNKNOWN: > case FF_PROFILE_MPEG4_ADVANCED_SIMPLE: > profile = VDP_DECODER_PROFILE_MPEG4_PART2_ASP; Could instead be the default case to set profile to ASP? Or to remove this whole logic? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] vdpau_mpeg4: Do not fail on unknown profile.
On Sat, 1 Nov 2014 13:45:46 +0100 Reimar Döffinger wrote: > This is a regression, previous code did not require > users to set the profile to use VDPAU. It still doesn't...? There are basically 3 ffmpeg vdpau hwaccel APIs now. > In addition, ASP is a superset of SP, so there is not > even a good reason to require it. > --- > libavcodec/vdpau_mpeg4.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavcodec/vdpau_mpeg4.c b/libavcodec/vdpau_mpeg4.c > index 2a036b6..02ed4d1 100644 > --- a/libavcodec/vdpau_mpeg4.c > +++ b/libavcodec/vdpau_mpeg4.c > @@ -119,6 +119,7 @@ static int vdpau_mpeg4_init(AVCodecContext *avctx) > case FF_PROFILE_MPEG4_SIMPLE: > profile = VDP_DECODER_PROFILE_MPEG4_PART2_SP; > break; > +case FF_PROFILE_UNKNOWN: > case FF_PROFILE_MPEG4_ADVANCED_SIMPLE: > profile = VDP_DECODER_PROFILE_MPEG4_PART2_ASP; > break; ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel