diff for duplicates of <4F3E47F2.2090400@ti.com> diff --git a/a/1.txt b/N1/1.txt index f594d2b..a814a1e 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -122,3 +122,13 @@ index 4285daa..9834851 100644 -- 1.7.4.1 + + + +-------------- next part -------------- +A non-text attachment was scrubbed... +Name: 0001-ARM-smp_twd-Don-t-register-CPUFREQ-notifiers-if-loca.patch +Type: text/x-patch +Size: 4478 bytes +Desc: not available +URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20120217/e144ee6b/attachment.bin> diff --git a/a/2.hdr b/a/2.hdr deleted file mode 100644 index f8e2a41..0000000 --- a/a/2.hdr +++ /dev/null @@ -1,6 +0,0 @@ -Content-Type: text/x-patch; - name="0001-ARM-smp_twd-Don-t-register-CPUFREQ-notifiers-if-loca.patch" -Content-Transfer-Encoding: 7bit -Content-Disposition: attachment; - filename*0="0001-ARM-smp_twd-Don-t-register-CPUFREQ-notifiers-if-loca.pa"; - filename*1="tch" diff --git a/a/2.txt b/a/2.txt deleted file mode 100644 index b5817e6..0000000 --- a/a/2.txt +++ /dev/null @@ -1,90 +0,0 @@ ->From f58c515a9d1a2c729c05a726d5176843381952ae Mon Sep 17 00:00:00 2001 -From: Santosh Shilimkar <santosh.shilimkar@ti.com> -Date: Fri, 17 Feb 2012 11:11:28 +0530 -Subject: [PATCH] ARM: smp_twd: Don't register CPUFREQ notifiers if local timers are not initialised. - -Current ARM local timer code registers CPUFREQ notifiers even in case -the twd_timer_setup() isn't called. That seems to be wrong and -would eventually lead to kernel crash on the CPU frequency transitions -on the SOCs where the local timer doesn't exist or broken because of -hardware BUG. Fix it by testing twd_evt and *__this_cpu_ptr(twd_evt). - -The issue was observed with v3.3-rc3 and building an OMAP2+ kernel -on OMAP3 SOC which doesn't have TWD. - -Below is the dump for reference : - - Unable to handle kernel paging request at virtual address 007e900 - pgd = cdc20000 - [007e9000] *pgd=00000000 - Internal error: Oops: 5 [#1] SMP - Modules linked in: - CPU: 0 Not tainted (3.3.0-rc3-pm+debug+initramfs #9) - PC is at twd_update_frequency+0x34/0x48 - LR is at twd_update_frequency+0x10/0x48 - pc : [<c001382c>] lr : [<c0013808>] psr: 60000093 - sp : ce311dd8 ip : 00000000 fp : 00000000 - r10: 00000000 r9 : 00000001 r8 : ce310000 - r7 : c0440458 r6 : c00137f8 r5 : 00000000 r4 : c0947a74 - r3 : 00000000 r2 : 007e9000 r1 : 00000000 r0 : 00000000 - Flags: nZCv IRQs off FIQs on Mode SVC_32 ISA ARM Segment usr - Control: 10c5387d Table: 8dc20019 DAC: 00000015 - Process sh (pid: 599, stack limit = 0xce3102f8) - Stack: (0xce311dd8 to 0xce312000) - 1dc0: 6000c - 1de0: 00000001 00000002 00000000 00000000 00000000 00000000 00000 - 1e00: ffffffff c093d8f0 00000000 ce311ebc 00000001 00000001 ce310 - 1e20: c001386c c0437c4c c0e95b60 c0e95ba8 00000001 c0e95bf8 ffff4 - 1e40: 00000000 00000000 c005ef74 ce310000 c0435cf0 ce311ebc 00000 - 1e60: ce352b40 0007a120 c08d5108 c08ba040 c08ba040 c005f030 00000 - 1e80: c08bc554 c032fe2c 0007a120 c08d4b64 ce352b40 c08d8618 ffff8 - 1ea0: c08ba040 c033364c ce311ecc c0433b50 00000002 ffffffea c0330 - 1ec0: 0007a120 0007a120 22222201 00000000 22222222 00000000 ce357 - 1ee0: ce3d6000 cdc2aed8 ce352ba0 c0470164 00000002 c032f47c 00034 - 1f00: c0331cac ce352b40 00000007 c032f6d0 ce352bbc 0003d090 c0930 - 1f20: c093d8bc c03306a4 00000007 ce311f80 00000007 cdc2aec0 ce358 - 1f40: ce8d20c0 00000007 b6fe5000 ce311f80 00000007 ce310000 0000c - 1f60: c000de74 ce987400 ce8d20c0 b6fe5000 00000000 00000000 0000c - 1f80: 00000000 00000000 001fbac8 00000000 00000007 001fbac8 00004 - 1fa0: c000df04 c000dd60 00000007 001fbac8 00000001 b6fe5000 00000 - 1fc0: 00000007 001fbac8 00000007 00000004 b6fe5000 00000000 00202 - 1fe0: 00000000 beb565f8 00101ffc 00008e8c 60000010 00000001 00000 - [<c001382c>] (twd_update_frequency+0x34/0x48) from [<c008ac4c>] ) - [<c008ac4c>] (smp_call_function_single+0x17c/0x1c8) from [<c0013) - [<c0013890>] (twd_cpufreq_transition+0x24/0x30) from [<c0437c4c>) - [<c0437c4c>] (notifier_call_chain+0x44/0x84) from [<c005efe4>] () - [<c005efe4>] (__srcu_notifier_call_chain+0x70/0xa4) from [<c005f) - [<c005f030>] (srcu_notifier_call_chain+0x18/0x20) from [<c032fe2) - [<c032fe2c>] (cpufreq_notify_transition+0xc8/0x1b0) from [<c0333) - [<c033364c>] (omap_target+0x1b4/0x28c) from [<c032f47c>] (__cpuf) - [<c032f47c>] (__cpufreq_driver_target+0x50/0x64) from [<c0331d24) - [<c0331d24>] (cpufreq_set+0x78/0x98) from [<c032f6d0>] (store_sc) - [<c032f6d0>] (store_scaling_setspeed+0x5c/0x74) from [<c03306a4>) - [<c03306a4>] (store+0x58/0x74) from [<c014d868>] (sysfs_write_fi) - [<c014d868>] (sysfs_write_file+0x80/0xb4) from [<c00f2c2c>] (vfs) - [<c00f2c2c>] (vfs_write+0xa8/0x138) from [<c00f2e9c>] (sys_write) - [<c00f2e9c>] (sys_write+0x40/0x6c) from [<c000dd60>] (ret_fast_s) - Code: e594300c e792210c e1a01000 e5840004 (e7930002) - ---[ end trace 5da3b5167c1ecdda ]--- - -Reported-by: Kevin Hilman <khilman@ti.com> -Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com> ---- - arch/arm/kernel/smp_twd.c | 2 +- - 1 files changed, 1 insertions(+), 1 deletions(-) - -diff --git a/arch/arm/kernel/smp_twd.c b/arch/arm/kernel/smp_twd.c -index 4285daa..9834851 100644 ---- a/arch/arm/kernel/smp_twd.c -+++ b/arch/arm/kernel/smp_twd.c -@@ -129,7 +129,7 @@ static struct notifier_block twd_cpufreq_nb = { - - static int twd_cpufreq_init(void) - { -- if (!IS_ERR(twd_clk)) -+ if (twd_evt && *__this_cpu_ptr(twd_evt) && !IS_ERR(twd_clk)) - return cpufreq_register_notifier(&twd_cpufreq_nb, - CPUFREQ_TRANSITION_NOTIFIER); - --- -1.7.4.1 diff --git a/a/content_digest b/N1/content_digest index 2e0218a..a0e327b 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -3,14 +3,11 @@ "ref\00401fed02a58f7bc48cc274050adbac7@localhost\0" "ref\0CAMQu2gwW6ES14jB8E6snE4pQRKg1q2ZF0pBedBtuPxeYV+7cow@mail.gmail.com\0" "ref\04F3E34A3.4000307@arm.com\0" - "From\0Santosh Shilimkar <santosh.shilimkar@ti.com>\0" - "Subject\0Re: SMP local timers and their CPUfreq notifiers setup on OMAP3\0" + "From\0santosh.shilimkar@ti.com (Santosh Shilimkar)\0" + "Subject\0SMP local timers and their CPUfreq notifiers setup on OMAP3\0" "Date\0Fri, 17 Feb 2012 17:58:34 +0530\0" - "To\0Marc Zyngier <marc.zyngier@arm.com>\0" - "Cc\0Kevin Hilman <khilman@ti.com>" - linux-omap <linux-omap@vger.kernel.org> - " linux-arm-kernel@lists.infradead.org <linux-arm-kernel@lists.infradead.org>\0" - "\01:1\0" + "To\0linux-arm-kernel@lists.infradead.org\0" + "\00:1\0" "b\0" "On Friday 17 February 2012 04:36 PM, Marc Zyngier wrote:\n" "> On 17/02/12 10:40, Shilimkar, Santosh wrote:\n" @@ -135,99 +132,16 @@ " \t\t\tCPUFREQ_TRANSITION_NOTIFIER);\n" "\n" "-- \n" - 1.7.4.1 - "\01:2\0" - "fn\00001-ARM-smp_twd-Don-t-register-CPUFREQ-notifiers-if-loca.patch\0" - "b\0" - ">From f58c515a9d1a2c729c05a726d5176843381952ae Mon Sep 17 00:00:00 2001\n" - "From: Santosh Shilimkar <santosh.shilimkar@ti.com>\n" - "Date: Fri, 17 Feb 2012 11:11:28 +0530\n" - "Subject: [PATCH] ARM: smp_twd: Don't register CPUFREQ notifiers if local timers are not initialised.\n" + "1.7.4.1\n" "\n" - "Current ARM local timer code registers CPUFREQ notifiers even in case\n" - "the twd_timer_setup() isn't called. That seems to be wrong and\n" - "would eventually lead to kernel crash on the CPU frequency transitions\n" - "on the SOCs where the local timer doesn't exist or broken because of\n" - "hardware BUG. Fix it by testing twd_evt and *__this_cpu_ptr(twd_evt).\n" - "\n" - "The issue was observed with v3.3-rc3 and building an OMAP2+ kernel\n" - "on OMAP3 SOC which doesn't have TWD.\n" - "\n" - "Below is the dump for reference :\n" - "\n" - " Unable to handle kernel paging request at virtual address 007e900\n" - " pgd = cdc20000\n" - " [007e9000] *pgd=00000000\n" - " Internal error: Oops: 5 [#1] SMP\n" - " Modules linked in:\n" - " CPU: 0 Not tainted (3.3.0-rc3-pm+debug+initramfs #9)\n" - " PC is at twd_update_frequency+0x34/0x48\n" - " LR is at twd_update_frequency+0x10/0x48\n" - " pc : [<c001382c>] lr : [<c0013808>] psr: 60000093\n" - " sp : ce311dd8 ip : 00000000 fp : 00000000\n" - " r10: 00000000 r9 : 00000001 r8 : ce310000\n" - " r7 : c0440458 r6 : c00137f8 r5 : 00000000 r4 : c0947a74\n" - " r3 : 00000000 r2 : 007e9000 r1 : 00000000 r0 : 00000000\n" - " Flags: nZCv IRQs off FIQs on Mode SVC_32 ISA ARM Segment usr\n" - " Control: 10c5387d Table: 8dc20019 DAC: 00000015\n" - " Process sh (pid: 599, stack limit = 0xce3102f8)\n" - " Stack: (0xce311dd8 to 0xce312000)\n" - " 1dc0: 6000c\n" - " 1de0: 00000001 00000002 00000000 00000000 00000000 00000000 00000\n" - " 1e00: ffffffff c093d8f0 00000000 ce311ebc 00000001 00000001 ce310\n" - " 1e20: c001386c c0437c4c c0e95b60 c0e95ba8 00000001 c0e95bf8 ffff4\n" - " 1e40: 00000000 00000000 c005ef74 ce310000 c0435cf0 ce311ebc 00000\n" - " 1e60: ce352b40 0007a120 c08d5108 c08ba040 c08ba040 c005f030 00000\n" - " 1e80: c08bc554 c032fe2c 0007a120 c08d4b64 ce352b40 c08d8618 ffff8\n" - " 1ea0: c08ba040 c033364c ce311ecc c0433b50 00000002 ffffffea c0330\n" - " 1ec0: 0007a120 0007a120 22222201 00000000 22222222 00000000 ce357\n" - " 1ee0: ce3d6000 cdc2aed8 ce352ba0 c0470164 00000002 c032f47c 00034\n" - " 1f00: c0331cac ce352b40 00000007 c032f6d0 ce352bbc 0003d090 c0930\n" - " 1f20: c093d8bc c03306a4 00000007 ce311f80 00000007 cdc2aec0 ce358\n" - " 1f40: ce8d20c0 00000007 b6fe5000 ce311f80 00000007 ce310000 0000c\n" - " 1f60: c000de74 ce987400 ce8d20c0 b6fe5000 00000000 00000000 0000c\n" - " 1f80: 00000000 00000000 001fbac8 00000000 00000007 001fbac8 00004\n" - " 1fa0: c000df04 c000dd60 00000007 001fbac8 00000001 b6fe5000 00000\n" - " 1fc0: 00000007 001fbac8 00000007 00000004 b6fe5000 00000000 00202\n" - " 1fe0: 00000000 beb565f8 00101ffc 00008e8c 60000010 00000001 00000\n" - " [<c001382c>] (twd_update_frequency+0x34/0x48) from [<c008ac4c>] )\n" - " [<c008ac4c>] (smp_call_function_single+0x17c/0x1c8) from [<c0013)\n" - " [<c0013890>] (twd_cpufreq_transition+0x24/0x30) from [<c0437c4c>)\n" - " [<c0437c4c>] (notifier_call_chain+0x44/0x84) from [<c005efe4>] ()\n" - " [<c005efe4>] (__srcu_notifier_call_chain+0x70/0xa4) from [<c005f)\n" - " [<c005f030>] (srcu_notifier_call_chain+0x18/0x20) from [<c032fe2)\n" - " [<c032fe2c>] (cpufreq_notify_transition+0xc8/0x1b0) from [<c0333)\n" - " [<c033364c>] (omap_target+0x1b4/0x28c) from [<c032f47c>] (__cpuf)\n" - " [<c032f47c>] (__cpufreq_driver_target+0x50/0x64) from [<c0331d24)\n" - " [<c0331d24>] (cpufreq_set+0x78/0x98) from [<c032f6d0>] (store_sc)\n" - " [<c032f6d0>] (store_scaling_setspeed+0x5c/0x74) from [<c03306a4>)\n" - " [<c03306a4>] (store+0x58/0x74) from [<c014d868>] (sysfs_write_fi)\n" - " [<c014d868>] (sysfs_write_file+0x80/0xb4) from [<c00f2c2c>] (vfs)\n" - " [<c00f2c2c>] (vfs_write+0xa8/0x138) from [<c00f2e9c>] (sys_write)\n" - " [<c00f2e9c>] (sys_write+0x40/0x6c) from [<c000dd60>] (ret_fast_s)\n" - " Code: e594300c e792210c e1a01000 e5840004 (e7930002)\n" - " ---[ end trace 5da3b5167c1ecdda ]---\n" "\n" - "Reported-by: Kevin Hilman <khilman@ti.com>\n" - "Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com>\n" - "---\n" - " arch/arm/kernel/smp_twd.c | 2 +-\n" - " 1 files changed, 1 insertions(+), 1 deletions(-)\n" "\n" - "diff --git a/arch/arm/kernel/smp_twd.c b/arch/arm/kernel/smp_twd.c\n" - "index 4285daa..9834851 100644\n" - "--- a/arch/arm/kernel/smp_twd.c\n" - "+++ b/arch/arm/kernel/smp_twd.c\n" - "@@ -129,7 +129,7 @@ static struct notifier_block twd_cpufreq_nb = {\n" - " \n" - " static int twd_cpufreq_init(void)\n" - " {\n" - "-\tif (!IS_ERR(twd_clk))\n" - "+\t if (twd_evt && *__this_cpu_ptr(twd_evt) && !IS_ERR(twd_clk))\n" - " \t\treturn cpufreq_register_notifier(&twd_cpufreq_nb,\n" - " \t\t\tCPUFREQ_TRANSITION_NOTIFIER);\n" - " \n" - "-- \n" - 1.7.4.1 + "-------------- next part --------------\n" + "A non-text attachment was scrubbed...\n" + "Name: 0001-ARM-smp_twd-Don-t-register-CPUFREQ-notifiers-if-loca.patch\n" + "Type: text/x-patch\n" + "Size: 4478 bytes\n" + "Desc: not available\n" + URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20120217/e144ee6b/attachment.bin> -2f2f8e227d8673447cae6e70f69adb07a215d9a1712ec901af6abb89b41a849c +488d0c0c7f1b1f3e1606f1592513b8bc8817eccd49e5c8f91b68e831a551ab04
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.