If the *module* is really a nonmodule, IMHO it should add the information to
or add other macros macros instead of reuse the MODULE_*.
BTW, for macros in this file, for a nonmodule "license GPL" is unnecessary at
all.
I really fail to see what you have against machine-and-human readable data
On 01/16/14 at 02:46pm, Dave Young wrote:
> On 01/15/14 at 08:04pm, H. Peter Anvin wrote:
> > On 01/15/2014 07:03 PM, Dave Young wrote:
> > >>
> > >> making something harder to grep and less standardized is hardly cleaner
> > >> and these things compile to nothing for non-modules.
> > >
> > > It's
On 01/15/14 at 08:04pm, H. Peter Anvin wrote:
> On 01/15/2014 07:03 PM, Dave Young wrote:
> >>
> >> making something harder to grep and less standardized is hardly cleaner
> >> and these things compile to nothing for non-modules.
> >
> > It's not nothing, just very small increasement:
> >text
On 01/15/2014 07:03 PM, Dave Young wrote:
>>
>> making something harder to grep and less standardized is hardly cleaner
>> and these things compile to nothing for non-modules.
>
> It's not nothing, just very small increasement:
>text data bss dec hex filename
> 7636121
On 01/15/14 at 06:41pm, Arjan van de Ven wrote:
> On 1/15/2014 5:44 PM, Dave Young wrote:
> >On 01/15/14 at 06:11am, H. Peter Anvin wrote:
> >>On 01/14/2014 05:16 PM, Dave Young wrote:
>
> Why the [Finnish] do you feel that information needs to be in a
> different form because it is (c
On 1/15/2014 5:44 PM, Dave Young wrote:
On 01/15/14 at 06:11am, H. Peter Anvin wrote:
On 01/14/2014 05:16 PM, Dave Young wrote:
Why the [Finnish] do you feel that information needs to be in a
different form because it is (currently!) not available as a module?
I think moving them to comment
On 01/15/14 at 06:11am, H. Peter Anvin wrote:
> On 01/14/2014 05:16 PM, Dave Young wrote:
> >>
> >> Why the [Finnish] do you feel that information needs to be in a
> >> different form because it is (currently!) not available as a module?
> >
> > I think moving them to comment can avoid including e
On 01/14/2014 05:16 PM, Dave Young wrote:
>>
>> Why the [Finnish] do you feel that information needs to be in a
>> different form because it is (currently!) not available as a module?
>
> I think moving them to comment can avoid including extra linux/module.h
>
And that matters, why?
-h
On 01/14/14 at 10:18am, H. Peter Anvin wrote:
> On 01/13/2014 05:40 PM, Dave Young wrote:
> > On 01/13/14 at 06:48am, Arjan van de Ven wrote:
> >> On 1/13/2014 4:23 AM, Dave Young wrote:
>
> > How about do not limit to only if (pgd) case, instead do something
> > like below: set dump_t
On 01/13/2014 05:40 PM, Dave Young wrote:
> On 01/13/14 at 06:48am, Arjan van de Ven wrote:
>> On 1/13/2014 4:23 AM, Dave Young wrote:
> How about do not limit to only if (pgd) case, instead do something
> like below: set dump_to_dmesg as a module parameter
X86_PTDUMP is not
On 01/13/14 at 06:48am, Arjan van de Ven wrote:
> On 1/13/2014 4:23 AM, Dave Young wrote:
> >>
> >>>How about do not limit to only if (pgd) case, instead do something
> >>>like below: set dump_to_dmesg as a module parameter
> >>
> >>X86_PTDUMP is not a module.
> >
> >Hmm, I just see the module macr
On Sat, 11 Jan, at 09:49:27PM, Borislav Petkov wrote:
> From: Borislav Petkov
>
> With reusing the ->trampoline_pgd page table for mapping EFI regions in
> order to use them after having switched to EFI virtual mode, it is very
> useful to be able to dump aforementioned page table in dmesg. This
On Mon, Jan 13, 2014 at 06:48:10AM -0800, Arjan van de Ven wrote:
> >MODULE_LICENSE("GPL");
> >MODULE_AUTHOR("Arjan van de Ven ");
> >MODULE_DESCRIPTION("Kernel debugging helper that dumps pagetables");
>
> personally I consider it good form to always have this kind of
> information in .c files, re
On 1/13/2014 4:23 AM, Dave Young wrote:
How about do not limit to only if (pgd) case, instead do something
like below: set dump_to_dmesg as a module parameter
X86_PTDUMP is not a module.
Hmm, I just see the module macros in the code, since it's a bool Kconfig
I think the dump_pagetables.c n
>
> > How about do not limit to only if (pgd) case, instead do something
> > like below: set dump_to_dmesg as a module parameter
>
> X86_PTDUMP is not a module.
Hmm, I just see the module macros in the code, since it's a bool Kconfig
I think the dump_pagetables.c need a cleanup,
remove the #incl
On Mon, Jan 13, 2014 at 01:32:40PM +0800, Dave Young wrote:
> pr_info? This is for debug purpose, maybe pr_debug is better?
I hate pr_debug as it depends on other defines I have to have defined
properly in order just so I get output.
> How about do not limit to only if (pgd) case, instead do some
On 01/11/14 at 09:49pm, Borislav Petkov wrote:
> From: Borislav Petkov
>
> With reusing the ->trampoline_pgd page table for mapping EFI regions in
> order to use them after having switched to EFI virtual mode, it is very
> useful to be able to dump aforementioned page table in dmesg. This adds
>
On Sat, 2014-01-11 at 21:49 +0100, Borislav Petkov wrote:
> From: Borislav Petkov
>
> With reusing the ->trampoline_pgd page table for mapping EFI regions in
> order to use them after having switched to EFI virtual mode, it is very
> useful to be able to dump aforementioned page table in dmesg. T
From: Borislav Petkov
With reusing the ->trampoline_pgd page table for mapping EFI regions in
order to use them after having switched to EFI virtual mode, it is very
useful to be able to dump aforementioned page table in dmesg. This adds
that functionality through the walk_pgd_level() interface w
19 matches
Mail list logo