[meta-arago][master/kirkstone][PATCH] packagegroup-arago-gst: Disable vpe and ducati plugins

[meta-arago][master/kirkstone][PATCH] packagegroup-arago-gst: Disable vpe and ducati plugins

[Ryan Eatmon]

The code for the plugins uses dma allocation code that we add in with a
patch.  Theoretically we no longer need that code and can use some
upstreamed dma buffer allocation code.  The work to rewrite these
plugins to support the newer gstreamer versions needs to be scheduled.

In the meantime, disable using these plugins that cause build errors.

Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
 .../recipes-core/packagegroups/packagegroup-arago-gst.bb      | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
index e8a639bd..2184ab58 100644
--- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
+++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
@@ -27,8 +27,8 @@ GSTREAMER_DEPS:append:dra7xx = " \
 "
 
 GSTREAMER_ACCEL_MM = ""
-GSTREAMER_ACCEL_MM:omap-a15 = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', "gstreamer1.0-plugins-ducati", '', d)}"
-GSTREAMER_ACCEL_MM:append:dra7xx = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', " gstreamer1.0-plugins-vpe", '', d)}"
+#GSTREAMER_ACCEL_MM:omap-a15 = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', "gstreamer1.0-plugins-ducati", '', d)}"
+#GSTREAMER_ACCEL_MM:append:dra7xx = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', " gstreamer1.0-plugins-vpe", '', d)}"
 
 RDEPENDS:${PN} = "\
     ${GSTREAMER_DEPS} \
-- 
2.17.1

Re: [meta-arago][master/kirkstone][PATCH] packagegroup-arago-gst: Disable vpe and ducati plugins

[Denys Dmytriyenko]

On Mon, May 01, 2023 at 11:39:15AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
> The code for the plugins uses dma allocation code that we add in with a
> patch.  Theoretically we no longer need that code and can use some
> upstreamed dma buffer allocation code.  The work to rewrite these
> plugins to support the newer gstreamer versions needs to be scheduled.

I'm guessing you are missing this in "ltsprep" and other brandings:
https://git.yoctoproject.org/meta-arago/tree/meta-arago-distro/conf/distro/include/branding-core.inc#n15

But it is a more broader approach, so maybe surgically disabling individual 
components and re-enabling "mmip" machine feature in "core" is better...


> In the meantime, disable using these plugins that cause build errors.
> 
> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> ---
>  .../recipes-core/packagegroups/packagegroup-arago-gst.bb      | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
> index e8a639bd..2184ab58 100644
> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
> @@ -27,8 +27,8 @@ GSTREAMER_DEPS:append:dra7xx = " \
>  "
>  
>  GSTREAMER_ACCEL_MM = ""
> -GSTREAMER_ACCEL_MM:omap-a15 = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', "gstreamer1.0-plugins-ducati", '', d)}"
> -GSTREAMER_ACCEL_MM:append:dra7xx = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', " gstreamer1.0-plugins-vpe", '', d)}"
> +#GSTREAMER_ACCEL_MM:omap-a15 = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', "gstreamer1.0-plugins-ducati", '', d)}"
> +#GSTREAMER_ACCEL_MM:append:dra7xx = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', " gstreamer1.0-plugins-vpe", '', d)}"
>  
>  RDEPENDS:${PN} = "\
>      ${GSTREAMER_DEPS} \
> -- 
> 2.17.1

Re: [meta-arago][master/kirkstone][PATCH] packagegroup-arago-gst: Disable vpe and ducati plugins

[Ryan Eatmon]

On 5/1/2023 11:55, Denys Dmytriyenko wrote:
> On Mon, May 01, 2023 at 11:39:15AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
>> The code for the plugins uses dma allocation code that we add in with a
>> patch.  Theoretically we no longer need that code and can use some
>> upstreamed dma buffer allocation code.  The work to rewrite these
>> plugins to support the newer gstreamer versions needs to be scheduled.
> 
> I'm guessing you are missing this in "ltsprep" and other brandings:
> https://git.yoctoproject.org/meta-arago/tree/meta-arago-distro/conf/distro/include/branding-core.inc#n15
> 
> But it is a more broader approach, so maybe surgically disabling individual
> components and re-enabling "mmip" machine feature in "core" is better...

It's not that I missed it, but evn running ltsprep the recipes were 
being included and they fail to build.  So something else must be going 
on to make them be included.  Let me look into it.

I still need to understand why we did not see these build failures in 
kirkstone-wip.  Because these errors are related to the version of 
gstreamer and the bbappends not being applied.  And the only time these 
tests were turned off was under ltsprep...  Odd.

