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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Thu, 06 Aug 2009 09:22:20 +0900
From:	Hidetoshi Seto <seto.hidetoshi@...fujitsu.com>
To:	Andi Kleen <ak@...ux.intel.com>
CC:	Jin Dongming <jin.dongming@...css.fujitsu.com>,
	"H.Peter Anvin" <hpa@...or.com>,
	LKLM <linux-kernel@...r.kernel.org>, Ingo Molnar <mingo@...e.hu>,
	Huang Ying <ying.huang@...el.com>,
	Kenji Kaneshige <kaneshige.kenji@...fujitsu.com>
Subject: Re: [PATCH] x86, mce: cleanup initcall of debugfs for mce

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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/