Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 17 Apr 2018 13:42:19 -0700
From:      John Baldwin <jhb@freebsd.org>
To:        Vitalij Satanivskij <satan@ukr.net>
Cc:        freebsd-current@freebsd.org, cem@freebsd.org, Stephen Hurd <shurd@llnw.com>, Matthew Macy <mmacy@nextbsd.org>, "freebsd-hackers@freebsd.org" <hackers@freebsd.org>, Stephen Hurd <shurd@freebsd.org>
Subject:   Re: Current panic on boot on H11DSI motherboard with epyc cpu (nexus_add_irq: failed)
Message-ID:  <3628282.XVdngBdGlp@ralph.baldwin.cx>
In-Reply-To: <20180417191553.GA95803@hell.ukr.net>
References:  <20180416102710.GA90028@hell.ukr.net> <3723755.0KMZDfyMWu@ralph.baldwin.cx> <20180417191553.GA95803@hell.ukr.net>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tuesday, April 17, 2018 10:15:53 PM Vitalij Satanivskij wrote:
> Dear John
> 
> I'm try patch with no success
> 
> http://hell.ukr.net/panic/recorder_patch165.webm
> 
> Also I'm enable verbose boot and record boot process (hpet was disabled so crash in another driver atach)
> http://hell.ukr.net/panic/recorder_patch_verbose.webm
> 
> root@test:/usr/src # svnlite diff
> Index: sys/x86/x86/msi.c
> ===================================================================
> --- sys/x86/x86/msi.c   (revision 332650)
> +++ sys/x86/x86/msi.c   (working copy)
> @@ -404,7 +404,7 @@
>         /* Do we need to create some new sources? */
>         if (cnt < count) {
>                 /* If we would exceed the max, give up. */
> -               if (i + (count - cnt) > FIRST_MSI_INT + NUM_MSI_INTS) {
> +               if (i + (count - cnt) >= FIRST_MSI_INT + NUM_MSI_INTS) {
>                         mtx_unlock(&msi_lock);
>                         free(mirqs, M_MSI);
>                         return (ENXIO);
> @@ -645,7 +645,7 @@
>         /* Do we need to create a new source? */
>         if (msi == NULL) {
>                 /* If we would exceed the max, give up. */
> -               if (i + 1 > FIRST_MSI_INT + NUM_MSI_INTS) {
> +               if (i + 1 >= FIRST_MSI_INT + NUM_MSI_INTS) {
>                         mtx_unlock(&msi_lock);
>                         return (ENXIO);
>                 }
> root@test:/usr/src
> 
> If you need any aditional information please tell me about. 

Can you perhaps turn off the stack trace on boot to not lose the panic messages
(remove KDB_TRACE from kernel config) and maybe modify the panic message to
include the IRQ number passed to nexus_add_irq?

-- 
John Baldwin



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?3628282.XVdngBdGlp>