| Message ID | 20230928093808.545770-1-costamagnagianfranco@yahoo.it |
|---|---|
| State | Under Review |
| Headers | show |
| Series | [meta-oe,1/2] dlt-daemon: | expand |
I have staged it with some modified commit msg summary please check master-next and see if it looks ok. On Thu, Sep 28, 2023 at 2:38 AM Gianfranco <costamagna.gianfranco@gmail.com> wrote: > > Make sure also without systemd enabled as default, the tools can be built and installed, > such as dlt-adaptor, dlt-adaptor-stdin, dlt-adaptor-udp, dlt-examples and dlt-console. > They have stub functionalities when systemd is not default, but they can still be used with > limited functionalities > > Reported-By: Federico Briata <federicobriata@gmail.com> > Reported-By: Andrea Reali <andrea.reali@cnhind.com> > Signed-off-by: Gianfranco Costamagna <costamagnagianfranco@yahoo.it> > Signed-off-by: Gianfranco Costamagna <locutusofborg@debian.org> > --- > .../dlt-daemon/dlt-daemon_2.18.10.bb | 13 +++++++------ > 1 file changed, 7 insertions(+), 6 deletions(-) > > diff --git a/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb b/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb > index 33fae0619..3811772a7 100644 > --- a/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb > +++ b/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb > @@ -22,12 +22,13 @@ SRCREV = "0f2d4cfffada6f8448a2cb27995b38eb4271044f" > > S = "${WORKDIR}/git" > > -PACKAGECONFIG ?= "${@bb.utils.contains('DISTRO_FEATURES', 'systemd', ' systemd systemd-watchdog systemd-journal dlt-examples dlt-adaptor dlt-adaptor-stdin dlt-adaptor-udp dlt-console ', '', d)} \ > +PACKAGECONFIG ?= "${@bb.utils.contains('DISTRO_FEATURES', 'systemd', ' systemd systemd-watchdog systemd-journal ', '', d)} \ > + dlt-examples dlt-adaptor dlt-adaptor-stdin dlt-adaptor-udp dlt-console \ > udp-connection dlt-system dlt-filetransfer " > # dlt-dbus > > # General options > -PACKAGECONFIG[dlt-examples] = "-DWITH_DLT_EXAMPLES=ON,-DWITH_DLT_EXAMPLES=OFF,,dlt-daemon-systemd" > +PACKAGECONFIG[dlt-examples] = "-DWITH_DLT_EXAMPLES=ON,-DWITH_DLT_EXAMPLES=OFF" > > # Linux options > PACKAGECONFIG[systemd] = "-DWITH_SYSTEMD=ON,-DWITH_SYSTEMD=OFF -DWITH_DLT_SYSTEM=OFF,systemd" > @@ -38,11 +39,11 @@ PACKAGECONFIG[udp-connection] = "-DWITH_UDP_CONNECTION=ON,-DWITH_UDP_CONNECTION= > > # Command line options > PACKAGECONFIG[dlt-system] = "-DWITH_DLT_SYSTEM=ON,-DWITH_DLT_SYSTEM=OFF" > -PACKAGECONFIG[dlt-adaptor] = "-DWITH_DLT_ADAPTOR=ON,-DWITH_DLT_ADAPTOR=OFF,,dlt-daemon-systemd" > -PACKAGECONFIG[dlt-adaptor-stdin] = "-DWITH_DLT_ADAPTOR_STDIN=ON,-DWITH_DLT_ADAPTOR_STDIN=OFF,,dlt-daemon-systemd" > -PACKAGECONFIG[dlt-adaptor-udp] = "-DWITH_DLT_ADAPTOR_UDP=ON,-DWITH_DLT_ADAPTOR_UDP=OFF,,dlt-daemon-systemd" > +PACKAGECONFIG[dlt-adaptor] = "-DWITH_DLT_ADAPTOR=ON,-DWITH_DLT_ADAPTOR=OFF" > +PACKAGECONFIG[dlt-adaptor-stdin] = "-DWITH_DLT_ADAPTOR_STDIN=ON,-DWITH_DLT_ADAPTOR_STDIN=OFF" > +PACKAGECONFIG[dlt-adaptor-udp] = "-DWITH_DLT_ADAPTOR_UDP=ON,-DWITH_DLT_ADAPTOR_UDP=OFF" > PACKAGECONFIG[dlt-filetransfer] = "-DWITH_DLT_FILETRANSFER=ON,-DWITH_DLT_FILETRANSFER=OFF" > -PACKAGECONFIG[dlt-console] = "-DWITH_DLT_CONSOLE=ON,-DWITH_DLT_CONSOLE=OFF,,dlt-daemon-systemd" > +PACKAGECONFIG[dlt-console] = "-DWITH_DLT_CONSOLE=ON,-DWITH_DLT_CONSOLE=OFF" > > inherit autotools gettext cmake pkgconfig systemd > > -- > 2.34.1 > > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#105219): https://lists.openembedded.org/g/openembedded-devel/message/105219 > Mute This Topic: https://lists.openembedded.org/mt/101634559/1997914 > Group Owner: openembedded-devel+owner@lists.openembedded.org > Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [raj.khem@gmail.com] > -=-=-=-=-=-=-=-=-=-=-=- >
hello, sure looks perfect to me! btw you will also need this one to build properly on some architecture https://patchwork.yoctoproject.org/project/oe/list/?series=17190 I'm polishing up the work, testing, and will try to update upstream PR. But please note, this is a never-tested and highly experimental dlt feature nobody is using right now. (and it's a new binary, so won't affect the normal dlt usage) Expect new commits soon :) G. On Thu, Sep 28, 2023 at 5:01 PM Khem Raj <raj.khem@gmail.com> wrote: > I have staged it with some modified commit msg summary please check > master-next and see if it looks ok. > > On Thu, Sep 28, 2023 at 2:38 AM Gianfranco > <costamagna.gianfranco@gmail.com> wrote: > > > > Make sure also without systemd enabled as default, the tools can be > built and installed, > > such as dlt-adaptor, dlt-adaptor-stdin, dlt-adaptor-udp, dlt-examples > and dlt-console. > > They have stub functionalities when systemd is not default, but they can > still be used with > > limited functionalities > > > > Reported-By: Federico Briata <federicobriata@gmail.com> > > Reported-By: Andrea Reali <andrea.reali@cnhind.com> > > Signed-off-by: Gianfranco Costamagna <costamagnagianfranco@yahoo.it> > > Signed-off-by: Gianfranco Costamagna <locutusofborg@debian.org> > > --- > > .../dlt-daemon/dlt-daemon_2.18.10.bb | 13 +++++++------ > > 1 file changed, 7 insertions(+), 6 deletions(-) > > > > diff --git a/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb > b/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb > > index 33fae0619..3811772a7 100644 > > --- a/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb > > +++ b/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb > > @@ -22,12 +22,13 @@ SRCREV = "0f2d4cfffada6f8448a2cb27995b38eb4271044f" > > > > S = "${WORKDIR}/git" > > > > -PACKAGECONFIG ?= "${@bb.utils.contains('DISTRO_FEATURES', 'systemd', ' > systemd systemd-watchdog systemd-journal dlt-examples dlt-adaptor > dlt-adaptor-stdin dlt-adaptor-udp dlt-console ', '', d)} \ > > +PACKAGECONFIG ?= "${@bb.utils.contains('DISTRO_FEATURES', 'systemd', ' > systemd systemd-watchdog systemd-journal ', '', d)} \ > > + dlt-examples dlt-adaptor dlt-adaptor-stdin dlt-adaptor-udp dlt-console > \ > > udp-connection dlt-system dlt-filetransfer " > > # dlt-dbus > > > > # General options > > -PACKAGECONFIG[dlt-examples] = > "-DWITH_DLT_EXAMPLES=ON,-DWITH_DLT_EXAMPLES=OFF,,dlt-daemon-systemd" > > +PACKAGECONFIG[dlt-examples] = > "-DWITH_DLT_EXAMPLES=ON,-DWITH_DLT_EXAMPLES=OFF" > > > > # Linux options > > PACKAGECONFIG[systemd] = "-DWITH_SYSTEMD=ON,-DWITH_SYSTEMD=OFF > -DWITH_DLT_SYSTEM=OFF,systemd" > > @@ -38,11 +39,11 @@ PACKAGECONFIG[udp-connection] = > "-DWITH_UDP_CONNECTION=ON,-DWITH_UDP_CONNECTION= > > > > # Command line options > > PACKAGECONFIG[dlt-system] = "-DWITH_DLT_SYSTEM=ON,-DWITH_DLT_SYSTEM=OFF" > > -PACKAGECONFIG[dlt-adaptor] = > "-DWITH_DLT_ADAPTOR=ON,-DWITH_DLT_ADAPTOR=OFF,,dlt-daemon-systemd" > > -PACKAGECONFIG[dlt-adaptor-stdin] = > "-DWITH_DLT_ADAPTOR_STDIN=ON,-DWITH_DLT_ADAPTOR_STDIN=OFF,,dlt-daemon-systemd" > > -PACKAGECONFIG[dlt-adaptor-udp] = > "-DWITH_DLT_ADAPTOR_UDP=ON,-DWITH_DLT_ADAPTOR_UDP=OFF,,dlt-daemon-systemd" > > +PACKAGECONFIG[dlt-adaptor] = > "-DWITH_DLT_ADAPTOR=ON,-DWITH_DLT_ADAPTOR=OFF" > > +PACKAGECONFIG[dlt-adaptor-stdin] = > "-DWITH_DLT_ADAPTOR_STDIN=ON,-DWITH_DLT_ADAPTOR_STDIN=OFF" > > +PACKAGECONFIG[dlt-adaptor-udp] = > "-DWITH_DLT_ADAPTOR_UDP=ON,-DWITH_DLT_ADAPTOR_UDP=OFF" > > PACKAGECONFIG[dlt-filetransfer] = > "-DWITH_DLT_FILETRANSFER=ON,-DWITH_DLT_FILETRANSFER=OFF" > > -PACKAGECONFIG[dlt-console] = > "-DWITH_DLT_CONSOLE=ON,-DWITH_DLT_CONSOLE=OFF,,dlt-daemon-systemd" > > +PACKAGECONFIG[dlt-console] = > "-DWITH_DLT_CONSOLE=ON,-DWITH_DLT_CONSOLE=OFF" > > > > inherit autotools gettext cmake pkgconfig systemd > > > > -- > > 2.34.1 > > > > > > -=-=-=-=-=-=-=-=-=-=-=- > > Links: You receive all messages sent to this group. > > View/Reply Online (#105219): > https://lists.openembedded.org/g/openembedded-devel/message/105219 > > Mute This Topic: https://lists.openembedded.org/mt/101634559/1997914 > > Group Owner: openembedded-devel+owner@lists.openembedded.org > > Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [ > raj.khem@gmail.com] > > -=-=-=-=-=-=-=-=-=-=-=- > > >
On Thu, Sep 28, 2023 at 8:35 AM Gianfranco Costamagna <costamagna.gianfranco@gmail.com> wrote: > > hello, sure looks perfect to me! > btw you will also need this one to build properly on some architecture > https://patchwork.yoctoproject.org/project/oe/list/?series=17190 OK > > I'm polishing up the work, testing, and will try to update upstream PR. > > But please note, this is a never-tested and highly experimental dlt feature nobody is using right now. > (and it's a new binary, so won't affect the normal dlt usage) This does not generate confidence. If you want this to be testing-only then mark the patches RFC so we can understand the intentions. > > Expect new commits soon :) sure > > G. > > On Thu, Sep 28, 2023 at 5:01 PM Khem Raj <raj.khem@gmail.com> wrote: >> >> I have staged it with some modified commit msg summary please check >> master-next and see if it looks ok. >> >> On Thu, Sep 28, 2023 at 2:38 AM Gianfranco >> <costamagna.gianfranco@gmail.com> wrote: >> > >> > Make sure also without systemd enabled as default, the tools can be built and installed, >> > such as dlt-adaptor, dlt-adaptor-stdin, dlt-adaptor-udp, dlt-examples and dlt-console. >> > They have stub functionalities when systemd is not default, but they can still be used with >> > limited functionalities >> > >> > Reported-By: Federico Briata <federicobriata@gmail.com> >> > Reported-By: Andrea Reali <andrea.reali@cnhind.com> >> > Signed-off-by: Gianfranco Costamagna <costamagnagianfranco@yahoo.it> >> > Signed-off-by: Gianfranco Costamagna <locutusofborg@debian.org> >> > --- >> > .../dlt-daemon/dlt-daemon_2.18.10.bb | 13 +++++++------ >> > 1 file changed, 7 insertions(+), 6 deletions(-) >> > >> > diff --git a/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb b/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb >> > index 33fae0619..3811772a7 100644 >> > --- a/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb >> > +++ b/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb >> > @@ -22,12 +22,13 @@ SRCREV = "0f2d4cfffada6f8448a2cb27995b38eb4271044f" >> > >> > S = "${WORKDIR}/git" >> > >> > -PACKAGECONFIG ?= "${@bb.utils.contains('DISTRO_FEATURES', 'systemd', ' systemd systemd-watchdog systemd-journal dlt-examples dlt-adaptor dlt-adaptor-stdin dlt-adaptor-udp dlt-console ', '', d)} \ >> > +PACKAGECONFIG ?= "${@bb.utils.contains('DISTRO_FEATURES', 'systemd', ' systemd systemd-watchdog systemd-journal ', '', d)} \ >> > + dlt-examples dlt-adaptor dlt-adaptor-stdin dlt-adaptor-udp dlt-console \ >> > udp-connection dlt-system dlt-filetransfer " >> > # dlt-dbus >> > >> > # General options >> > -PACKAGECONFIG[dlt-examples] = "-DWITH_DLT_EXAMPLES=ON,-DWITH_DLT_EXAMPLES=OFF,,dlt-daemon-systemd" >> > +PACKAGECONFIG[dlt-examples] = "-DWITH_DLT_EXAMPLES=ON,-DWITH_DLT_EXAMPLES=OFF" >> > >> > # Linux options >> > PACKAGECONFIG[systemd] = "-DWITH_SYSTEMD=ON,-DWITH_SYSTEMD=OFF -DWITH_DLT_SYSTEM=OFF,systemd" >> > @@ -38,11 +39,11 @@ PACKAGECONFIG[udp-connection] = "-DWITH_UDP_CONNECTION=ON,-DWITH_UDP_CONNECTION= >> > >> > # Command line options >> > PACKAGECONFIG[dlt-system] = "-DWITH_DLT_SYSTEM=ON,-DWITH_DLT_SYSTEM=OFF" >> > -PACKAGECONFIG[dlt-adaptor] = "-DWITH_DLT_ADAPTOR=ON,-DWITH_DLT_ADAPTOR=OFF,,dlt-daemon-systemd" >> > -PACKAGECONFIG[dlt-adaptor-stdin] = "-DWITH_DLT_ADAPTOR_STDIN=ON,-DWITH_DLT_ADAPTOR_STDIN=OFF,,dlt-daemon-systemd" >> > -PACKAGECONFIG[dlt-adaptor-udp] = "-DWITH_DLT_ADAPTOR_UDP=ON,-DWITH_DLT_ADAPTOR_UDP=OFF,,dlt-daemon-systemd" >> > +PACKAGECONFIG[dlt-adaptor] = "-DWITH_DLT_ADAPTOR=ON,-DWITH_DLT_ADAPTOR=OFF" >> > +PACKAGECONFIG[dlt-adaptor-stdin] = "-DWITH_DLT_ADAPTOR_STDIN=ON,-DWITH_DLT_ADAPTOR_STDIN=OFF" >> > +PACKAGECONFIG[dlt-adaptor-udp] = "-DWITH_DLT_ADAPTOR_UDP=ON,-DWITH_DLT_ADAPTOR_UDP=OFF" >> > PACKAGECONFIG[dlt-filetransfer] = "-DWITH_DLT_FILETRANSFER=ON,-DWITH_DLT_FILETRANSFER=OFF" >> > -PACKAGECONFIG[dlt-console] = "-DWITH_DLT_CONSOLE=ON,-DWITH_DLT_CONSOLE=OFF,,dlt-daemon-systemd" >> > +PACKAGECONFIG[dlt-console] = "-DWITH_DLT_CONSOLE=ON,-DWITH_DLT_CONSOLE=OFF" >> > >> > inherit autotools gettext cmake pkgconfig systemd >> > >> > -- >> > 2.34.1 >> > >> > >> > -=-=-=-=-=-=-=-=-=-=-=- >> > Links: You receive all messages sent to this group. >> > View/Reply Online (#105219): https://lists.openembedded.org/g/openembedded-devel/message/105219 >> > Mute This Topic: https://lists.openembedded.org/mt/101634559/1997914 >> > Group Owner: openembedded-devel+owner@lists.openembedded.org >> > Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [raj.khem@gmail.com] >> > -=-=-=-=-=-=-=-=-=-=-=- >> >
The functionality works already. The idea is to avoid many files with custom cpu code, and try to use perf native functions to get pc/sp and so on. But I did test, coredumps are correctly sent as well as context files. G. On Thu, Sep 28, 2023, 18:30 Khem Raj <raj.khem@gmail.com> wrote: > On Thu, Sep 28, 2023 at 8:35 AM Gianfranco Costamagna > <costamagna.gianfranco@gmail.com> wrote: > > > > hello, sure looks perfect to me! > > btw you will also need this one to build properly on some architecture > > https://patchwork.yoctoproject.org/project/oe/list/?series=17190 > > OK > > > > > I'm polishing up the work, testing, and will try to update upstream PR. > > > > But please note, this is a never-tested and highly experimental dlt > feature nobody is using right now. > > (and it's a new binary, so won't affect the normal dlt usage) > > This does not generate confidence. If you want this to be testing-only > then mark the patches RFC so > we can understand the intentions. > > > > > Expect new commits soon :) > > sure > > > > > G. > > > > On Thu, Sep 28, 2023 at 5:01 PM Khem Raj <raj.khem@gmail.com> wrote: > >> > >> I have staged it with some modified commit msg summary please check > >> master-next and see if it looks ok. > >> > >> On Thu, Sep 28, 2023 at 2:38 AM Gianfranco > >> <costamagna.gianfranco@gmail.com> wrote: > >> > > >> > Make sure also without systemd enabled as default, the tools can be > built and installed, > >> > such as dlt-adaptor, dlt-adaptor-stdin, dlt-adaptor-udp, dlt-examples > and dlt-console. > >> > They have stub functionalities when systemd is not default, but they > can still be used with > >> > limited functionalities > >> > > >> > Reported-By: Federico Briata <federicobriata@gmail.com> > >> > Reported-By: Andrea Reali <andrea.reali@cnhind.com> > >> > Signed-off-by: Gianfranco Costamagna <costamagnagianfranco@yahoo.it> > >> > Signed-off-by: Gianfranco Costamagna <locutusofborg@debian.org> > >> > --- > >> > .../dlt-daemon/dlt-daemon_2.18.10.bb | 13 > +++++++------ > >> > 1 file changed, 7 insertions(+), 6 deletions(-) > >> > > >> > diff --git a/meta-oe/recipes-extended/dlt-daemon/ > dlt-daemon_2.18.10.bb b/meta-oe/recipes-extended/dlt-daemon/ > dlt-daemon_2.18.10.bb > >> > index 33fae0619..3811772a7 100644 > >> > --- a/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb > >> > +++ b/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb > >> > @@ -22,12 +22,13 @@ SRCREV = > "0f2d4cfffada6f8448a2cb27995b38eb4271044f" > >> > > >> > S = "${WORKDIR}/git" > >> > > >> > -PACKAGECONFIG ?= "${@bb.utils.contains('DISTRO_FEATURES', 'systemd', > ' systemd systemd-watchdog systemd-journal dlt-examples dlt-adaptor > dlt-adaptor-stdin dlt-adaptor-udp dlt-console ', '', d)} \ > >> > +PACKAGECONFIG ?= "${@bb.utils.contains('DISTRO_FEATURES', 'systemd', > ' systemd systemd-watchdog systemd-journal ', '', d)} \ > >> > + dlt-examples dlt-adaptor dlt-adaptor-stdin dlt-adaptor-udp > dlt-console \ > >> > udp-connection dlt-system dlt-filetransfer " > >> > # dlt-dbus > >> > > >> > # General options > >> > -PACKAGECONFIG[dlt-examples] = > "-DWITH_DLT_EXAMPLES=ON,-DWITH_DLT_EXAMPLES=OFF,,dlt-daemon-systemd" > >> > +PACKAGECONFIG[dlt-examples] = > "-DWITH_DLT_EXAMPLES=ON,-DWITH_DLT_EXAMPLES=OFF" > >> > > >> > # Linux options > >> > PACKAGECONFIG[systemd] = "-DWITH_SYSTEMD=ON,-DWITH_SYSTEMD=OFF > -DWITH_DLT_SYSTEM=OFF,systemd" > >> > @@ -38,11 +39,11 @@ PACKAGECONFIG[udp-connection] = > "-DWITH_UDP_CONNECTION=ON,-DWITH_UDP_CONNECTION= > >> > > >> > # Command line options > >> > PACKAGECONFIG[dlt-system] = > "-DWITH_DLT_SYSTEM=ON,-DWITH_DLT_SYSTEM=OFF" > >> > -PACKAGECONFIG[dlt-adaptor] = > "-DWITH_DLT_ADAPTOR=ON,-DWITH_DLT_ADAPTOR=OFF,,dlt-daemon-systemd" > >> > -PACKAGECONFIG[dlt-adaptor-stdin] = > "-DWITH_DLT_ADAPTOR_STDIN=ON,-DWITH_DLT_ADAPTOR_STDIN=OFF,,dlt-daemon-systemd" > >> > -PACKAGECONFIG[dlt-adaptor-udp] = > "-DWITH_DLT_ADAPTOR_UDP=ON,-DWITH_DLT_ADAPTOR_UDP=OFF,,dlt-daemon-systemd" > >> > +PACKAGECONFIG[dlt-adaptor] = > "-DWITH_DLT_ADAPTOR=ON,-DWITH_DLT_ADAPTOR=OFF" > >> > +PACKAGECONFIG[dlt-adaptor-stdin] = > "-DWITH_DLT_ADAPTOR_STDIN=ON,-DWITH_DLT_ADAPTOR_STDIN=OFF" > >> > +PACKAGECONFIG[dlt-adaptor-udp] = > "-DWITH_DLT_ADAPTOR_UDP=ON,-DWITH_DLT_ADAPTOR_UDP=OFF" > >> > PACKAGECONFIG[dlt-filetransfer] = > "-DWITH_DLT_FILETRANSFER=ON,-DWITH_DLT_FILETRANSFER=OFF" > >> > -PACKAGECONFIG[dlt-console] = > "-DWITH_DLT_CONSOLE=ON,-DWITH_DLT_CONSOLE=OFF,,dlt-daemon-systemd" > >> > +PACKAGECONFIG[dlt-console] = > "-DWITH_DLT_CONSOLE=ON,-DWITH_DLT_CONSOLE=OFF" > >> > > >> > inherit autotools gettext cmake pkgconfig systemd > >> > > >> > -- > >> > 2.34.1 > >> > > >> > > >> > -=-=-=-=-=-=-=-=-=-=-=- > >> > Links: You receive all messages sent to this group. > >> > View/Reply Online (#105219): > https://lists.openembedded.org/g/openembedded-devel/message/105219 > >> > Mute This Topic: https://lists.openembedded.org/mt/101634559/1997914 > >> > Group Owner: openembedded-devel+owner@lists.openembedded.org > >> > Unsubscribe: > https://lists.openembedded.org/g/openembedded-devel/unsub [ > raj.khem@gmail.com] > >> > -=-=-=-=-=-=-=-=-=-=-=- > >> > >
fails to build on AB https://autobuilder.yoctoproject.org/typhoon/#/builders/88/builds/3186/steps/14/logs/stdio On Thu, Sep 28, 2023 at 11:57 AM Federico Pietro Briata <federicobriata@gmail.com> wrote: > > thanks guys, > we'll test it and give you a feedback very soon. > best regards > Federico > > > Il gio 28 set 2023, 6:58 PM Gianfranco Costamagna <costamagna.gianfranco@gmail.com> ha scritto: >> >> The functionality works already. >> The idea is to avoid many files with custom cpu code, and try to use perf native functions to get pc/sp and so on. But I did test, coredumps are correctly sent as well as context files. >> >> G. >> >> On Thu, Sep 28, 2023, 18:30 Khem Raj <raj.khem@gmail.com> wrote: >>> >>> On Thu, Sep 28, 2023 at 8:35 AM Gianfranco Costamagna >>> <costamagna.gianfranco@gmail.com> wrote: >>> > >>> > hello, sure looks perfect to me! >>> > btw you will also need this one to build properly on some architecture >>> > https://patchwork.yoctoproject.org/project/oe/list/?series=17190 >>> >>> OK >>> >>> > >>> > I'm polishing up the work, testing, and will try to update upstream PR. >>> > >>> > But please note, this is a never-tested and highly experimental dlt feature nobody is using right now. >>> > (and it's a new binary, so won't affect the normal dlt usage) >>> >>> This does not generate confidence. If you want this to be testing-only >>> then mark the patches RFC so >>> we can understand the intentions. >>> >>> > >>> > Expect new commits soon :) >>> >>> sure >>> >>> > >>> > G. >>> > >>> > On Thu, Sep 28, 2023 at 5:01 PM Khem Raj <raj.khem@gmail.com> wrote: >>> >> >>> >> I have staged it with some modified commit msg summary please check >>> >> master-next and see if it looks ok. >>> >> >>> >> On Thu, Sep 28, 2023 at 2:38 AM Gianfranco >>> >> <costamagna.gianfranco@gmail.com> wrote: >>> >> > >>> >> > Make sure also without systemd enabled as default, the tools can be built and installed, >>> >> > such as dlt-adaptor, dlt-adaptor-stdin, dlt-adaptor-udp, dlt-examples and dlt-console. >>> >> > They have stub functionalities when systemd is not default, but they can still be used with >>> >> > limited functionalities >>> >> > >>> >> > Reported-By: Federico Briata <federicobriata@gmail.com> >>> >> > Reported-By: Andrea Reali <andrea.reali@cnhind.com> >>> >> > Signed-off-by: Gianfranco Costamagna <costamagnagianfranco@yahoo.it> >>> >> > Signed-off-by: Gianfranco Costamagna <locutusofborg@debian.org> >>> >> > --- >>> >> > .../dlt-daemon/dlt-daemon_2.18.10.bb | 13 +++++++------ >>> >> > 1 file changed, 7 insertions(+), 6 deletions(-) >>> >> > >>> >> > diff --git a/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb b/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb >>> >> > index 33fae0619..3811772a7 100644 >>> >> > --- a/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb >>> >> > +++ b/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb >>> >> > @@ -22,12 +22,13 @@ SRCREV = "0f2d4cfffada6f8448a2cb27995b38eb4271044f" >>> >> > >>> >> > S = "${WORKDIR}/git" >>> >> > >>> >> > -PACKAGECONFIG ?= "${@bb.utils.contains('DISTRO_FEATURES', 'systemd', ' systemd systemd-watchdog systemd-journal dlt-examples dlt-adaptor dlt-adaptor-stdin dlt-adaptor-udp dlt-console ', '', d)} \ >>> >> > +PACKAGECONFIG ?= "${@bb.utils.contains('DISTRO_FEATURES', 'systemd', ' systemd systemd-watchdog systemd-journal ', '', d)} \ >>> >> > + dlt-examples dlt-adaptor dlt-adaptor-stdin dlt-adaptor-udp dlt-console \ >>> >> > udp-connection dlt-system dlt-filetransfer " >>> >> > # dlt-dbus >>> >> > >>> >> > # General options >>> >> > -PACKAGECONFIG[dlt-examples] = "-DWITH_DLT_EXAMPLES=ON,-DWITH_DLT_EXAMPLES=OFF,,dlt-daemon-systemd" >>> >> > +PACKAGECONFIG[dlt-examples] = "-DWITH_DLT_EXAMPLES=ON,-DWITH_DLT_EXAMPLES=OFF" >>> >> > >>> >> > # Linux options >>> >> > PACKAGECONFIG[systemd] = "-DWITH_SYSTEMD=ON,-DWITH_SYSTEMD=OFF -DWITH_DLT_SYSTEM=OFF,systemd" >>> >> > @@ -38,11 +39,11 @@ PACKAGECONFIG[udp-connection] = "-DWITH_UDP_CONNECTION=ON,-DWITH_UDP_CONNECTION= >>> >> > >>> >> > # Command line options >>> >> > PACKAGECONFIG[dlt-system] = "-DWITH_DLT_SYSTEM=ON,-DWITH_DLT_SYSTEM=OFF" >>> >> > -PACKAGECONFIG[dlt-adaptor] = "-DWITH_DLT_ADAPTOR=ON,-DWITH_DLT_ADAPTOR=OFF,,dlt-daemon-systemd" >>> >> > -PACKAGECONFIG[dlt-adaptor-stdin] = "-DWITH_DLT_ADAPTOR_STDIN=ON,-DWITH_DLT_ADAPTOR_STDIN=OFF,,dlt-daemon-systemd" >>> >> > -PACKAGECONFIG[dlt-adaptor-udp] = "-DWITH_DLT_ADAPTOR_UDP=ON,-DWITH_DLT_ADAPTOR_UDP=OFF,,dlt-daemon-systemd" >>> >> > +PACKAGECONFIG[dlt-adaptor] = "-DWITH_DLT_ADAPTOR=ON,-DWITH_DLT_ADAPTOR=OFF" >>> >> > +PACKAGECONFIG[dlt-adaptor-stdin] = "-DWITH_DLT_ADAPTOR_STDIN=ON,-DWITH_DLT_ADAPTOR_STDIN=OFF" >>> >> > +PACKAGECONFIG[dlt-adaptor-udp] = "-DWITH_DLT_ADAPTOR_UDP=ON,-DWITH_DLT_ADAPTOR_UDP=OFF" >>> >> > PACKAGECONFIG[dlt-filetransfer] = "-DWITH_DLT_FILETRANSFER=ON,-DWITH_DLT_FILETRANSFER=OFF" >>> >> > -PACKAGECONFIG[dlt-console] = "-DWITH_DLT_CONSOLE=ON,-DWITH_DLT_CONSOLE=OFF,,dlt-daemon-systemd" >>> >> > +PACKAGECONFIG[dlt-console] = "-DWITH_DLT_CONSOLE=ON,-DWITH_DLT_CONSOLE=OFF" >>> >> > >>> >> > inherit autotools gettext cmake pkgconfig systemd >>> >> > >>> >> > -- >>> >> > 2.34.1 >>> >> > >>> >> > >>> >> > -=-=-=-=-=-=-=-=-=-=-=- >>> >> > Links: You receive all messages sent to this group. >>> >> > View/Reply Online (#105219): https://lists.openembedded.org/g/openembedded-devel/message/105219 >>> >> > Mute This Topic: https://lists.openembedded.org/mt/101634559/1997914 >>> >> > Group Owner: openembedded-devel+owner@lists.openembedded.org >>> >> > Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [raj.khem@gmail.com] >>> >> > -=-=-=-=-=-=-=-=-=-=-=- >>> >> >
Hello, the additional fix is already on patchwork! On Fri, Sep 29, 2023, 06:44 Khem Raj <raj.khem@gmail.com> wrote: > fails to build on AB > > https://autobuilder.yoctoproject.org/typhoon/#/builders/88/builds/3186/steps/14/logs/stdio > > On Thu, Sep 28, 2023 at 11:57 AM Federico Pietro Briata > <federicobriata@gmail.com> wrote: > > > > thanks guys, > > we'll test it and give you a feedback very soon. > > best regards > > Federico > > > > > > Il gio 28 set 2023, 6:58 PM Gianfranco Costamagna < > costamagna.gianfranco@gmail.com> ha scritto: > >> > >> The functionality works already. > >> The idea is to avoid many files with custom cpu code, and try to use > perf native functions to get pc/sp and so on. But I did test, coredumps are > correctly sent as well as context files. > >> > >> G. > >> > >> On Thu, Sep 28, 2023, 18:30 Khem Raj <raj.khem@gmail.com> wrote: > >>> > >>> On Thu, Sep 28, 2023 at 8:35 AM Gianfranco Costamagna > >>> <costamagna.gianfranco@gmail.com> wrote: > >>> > > >>> > hello, sure looks perfect to me! > >>> > btw you will also need this one to build properly on some > architecture > >>> > https://patchwork.yoctoproject.org/project/oe/list/?series=17190 > >>> > >>> OK > >>> > >>> > > >>> > I'm polishing up the work, testing, and will try to update upstream > PR. > >>> > > >>> > But please note, this is a never-tested and highly experimental dlt > feature nobody is using right now. > >>> > (and it's a new binary, so won't affect the normal dlt usage) > >>> > >>> This does not generate confidence. If you want this to be testing-only > >>> then mark the patches RFC so > >>> we can understand the intentions. > >>> > >>> > > >>> > Expect new commits soon :) > >>> > >>> sure > >>> > >>> > > >>> > G. > >>> > > >>> > On Thu, Sep 28, 2023 at 5:01 PM Khem Raj <raj.khem@gmail.com> wrote: > >>> >> > >>> >> I have staged it with some modified commit msg summary please check > >>> >> master-next and see if it looks ok. > >>> >> > >>> >> On Thu, Sep 28, 2023 at 2:38 AM Gianfranco > >>> >> <costamagna.gianfranco@gmail.com> wrote: > >>> >> > > >>> >> > Make sure also without systemd enabled as default, the tools can > be built and installed, > >>> >> > such as dlt-adaptor, dlt-adaptor-stdin, dlt-adaptor-udp, > dlt-examples and dlt-console. > >>> >> > They have stub functionalities when systemd is not default, but > they can still be used with > >>> >> > limited functionalities > >>> >> > > >>> >> > Reported-By: Federico Briata <federicobriata@gmail.com> > >>> >> > Reported-By: Andrea Reali <andrea.reali@cnhind.com> > >>> >> > Signed-off-by: Gianfranco Costamagna < > costamagnagianfranco@yahoo.it> > >>> >> > Signed-off-by: Gianfranco Costamagna <locutusofborg@debian.org> > >>> >> > --- > >>> >> > .../dlt-daemon/dlt-daemon_2.18.10.bb | 13 > +++++++------ > >>> >> > 1 file changed, 7 insertions(+), 6 deletions(-) > >>> >> > > >>> >> > diff --git a/meta-oe/recipes-extended/dlt-daemon/ > dlt-daemon_2.18.10.bb b/meta-oe/recipes-extended/dlt-daemon/ > dlt-daemon_2.18.10.bb > >>> >> > index 33fae0619..3811772a7 100644 > >>> >> > --- a/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb > >>> >> > +++ b/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb > >>> >> > @@ -22,12 +22,13 @@ SRCREV = > "0f2d4cfffada6f8448a2cb27995b38eb4271044f" > >>> >> > > >>> >> > S = "${WORKDIR}/git" > >>> >> > > >>> >> > -PACKAGECONFIG ?= "${@bb.utils.contains('DISTRO_FEATURES', > 'systemd', ' systemd systemd-watchdog systemd-journal dlt-examples > dlt-adaptor dlt-adaptor-stdin dlt-adaptor-udp dlt-console ', '', d)} \ > >>> >> > +PACKAGECONFIG ?= "${@bb.utils.contains('DISTRO_FEATURES', > 'systemd', ' systemd systemd-watchdog systemd-journal ', '', d)} \ > >>> >> > + dlt-examples dlt-adaptor dlt-adaptor-stdin dlt-adaptor-udp > dlt-console \ > >>> >> > udp-connection dlt-system dlt-filetransfer " > >>> >> > # dlt-dbus > >>> >> > > >>> >> > # General options > >>> >> > -PACKAGECONFIG[dlt-examples] = > "-DWITH_DLT_EXAMPLES=ON,-DWITH_DLT_EXAMPLES=OFF,,dlt-daemon-systemd" > >>> >> > +PACKAGECONFIG[dlt-examples] = > "-DWITH_DLT_EXAMPLES=ON,-DWITH_DLT_EXAMPLES=OFF" > >>> >> > > >>> >> > # Linux options > >>> >> > PACKAGECONFIG[systemd] = "-DWITH_SYSTEMD=ON,-DWITH_SYSTEMD=OFF > -DWITH_DLT_SYSTEM=OFF,systemd" > >>> >> > @@ -38,11 +39,11 @@ PACKAGECONFIG[udp-connection] = > "-DWITH_UDP_CONNECTION=ON,-DWITH_UDP_CONNECTION= > >>> >> > > >>> >> > # Command line options > >>> >> > PACKAGECONFIG[dlt-system] = > "-DWITH_DLT_SYSTEM=ON,-DWITH_DLT_SYSTEM=OFF" > >>> >> > -PACKAGECONFIG[dlt-adaptor] = > "-DWITH_DLT_ADAPTOR=ON,-DWITH_DLT_ADAPTOR=OFF,,dlt-daemon-systemd" > >>> >> > -PACKAGECONFIG[dlt-adaptor-stdin] = > "-DWITH_DLT_ADAPTOR_STDIN=ON,-DWITH_DLT_ADAPTOR_STDIN=OFF,,dlt-daemon-systemd" > >>> >> > -PACKAGECONFIG[dlt-adaptor-udp] = > "-DWITH_DLT_ADAPTOR_UDP=ON,-DWITH_DLT_ADAPTOR_UDP=OFF,,dlt-daemon-systemd" > >>> >> > +PACKAGECONFIG[dlt-adaptor] = > "-DWITH_DLT_ADAPTOR=ON,-DWITH_DLT_ADAPTOR=OFF" > >>> >> > +PACKAGECONFIG[dlt-adaptor-stdin] = > "-DWITH_DLT_ADAPTOR_STDIN=ON,-DWITH_DLT_ADAPTOR_STDIN=OFF" > >>> >> > +PACKAGECONFIG[dlt-adaptor-udp] = > "-DWITH_DLT_ADAPTOR_UDP=ON,-DWITH_DLT_ADAPTOR_UDP=OFF" > >>> >> > PACKAGECONFIG[dlt-filetransfer] = > "-DWITH_DLT_FILETRANSFER=ON,-DWITH_DLT_FILETRANSFER=OFF" > >>> >> > -PACKAGECONFIG[dlt-console] = > "-DWITH_DLT_CONSOLE=ON,-DWITH_DLT_CONSOLE=OFF,,dlt-daemon-systemd" > >>> >> > +PACKAGECONFIG[dlt-console] = > "-DWITH_DLT_CONSOLE=ON,-DWITH_DLT_CONSOLE=OFF" > >>> >> > > >>> >> > inherit autotools gettext cmake pkgconfig systemd > >>> >> > > >>> >> > -- > >>> >> > 2.34.1 > >>> >> > > >>> >> > > >>> >> > -=-=-=-=-=-=-=-=-=-=-=- > >>> >> > Links: You receive all messages sent to this group. > >>> >> > View/Reply Online (#105219): > https://lists.openembedded.org/g/openembedded-devel/message/105219 > >>> >> > Mute This Topic: > https://lists.openembedded.org/mt/101634559/1997914 > >>> >> > Group Owner: openembedded-devel+owner@lists.openembedded.org > >>> >> > Unsubscribe: > https://lists.openembedded.org/g/openembedded-devel/unsub [ > raj.khem@gmail.com] > >>> >> > -=-=-=-=-=-=-=-=-=-=-=- > >>> >> > >
thanks, do we still need the arm fix? it would be good if you resend the whole series as v2 and so on. On Thu, Sep 28, 2023 at 10:24 PM Gianfranco Costamagna <costamagna.gianfranco@gmail.com> wrote: > > Hello, the additional fix is already on patchwork! > > On Fri, Sep 29, 2023, 06:44 Khem Raj <raj.khem@gmail.com> wrote: >> >> fails to build on AB >> https://autobuilder.yoctoproject.org/typhoon/#/builders/88/builds/3186/steps/14/logs/stdio >> >> On Thu, Sep 28, 2023 at 11:57 AM Federico Pietro Briata >> <federicobriata@gmail.com> wrote: >> > >> > thanks guys, >> > we'll test it and give you a feedback very soon. >> > best regards >> > Federico >> > >> > >> > Il gio 28 set 2023, 6:58 PM Gianfranco Costamagna <costamagna.gianfranco@gmail.com> ha scritto: >> >> >> >> The functionality works already. >> >> The idea is to avoid many files with custom cpu code, and try to use perf native functions to get pc/sp and so on. But I did test, coredumps are correctly sent as well as context files. >> >> >> >> G. >> >> >> >> On Thu, Sep 28, 2023, 18:30 Khem Raj <raj.khem@gmail.com> wrote: >> >>> >> >>> On Thu, Sep 28, 2023 at 8:35 AM Gianfranco Costamagna >> >>> <costamagna.gianfranco@gmail.com> wrote: >> >>> > >> >>> > hello, sure looks perfect to me! >> >>> > btw you will also need this one to build properly on some architecture >> >>> > https://patchwork.yoctoproject.org/project/oe/list/?series=17190 >> >>> >> >>> OK >> >>> >> >>> > >> >>> > I'm polishing up the work, testing, and will try to update upstream PR. >> >>> > >> >>> > But please note, this is a never-tested and highly experimental dlt feature nobody is using right now. >> >>> > (and it's a new binary, so won't affect the normal dlt usage) >> >>> >> >>> This does not generate confidence. If you want this to be testing-only >> >>> then mark the patches RFC so >> >>> we can understand the intentions. >> >>> >> >>> > >> >>> > Expect new commits soon :) >> >>> >> >>> sure >> >>> >> >>> > >> >>> > G. >> >>> > >> >>> > On Thu, Sep 28, 2023 at 5:01 PM Khem Raj <raj.khem@gmail.com> wrote: >> >>> >> >> >>> >> I have staged it with some modified commit msg summary please check >> >>> >> master-next and see if it looks ok. >> >>> >> >> >>> >> On Thu, Sep 28, 2023 at 2:38 AM Gianfranco >> >>> >> <costamagna.gianfranco@gmail.com> wrote: >> >>> >> > >> >>> >> > Make sure also without systemd enabled as default, the tools can be built and installed, >> >>> >> > such as dlt-adaptor, dlt-adaptor-stdin, dlt-adaptor-udp, dlt-examples and dlt-console. >> >>> >> > They have stub functionalities when systemd is not default, but they can still be used with >> >>> >> > limited functionalities >> >>> >> > >> >>> >> > Reported-By: Federico Briata <federicobriata@gmail.com> >> >>> >> > Reported-By: Andrea Reali <andrea.reali@cnhind.com> >> >>> >> > Signed-off-by: Gianfranco Costamagna <costamagnagianfranco@yahoo.it> >> >>> >> > Signed-off-by: Gianfranco Costamagna <locutusofborg@debian.org> >> >>> >> > --- >> >>> >> > .../dlt-daemon/dlt-daemon_2.18.10.bb | 13 +++++++------ >> >>> >> > 1 file changed, 7 insertions(+), 6 deletions(-) >> >>> >> > >> >>> >> > diff --git a/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb b/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb >> >>> >> > index 33fae0619..3811772a7 100644 >> >>> >> > --- a/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb >> >>> >> > +++ b/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb >> >>> >> > @@ -22,12 +22,13 @@ SRCREV = "0f2d4cfffada6f8448a2cb27995b38eb4271044f" >> >>> >> > >> >>> >> > S = "${WORKDIR}/git" >> >>> >> > >> >>> >> > -PACKAGECONFIG ?= "${@bb.utils.contains('DISTRO_FEATURES', 'systemd', ' systemd systemd-watchdog systemd-journal dlt-examples dlt-adaptor dlt-adaptor-stdin dlt-adaptor-udp dlt-console ', '', d)} \ >> >>> >> > +PACKAGECONFIG ?= "${@bb.utils.contains('DISTRO_FEATURES', 'systemd', ' systemd systemd-watchdog systemd-journal ', '', d)} \ >> >>> >> > + dlt-examples dlt-adaptor dlt-adaptor-stdin dlt-adaptor-udp dlt-console \ >> >>> >> > udp-connection dlt-system dlt-filetransfer " >> >>> >> > # dlt-dbus >> >>> >> > >> >>> >> > # General options >> >>> >> > -PACKAGECONFIG[dlt-examples] = "-DWITH_DLT_EXAMPLES=ON,-DWITH_DLT_EXAMPLES=OFF,,dlt-daemon-systemd" >> >>> >> > +PACKAGECONFIG[dlt-examples] = "-DWITH_DLT_EXAMPLES=ON,-DWITH_DLT_EXAMPLES=OFF" >> >>> >> > >> >>> >> > # Linux options >> >>> >> > PACKAGECONFIG[systemd] = "-DWITH_SYSTEMD=ON,-DWITH_SYSTEMD=OFF -DWITH_DLT_SYSTEM=OFF,systemd" >> >>> >> > @@ -38,11 +39,11 @@ PACKAGECONFIG[udp-connection] = "-DWITH_UDP_CONNECTION=ON,-DWITH_UDP_CONNECTION= >> >>> >> > >> >>> >> > # Command line options >> >>> >> > PACKAGECONFIG[dlt-system] = "-DWITH_DLT_SYSTEM=ON,-DWITH_DLT_SYSTEM=OFF" >> >>> >> > -PACKAGECONFIG[dlt-adaptor] = "-DWITH_DLT_ADAPTOR=ON,-DWITH_DLT_ADAPTOR=OFF,,dlt-daemon-systemd" >> >>> >> > -PACKAGECONFIG[dlt-adaptor-stdin] = "-DWITH_DLT_ADAPTOR_STDIN=ON,-DWITH_DLT_ADAPTOR_STDIN=OFF,,dlt-daemon-systemd" >> >>> >> > -PACKAGECONFIG[dlt-adaptor-udp] = "-DWITH_DLT_ADAPTOR_UDP=ON,-DWITH_DLT_ADAPTOR_UDP=OFF,,dlt-daemon-systemd" >> >>> >> > +PACKAGECONFIG[dlt-adaptor] = "-DWITH_DLT_ADAPTOR=ON,-DWITH_DLT_ADAPTOR=OFF" >> >>> >> > +PACKAGECONFIG[dlt-adaptor-stdin] = "-DWITH_DLT_ADAPTOR_STDIN=ON,-DWITH_DLT_ADAPTOR_STDIN=OFF" >> >>> >> > +PACKAGECONFIG[dlt-adaptor-udp] = "-DWITH_DLT_ADAPTOR_UDP=ON,-DWITH_DLT_ADAPTOR_UDP=OFF" >> >>> >> > PACKAGECONFIG[dlt-filetransfer] = "-DWITH_DLT_FILETRANSFER=ON,-DWITH_DLT_FILETRANSFER=OFF" >> >>> >> > -PACKAGECONFIG[dlt-console] = "-DWITH_DLT_CONSOLE=ON,-DWITH_DLT_CONSOLE=OFF,,dlt-daemon-systemd" >> >>> >> > +PACKAGECONFIG[dlt-console] = "-DWITH_DLT_CONSOLE=ON,-DWITH_DLT_CONSOLE=OFF" >> >>> >> > >> >>> >> > inherit autotools gettext cmake pkgconfig systemd >> >>> >> > >> >>> >> > -- >> >>> >> > 2.34.1 >> >>> >> > >> >>> >> > >> >>> >> > -=-=-=-=-=-=-=-=-=-=-=- >> >>> >> > Links: You receive all messages sent to this group. >> >>> >> > View/Reply Online (#105219): https://lists.openembedded.org/g/openembedded-devel/message/105219 >> >>> >> > Mute This Topic: https://lists.openembedded.org/mt/101634559/1997914 >> >>> >> > Group Owner: openembedded-devel+owner@lists.openembedded.org >> >>> >> > Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [raj.khem@gmail.com] >> >>> >> > -=-=-=-=-=-=-=-=-=-=-=- >> >>> >> >
I sent a new patch serie. For now better keep it disabled as default until I finish writing the code. On Fri, Sep 29, 2023, 15:22 Khem Raj <raj.khem@gmail.com> wrote: > thanks, do we still need the arm fix? it would be good if you resend > the whole series as v2 and so on. > > On Thu, Sep 28, 2023 at 10:24 PM Gianfranco Costamagna > <costamagna.gianfranco@gmail.com> wrote: > > > > Hello, the additional fix is already on patchwork! > > > > On Fri, Sep 29, 2023, 06:44 Khem Raj <raj.khem@gmail.com> wrote: > >> > >> fails to build on AB > >> > https://autobuilder.yoctoproject.org/typhoon/#/builders/88/builds/3186/steps/14/logs/stdio > >> > >> On Thu, Sep 28, 2023 at 11:57 AM Federico Pietro Briata > >> <federicobriata@gmail.com> wrote: > >> > > >> > thanks guys, > >> > we'll test it and give you a feedback very soon. > >> > best regards > >> > Federico > >> > > >> > > >> > Il gio 28 set 2023, 6:58 PM Gianfranco Costamagna < > costamagna.gianfranco@gmail.com> ha scritto: > >> >> > >> >> The functionality works already. > >> >> The idea is to avoid many files with custom cpu code, and try to use > perf native functions to get pc/sp and so on. But I did test, coredumps are > correctly sent as well as context files. > >> >> > >> >> G. > >> >> > >> >> On Thu, Sep 28, 2023, 18:30 Khem Raj <raj.khem@gmail.com> wrote: > >> >>> > >> >>> On Thu, Sep 28, 2023 at 8:35 AM Gianfranco Costamagna > >> >>> <costamagna.gianfranco@gmail.com> wrote: > >> >>> > > >> >>> > hello, sure looks perfect to me! > >> >>> > btw you will also need this one to build properly on some > architecture > >> >>> > https://patchwork.yoctoproject.org/project/oe/list/?series=17190 > >> >>> > >> >>> OK > >> >>> > >> >>> > > >> >>> > I'm polishing up the work, testing, and will try to update > upstream PR. > >> >>> > > >> >>> > But please note, this is a never-tested and highly experimental > dlt feature nobody is using right now. > >> >>> > (and it's a new binary, so won't affect the normal dlt usage) > >> >>> > >> >>> This does not generate confidence. If you want this to be > testing-only > >> >>> then mark the patches RFC so > >> >>> we can understand the intentions. > >> >>> > >> >>> > > >> >>> > Expect new commits soon :) > >> >>> > >> >>> sure > >> >>> > >> >>> > > >> >>> > G. > >> >>> > > >> >>> > On Thu, Sep 28, 2023 at 5:01 PM Khem Raj <raj.khem@gmail.com> > wrote: > >> >>> >> > >> >>> >> I have staged it with some modified commit msg summary please > check > >> >>> >> master-next and see if it looks ok. > >> >>> >> > >> >>> >> On Thu, Sep 28, 2023 at 2:38 AM Gianfranco > >> >>> >> <costamagna.gianfranco@gmail.com> wrote: > >> >>> >> > > >> >>> >> > Make sure also without systemd enabled as default, the tools > can be built and installed, > >> >>> >> > such as dlt-adaptor, dlt-adaptor-stdin, dlt-adaptor-udp, > dlt-examples and dlt-console. > >> >>> >> > They have stub functionalities when systemd is not default, > but they can still be used with > >> >>> >> > limited functionalities > >> >>> >> > > >> >>> >> > Reported-By: Federico Briata <federicobriata@gmail.com> > >> >>> >> > Reported-By: Andrea Reali <andrea.reali@cnhind.com> > >> >>> >> > Signed-off-by: Gianfranco Costamagna < > costamagnagianfranco@yahoo.it> > >> >>> >> > Signed-off-by: Gianfranco Costamagna <locutusofborg@debian.org > > > >> >>> >> > --- > >> >>> >> > .../dlt-daemon/dlt-daemon_2.18.10.bb | 13 > +++++++------ > >> >>> >> > 1 file changed, 7 insertions(+), 6 deletions(-) > >> >>> >> > > >> >>> >> > diff --git a/meta-oe/recipes-extended/dlt-daemon/ > dlt-daemon_2.18.10.bb b/meta-oe/recipes-extended/dlt-daemon/ > dlt-daemon_2.18.10.bb > >> >>> >> > index 33fae0619..3811772a7 100644 > >> >>> >> > --- a/meta-oe/recipes-extended/dlt-daemon/ > dlt-daemon_2.18.10.bb > >> >>> >> > +++ b/meta-oe/recipes-extended/dlt-daemon/ > dlt-daemon_2.18.10.bb > >> >>> >> > @@ -22,12 +22,13 @@ SRCREV = > "0f2d4cfffada6f8448a2cb27995b38eb4271044f" > >> >>> >> > > >> >>> >> > S = "${WORKDIR}/git" > >> >>> >> > > >> >>> >> > -PACKAGECONFIG ?= "${@bb.utils.contains('DISTRO_FEATURES', > 'systemd', ' systemd systemd-watchdog systemd-journal dlt-examples > dlt-adaptor dlt-adaptor-stdin dlt-adaptor-udp dlt-console ', '', d)} \ > >> >>> >> > +PACKAGECONFIG ?= "${@bb.utils.contains('DISTRO_FEATURES', > 'systemd', ' systemd systemd-watchdog systemd-journal ', '', d)} \ > >> >>> >> > + dlt-examples dlt-adaptor dlt-adaptor-stdin dlt-adaptor-udp > dlt-console \ > >> >>> >> > udp-connection dlt-system dlt-filetransfer " > >> >>> >> > # dlt-dbus > >> >>> >> > > >> >>> >> > # General options > >> >>> >> > -PACKAGECONFIG[dlt-examples] = > "-DWITH_DLT_EXAMPLES=ON,-DWITH_DLT_EXAMPLES=OFF,,dlt-daemon-systemd" > >> >>> >> > +PACKAGECONFIG[dlt-examples] = > "-DWITH_DLT_EXAMPLES=ON,-DWITH_DLT_EXAMPLES=OFF" > >> >>> >> > > >> >>> >> > # Linux options > >> >>> >> > PACKAGECONFIG[systemd] = > "-DWITH_SYSTEMD=ON,-DWITH_SYSTEMD=OFF -DWITH_DLT_SYSTEM=OFF,systemd" > >> >>> >> > @@ -38,11 +39,11 @@ PACKAGECONFIG[udp-connection] = > "-DWITH_UDP_CONNECTION=ON,-DWITH_UDP_CONNECTION= > >> >>> >> > > >> >>> >> > # Command line options > >> >>> >> > PACKAGECONFIG[dlt-system] = > "-DWITH_DLT_SYSTEM=ON,-DWITH_DLT_SYSTEM=OFF" > >> >>> >> > -PACKAGECONFIG[dlt-adaptor] = > "-DWITH_DLT_ADAPTOR=ON,-DWITH_DLT_ADAPTOR=OFF,,dlt-daemon-systemd" > >> >>> >> > -PACKAGECONFIG[dlt-adaptor-stdin] = > "-DWITH_DLT_ADAPTOR_STDIN=ON,-DWITH_DLT_ADAPTOR_STDIN=OFF,,dlt-daemon-systemd" > >> >>> >> > -PACKAGECONFIG[dlt-adaptor-udp] = > "-DWITH_DLT_ADAPTOR_UDP=ON,-DWITH_DLT_ADAPTOR_UDP=OFF,,dlt-daemon-systemd" > >> >>> >> > +PACKAGECONFIG[dlt-adaptor] = > "-DWITH_DLT_ADAPTOR=ON,-DWITH_DLT_ADAPTOR=OFF" > >> >>> >> > +PACKAGECONFIG[dlt-adaptor-stdin] = > "-DWITH_DLT_ADAPTOR_STDIN=ON,-DWITH_DLT_ADAPTOR_STDIN=OFF" > >> >>> >> > +PACKAGECONFIG[dlt-adaptor-udp] = > "-DWITH_DLT_ADAPTOR_UDP=ON,-DWITH_DLT_ADAPTOR_UDP=OFF" > >> >>> >> > PACKAGECONFIG[dlt-filetransfer] = > "-DWITH_DLT_FILETRANSFER=ON,-DWITH_DLT_FILETRANSFER=OFF" > >> >>> >> > -PACKAGECONFIG[dlt-console] = > "-DWITH_DLT_CONSOLE=ON,-DWITH_DLT_CONSOLE=OFF,,dlt-daemon-systemd" > >> >>> >> > +PACKAGECONFIG[dlt-console] = > "-DWITH_DLT_CONSOLE=ON,-DWITH_DLT_CONSOLE=OFF" > >> >>> >> > > >> >>> >> > inherit autotools gettext cmake pkgconfig systemd > >> >>> >> > > >> >>> >> > -- > >> >>> >> > 2.34.1 > >> >>> >> > > >> >>> >> > > >> >>> >> > -=-=-=-=-=-=-=-=-=-=-=- > >> >>> >> > Links: You receive all messages sent to this group. > >> >>> >> > View/Reply Online (#105219): > https://lists.openembedded.org/g/openembedded-devel/message/105219 > >> >>> >> > Mute This Topic: > https://lists.openembedded.org/mt/101634559/1997914 > >> >>> >> > Group Owner: openembedded-devel+owner@lists.openembedded.org > >> >>> >> > Unsubscribe: > https://lists.openembedded.org/g/openembedded-devel/unsub [ > raj.khem@gmail.com] > >> >>> >> > -=-=-=-=-=-=-=-=-=-=-=- > >> >>> >> > >
diff --git a/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb b/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb index 33fae0619..3811772a7 100644 --- a/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb +++ b/meta-oe/recipes-extended/dlt-daemon/dlt-daemon_2.18.10.bb @@ -22,12 +22,13 @@ SRCREV = "0f2d4cfffada6f8448a2cb27995b38eb4271044f" S = "${WORKDIR}/git" -PACKAGECONFIG ?= "${@bb.utils.contains('DISTRO_FEATURES', 'systemd', ' systemd systemd-watchdog systemd-journal dlt-examples dlt-adaptor dlt-adaptor-stdin dlt-adaptor-udp dlt-console ', '', d)} \ +PACKAGECONFIG ?= "${@bb.utils.contains('DISTRO_FEATURES', 'systemd', ' systemd systemd-watchdog systemd-journal ', '', d)} \ + dlt-examples dlt-adaptor dlt-adaptor-stdin dlt-adaptor-udp dlt-console \ udp-connection dlt-system dlt-filetransfer " # dlt-dbus # General options -PACKAGECONFIG[dlt-examples] = "-DWITH_DLT_EXAMPLES=ON,-DWITH_DLT_EXAMPLES=OFF,,dlt-daemon-systemd" +PACKAGECONFIG[dlt-examples] = "-DWITH_DLT_EXAMPLES=ON,-DWITH_DLT_EXAMPLES=OFF" # Linux options PACKAGECONFIG[systemd] = "-DWITH_SYSTEMD=ON,-DWITH_SYSTEMD=OFF -DWITH_DLT_SYSTEM=OFF,systemd" @@ -38,11 +39,11 @@ PACKAGECONFIG[udp-connection] = "-DWITH_UDP_CONNECTION=ON,-DWITH_UDP_CONNECTION= # Command line options PACKAGECONFIG[dlt-system] = "-DWITH_DLT_SYSTEM=ON,-DWITH_DLT_SYSTEM=OFF" -PACKAGECONFIG[dlt-adaptor] = "-DWITH_DLT_ADAPTOR=ON,-DWITH_DLT_ADAPTOR=OFF,,dlt-daemon-systemd" -PACKAGECONFIG[dlt-adaptor-stdin] = "-DWITH_DLT_ADAPTOR_STDIN=ON,-DWITH_DLT_ADAPTOR_STDIN=OFF,,dlt-daemon-systemd" -PACKAGECONFIG[dlt-adaptor-udp] = "-DWITH_DLT_ADAPTOR_UDP=ON,-DWITH_DLT_ADAPTOR_UDP=OFF,,dlt-daemon-systemd" +PACKAGECONFIG[dlt-adaptor] = "-DWITH_DLT_ADAPTOR=ON,-DWITH_DLT_ADAPTOR=OFF" +PACKAGECONFIG[dlt-adaptor-stdin] = "-DWITH_DLT_ADAPTOR_STDIN=ON,-DWITH_DLT_ADAPTOR_STDIN=OFF" +PACKAGECONFIG[dlt-adaptor-udp] = "-DWITH_DLT_ADAPTOR_UDP=ON,-DWITH_DLT_ADAPTOR_UDP=OFF" PACKAGECONFIG[dlt-filetransfer] = "-DWITH_DLT_FILETRANSFER=ON,-DWITH_DLT_FILETRANSFER=OFF" -PACKAGECONFIG[dlt-console] = "-DWITH_DLT_CONSOLE=ON,-DWITH_DLT_CONSOLE=OFF,,dlt-daemon-systemd" +PACKAGECONFIG[dlt-console] = "-DWITH_DLT_CONSOLE=ON,-DWITH_DLT_CONSOLE=OFF" inherit autotools gettext cmake pkgconfig systemd