Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 74770 - speedtouch 1.3.1: lots of "Clp bit is ON" messages
Summary: speedtouch 1.3.1: lots of "Clp bit is ON" messages
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Unspecified (show other bugs)
Hardware: x86 Linux
: High normal (vote)
Assignee: Gentoo Dialup Developers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-12-17 12:43 UTC by Johan Verrept
Modified: 2004-12-22 13:46 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
patch to move message to verbose level 1 (speedtouch-1.3.1-no-clp-p1.patch,374 bytes, patch)
2004-12-17 12:44 UTC, Johan Verrept
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Johan Verrept 2004-12-17 12:43:50 UTC
Lots and lots of Clp bit is on messages.
When pppoa3 is using about 40% CPU, syslog is using almost 20%.
Has negative influence on bw.

Added patch moves the "Clip bit is ON" message to verbose level 1.

Reproducible: Always
Steps to Reproduce:
1.
2.
3.




the ISP controls the setting of this bit in the atm cells. This patch just makes
the driver ignore the bit iso printing a warning every time.
Comment 1 Johan Verrept 2004-12-17 12:44:40 UTC
Created attachment 46229 [details, diff]
patch to move message to verbose level 1
Comment 2 Alin Năstac (RETIRED) gentoo-dev 2004-12-18 07:15:04 UTC
I've increased verbosity level to 1 (using sed) on all report calls in atm.c.
Version 1.3.1-r1 commited with unchanged KEYWORDS since it is a trivial change that does not require testing.
Deleted any other versions since the last one is also the stable one.
Comment 3 Johan Verrept 2004-12-22 08:13:17 UTC
bad fix.

unpack function should be named src_unpack and not pkg_unpack

and 
   "sed -i -e 's/report(0/report(1/' src/atm.c"

should be 
   "sed -i -e 's/report(0/report(1/' ${S}/src/atm.c"
Comment 4 Alin Năstac (RETIRED) gentoo-dev 2004-12-22 13:46:22 UTC
r2 commited.

Sorry about that. I did test that sed in src_compile, but then I thought it isn't the right place for that so I put it in *_unpack.