> 
>> In the meantime, disable using these plugins that cause build errors.
>>
>> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
>> ---
>>   .../recipes-core/packagegroups/packagegroup-arago-gst.bb      | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
>> index e8a639bd..2184ab58 100644
>> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
>> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
>> @@ -27,8 +27,8 @@ GSTREAMER_DEPS:append:dra7xx = " \
>>   "
>>   
>>   GSTREAMER_ACCEL_MM = ""
>> -GSTREAMER_ACCEL_MM:omap-a15 = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', "gstreamer1.0-plugins-ducati", '', d)}"
>> -GSTREAMER_ACCEL_MM:append:dra7xx = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', " gstreamer1.0-plugins-vpe", '', d)}"
>> +#GSTREAMER_ACCEL_MM:omap-a15 = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', "gstreamer1.0-plugins-ducati", '', d)}"
>> +#GSTREAMER_ACCEL_MM:append:dra7xx = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', " gstreamer1.0-plugins-vpe", '', d)}"
>>   
>>   RDEPENDS:${PN} = "\
>>       ${GSTREAMER_DEPS} \
>> -- 
>> 2.17.1

-- 
Ryan Eatmon                reatmon@ti.com
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS

Re: [meta-arago][master/kirkstone][PATCH] packagegroup-arago-gst: Disable vpe and ducati plugins

[Denys Dmytriyenko]

On Mon, May 01, 2023 at 01:39:12PM -0500, Ryan Eatmon wrote:
> 
> 
> On 5/1/2023 11:55, Denys Dmytriyenko wrote:
> >On Mon, May 01, 2023 at 11:39:15AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
> >>The code for the plugins uses dma allocation code that we add in with a
> >>patch.  Theoretically we no longer need that code and can use some
> >>upstreamed dma buffer allocation code.  The work to rewrite these
> >>plugins to support the newer gstreamer versions needs to be scheduled.
> >
> >I'm guessing you are missing this in "ltsprep" and other brandings:
> >https://git.yoctoproject.org/meta-arago/tree/meta-arago-distro/conf/distro/include/branding-core.inc#n15
> >
> >But it is a more broader approach, so maybe surgically disabling individual
> >components and re-enabling "mmip" machine feature in "core" is better...
> 
> It's not that I missed it, but evn running ltsprep the recipes were
> being included and they fail to build.  So something else must be
> going on to make them be included.  Let me look into it.
> 
> I still need to understand why we did not see these build failures
> in kirkstone-wip.  Because these errors are related to the version
> of gstreamer and the bbappends not being applied.  And the only time
> these tests were turned off was under ltsprep...  Odd.

I think you got it in reverse: "mmip" is being completely disabled for default 
"core" builds, but not for "ltsprep" ones, hence you see the breakage there.


> >>In the meantime, disable using these plugins that cause build errors.
> >>
> >>Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> >>---
> >>  .../recipes-core/packagegroups/packagegroup-arago-gst.bb      | 4 ++--
> >>  1 file changed, 2 insertions(+), 2 deletions(-)
> >>
> >>diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
> >>index e8a639bd..2184ab58 100644
> >>--- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
> >>+++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
> >>@@ -27,8 +27,8 @@ GSTREAMER_DEPS:append:dra7xx = " \
> >>  "
> >>  GSTREAMER_ACCEL_MM = ""
> >>-GSTREAMER_ACCEL_MM:omap-a15 = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', "gstreamer1.0-plugins-ducati", '', d)}"
> >>-GSTREAMER_ACCEL_MM:append:dra7xx = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', " gstreamer1.0-plugins-vpe", '', d)}"
> >>+#GSTREAMER_ACCEL_MM:omap-a15 = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', "gstreamer1.0-plugins-ducati", '', d)}"
> >>+#GSTREAMER_ACCEL_MM:append:dra7xx = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', " gstreamer1.0-plugins-vpe", '', d)}"
> >>  RDEPENDS:${PN} = "\
> >>      ${GSTREAMER_DEPS} \
> >>-- 
> >>2.17.1

Re: [meta-arago][master/kirkstone][PATCH] packagegroup-arago-gst: Disable vpe and ducati plugins

[Ryan Eatmon]

On 5/1/2023 13:49, Denys Dmytriyenko wrote:
> On Mon, May 01, 2023 at 01:39:12PM -0500, Ryan Eatmon wrote:
>>
>>
>> On 5/1/2023 11:55, Denys Dmytriyenko wrote:
>>> On Mon, May 01, 2023 at 11:39:15AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
>>>> The code for the plugins uses dma allocation code that we add in with a
>>>> patch.  Theoretically we no longer need that code and can use some
>>>> upstreamed dma buffer allocation code.  The work to rewrite these
>>>> plugins to support the newer gstreamer versions needs to be scheduled.
>>>
>>> I'm guessing you are missing this in "ltsprep" and other brandings:
>>> https://git.yoctoproject.org/meta-arago/tree/meta-arago-distro/conf/distro/include/branding-core.inc#n15
>>>
>>> But it is a more broader approach, so maybe surgically disabling individual
>>> components and re-enabling "mmip" machine feature in "core" is better...
>>
>> It's not that I missed it, but evn running ltsprep the recipes were
>> being included and they fail to build.  So something else must be
>> going on to make them be included.  Let me look into it.
>>
>> I still need to understand why we did not see these build failures
>> in kirkstone-wip.  Because these errors are related to the version
>> of gstreamer and the bbappends not being applied.  And the only time
>> these tests were turned off was under ltsprep...  Odd.
> 
> I think you got it in reverse: "mmip" is being completely disabled for default
> "core" builds, but not for "ltsprep" ones, hence you see the breakage there.

Ahhhh... gotcha... Yes.  That is what is missing.  I'll remove this 
patch and add that to the ltsprep brand file.

> 
>>>> In the meantime, disable using these plugins that cause build errors.
>>>>
>>>> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
>>>> ---
>>>>   .../recipes-core/packagegroups/packagegroup-arago-gst.bb      | 4 ++--
>>>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>>>
>>>> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
>>>> index e8a639bd..2184ab58 100644
>>>> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
>>>> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
>>>> @@ -27,8 +27,8 @@ GSTREAMER_DEPS:append:dra7xx = " \
>>>>   "
>>>>   GSTREAMER_ACCEL_MM = ""
>>>> -GSTREAMER_ACCEL_MM:omap-a15 = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', "gstreamer1.0-plugins-ducati", '', d)}"
>>>> -GSTREAMER_ACCEL_MM:append:dra7xx = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', " gstreamer1.0-plugins-vpe", '', d)}"
>>>> +#GSTREAMER_ACCEL_MM:omap-a15 = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', "gstreamer1.0-plugins-ducati", '', d)}"
>>>> +#GSTREAMER_ACCEL_MM:append:dra7xx = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', " gstreamer1.0-plugins-vpe", '', d)}"
>>>>   RDEPENDS:${PN} = "\
>>>>       ${GSTREAMER_DEPS} \
>>>> -- 
>>>> 2.17.1

-- 
Ryan Eatmon                reatmon@ti.com
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS

Re: [meta-arago][master/kirkstone][PATCH] packagegroup-arago-gst: Disable vpe and ducati plugins

[Denys Dmytriyenko]

On Mon, May 01, 2023 at 02:08:40PM -0500, Ryan Eatmon wrote:
> 
> 
> On 5/1/2023 13:49, Denys Dmytriyenko wrote:
> >On Mon, May 01, 2023 at 01:39:12PM -0500, Ryan Eatmon wrote:
> >>
> >>
> >>On 5/1/2023 11:55, Denys Dmytriyenko wrote:
> >>>On Mon, May 01, 2023 at 11:39:15AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
> >>>>The code for the plugins uses dma allocation code that we add in with a
> >>>>patch.  Theoretically we no longer need that code and can use some
> >>>>upstreamed dma buffer allocation code.  The work to rewrite these
> >>>>plugins to support the newer gstreamer versions needs to be scheduled.
> >>>
> >>>I'm guessing you are missing this in "ltsprep" and other brandings:
> >>>https://git.yoctoproject.org/meta-arago/tree/meta-arago-distro/conf/distro/include/branding-core.inc#n15
> >>>
> >>>But it is a more broader approach, so maybe surgically disabling individual
> >>>components and re-enabling "mmip" machine feature in "core" is better...
> >>
> >>It's not that I missed it, but evn running ltsprep the recipes were
> >>being included and they fail to build.  So something else must be
> >>going on to make them be included.  Let me look into it.
> >>
> >>I still need to understand why we did not see these build failures
> >>in kirkstone-wip.  Because these errors are related to the version
> >>of gstreamer and the bbappends not being applied.  And the only time
> >>these tests were turned off was under ltsprep...  Odd.
> >
> >I think you got it in reverse: "mmip" is being completely disabled for default
> >"core" builds, but not for "ltsprep" ones, hence you see the breakage there.
> 
> Ahhhh... gotcha... Yes.  That is what is missing.  I'll remove this
> patch and add that to the ltsprep brand file.

What about this one:
https://patchwork.yoctoproject.org/project/arago/patch/20230429144445.32401-1-reatmon@ti.com/


