Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 8 May 2018 16:31:34 -0700
From:      Matthew Macy <mmacy@freebsd.org>
To:        Mark Johnston <markj@freebsd.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org,  svn-src-head@freebsd.org, Stephen Hurd <shurd@freebsd.org>
Subject:   Re: svn commit: r333345 - head/sys/dev/e1000
Message-ID:  <CAPrugNpu6XuR3sg3r1cjLGROmAh57%2BBmNVE0GkiG2Esk0rv_aA@mail.gmail.com>
In-Reply-To: <20180508221025.GA4249@raichu>
References:  <201805080139.w481djMX062724@repo.freebsd.org> <20180508221025.GA4249@raichu>

next in thread | previous in thread | raw e-mail | index | archive | help
--000000000000e2381a056bba2f42
Content-Type: text/plain; charset="UTF-8"

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 <markj@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

--000000000000e2381a056bba2f42
Content-Type: text/x-patch; charset="US-ASCII"; name="e1000_hack.diff"
Content-Disposition: attachment; filename="e1000_hack.diff"
Content-Transfer-Encoding: base64
X-Attachment-Id: f_jgyb6gb20

ZGlmZiAtLWdpdCBhL3N5cy9kZXYvZTEwMDAvZTEwMDBfbWFjLmMgYi9zeXMvZGV2L2UxMDAwL2Ux
MDAwX21hYy5jCmluZGV4IGRhNzkzNGY5ZmI2Li43OTI4M2NmMzI5NiAxMDA2NDQKLS0tIGEvc3lz
L2Rldi9lMTAwMC9lMTAwMF9tYWMuYworKysgYi9zeXMvZGV2L2UxMDAwL2UxMDAwX21hYy5jCkBA
IC0yMTk2LDcgKzIxOTYsNiBAQCBzMzIgZTEwMDBfZ2V0X2h3X3NlbWFwaG9yZShzdHJ1Y3QgZTEw
MDBfaHcgKmh3KQogCXMzMiBpID0gMDsKIAkKIAlERUJVR0ZVTkMoImUxMDAwX2dldF9od19zZW1h
cGhvcmUiKTsKLSNpZmRlZiBub3R5ZXQKIAkvKiBfODI1NzEgKi8KIAkvKiBJZiB3ZSBoYXZlIHRp
bWVkb3V0IDMgdGltZXMgb24gdHJ5aW5nIHRvIGFjcXVpcmUKIAkgKiB0aGUgaW50ZXItcG9ydCBT
TUJJIHNlbWFwaG9yZSwgdGhlcmUgaXMgb2xkIGNvZGUKQEAgLTIyMDgsNyArMjIwNyw2IEBAIHMz
MiBlMTAwMF9nZXRfaHdfc2VtYXBob3JlKHN0cnVjdCBlMTAwMF9odyAqaHcpCiAJaWYgKGh3LT5k
ZXZfc3BlYy5fODI1NzEuc21iX2NvdW50ZXIgPiAyKQogCQlzd190aW1lb3V0ID0gMTsKIAotI2Vu
ZGlmCiAJLyogR2V0IHRoZSBTVyBzZW1hcGhvcmUgKi8KIAl3aGlsZSAoaSA8IHRpbWVvdXQpIHsK
IAkJc3dzbSA9IEUxMDAwX1JFQURfUkVHKGh3LCBFMTAwMF9TV1NNKTsKQEAgLTIyMjAsNyArMjIx
OCw3IEBAIHMzMiBlMTAwMF9nZXRfaHdfc2VtYXBob3JlKHN0cnVjdCBlMTAwMF9odyAqaHcpCiAJ
fQogCiAJaWYgKGkgPT0gdGltZW91dCkgewotI2lmZGVmIG5vdHlldAorCiAJCS8qCiAJCSAqIFhY
WCBUaGlzIHNvdW5kcyBtb3JlIGxpa2UgYSBkcml2ZXIgYnVnIHdoZXJlYnkgd2UgZWl0aGVyCiAJ
CSAqIHJlY3Vyc2VkIGFjY2lkZW50YWxseSBvciBtaXNzZWQgY2xlYXJpbmcgaXQgcHJldmlvdXNs
eQpAQCAtMjIzOSw3ICsyMjM3LDYgQEAgczMyIGUxMDAwX2dldF9od19zZW1hcGhvcmUoc3RydWN0
IGUxMDAwX2h3ICpodykKICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICB1c2VjX2RlbGF5
KDUwKTsKICAgICAgICAgICAgICAgICAgICAgICAgfQogICAgICAgICAgICAgICAgfQotI2VuZGlm
CiAKIAkJREVCVUdPVVQoIkRyaXZlciBjYW4ndCBhY2Nlc3MgZGV2aWNlIC0gU01CSSBiaXQgaXMg
c2V0LlxuIik7CiAJCXJldHVybiAtRTEwMDBfRVJSX05WTTsK
--000000000000e2381a056bba2f42--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAPrugNpu6XuR3sg3r1cjLGROmAh57%2BBmNVE0GkiG2Esk0rv_aA>