[Rpm-maint] [PATCH] - Added support for ppc64p7 arch (Power7 and higher optimized)
pmatilai at laiskiainen.org
Thu Apr 12 09:48:09 UTC 2012
On 04/11/2012 12:58 PM, Phil Knirsch wrote:
> Attached is the updated version of the ppc64p7 support patch. I've got
> rid of the AUXV env reading and doing it properly now via reading and
> parsing /proc/self/auxv. Additionally the patch now checks if the
> AT_PLATFORM is power7 or higher via a sscanf(), allowing future power
> architectures to benefit from it as well.
The patch leaks file descriptors, the fd it opens is never closed.
In the cosmetics department you might want to split the long if with
sscanf() into two separate if's (or otherwise reformat) to avoid the
large line-wrap which is easily avoidable.
Other than that, looks okay to me.
- Panu -
More information about the Rpm-maint