Review Request 98: Hardware: Added Processor Cache Memory Provider

Jan Safranek reviewboard at gallagherhome.com
Fri Apr 5 11:49:24 UTC 2013



> On April 5, 2013, 1:29 p.m., Tomas Smetana wrote:
> > src/hardware/dmidecode.c, lines 105-110
> > <http://reviewboard-openlmi.rhcloud.com/r/98/diff/1/?file=508#file508line105>
> >
> >     I think using '&&' would make the code more readable than the if nesting.

It wouldn't, 'goto done' is executed only if strdup fails, otherwise the function continues with cpu->l1_cache_handle=="".


- Jan


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/98/#review97
-----------------------------------------------------------


On April 4, 2013, 7:24 p.m., Peter Schiffer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard-openlmi.rhcloud.com/r/98/
> -----------------------------------------------------------
> 
> (Updated April 4, 2013, 7:24 p.m.)
> 
> 
> Review request for OpenLMI Developers.
> 
> 
> Repository: openlmi-providers
> 
> 
> Description
> -------
> 
> Hardware: Added Processor Cache Memory Provider
> 
> New Providers:
>     * LMI_ProcessorCacheMemoryProvider
>     * LMI_AssociatedProcessorCacheMemoryProvider
> 
> Other Changes:
>     * Optimized usage of string constants
>     * Fixed wrong usage of pointers in dmidecode.c
>     * Filled unknown mandatory fields in providers with "Unknown" value
>     * Replaced hard coded numbers with LMI constants
>     * Minor optimization - don't gather data which won't be used
> 
> (Sorry for unrelated changes in one patch. I'll be more cautious next time!)
> 
> 
> Diffs
> -----
> 
>   mof/LMI_Hardware.mof e655c012f542509a313e496a68ea7b85957aae6d 
>   src/hardware/CMakeLists.txt 352cebec53d9d6b877f81fd79a16219bb7e9c946 
>   src/hardware/LMI_AssociatedProcessorCacheMemoryProvider.c PRE-CREATION 
>   src/hardware/LMI_Hardware.h e93f9fe292e82d7761114425877efc7dfc38f3b9 
>   src/hardware/LMI_ProcessorCacheMemoryProvider.c PRE-CREATION 
>   src/hardware/LMI_ProcessorCapabilitiesProvider.c c9f7e14675de5c4eab104d02230e56afc0a4b094 
>   src/hardware/LMI_ProcessorElementCapabilitiesProvider.c a477717789026c8522415f7114cc2070eaca22e3 
>   src/hardware/LMI_ProcessorProvider.c c3b78e2974c9d2da3da8ff1a9595102c0c14d590 
>   src/hardware/dmidecode.h 9a8c6c8c4cf745edc313f92150272cc9a3661007 
>   src/hardware/dmidecode.c 14e4ac9fb21aa2467ac04c5830c9ba494c8cf80e 
>   src/hardware/sysfs.h PRE-CREATION 
>   src/hardware/sysfs.c PRE-CREATION 
> 
> Diff: http://reviewboard-openlmi.rhcloud.com/r/98/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Peter Schiffer
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.fedorahosted.org/pipermail/openlmi-devel/attachments/20130405/a3ea67e5/attachment.html>


More information about the openlmi-devel mailing list