On Wednesday 12 February 2014 05:13 PM, Kirill Tkhai wrote: > 12.02.2014, 15:29, "Allen Pais" <allen.pais@xxxxxxxxxx>: >>>>>> [ 1487.027884] I7: <rt_mutex_setprio+0x3c/0x2c0> >>>>>> [ 1487.027885] Call Trace: >>>>>> [ 1487.027887] [00000000004967dc] rt_mutex_setprio+0x3c/0x2c0 >>>>>> [ 1487.027892] [00000000004afe20] task_blocks_on_rt_mutex+0x180/0x200 >>>>>> [ 1487.027895] [0000000000819114] rt_spin_lock_slowlock+0x94/0x300 >>>>>> [ 1487.027897] [0000000000817ebc] __schedule+0x39c/0x53c >>>>>> [ 1487.027899] [00000000008185fc] schedule+0x1c/0xc0 >>>>>> [ 1487.027908] [000000000048fff4] smpboot_thread_fn+0x154/0x2e0 >>>>>> [ 1487.027913] [000000000048753c] kthread+0x7c/0xa0 >>>>>> [ 1487.027920] [00000000004060c4] ret_from_syscall+0x1c/0x2c >>>>>> [ 1487.027922] [0000000000000000] (null) >> >> Kirill, Well the change works. So far the machine is up and no stall or crashes >> with Hackbench. I'll run it for longer period and check. > > Ok, good. > > But I don't know is this the best fix. May we have to implement another optimization > for RT. No, unfortunately, the system hit a stall on about 8 cpu's. CPU: 31 PID: 28675 Comm: hackbench Tainted: G D W 3.10.24-rt22+ #13 [ 5725.097645] task: fffff80f929da8c0 ti: fffff80f8a4fc000 task.ti: fffff80f8a4fc000 [ 5725.097649] TSTATE: 0000000011001604 TPC: 0000000000671e54 TNPC: 0000000000671e58 Y: 00000000 Tainted: G D W TPC: <do_raw_spin_lock+0xb4/0x120> [ 5725.097657] g0: 0000000000671e4c g1: 00000000000000ff g2: 0000000002625010 g3: 0000000000000000 [ 5725.097661] g4: fffff80f929da8c0 g5: fffff80fd649c000 g6: fffff80f8a4fc000 g7: 0000000000000000 [ 5725.097664] o0: 0000000000000001 o1: 00000000009dfc00 o2: 0000000000000000 o3: 0000000000000000 [ 5725.097667] o4: 0000000000000002 o5: 0000000000000000 sp: fffff80f8a4fee21 ret_pc: 0000000000671e58 [ 5725.097671] RPC: <do_raw_spin_lock+0xb8/0x120> [ 5725.097675] l0: 000000000933b401 l1: 000000003b99d190 l2: 0000000000e25c00 l3: 0000000000000000 [ 5725.097678] l4: 0000000000000000 l5: 0000000000000000 l6: 0000000000000000 l7: fffff801001254c8 [ 5725.097682] i0: fffff80f89a367c8 i1: 0000000000878be4 i2: 0000000000000000 i3: 0000000000000000 [ 5725.097685] i4: 0000000000000002 i5: 0000000000000000 i6: fffff80f8a4feed1 i7: 0000000000879b14 [ 5725.097690] I7: <_raw_spin_lock+0x54/0x80> [ 5725.097692] Call Trace: [ 5725.097697] [0000000000879b14] _raw_spin_lock+0x54/0x80 [ 5725.097702] [0000000000878be4] rt_spin_lock_slowlock+0x24/0x340 [ 5725.097707] [00000000008790ac] rt_spin_lock+0xc/0x40 [ 5725.097712] [00000000008610bc] unix_stream_sendmsg+0x15c/0x380 [ 5725.097717] [00000000007ac114] sock_aio_write+0xf4/0x120 [ 5725.097722] [000000000055891c] do_sync_write+0x5c/0xa0 [ 5725.097727] [0000000000559e1c] vfs_write+0x15c/0x180 [ 5725.097732] [0000000000559ef8] SyS_write+0x38/0x80 [ 5725.097738] [0000000000406234] linux_sparc_syscall+0x34/0x44 This(above) on a few cpu's and this(below) on the other BUG: soft lockup - CPU#13 stuck for 22s! [hackbench:28701] [ 5728.378345] Modules linked in: binfmt_misc usb_storage ehci_pci ehci_hcd sg n2_rng rng_core ext4 jbd2 crc16 sr_mod mpt2sas scsi_transport_sas raid_class sunvnet sunvdc dm_mirror dm_region_hash dm_log dm_mod be2iscsi iscsi_boot_sysfs bnx2i cnic uio ipv6 cxgb4i cxgb4 cxgb3i libcxgbi cxgb3 mdio libiscsi_tcp libiscsi scsi_transport_iscsi [ 5728.378347] irq event stamp: 0 [ 5728.378350] hardirqs last enabled at (0): [< (null)>] (null) [ 5728.378356] hardirqs last disabled at (0): [<000000000045eb38>] copy_process+0x418/0x1080 [ 5728.378361] softirqs last enabled at (0): [<000000000045eb38>] copy_process+0x418/0x1080 [ 5728.378364] softirqs last disabled at (0): [< (null)>] (null) [ 5728.378368] CPU: 13 PID: 28701 Comm: hackbench Tainted: G D W 3.10.24-rt22+ #13 [ 5728.378371] task: fffff80f90efbb80 ti: fffff80f925ac000 task.ti: fffff80f925ac000 [ 5728.378374] TSTATE: 0000000011001604 TPC: 00000000004668b4 TNPC: 00000000004668b8 Y: 00000000 Tainted: G D W [ 5728.378378] TPC: <do_exit+0xb4/0xa40> [ 5728.378380] g0: 0000000000003f40 g1: 00000000000000ff g2: fffff80f90efbeb0 g3: 0000000000000002 [ 5728.378383] g4: fffff80f90efbb80 g5: fffff80fd1c9c000 g6: fffff80f925ac000 g7: 0000000000000000 [ 5728.378385] o0: fffff80f90efbb80 o1: fffff80f925ac400 o2: 000000000087a654 o3: 0000000000000000 [ 5728.378387] o4: 0000000000000000 o5: fffff80f925aff40 sp: fffff80fff98f671 ret_pc: 000000000046689c [ 5728.378390] RPC: <do_exit+0x9c/0xa40> [ 5728.378393] l0: fffff80f90efbb80 l1: 0000004480001603 l2: 000000000087a650 l3: 0000000000000400 [ 5728.378395] l4: 0000000000000000 l5: 0000000000000003 l6: 0000000000000000 l7: 0000000000000008 [ 5728.378397] i0: 000000000000000a i1: 000000000000000d i2: 000000000042f608 i3: 0000000000000000 [ 5728.378400] i4: 000000000000004f i5: 0000000000000002 i6: fffff80fff98f741 i7: 000000000087a650 [ 5728.378405] I7: <perfctr_irq+0x3d0/0x420> [ 5728.378406] Call Trace: [ 5728.378410] [000000000087a650] perfctr_irq+0x3d0/0x420 [ 5728.378415] [00000000004209f4] tl0_irq15+0x14/0x20 [ 5728.378419] [000000000042f608] stick_get_tick+0x8/0x20 [ 5728.378422] [000000000042fa24] __delay+0x24/0x60 [ 5728.378426] [0000000000671e58] do_raw_spin_lock+0xb8/0x120 [ 5728.378430] [0000000000879b14] _raw_spin_lock+0x54/0x80 [ 5728.378435] [00000000004a1978] load_balance+0x538/0x860 [ 5728.378438] [00000000004a2154] idle_balance+0x134/0x1c0 [ 5728.378442] [0000000000877d54] switch_to_pc+0x1f4/0x2c0 [ 5728.378445] [0000000000877ec4] schedule+0x24/0xc0 [ 5728.378449] [0000000000876860] schedule_timeout+0x1c0/0x2a0 [ 5728.378452] [0000000000860ac0] unix_stream_recvmsg+0x240/0x6e0 [ 5728.378456] [00000000007ac23c] sock_aio_read+0xfc/0x120 [ 5728.378460] [0000000000558adc] do_sync_read+0x5c/0xa0 [ 5728.378464] [000000000055a04c] vfs_read+0x10c/0x120 [ 5728.378467] [000000000055a118] SyS_read+0x38/0x80 > > For example, collect only batches which does not require smp call function. Or the > main goal of lazy tlb was to prevent smp calls?! It's good to discover this.. > > The other serious thing is to know does __set_pte_at() execute in preemption disable > context on !RT kernel. Because the place is interesting. > > If yes, we have to do the same for RT. If not, then no. I am not convinced that I've covered all tlb/smp code. Guess I'll need to dig more. Thanks, Allen -- To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
