Re: [RFC][PATCH] module: Limit line length of module prints
On 12/13/2015 05:06 PM, Rusty Russell wrote: Laura Abbott writes: On 12/11/2015 01:39 AM, Rusty Russell wrote: Laura Abbott writes: print_modules currently uses pr_cont to print all module information. This has the side effect of printing lots of modules on one very long line. This makes copy/pasting oopses more effort if manual wrapping is required. Place a reasonable limit (80 chars) on the number of modules on each line. Signed-off-by: Laura Abbott --- Does this bother anyone else or am I the only one who hates dealing with the long lines of "Modules linked in"? Never bothered me, but I'm a bit odd :) I worry more about the effect on machine parsing. Yes, that was a concern I had as well, but the module list seems to get wrapped eventually (although at a much longer length) so it seems like if machine parsing can handle one wrap it can handle multiple wraps. Does it? I find that code very hard to parse, but seems like something happens at 1024 chars. But my testing here doesn't show any such break in dmesg, nor on serial console. diff --git a/kernel/module.c b/kernel/module.c index 912e891e0e2f..f882d9d99844 100644 --- a/kernel/module.c +++ b/kernel/module.c @@ -3925,6 +3925,12 @@ static const struct file_operations proc_modules_operations = { static int __init proc_modules_init(void) { + int x; + printk("Test of long line:"); + for (x = 0; x < 1024; x++) + pr_cont("%c%c", (x % 26) + 'A', (x % 26) + 'A'); + pr_cont("\n"); + proc_create("modules", 0, NULL, _modules_operations); return 0; } Confused, Rusty. So this is what I see from an example warning: [ 51.432399] [ cut here ] [ 51.432406] WARNING: CPU: 2 PID: 77 at block/genhd.c:626 add_disk+0x480/0x4e0() [ 51.432407] Modules linked in: uas usb_storage scsi_dh_alua rfcomm fuse cmac ccm xt_CHECKSUM ipt_MASQUERADE nf_nat_masquerade_ipv4 tun nf_conntrack_netbios_ns nf_conntrack_broadcast ip6t_rpfilter ip6t_REJECT nf_reject_ipv6 xt_conntrack ebtable_filter ebtable_nat ebtable_broute bridge stp llc ebtables ip6table_mangle ip6table_nat nf_conntrack_ipv6 nf_defrag_ipv6 nf_nat_ipv6 ip6table_raw ip6table_security ip6table_filter ip6_tables iptable_mangle iptable_nat nf_conntrack_ipv4 nf_defrag_ipv4 nf_nat_ipv4 nf_nat nf_conntrack iptable_raw iptable_security bnep intel_rapl iTCO_wdt iTCO_vendor_support iosf_mbi x86_pkg_temp_thermal coretemp kvm_intel kvm irqbypass crct10dif_pclmul crc32_pclmul crc32c_intel arc4 uvcvideo iwlmvm videobuf2_vmalloc videobuf2_v4l2 btusb videobuf2_core mac80211 btrtl videobuf2_memops [ 51.432460] v4l2_common joydev btbcm videodev btintel iwlwifi snd_hda_codec_realtek media bluetooth cfg80211 snd_hda_codec_generic snd_hda_codec_hdmi lpc_ich intel_pch_thermal i2c_i801 shpchp snd_hda_intel snd_hda_codec snd_hda_core snd_hwdep wmi snd_seq thinkpad_acpi snd_seq_device rfkill snd_pcm mei_me snd_timer snd soundcore mei tpm_tis tpm nfsd auth_rpcgss nfs_acl lockd grace sunrpc i915 i2c_algo_bit drm_kms_helper serio_raw e1000e drm ptp pps_core video fjes [ 51.432501] CPU: 2 PID: 77 Comm: kworker/u16:1 Tainted: GW 4.4.0-0.rc4.git2.1.fc24.x86_64 #1 [ 51.432503] Hardware name: LENOVO 20BTS1N700/20BTS1N700, BIOS N14ET28W (1.06 ) 03/12/2015 [ 51.432507] Workqueue: events_unbound async_run_entry_fn [ 51.432509] 2902d154 88021f893c38 81427dc9 [ 51.432513] 88021f893c70 810add82 880203baa000 [ 51.432517] 880203baa0b0 880203baa00c 880215028000 880036108600 [ 51.432520] Call Trace: [ 51.432525] [] dump_stack+0x4b/0x72 [ 51.432529] [] warn_slowpath_common+0x82/0xc0 [ 51.432531] [] warn_slowpath_null+0x1a/0x20 [ 51.432534] [] add_disk+0x480/0x4e0 [ 51.432538] [] sd_probe_async+0x115/0x1d0 [ 51.432541] [] async_run_entry_fn+0x4a/0x140 [ 51.432544] [] process_one_work+0x230/0x6a0 [ 51.432546] [] ? process_one_work+0x199/0x6a0 [ 51.432549] [] worker_thread+0x4e/0x450 [ 51.432552] [] ? process_one_work+0x6a0/0x6a0 [ 51.432554] [] kthread+0x101/0x120 [ 51.432558] [] ? trace_hardirqs_on_caller+0x129/0x1b0 [ 51.432560] [] ? kthread_create_on_node+0x250/0x250 [ 51.432564] [] ret_from_fork+0x3f/0x70 [ 51.432566] [] ? kthread_create_on_node+0x250/0x250 [ 51.432568] ---[ end trace ea38cc3cfd710baf ]--- The modules are split over two lines judging by the time stamps but I'm not sure why. I don't see it when using pr_cont in the example. Thanks, Laura -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [RFC][PATCH] module: Limit line length of module prints
On 12/13/2015 05:06 PM, Rusty Russell wrote: Laura Abbottwrites: On 12/11/2015 01:39 AM, Rusty Russell wrote: Laura Abbott writes: print_modules currently uses pr_cont to print all module information. This has the side effect of printing lots of modules on one very long line. This makes copy/pasting oopses more effort if manual wrapping is required. Place a reasonable limit (80 chars) on the number of modules on each line. Signed-off-by: Laura Abbott --- Does this bother anyone else or am I the only one who hates dealing with the long lines of "Modules linked in"? Never bothered me, but I'm a bit odd :) I worry more about the effect on machine parsing. Yes, that was a concern I had as well, but the module list seems to get wrapped eventually (although at a much longer length) so it seems like if machine parsing can handle one wrap it can handle multiple wraps. Does it? I find that code very hard to parse, but seems like something happens at 1024 chars. But my testing here doesn't show any such break in dmesg, nor on serial console. diff --git a/kernel/module.c b/kernel/module.c index 912e891e0e2f..f882d9d99844 100644 --- a/kernel/module.c +++ b/kernel/module.c @@ -3925,6 +3925,12 @@ static const struct file_operations proc_modules_operations = { static int __init proc_modules_init(void) { + int x; + printk("Test of long line:"); + for (x = 0; x < 1024; x++) + pr_cont("%c%c", (x % 26) + 'A', (x % 26) + 'A'); + pr_cont("\n"); + proc_create("modules", 0, NULL, _modules_operations); return 0; } Confused, Rusty. So this is what I see from an example warning: [ 51.432399] [ cut here ] [ 51.432406] WARNING: CPU: 2 PID: 77 at block/genhd.c:626 add_disk+0x480/0x4e0() [ 51.432407] Modules linked in: uas usb_storage scsi_dh_alua rfcomm fuse cmac ccm xt_CHECKSUM ipt_MASQUERADE nf_nat_masquerade_ipv4 tun nf_conntrack_netbios_ns nf_conntrack_broadcast ip6t_rpfilter ip6t_REJECT nf_reject_ipv6 xt_conntrack ebtable_filter ebtable_nat ebtable_broute bridge stp llc ebtables ip6table_mangle ip6table_nat nf_conntrack_ipv6 nf_defrag_ipv6 nf_nat_ipv6 ip6table_raw ip6table_security ip6table_filter ip6_tables iptable_mangle iptable_nat nf_conntrack_ipv4 nf_defrag_ipv4 nf_nat_ipv4 nf_nat nf_conntrack iptable_raw iptable_security bnep intel_rapl iTCO_wdt iTCO_vendor_support iosf_mbi x86_pkg_temp_thermal coretemp kvm_intel kvm irqbypass crct10dif_pclmul crc32_pclmul crc32c_intel arc4 uvcvideo iwlmvm videobuf2_vmalloc videobuf2_v4l2 btusb videobuf2_core mac80211 btrtl videobuf2_memops [ 51.432460] v4l2_common joydev btbcm videodev btintel iwlwifi snd_hda_codec_realtek media bluetooth cfg80211 snd_hda_codec_generic snd_hda_codec_hdmi lpc_ich intel_pch_thermal i2c_i801 shpchp snd_hda_intel snd_hda_codec snd_hda_core snd_hwdep wmi snd_seq thinkpad_acpi snd_seq_device rfkill snd_pcm mei_me snd_timer snd soundcore mei tpm_tis tpm nfsd auth_rpcgss nfs_acl lockd grace sunrpc i915 i2c_algo_bit drm_kms_helper serio_raw e1000e drm ptp pps_core video fjes [ 51.432501] CPU: 2 PID: 77 Comm: kworker/u16:1 Tainted: GW 4.4.0-0.rc4.git2.1.fc24.x86_64 #1 [ 51.432503] Hardware name: LENOVO 20BTS1N700/20BTS1N700, BIOS N14ET28W (1.06 ) 03/12/2015 [ 51.432507] Workqueue: events_unbound async_run_entry_fn [ 51.432509] 2902d154 88021f893c38 81427dc9 [ 51.432513] 88021f893c70 810add82 880203baa000 [ 51.432517] 880203baa0b0 880203baa00c 880215028000 880036108600 [ 51.432520] Call Trace: [ 51.432525] [] dump_stack+0x4b/0x72 [ 51.432529] [] warn_slowpath_common+0x82/0xc0 [ 51.432531] [] warn_slowpath_null+0x1a/0x20 [ 51.432534] [] add_disk+0x480/0x4e0 [ 51.432538] [] sd_probe_async+0x115/0x1d0 [ 51.432541] [] async_run_entry_fn+0x4a/0x140 [ 51.432544] [] process_one_work+0x230/0x6a0 [ 51.432546] [] ? process_one_work+0x199/0x6a0 [ 51.432549] [] worker_thread+0x4e/0x450 [ 51.432552] [] ? process_one_work+0x6a0/0x6a0 [ 51.432554] [] kthread+0x101/0x120 [ 51.432558] [] ? trace_hardirqs_on_caller+0x129/0x1b0 [ 51.432560] [] ? kthread_create_on_node+0x250/0x250 [ 51.432564] [] ret_from_fork+0x3f/0x70 [ 51.432566] [] ? kthread_create_on_node+0x250/0x250 [ 51.432568] ---[ end trace ea38cc3cfd710baf ]--- The modules are split over two lines judging by the time stamps but I'm not sure why. I don't see it when using pr_cont in the example. Thanks, Laura -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [RFC][PATCH] module: Limit line length of module prints
Laura Abbott writes: > On 12/11/2015 01:39 AM, Rusty Russell wrote: >> Laura Abbott writes: >>> print_modules currently uses pr_cont to print all module information. >>> This has the side effect of printing lots of modules on one very long >>> line. This makes copy/pasting oopses more effort if manual wrapping is >>> required. Place a reasonable limit (80 chars) on the number of modules >>> on each line. >>> >>> Signed-off-by: Laura Abbott >>> --- >>> Does this bother anyone else or am I the only one who hates dealing >>> with the long lines of "Modules linked in"? >> >> Never bothered me, but I'm a bit odd :) I worry more about the effect >> on machine parsing. >> > > Yes, that was a concern I had as well, but the module list seems to get > wrapped eventually (although at a much longer length) so it seems like > if machine parsing can handle one wrap it can handle multiple wraps. Does it? I find that code very hard to parse, but seems like something happens at 1024 chars. But my testing here doesn't show any such break in dmesg, nor on serial console. diff --git a/kernel/module.c b/kernel/module.c index 912e891e0e2f..f882d9d99844 100644 --- a/kernel/module.c +++ b/kernel/module.c @@ -3925,6 +3925,12 @@ static const struct file_operations proc_modules_operations = { static int __init proc_modules_init(void) { + int x; + printk("Test of long line:"); + for (x = 0; x < 1024; x++) + pr_cont("%c%c", (x % 26) + 'A', (x % 26) + 'A'); + pr_cont("\n"); + proc_create("modules", 0, NULL, _modules_operations); return 0; } Confused, Rusty. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [RFC][PATCH] module: Limit line length of module prints
Laura Abbottwrites: > On 12/11/2015 01:39 AM, Rusty Russell wrote: >> Laura Abbott writes: >>> print_modules currently uses pr_cont to print all module information. >>> This has the side effect of printing lots of modules on one very long >>> line. This makes copy/pasting oopses more effort if manual wrapping is >>> required. Place a reasonable limit (80 chars) on the number of modules >>> on each line. >>> >>> Signed-off-by: Laura Abbott >>> --- >>> Does this bother anyone else or am I the only one who hates dealing >>> with the long lines of "Modules linked in"? >> >> Never bothered me, but I'm a bit odd :) I worry more about the effect >> on machine parsing. >> > > Yes, that was a concern I had as well, but the module list seems to get > wrapped eventually (although at a much longer length) so it seems like > if machine parsing can handle one wrap it can handle multiple wraps. Does it? I find that code very hard to parse, but seems like something happens at 1024 chars. But my testing here doesn't show any such break in dmesg, nor on serial console. diff --git a/kernel/module.c b/kernel/module.c index 912e891e0e2f..f882d9d99844 100644 --- a/kernel/module.c +++ b/kernel/module.c @@ -3925,6 +3925,12 @@ static const struct file_operations proc_modules_operations = { static int __init proc_modules_init(void) { + int x; + printk("Test of long line:"); + for (x = 0; x < 1024; x++) + pr_cont("%c%c", (x % 26) + 'A', (x % 26) + 'A'); + pr_cont("\n"); + proc_create("modules", 0, NULL, _modules_operations); return 0; } Confused, Rusty. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [RFC][PATCH] module: Limit line length of module prints
On 12/11/2015 01:39 AM, Rusty Russell wrote: Laura Abbott writes: print_modules currently uses pr_cont to print all module information. This has the side effect of printing lots of modules on one very long line. This makes copy/pasting oopses more effort if manual wrapping is required. Place a reasonable limit (80 chars) on the number of modules on each line. Signed-off-by: Laura Abbott --- Does this bother anyone else or am I the only one who hates dealing with the long lines of "Modules linked in"? Never bothered me, but I'm a bit odd :) I worry more about the effect on machine parsing. Yes, that was a concern I had as well, but the module list seems to get wrapped eventually (although at a much longer length) so it seems like if machine parsing can handle one wrap it can handle multiple wraps. Cheers, Rusty. Thanks, Laura -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [RFC][PATCH] module: Limit line length of module prints
On 12/11/2015 01:36 AM, Geyslan G. Bem wrote: Hello, What do you think of get the strlen of name and module_flags before printing them? So the check against MAX CHARS would be more precise and uniform. Or the limit is not strictly necessary? It's not really necessary. If it goes over a little bit that's okay. The goal here is to try and introduce some reasonable wrapping. Thanks, Laura -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [RFC][PATCH] module: Limit line length of module prints
Laura Abbott writes: > print_modules currently uses pr_cont to print all module information. > This has the side effect of printing lots of modules on one very long > line. This makes copy/pasting oopses more effort if manual wrapping is > required. Place a reasonable limit (80 chars) on the number of modules > on each line. > > Signed-off-by: Laura Abbott > --- > Does this bother anyone else or am I the only one who hates dealing > with the long lines of "Modules linked in"? Never bothered me, but I'm a bit odd :) I worry more about the effect on machine parsing. Cheers, Rusty. > --- > kernel/module.c | 11 ++- > 1 file changed, 10 insertions(+), 1 deletion(-) > > diff --git a/kernel/module.c b/kernel/module.c > index 8f051a1..ace82f1 100644 > --- a/kernel/module.c > +++ b/kernel/module.c > @@ -4059,11 +4059,14 @@ struct module *__module_text_address(unsigned long > addr) > } > EXPORT_SYMBOL_GPL(__module_text_address); > > +#define MAX_LINE_CHARS 80 > + > /* Don't grab lock, we're oopsing. */ > void print_modules(void) > { > struct module *mod; > char buf[8]; > + int cnt = 0; > > printk(KERN_DEFAULT "Modules linked in:"); > /* Most callers should already have preempt disabled, but make sure */ > @@ -4071,7 +4074,13 @@ void print_modules(void) > list_for_each_entry_rcu(mod, , list) { > if (mod->state == MODULE_STATE_UNFORMED) > continue; > - pr_cont(" %s%s", mod->name, module_flags(mod, buf)); > + > + if (cnt > MAX_LINE_CHARS) { > + cnt = 0; > + pr_cont("\n"); > + } > + > + cnt += pr_cont(" %s%s", mod->name, module_flags(mod, buf)); > } > preempt_enable(); > if (last_unloaded_module[0]) > -- > 2.5.0 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [RFC][PATCH] module: Limit line length of module prints
Laura Abbottwrites: > print_modules currently uses pr_cont to print all module information. > This has the side effect of printing lots of modules on one very long > line. This makes copy/pasting oopses more effort if manual wrapping is > required. Place a reasonable limit (80 chars) on the number of modules > on each line. > > Signed-off-by: Laura Abbott > --- > Does this bother anyone else or am I the only one who hates dealing > with the long lines of "Modules linked in"? Never bothered me, but I'm a bit odd :) I worry more about the effect on machine parsing. Cheers, Rusty. > --- > kernel/module.c | 11 ++- > 1 file changed, 10 insertions(+), 1 deletion(-) > > diff --git a/kernel/module.c b/kernel/module.c > index 8f051a1..ace82f1 100644 > --- a/kernel/module.c > +++ b/kernel/module.c > @@ -4059,11 +4059,14 @@ struct module *__module_text_address(unsigned long > addr) > } > EXPORT_SYMBOL_GPL(__module_text_address); > > +#define MAX_LINE_CHARS 80 > + > /* Don't grab lock, we're oopsing. */ > void print_modules(void) > { > struct module *mod; > char buf[8]; > + int cnt = 0; > > printk(KERN_DEFAULT "Modules linked in:"); > /* Most callers should already have preempt disabled, but make sure */ > @@ -4071,7 +4074,13 @@ void print_modules(void) > list_for_each_entry_rcu(mod, , list) { > if (mod->state == MODULE_STATE_UNFORMED) > continue; > - pr_cont(" %s%s", mod->name, module_flags(mod, buf)); > + > + if (cnt > MAX_LINE_CHARS) { > + cnt = 0; > + pr_cont("\n"); > + } > + > + cnt += pr_cont(" %s%s", mod->name, module_flags(mod, buf)); > } > preempt_enable(); > if (last_unloaded_module[0]) > -- > 2.5.0 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [RFC][PATCH] module: Limit line length of module prints
On 12/11/2015 01:39 AM, Rusty Russell wrote: Laura Abbottwrites: print_modules currently uses pr_cont to print all module information. This has the side effect of printing lots of modules on one very long line. This makes copy/pasting oopses more effort if manual wrapping is required. Place a reasonable limit (80 chars) on the number of modules on each line. Signed-off-by: Laura Abbott --- Does this bother anyone else or am I the only one who hates dealing with the long lines of "Modules linked in"? Never bothered me, but I'm a bit odd :) I worry more about the effect on machine parsing. Yes, that was a concern I had as well, but the module list seems to get wrapped eventually (although at a much longer length) so it seems like if machine parsing can handle one wrap it can handle multiple wraps. Cheers, Rusty. Thanks, Laura -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [RFC][PATCH] module: Limit line length of module prints
On 12/11/2015 01:36 AM, Geyslan G. Bem wrote: Hello, What do you think of get the strlen of name and module_flags before printing them? So the check against MAX CHARS would be more precise and uniform. Or the limit is not strictly necessary? It's not really necessary. If it goes over a little bit that's okay. The goal here is to try and introduce some reasonable wrapping. Thanks, Laura -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[RFC][PATCH] module: Limit line length of module prints
print_modules currently uses pr_cont to print all module information. This has the side effect of printing lots of modules on one very long line. This makes copy/pasting oopses more effort if manual wrapping is required. Place a reasonable limit (80 chars) on the number of modules on each line. Signed-off-by: Laura Abbott --- Does this bother anyone else or am I the only one who hates dealing with the long lines of "Modules linked in"? --- kernel/module.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/kernel/module.c b/kernel/module.c index 8f051a1..ace82f1 100644 --- a/kernel/module.c +++ b/kernel/module.c @@ -4059,11 +4059,14 @@ struct module *__module_text_address(unsigned long addr) } EXPORT_SYMBOL_GPL(__module_text_address); +#define MAX_LINE_CHARS 80 + /* Don't grab lock, we're oopsing. */ void print_modules(void) { struct module *mod; char buf[8]; + int cnt = 0; printk(KERN_DEFAULT "Modules linked in:"); /* Most callers should already have preempt disabled, but make sure */ @@ -4071,7 +4074,13 @@ void print_modules(void) list_for_each_entry_rcu(mod, , list) { if (mod->state == MODULE_STATE_UNFORMED) continue; - pr_cont(" %s%s", mod->name, module_flags(mod, buf)); + + if (cnt > MAX_LINE_CHARS) { + cnt = 0; + pr_cont("\n"); + } + + cnt += pr_cont(" %s%s", mod->name, module_flags(mod, buf)); } preempt_enable(); if (last_unloaded_module[0]) -- 2.5.0 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[RFC][PATCH] module: Limit line length of module prints
print_modules currently uses pr_cont to print all module information. This has the side effect of printing lots of modules on one very long line. This makes copy/pasting oopses more effort if manual wrapping is required. Place a reasonable limit (80 chars) on the number of modules on each line. Signed-off-by: Laura Abbott--- Does this bother anyone else or am I the only one who hates dealing with the long lines of "Modules linked in"? --- kernel/module.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/kernel/module.c b/kernel/module.c index 8f051a1..ace82f1 100644 --- a/kernel/module.c +++ b/kernel/module.c @@ -4059,11 +4059,14 @@ struct module *__module_text_address(unsigned long addr) } EXPORT_SYMBOL_GPL(__module_text_address); +#define MAX_LINE_CHARS 80 + /* Don't grab lock, we're oopsing. */ void print_modules(void) { struct module *mod; char buf[8]; + int cnt = 0; printk(KERN_DEFAULT "Modules linked in:"); /* Most callers should already have preempt disabled, but make sure */ @@ -4071,7 +4074,13 @@ void print_modules(void) list_for_each_entry_rcu(mod, , list) { if (mod->state == MODULE_STATE_UNFORMED) continue; - pr_cont(" %s%s", mod->name, module_flags(mod, buf)); + + if (cnt > MAX_LINE_CHARS) { + cnt = 0; + pr_cont("\n"); + } + + cnt += pr_cont(" %s%s", mod->name, module_flags(mod, buf)); } preempt_enable(); if (last_unloaded_module[0]) -- 2.5.0 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/