Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

8870 do not lock cpu when waiting for IBF or OBF #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

procmonk
Copy link

commit 8ad8a2c4a4abef9af95f1033b18eecffe4860896
Author: jhb [email protected]
Date: Mon Dec 22 16:53:04 2014 +0000

Explicitly treat timeouts when waiting for IBF or OBF to change state as an
error.  This fixes occasional hangs in the IPMI kcs thread when using
ipmitool locally.

MFC after:  1 week

commit 8ad8a2c4a4abef9af95f1033b18eecffe4860896
Author: jhb <[email protected]>
Date:   Mon Dec 22 16:53:04 2014 +0000

    Explicitly treat timeouts when waiting for IBF or OBF to change state as an
    error.  This fixes occasional hangs in the IPMI kcs thread when using
    ipmitool locally.

    MFC after:  1 week
@danmcd
Copy link
Member

danmcd commented Nov 30, 2017

We generally don't do PRs directly, but if you're doing this to provide a place for review, that's fine.

@procmonk
Copy link
Author

procmonk commented Dec 1, 2017

For the moment, my only success is to panic my test node.
``

$C
ffffff00202e9010 lwp_kernel_create+0x31(0, fffffffffbb17680, ffffff0502a4b6a8, 2, 3c)
ffffff00202e9060 taskq_thread_create+0x62(ffffff0502a4b6a8)
ffffff00202e9110 taskq_create_common+0x1af(fffffffff81c5cb6, 0, 1, 3c, 1, 1, 0, 0, 40001)
ffffff00202e9170 taskq_create_proc+0x4d(fffffffff81c5cb6, 1, 3c, 1, 1, 0, ffffffff00000001)
ffffff00202e91b0 kcs_startup+0x5b(ffffffffc01abea0)
ffffff00202e9210 ipmi_startup+0x52(ffffffffc01abea0)
ffffff00202e9240 ipmi_attach+0x10f(ffffff04ff7eb008, 0)
ffffff00202e92b0 devi_attach+0x92(ffffff04ff7eb008, 0)
ffffff00202e92f0 attach_node+0xa7(ffffff04ff7eb008)
ffffff00202e9340 i_ndi_config_node+0x7d(ffffff04ff7eb008, 6, 0)
ffffff00202e9370 i_ddi_attachchild+0x48(ffffff04ff7eb008)
ffffff00202e93b0 devi_attach_node+0x5e(ffffff04ff7eb008, 4000)
ffffff00202e94b0 devi_config_one+0x294(ffffff04e59bcaa8, ffffff00202e97a0, ffffff00202e9588, 4000, 0)
ffffff00202e9520 ndi_devi_config_one+0xa8(ffffff04e59bcaa8, ffffff00202e97a0, ffffff00202e9588, 4000)
ffffff00202e9600 dv_find+0x233(ffffff04ea8cbdc8, ffffff00202e97a0, ffffff00202e98e8, ffffff00202e9a70,
ffffff04ea81cc00, ffffff04ff939460, ffffff0000000000)
ffffff00202e9650 devfs_lookup+0x37(ffffff04eab35c00, ffffff00202e97a0, ffffff00202e98e8, ffffff00202e9a70, 0,
ffffff04ea81cc00, ffffff04ff939460, 0, 0, 0)
ffffff00202e9700 fop_lookup+0xa2(ffffff04eab35c00, ffffff00202e97a0, ffffff00202e98e8, ffffff00202e9a70, 0,
ffffff04ea81cc00, ffffff04ff939460, 0, 0, 0)
ffffff00202e9950 lookuppnvp+0x1fe(ffffff00202e9a70, 0, 1, 0, ffffff00202e9c88, ffffff04ea81cc00, ffffff04ea81cc00,
ffffff04ff939460)
ffffff00202e99f0 lookuppnatcred+0x15e(ffffff00202e9a70, 0, 1, 0, ffffff00202e9c88, 0, ffffff04ff939460)
ffffff00202e9af0 lookupnameatcred+0xad(fe6b9430, 0, 1, 0, ffffff00202e9c88, 0, ffffff04ff939460)
ffffff00202e9b40 lookupnameat+0x39(fe6b9430, 0, 1, 0, ffffff00202e9c88, 0)
ffffff00202e9cf0 vn_openat+0x315(fe6b9430, 0, 3, 840, ffffff00202e9e00, 0, ffffff0000000012, 0, ffffff050000000c)
ffffff00202e9e60 copen+0x20c(ffd19553, fe6b9430, 3, 840)
ffffff00202e9e90 openat32+0x27(ffd19553, fe6b9430, 2, 840)
ffffff00202e9ec0 open32+0x25(fe6b9430, 2, 840)
ffffff00202e9f10 sys_syscall32+0xff()

``

@procmonk
Copy link
Author

procmonk commented Dec 1, 2017

Will build via the correct release, as you suggested by email.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants