[PATCH 1/2] packagegroup-core-sdk.bb: Drop distcc from the core-sdk.

[PATCH 1/2] packagegroup-core-sdk.bb: Drop distcc from the core-sdk.

[Philip Balister]

distcc has been used in the past to speed up native compile across
several machines. This is less an issue on modern embedded systems.

Also, improvements in sdk generation have reduced the need for on target
compile of large projects.

Since the distcc packages start a daemon at boot, drop it from the
packagegroup to avoid having some slight runtime impact on systems
installing an sdk.

Signed-off-by: Philip Balister <philip@balister.org>
---
 meta/recipes-core/packagegroups/packagegroup-core-sdk.bb | 1 -
 1 file changed, 1 deletion(-)

diff --git a/meta/recipes-core/packagegroups/packagegroup-core-sdk.bb b/meta/recipes-core/packagegroups/packagegroup-core-sdk.bb
index a41eada..9b26387 100644
--- a/meta/recipes-core/packagegroups/packagegroup-core-sdk.bb
+++ b/meta/recipes-core/packagegroups/packagegroup-core-sdk.bb
@@ -23,7 +23,6 @@ RDEPENDS_packagegroup-core-sdk = "\
     findutils \
     quilt \
     less \
-    distcc \
     ldd \
     file \
     tcl"
-- 
2.1.0

[PATCH 2/2] packagegroup-core-tools-profile.bb: Remove oprofile.

[Philip Balister]

Oprofile functions duplicate perf. Since perf is easier to use drop oprofile
from the core-tools-profile package.

Also, the oprofile-ui package opens a port on the device. removing this
package reduces a possible security issue for devices with profiling tools.

Signed-off-by: Philip Balister <philip@balister.org>
---
 meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb b/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
index 6f4842f..80f5933 100644
--- a/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
+++ b/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
@@ -19,15 +19,12 @@ PROFILE_TOOLS_SYSTEMD = "${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'sys
 RRECOMMENDS_${PN} = "\
     perf \
     trace-cmd \
-    kernel-module-oprofile \
     blktrace \
     ${PROFILE_TOOLS_X} \
     ${PROFILE_TOOLS_SYSTEMD} \
     "
 
 PROFILETOOLS = "\
-    oprofile \
-    oprofileui-server \
     powertop \
     latencytop \
     "
-- 
2.1.0

Re: [PATCH 2/2] packagegroup-core-tools-profile.bb: Remove oprofile.

[Philip Balister]

Ping

On 06/01/2015 04:17 PM, Philip Balister wrote:
> Oprofile functions duplicate perf. Since perf is easier to use drop oprofile
> from the core-tools-profile package.
> 
> Also, the oprofile-ui package opens a port on the device. removing this
> package reduces a possible security issue for devices with profiling tools.
> 
> Signed-off-by: Philip Balister <philip@balister.org>
> ---
>  meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb b/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
> index 6f4842f..80f5933 100644
> --- a/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
> +++ b/meta/recipes-core/packagegroups/packagegroup-core-tools-profile.bb
> @@ -19,15 +19,12 @@ PROFILE_TOOLS_SYSTEMD = "${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'sys
>  RRECOMMENDS_${PN} = "\
>      perf \
>      trace-cmd \
> -    kernel-module-oprofile \
>      blktrace \
>      ${PROFILE_TOOLS_X} \
>      ${PROFILE_TOOLS_SYSTEMD} \
>      "
>  
>  PROFILETOOLS = "\
> -    oprofile \
> -    oprofileui-server \
>      powertop \
>      latencytop \
>      "
> 

Re: [PATCH 2/2] packagegroup-core-tools-profile.bb: Remove oprofile.

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 696 bytes --]

On 1 June 2015 at 21:17, Philip Balister <philip@balister.org> wrote:

> Oprofile functions duplicate perf. Since perf is easier to use drop
> oprofile
> from the core-tools-profile package.
>
> Also, the oprofile-ui package opens a port on the device. removing this
> package reduces a possible security issue for devices with profiling tools.
>

I filed this in the "think about" box when you submitted it, but then
failed to think.  My concern was "oprofile is the profiling tool we support
with oprofileui on the host" but that point is fairly moot right now as
oprofileui doesn't work anymore... (#7054).

When that's fixed, we can re-add the tooling as appropriate.

Ross

[-- Attachment #2: Type: text/html, Size: 1191 bytes --]

Re: [PATCH 2/2] packagegroup-core-tools-profile.bb: Remove oprofile.

[Philip Balister]

On 06/16/2015 08:08 AM, Burton, Ross wrote:
> On 1 June 2015 at 21:17, Philip Balister <philip@balister.org> wrote:
> 
>> Oprofile functions duplicate perf. Since perf is easier to use drop
>> oprofile
>> from the core-tools-profile package.
>>
>> Also, the oprofile-ui package opens a port on the device. removing this
>> package reduces a possible security issue for devices with profiling tools.
>>
> 
> I filed this in the "think about" box when you submitted it, but then
> failed to think.  My concern was "oprofile is the profiling tool we support
> with oprofileui on the host" but that point is fairly moot right now as
> oprofileui doesn't work anymore... (#7054).
> 
> When that's fixed, we can re-add the tooling as appropriate.

My gut feeling is people are moving to perf, which is also part of
tools-profile.

Philip

> 
> Ross
> 

Re: [PATCH 2/2] packagegroup-core-tools-profile.bb: Remove oprofile.

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 329 bytes --]

On 16 June 2015 at 13:33, Philip Balister <philip@balister.org> wrote:

> My gut feeling is people are moving to perf, which is also part of
> tools-profile.
>

Indeed.  Porting oprofileui to perf is one option here - as usual I've
forgotten why oprofile is still a thing considering perf is also in-tree
now...

Ross

[-- Attachment #2: Type: text/html, Size: 740 bytes --]

Re: [PATCH 1/2] packagegroup-core-sdk.bb: Drop distcc from the core-sdk.

[Philip Balister]

Ping

On 06/01/2015 04:17 PM, Philip Balister wrote:
> distcc has been used in the past to speed up native compile across
> several machines. This is less an issue on modern embedded systems.
> 
> Also, improvements in sdk generation have reduced the need for on target
> compile of large projects.
> 
> Since the distcc packages start a daemon at boot, drop it from the
> packagegroup to avoid having some slight runtime impact on systems
> installing an sdk.
> 
> Signed-off-by: Philip Balister <philip@balister.org>
> ---
>  meta/recipes-core/packagegroups/packagegroup-core-sdk.bb | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/meta/recipes-core/packagegroups/packagegroup-core-sdk.bb b/meta/recipes-core/packagegroups/packagegroup-core-sdk.bb
> index a41eada..9b26387 100644
> --- a/meta/recipes-core/packagegroups/packagegroup-core-sdk.bb
> +++ b/meta/recipes-core/packagegroups/packagegroup-core-sdk.bb
> @@ -23,7 +23,6 @@ RDEPENDS_packagegroup-core-sdk = "\
>      findutils \
>      quilt \
>      less \
> -    distcc \
>      ldd \
>      file \
>      tcl"
>