----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/5892/#review8992 -----------------------------------------------------------
Ship it! Good to go IMO. - Kevin On 2010-11-25 22:34:50, Alex Merry wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://svn.reviewboard.kde.org/r/5892/ > ----------------------------------------------------------- > > (Updated 2010-11-25 22:34:50) > > > Review request for Solid and Rafael Fernández López. > > > 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 > ----- > > /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
