lkml.org 
[lkml]   [2009]   [Jul]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 01/14] amd64_edac: simplify error type bits extractors
Hi Borislav,
> - /* If this was an 'observed' error, early out */
> - if (ec_pp == K8_NBSL_PP_OBS)
> - return; /* We aren't the node involved */
> +
> + /* Bail early out if this was an 'observed' error */
> + if (((ec >> 9) & 0x3) == K8_NBSL_PP_OBS)
> + return;
minor thing:
this patch and others makes me think if it wouldn't be better to have:
#define PP(x) (((x) >> 9) & 0x3)
#define PP_MSG(x) pp_msgs[PP(x)]
and the same for LL, TT, RRRR.

the rest of the patch series looks good to me

--
Aristeu



\
 
 \ /
  Last update: 2009-07-20 20:03    [W:0.153 / U:0.088 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site