> >>>>In the meantime, disable using these plugins that cause build errors.
> >>>>
> >>>>Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> >>>>---
> >>>>  .../recipes-core/packagegroups/packagegroup-arago-gst.bb      | 4 ++--
> >>>>  1 file changed, 2 insertions(+), 2 deletions(-)
> >>>>
> >>>>diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
> >>>>index e8a639bd..2184ab58 100644
> >>>>--- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
> >>>>+++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
> >>>>@@ -27,8 +27,8 @@ GSTREAMER_DEPS:append:dra7xx = " \
> >>>>  "
> >>>>  GSTREAMER_ACCEL_MM = ""
> >>>>-GSTREAMER_ACCEL_MM:omap-a15 = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', "gstreamer1.0-plugins-ducati", '', d)}"
> >>>>-GSTREAMER_ACCEL_MM:append:dra7xx = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', " gstreamer1.0-plugins-vpe", '', d)}"
> >>>>+#GSTREAMER_ACCEL_MM:omap-a15 = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', "gstreamer1.0-plugins-ducati", '', d)}"
> >>>>+#GSTREAMER_ACCEL_MM:append:dra7xx = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', " gstreamer1.0-plugins-vpe", '', d)}"
> >>>>  RDEPENDS:${PN} = "\
> >>>>      ${GSTREAMER_DEPS} \
> >>>>-- 
> >>>>2.17.1

Re: [meta-arago][master/kirkstone][PATCH] packagegroup-arago-gst: Disable vpe and ducati plugins

[Ryan Eatmon]

On 5/1/2023 14:10, Denys Dmytriyenko wrote:
> On Mon, May 01, 2023 at 02:08:40PM -0500, Ryan Eatmon wrote:
>>
>>
>> On 5/1/2023 13:49, Denys Dmytriyenko wrote:
>>> On Mon, May 01, 2023 at 01:39:12PM -0500, Ryan Eatmon wrote:
>>>>
>>>>
>>>> On 5/1/2023 11:55, Denys Dmytriyenko wrote:
>>>>> On Mon, May 01, 2023 at 11:39:15AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
>>>>>> The code for the plugins uses dma allocation code that we add in with a
>>>>>> patch.  Theoretically we no longer need that code and can use some
>>>>>> upstreamed dma buffer allocation code.  The work to rewrite these
>>>>>> plugins to support the newer gstreamer versions needs to be scheduled.
>>>>>
>>>>> I'm guessing you are missing this in "ltsprep" and other brandings:
>>>>> https://git.yoctoproject.org/meta-arago/tree/meta-arago-distro/conf/distro/include/branding-core.inc#n15
>>>>>
>>>>> But it is a more broader approach, so maybe surgically disabling individual
>>>>> components and re-enabling "mmip" machine feature in "core" is better...
>>>>
>>>> It's not that I missed it, but evn running ltsprep the recipes were
>>>> being included and they fail to build.  So something else must be
>>>> going on to make them be included.  Let me look into it.
>>>>
>>>> I still need to understand why we did not see these build failures
>>>> in kirkstone-wip.  Because these errors are related to the version
>>>> of gstreamer and the bbappends not being applied.  And the only time
>>>> these tests were turned off was under ltsprep...  Odd.
>>>
>>> I think you got it in reverse: "mmip" is being completely disabled for default
>>> "core" builds, but not for "ltsprep" ones, hence you see the breakage there.
>>
>> Ahhhh... gotcha... Yes.  That is what is missing.  I'll remove this
>> patch and add that to the ltsprep brand file.
> 
> What about this one:
> https://patchwork.yoctoproject.org/project/arago/patch/20230429144445.32401-1-reatmon@ti.com/

Yes.  It falls into the same bucket.  Turning off mmip in ltsprep will 
mean that patch can be removed as well.

> 
>>>>>> In the meantime, disable using these plugins that cause build errors.
>>>>>>
>>>>>> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
>>>>>> ---
>>>>>>   .../recipes-core/packagegroups/packagegroup-arago-gst.bb      | 4 ++--
>>>>>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>>>>>
>>>>>> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
>>>>>> index e8a639bd..2184ab58 100644
>>>>>> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
>>>>>> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
>>>>>> @@ -27,8 +27,8 @@ GSTREAMER_DEPS:append:dra7xx = " \
>>>>>>   "
>>>>>>   GSTREAMER_ACCEL_MM = ""
>>>>>> -GSTREAMER_ACCEL_MM:omap-a15 = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', "gstreamer1.0-plugins-ducati", '', d)}"
>>>>>> -GSTREAMER_ACCEL_MM:append:dra7xx = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', " gstreamer1.0-plugins-vpe", '', d)}"
>>>>>> +#GSTREAMER_ACCEL_MM:omap-a15 = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', "gstreamer1.0-plugins-ducati", '', d)}"
>>>>>> +#GSTREAMER_ACCEL_MM:append:dra7xx = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', " gstreamer1.0-plugins-vpe", '', d)}"
>>>>>>   RDEPENDS:${PN} = "\
>>>>>>       ${GSTREAMER_DEPS} \
>>>>>> -- 
>>>>>> 2.17.1

-- 
Ryan Eatmon                reatmon@ti.com
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS