[genivi-dlt] Updates on 2.15 DLT daemon

Lipka, Christoph (ADITJ/SWG) clipka at jp.adit-jv.com
Fri Apr 15 03:30:47 EDT 2016


Hi,

As Gunnar discussed, here the changes again, but split into separated patches.

Please have a look and let me know if any clarification is needed.
Thanks.

Best regards

Christoph Lipka
Software Group (ADITJ/SWG)

Tel. +81-(0)566 61-5124


-----Original Message-----
From: genivi-diagnostic-log-and-trace-bounces at lists.genivi.org [mailto:genivi-diagnostic-log-and-trace-bounces at lists.genivi.org] On Behalf Of Lipka, Christoph (ADITJ/SWG)
Sent: Friday, April 15, 2016 10:21 AM
To: genivi-diagnostic-log-and-trace at lists.genivi.org
Subject: Re: [genivi-dlt] Updates on 2.15 DLT daemon

Hi Gunnar,

Thanks for your comment. I very appreciate that someone is looking into it in detail. The stuff works for us and is already in practical use, but of course might raise some problems if someone else using it in a different way which was not expected by us.


We though it is a better idea to squash things together to not have a list of ~ 20 commits. But we can also provide the same thinks as list of more separated patches - no problem at all, we have it in place for internal reference.

We will recheck and send the patches again by today.

Best regards

Christoph Lipka
Software Group (ADITJ/SWG)

Tel. +81-(0)566 61-5124

-----Original Message-----
From: genivi-diagnostic-log-and-trace-bounces at lists.genivi.org [mailto:genivi-diagnostic-log-and-trace-bounces at lists.genivi.org] On Behalf Of Andersson, Gunnar
Sent: Thursday, April 14, 2016 5:46 PM
To: Lipka, Christoph (ADITJ/SWG)
Cc: genivi-diagnostic-log-and-trace at lists.genivi.org
Subject: Re: [genivi-dlt] Updates on 2.15 DLT daemon

Hi Christoph

> If you have any comments or questions, please let me know.
>

This looks like great stuff.  But why do you squash so many different changes together?

First thing I noticed browsing patch 1 was:  Introducing long options
 - a really nice little thing, but that change is not mentioned in the commit comment at all.

A separate commit would make it difficult to forget to mention it.

Patch 3: dlt-daemon: Connection handling improvements and other updates

 "... and other updates?"

Bugfixes are in there too - good stuff!  But sometimes that introduces new bugs -  it can be tricky. So keeping those commits separate really help in tracking down regressions with bisect etc.

Finally it's about the review process.  It's hard enough to get volunteers and at least I get overwhelmed to review it all....  What if someone only has time (or competence) to review one of the changes - It would be nice to 
be able to sign off on that commit.   At least I don't want to sign off on the 
really big commit if I've only studied part of it in detail.

If this is the only way you can send because of lack of time then thanks - it's still great to get this functionality and bugfixes, but it would be even better if different issues are separated in different commits, in my opinion.

My 5c,

Best Regards
- Gunnar

