[PATCH v2 0/12] add ubus support to ltq-[v|a]dsl-app

Martin Schiller ms at dev.tdt.de
Wed Jan 13 09:00:27 EST 2021


On 2021-01-09 09:40, Andre Heider wrote:
> Hi,
> 
> (cc'ed some recent commiters for lantiq)
> 
> On 15/12/2020 10:35, Andre Heider wrote:
>> v2:
>> - drop 0002-ltq-vdsl-app-fix-Wundef-warnings.patch
>> - use "/dev/dsl_cpe_api" without the "0" suffix for the adsl daemon:
>>    package/kernel/lantiq/ltq-adsl/patches/100-dsl_compat.patch:+   
>> device_create(dsl_class, NULL, MKDEV(DRV_DSL_CPE_API_DEV_MAJOR, 0), 
>> NULL, "dsl_cpe_api");
>>    package/kernel/lantiq/ltq-vdsl/patches/100-compat.patch:+   
>> device_create(dsl_class, NULL, dsl_devt, NULL, "dsl_cpe_api0");
>> - use callDSLMetrics() for luci, per jo
>> - add Tested-by tags
>> 
>> This is to significantly speed up the generation of the metrics.
>> 
>> The motivation comes from the fact that ~2.6s is just way too
>> ineffcient for interval based metric collectors like prometheus or
>> collectd.
>> 
>> The luci status page also loads/refreshes alot faster.
>> 
>> $ time /etc/init.d/dsl_control dslstat
>> real	0m 2.66s
>> user	0m 0.90s
>> sys	0m 1.76s
>> 
>> $ time ubus call dsl metrics
>> real	0m 0.02s
>> user	0m 0.00s
>> sys	0m 0.01s
>> 
>> The ltq-adsl-app changes are only compile time tested.
> 
> who do I need to convince to review this? ;)

I would also like this great work to be merged.

Martin



More information about the openwrt-devel mailing list