On Tue, May 08, 2018 at 04:37:52PM -0700, Matthew Macy wrote:
> Nix that. The panic is incorrect, we simply don't have anything to do
> in the default case.

Indeed, getting rid of the panic allows my workstation to boot.

> On Tue, May 8, 2018 at 4:31 PM, Matthew Macy <mm...@freebsd.org> wrote:
> > Can you please try the attached patch? (Note that gmail may insert
> > carriage returns)
> >
> > -M
> >
> > On Tue, May 8, 2018 at 3:10 PM, Mark Johnston <ma...@freebsd.org> wrote:
> >> On Tue, May 08, 2018 at 01:39:45AM +0000, Matt Macy wrote:
> >>> Author: mmacy
> >>> Date: Tue May  8 01:39:45 2018
> >>> New Revision: 333345
> >>> URL: https://svnweb.freebsd.org/changeset/base/333345
> >>>
> >>> Log:
> >>>   Sleep rather than spin in e1000 when doing long running config 
> >>> operations.
> >>>
> >>>   With r333218 it is now possible for drivers to use an sx lock and thus 
> >>> sleep while
> >>>   waiting on long running operations rather than DELAY().
> >>>
> >>>   Reported by:        gallatin
> >>>   Reviewed by:        sbruno
> >>>   Approved by:        sbruno
> >>>   MFC after:  1 month
> >>>   Sponsored by:       Limelight Networks
> >>>   Differential Revision:      https://reviews.freebsd.org/D14984
> >>
> >> I'm getting a panic during boot with this change: "unknown mac type d".
> >>
> >> It occurs while the driver is attaching to:
> >>
> >> em0@pci0:1:0:0: class=0x020000 card=0x7044103c chip=0x105e8086 rev=0x06 
> >> hdr=0x00
> >>     vendor     = 'Intel Corporation'
> >>     device     = '82571EB Gigabit Ethernet Controller'
> >>     class      = network
> >>     subclass   = ethernet

> diff --git a/sys/dev/e1000/e1000_82571.c b/sys/dev/e1000/e1000_82571.c
> index 57c47a5ebd6..348a3daa3c0 100644
> --- a/sys/dev/e1000/e1000_82571.c
> +++ b/sys/dev/e1000/e1000_82571.c
> @@ -1000,7 +1000,7 @@ static s32 e1000_reset_hw_82571(struct e1000_hw *hw)
>                       e1000_put_hw_semaphore_82574(hw);
>               break;
>       default:
> -             panic("unknown mac type %x\n", hw->mac.type);
> +             /* we didn't get the semaphore no need to put it */
>               break;
>       }
>  

_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to