> From: genivi-diagnostic-log-and-trace-bounces at lists.genivi.org
> [mailto:genivi-diagnostic-log-and-trace-bounces at lists.genivi.org] On 
> Behalf Of Lipka, Christoph (ADITJ/SWG) Sent: den 14 april 2016 09:58 To:
> genivi-diagnostic-log-and-trace at lists.genivi.org Subject: [genivi-dlt] 
> Updates on 2.15 DLT daemon
>
> Dear all,
>
> Attached are a couple of updates we would like to share with you. The 
> commit messages contain (hopefully) enough information to understand 
> the changes.
>
> 0001-Offlinelogstorage-Functionality-improvements.patch
> 0002-MultiNode-Functionality-improvements.patch
> 0003-dlt-daemon-Connection-handling-improvements-and-othe.patch
> 0004-CommonControl-dlt-control-application-improvements.patch
> 0005-Offline-trace-Make-search-more-precise.patch
> 0006-DLT_PTR-User-macro-to-print-pointers.patch
>
> If you have any comments or questions, please let me know.
>
> Best regards
>
> Christoph Lipka Advanced Driver Information Technology Software Group
> (ADITJ/SWG) 1-1 Showa-cho, Kariya-shi Aichi-ken 448-8661, Japan Tel.
> +81-(0)566 61-5124 Fax +81-(0)566 25-4774 clipka at jp.adit-jv.com
>
> ADIT is joint venture company of DENSO Corporation and Bosch GmbH
>
>
_______________________________________________
genivi-diagnostic-log-and-trace mailing list genivi-diagnostic-log-and-trace at lists.genivi.org
https://lists.genivi.org/mailman/listinfo/genivi-diagnostic-log-and-trace
_______________________________________________
genivi-diagnostic-log-and-trace mailing list genivi-diagnostic-log-and-trace at lists.genivi.org
https://lists.genivi.org/mailman/listinfo/genivi-diagnostic-log-and-trace
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Offline-logstorage-Fix-invalid-filter-configuration-.patch
Type: application/octet-stream
Size: 2744 bytes
Desc: 0001-Offline-logstorage-Fix-invalid-filter-configuration-.patch
URL: <http://lists.genivi.org/pipermail/genivi-diagnostic-log-and-trace_lists.genivi.org/attachments/20160415/dda14a8c/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Offline-logstorage-Fixed-extended-header-size-check-.patch
Type: application/octet-stream
Size: 1195 bytes
Desc: 0002-Offline-logstorage-Fixed-extended-header-size-check-.patch
URL: <http://lists.genivi.org/pipermail/genivi-diagnostic-log-and-trace_lists.genivi.org/attachments/20160415/dda14a8c/attachment-0001.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-Offline-logstorage-Refactor-filter-storage-functiona.patch
Type: application/octet-stream
Size: 12555 bytes
Desc: 0003-Offline-logstorage-Refactor-filter-storage-functiona.patch
URL: <http://lists.genivi.org/pipermail/genivi-diagnostic-log-and-trace_lists.genivi.org/attachments/20160415/dda14a8c/attachment-0002.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0004-Offline-logstorage-Fix-to-resetting-of-Syncbehavior-.patch
Type: application/octet-stream
Size: 1271 bytes
Desc: 0004-Offline-logstorage-Fix-to-resetting-of-Syncbehavior-.patch
URL: <http://lists.genivi.org/pipermail/genivi-diagnostic-log-and-trace_lists.genivi.org/attachments/20160415/dda14a8c/attachment-0003.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0005-Offline-logstorage-On-Demand-triggering-for-syncing-.patch
Type: application/octet-stream
Size: 31741 bytes
Desc: 0005-Offline-logstorage-On-Demand-triggering-for-syncing-.patch
URL: <http://lists.genivi.org/pipermail/genivi-diagnostic-log-and-trace_lists.genivi.org/attachments/20160415/dda14a8c/attachment-0004.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0006-MultiNode-Send-serialheader-if-specified-in-dlt.conf.patch
Type: application/octet-stream
Size: 1187 bytes
Desc: 0006-MultiNode-Send-serialheader-if-specified-in-dlt.conf.patch
URL: <http://lists.genivi.org/pipermail/genivi-diagnostic-log-and-trace_lists.genivi.org/attachments/20160415/dda14a8c/attachment-0005.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0007-MultiNode-Send-control-messages-after-connection.patch
Type: application/octet-stream
Size: 13791 bytes
Desc: 0007-MultiNode-Send-control-messages-after-connection.patch
URL: <http://lists.genivi.org/pipermail/genivi-diagnostic-log-and-trace_lists.genivi.org/attachments/20160415/dda14a8c/attachment-0006.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0008-MultiNode-Add-support-for-port-configuration.patch
Type: application/octet-stream
Size: 6254 bytes
Desc: 0008-MultiNode-Add-support-for-port-configuration.patch
URL: <http://lists.genivi.org/pipermail/genivi-diagnostic-log-and-trace_lists.genivi.org/attachments/20160415/dda14a8c/attachment-0007.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0009-MultiNode-Add-support-for-SerialHeader-conf.patch
Type: application/octet-stream
Size: 8234 bytes
Desc: 0009-MultiNode-Add-support-for-SerialHeader-conf.patch
URL: <http://lists.genivi.org/pipermail/genivi-diagnostic-log-and-trace_lists.genivi.org/attachments/20160415/dda14a8c/attachment-0008.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0010-MultiNode-Add-support-for-mandatory-configurations.patch
Type: application/octet-stream
Size: 19104 bytes
Desc: 0010-MultiNode-Add-support-for-mandatory-configurations.patch
URL: <http://lists.genivi.org/pipermail/genivi-diagnostic-log-and-trace_lists.genivi.org/attachments/20160415/dda14a8c/attachment-0009.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0011-Offline-trace-Make-search-more-precise.patch
Type: application/octet-stream
Size: 1187 bytes
Desc: 0011-Offline-trace-Make-search-more-precise.patch
URL: <http://lists.genivi.org/pipermail/genivi-diagnostic-log-and-trace_lists.genivi.org/attachments/20160415/dda14a8c/attachment-0010.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0012-Fix-connection-handling-of-serial-interface.patch
Type: application/octet-stream
Size: 1364 bytes
Desc: 0012-Fix-connection-handling-of-serial-interface.patch
URL: <http://lists.genivi.org/pipermail/genivi-diagnostic-log-and-trace_lists.genivi.org/attachments/20160415/dda14a8c/attachment-0011.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0013-dlt-daemon-receiver-rework.patch
Type: application/octet-stream
Size: 96136 bytes
Desc: 0013-dlt-daemon-receiver-rework.patch
URL: <http://lists.genivi.org/pipermail/genivi-diagnostic-log-and-trace_lists.genivi.org/attachments/20160415/dda14a8c/attachment-0012.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0014-dlt-daemon-Connection-activation-rework.patch
Type: application/octet-stream
Size: 11887 bytes
Desc: 0014-dlt-daemon-Connection-activation-rework.patch
URL: <http://lists.genivi.org/pipermail/genivi-diagnostic-log-and-trace_lists.genivi.org/attachments/20160415/dda14a8c/attachment-0013.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0015-dlt-daemon-Fix-user-log-handler-return-value.patch
Type: application/octet-stream
Size: 3849 bytes
Desc: 0015-dlt-daemon-Fix-user-log-handler-return-value.patch
URL: <http://lists.genivi.org/pipermail/genivi-diagnostic-log-and-trace_lists.genivi.org/attachments/20160415/dda14a8c/attachment-0014.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0016-DLT_PTR-User-macro-to-print-pointers.patch
Type: application/octet-stream
Size: 2169 bytes
Desc: 0016-DLT_PTR-User-macro-to-print-pointers.patch
URL: <http://lists.genivi.org/pipermail/genivi-diagnostic-log-and-trace_lists.genivi.org/attachments/20160415/dda14a8c/attachment-0015.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0017-dlt-control-Provision-to-control-entire-system-log-l.patch
Type: application/octet-stream
Size: 23777 bytes
Desc: 0017-dlt-control-Provision-to-control-entire-system-log-l.patch
URL: <http://lists.genivi.org/pipermail/genivi-diagnostic-log-and-trace_lists.genivi.org/attachments/20160415/dda14a8c/attachment-0016.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0018-CommonControl-Unix-socket-path-and-ecuid-parsing-for.patch
Type: application/octet-stream
Size: 10308 bytes
Desc: 0018-CommonControl-Unix-socket-path-and-ecuid-parsing-for.patch
URL: <http://lists.genivi.org/pipermail/genivi-diagnostic-log-and-trace_lists.genivi.org/attachments/20160415/dda14a8c/attachment-0017.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0019-CommonControl-Fix-for-commands-not-working-with-unix.patch
Type: application/octet-stream
Size: 6792 bytes
Desc: 0019-CommonControl-Fix-for-commands-not-working-with-unix.patch
URL: <http://lists.genivi.org/pipermail/genivi-diagnostic-log-and-trace_lists.genivi.org/attachments/20160415/dda14a8c/attachment-0018.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0020-dlt-daemon-Free-DltDaemon-structure-on-exit.patch
Type: application/octet-stream
Size: 1209 bytes
Desc: 0020-dlt-daemon-Free-DltDaemon-structure-on-exit.patch
URL: <http://lists.genivi.org/pipermail/genivi-diagnostic-log-and-trace_lists.genivi.org/attachments/20160415/dda14a8c/attachment-0019.patch>


More information about the genivi-diagnostic-log-and-trace mailing list