[PATCH] powerpc: Add MSR[CE, DE] to the MSR bits we print on show_regs()

Kumar Gala galak at kernel.crashing.org
Fri Nov 21 02:31:09 EST 2008


On Nov 19, 2008, at 10:46 PM, Benjamin Herrenschmidt wrote:

> On Wed, 2008-11-19 at 08:39 -0600, Kumar Gala wrote:
>> Signed-off-by: Kumar Gala <galak at kernel.crashing.org>
>> ---
>
> Shouldn't this be protected by some ifdef ?
>
> Ben.
>
>> arch/powerpc/kernel/process.c |    2 ++
>> 1 files changed, 2 insertions(+), 0 deletions(-)
>>
>> diff --git a/arch/powerpc/kernel/process.c b/arch/powerpc/kernel/ 
>> process.c
>> index 957bded..b038323 100644
>> --- a/arch/powerpc/kernel/process.c
>> +++ b/arch/powerpc/kernel/process.c
>> @@ -467,6 +467,8 @@ static struct regbit {
>> 	{MSR_VEC,	"VEC"},
>> 	{MSR_VSX,	"VSX"},
>> 	{MSR_ME,	"ME"},
>> +	{MSR_CE,	"CE"},
>> +	{MSR_DE,	"DE"},
>> 	{MSR_IR,	"IR"},
>> 	{MSR_DR,	"DR"},
>> 	{0,		NULL}

Why?  The bits are always defined and don't seem to be reused (and we  
don't ifdef VEC, VSX, etc..) so why bother?

- k



More information about the Linuxppc-dev mailing list