linux-kernel - Re: + kernelh-printk-panic-string-cleanup.patch added to -mm tree

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <a8e1da1002241755j3cc1738agdb414b9572f48e95@mail.gmail.com>
Date:	Thu, 25 Feb 2010 09:55:51 +0800
From:	Dave Young <hidave.darkstar@...il.com>
To:	Frederic Weisbecker <fweisbec@...il.com>
Cc:	akpm@...ux-foundation.org, Steven Rostedt <rostedt@...dmis.org>,
	mm-commits@...r.kernel.org, gregkh@...e.de,
	kosaki.motohiro@...fujitsu.com, lizf@...fujitsu.com, mingo@...e.hu,
	linux-kernel@...r.kernel.org
Subject: Re: + kernelh-printk-panic-string-cleanup.patch added to -mm tree

On Thu, Feb 25, 2010 at 5:34 AM, Frederic Weisbecker <fweisbec@...il.com> wrote:
> On Wed, Feb 24, 2010 at 01:07:15PM -0800, akpm@...ux-foundation.org wrote:
>>
>> The patch titled
>>      kernel.h: printk/panic/string cleanup
>> has been added to the -mm tree.  Its filename is
>>      kernelh-printk-panic-string-cleanup.patch
>>
>> Before you just go and hit "reply", please:
>>    a) Consider who else should be cc'ed
>>    b) Prefer to cc a suitable mailing list as well
>>    c) Ideally: find the original patch on the mailing list and do a
>>       reply-to-all to that, adding suitable additional cc's
>>
>> *** Remember to use Documentation/SubmitChecklist when testing your code ***
>>
>> See http://userweb.kernel.org/~akpm/stuff/added-to-mm.txt to find
>> out what to do about this
>>
>> The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/
>
>
>
> Andrew, please don't merge this patch.
>
> I'm not against the idea kernel.h gets split up a bit, by having
> a prink.h and panic.h included from kernel.h for example.
>
> But this patch breaks bisection: some tracing general helpers are
> removed here but re-integrated in a different header only in a subsequent
> patch in the series. This should be done in the same patch, otherwise it
> breaks the build in the middle of the set.

Yeah, my wrong, I will send a update version of this patch. Thank you.

>
> The other problem, as Steve noticed, is that the tracing prototypes are
> moved in ring_buffer.h (in the subsequent patch), which is not the right
> place as the ring buffer is not only for tracing purpose. And also
> it moves functions out of kernel.h while these are really useful for
> general purposes.
>
> That said, it's probably sane to think about a new header to put
> these tracing prototypes, if it's included by kernel.h,
> linux/ftrace.h is already filled with non-general purpose things.
> So may be linux/trace.h ? Yeah this could be confusing, Steve what
> do you think?
>
> And all in one, it would probably better to split this in three
> patches: one that moves printk helpers out of kernel.h to printk.h,
> another following the same pattern for panic things and another one
> for tracing things.

Fine to me.

>
> Thanks.
>
>



-- 
Regards
dave
--
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/