Change in simtrace2[master]: sniff: increase debug output

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Harald Welte gerrit-no-reply at lists.osmocom.org
Sat Jul 28 21:21:53 UTC 2018


Harald Welte has posted comments on this change. ( https://gerrit.osmocom.org/10213 )

Change subject: sniff: increase debug output
......................................................................


Patch Set 2: Code-Review-1

(1 comment)

https://gerrit.osmocom.org/#/c/10213/2/firmware/libcommon/source/sniffer.c
File firmware/libcommon/source/sniffer.c:

https://gerrit.osmocom.org/#/c/10213/2/firmware/libcommon/source/sniffer.c@302
PS2, Line 302: don't print since this is function is also called by ISRs
is the function now no longer called by ISRs? If not, then remove the comment.  If it's still called from ISRs, enabling debug will likely create timing problems :/



-- 
To view, visit https://gerrit.osmocom.org/10213
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: simtrace2
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ife0bbcf4a25aaa445a36768c00004e6e0d9a4947
Gerrit-Change-Number: 10213
Gerrit-PatchSet: 2
Gerrit-Owner: Kévin Redon <kredon at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Comment-Date: Sat, 28 Jul 2018 21:21:53 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: Yes
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20180728/c5ef22de/attachment.htm>


More information about the gerrit-log mailing list