Re: [PATCH] x86, mce: cleanup initcall of debugfs for mce

[Hidetoshi Seto]

Andi Kleen wrote:
> Jin Dongming wrote:
>> Andi Kleen wrote:
>>> Sorry but I think it's better to have an own initcall per file,
>>> that makes everything more self contained and easier to read.
>>>
>>> -Andi
>>>
>> Thank you for your comment.
>>
>> That's OK, but this patch is also used for resolving compile error
>> without CONFIG_DEBUG_FS.  Then how about this patch?
> 
> 
> Compiler errors should be always fixed of course. But did you check
> the compile error is still in the latest version of the x86/mce branch
> of
> git://git.kernel.org/pub/scm/linux/kernel/git/x86/linux-2.6-tip.git
> It might be fixed already.

The compiler error pointed by Jin will be introduced by Huang's patch:
 3.x86: MCE: Move debugfs mce dir creating to mce.c
   URL: http://marc.info/?l=linux-kernel&m=124900459100790&w=2
and it seems that none of tip branches have Huang's debug patch set yet.
At least I could not find them in today's tip/x86/mce and tip/master.
(Or I might have trouble on syncing of git.)

I think that the previous change to unify initcalls is worth to be in
an incremental patch, but this trivial change to add ifdef could be
done by updating original patch if you'd like to do so.


Thanks,
H.Seto

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/