-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5892/
-----------------------------------------------------------
(Updated 2010-11-24 23:30:04.365050)
Review request for Solid and Rafael Fernández López.
Changes
-------
Now with caching (and without the bug). Also, check that minFreq isn't 0 when
calculating canChangeFrequency() (0 means "unknown").
I'm not sure that caching is really worth the bother, since sysfs reads are
fast and I don't imagine clients will be repeatedly calling these methods. But
the caching code isn't exactly complicated. *shrugs*
Summary
-------
Implement support for all attributes of the Processor interface when using UDev.
With this patch, the udev backend does better than the hal one (notably, with
linux 2.6.36, HAL's reporting of the throttling ability of a processor will
break, as /proc/acpi/CPUX/info is removed in that kernel version; also, HAL
does not report the processor vendor).
It might be good to do some caching of values, since there is a lot of opening
and parsing files (although the opening and reading part should be fast on
virtual filesystems, and there isn't a whole lot of text to parse).
This addresses bug 253667.
https://bugs.kde.org/show_bug.cgi?id=253667
Diffs (updated)
-----
/trunk/KDE/kdelibs/solid/solid/CMakeLists.txt 1199450
/trunk/KDE/kdelibs/solid/solid/backends/shared/udevqt.h 1199450
/trunk/KDE/kdelibs/solid/solid/backends/shared/udevqtdevice.cpp 1199450
/trunk/KDE/kdelibs/solid/solid/backends/udev/cpuinfo.h PRE-CREATION
/trunk/KDE/kdelibs/solid/solid/backends/udev/cpuinfo.cpp PRE-CREATION
/trunk/KDE/kdelibs/solid/solid/backends/udev/udevdevice.h 1199450
/trunk/KDE/kdelibs/solid/solid/backends/udev/udevdevice.cpp 1199450
/trunk/KDE/kdelibs/solid/solid/backends/udev/udevmanager.cpp 1199450
/trunk/KDE/kdelibs/solid/solid/backends/udev/udevprocessor.h 1199450
/trunk/KDE/kdelibs/solid/solid/backends/udev/udevprocessor.cpp 1199450
Diff: http://svn.reviewboard.kde.org/r/5892/diff
Testing
-------
Reports information correctly on my computer.
Thanks,
Alex
_______________________________________________
Kde-hardware-devel mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kde-hardware-devel
