Bug 99475

Summary: [patch] allow disabling XvMC
Product: xorg Reporter: Harri Nieminen <moikkis>
Component: Driver/openchromeAssignee: Openchrome development list <openchrome-devel>
Status: RESOLVED WONTFIX QA Contact:
Severity: enhancement    
Priority: medium CC: kevinbrace
Version: unspecified   
Hardware: All   
OS: All   
Whiteboard:
i915 platform: i915 features:
Attachments:
Description Flags
Allow disabling XvMC none

Description Harri Nieminen 2017-01-20 15:07:30 UTC
Created attachment 129073 [details] [review]
Allow disabling XvMC
Comment 1 Kevin Brace 2017-01-22 18:38:02 UTC
What is the purpose of this patch?
I have not really dealt with Xv or XvMC code ever since I took over the development.
Comment 2 Harri Nieminen 2017-01-23 06:56:16 UTC
(In reply to Kevin Brace from comment #1)
> What is the purpose of this patch?
> I have not really dealt with Xv or XvMC code ever since I took over the
> development.

XvMC is pretty useless today. It can only decode mpeg2.
Some users want to disable it. 
Modern video players don't even support it.
Comment 4 Kevin Brace 2017-01-24 00:34:44 UTC
(In reply to Harri Nieminen from comment #2)

XvMC code was written by previous developers 7 to 10 years ago.
I had no involvement with it.
Please understand that OpenChrome's project development resource is easily 1/30th of Intel graphics stack.
I am the only developer working on the code, and my hands are completely tied up for the next few years.
I am trying to fix various flaws of existing OpenChrome DDX code, and trying to fix the problems of OpenChrome DRM (KMS / TTM supporting version) so that it can be mainlined with the Linux kernel eventually.
As a result, I have virtually no time to work on issues other than those two.
I am leaning towards not incorporating this patch.
I do not see the urgency of why this patch should be incorporated in the first place.

> 
> XvMC is pretty useless today. It can only decode mpeg2.
> Some users want to disable it. 
> Modern video players don't even support it.
Comment 5 Harri Nieminen 2017-01-24 09:23:22 UTC
(In reply to Kevin Brace from comment #4)
> (In reply to Harri Nieminen from comment #2)
> 
> XvMC code was written by previous developers 7 to 10 years ago.
> I had no involvement with it.
> Please understand that OpenChrome's project development resource is easily
> 1/30th of Intel graphics stack.
> I am the only developer working on the code, and my hands are completely
> tied up for the next few years.
> I am trying to fix various flaws of existing OpenChrome DDX code, and trying
> to fix the problems of OpenChrome DRM (KMS / TTM supporting version) so that
> it can be mainlined with the Linux kernel eventually.
> As a result, I have virtually no time to work on issues other than those two.
> I am leaning towards not incorporating this patch.
> I do not see the urgency of why this patch should be incorporated in the
> first place.
> 
> > 
> > XvMC is pretty useless today. It can only decode mpeg2.
> > Some users want to disable it. 
> > Modern video players don't even support it.

Okay thanks.
Comment 6 Harri Nieminen 2017-01-24 09:34:52 UTC
Also good luck with your work. Mainlining DRM driver sounds lots of work.
This patch was very low priority for me too.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.