Re: kern/42736

[Paul Goyette <paul%whooppee.com@localhost>]

The following reply was made to PR kern/42736; it has been noted by GNATS.

From: Paul Goyette <paul%whooppee.com@localhost>
To: fair%netbsd.org@localhost, gnats-bugs%netbsd.org@localhost
Cc: 
Subject: Re: kern/42736
Date: Mon, 22 Feb 2010 09:42:00 -0800 (PST)

   This message is in MIME format.  The first part should be readable text,
   while the remaining parts are likely unreadable without MIME-aware tools.
 
 --0-2007850013-1266860520=:20770
 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed
 
 On Mon, 22 Feb 2010, Paul Goyette wrote:
 
 > Eric, can you provide the output of
 >
 >      sysctl -r hw.spdmem0.spd_data | hexdump
 
 Actually, this is not necessary.  I found the resolution of this PR in 
 the OpenBSD version of code (they fixed it about two years ago).
 
 Please try the attached patch.
 
 
 
 -------------------------------------------------------------------------
 |   Paul Goyette   | PGP DSS Key fingerprint: |  E-mail addresses:      |
 | Customer Service | FA29 0E3B 35AF E8AE 6651 |  paul at whooppee.com   |
 | Network Engineer | 0786 F758 55DE 53BA 7731 | pgoyette at juniper.net |
 | Kernel Developer |                          | pgoyette at netbsd.org  |
 -------------------------------------------------------------------------
 --0-2007850013-1266860520=:20770
 Content-Type: TEXT/PLAIN; charset=US-ASCII; name=spdmem.c.diff
 Content-Transfer-Encoding: BASE64
 Content-ID: <Pine.NEB.4.64.1002220942000.20770%quicky.whooppee.com@localhost>
 Content-Description: 
 Content-Disposition: attachment; filename=spdmem.c.diff
 
 SW5kZXg6IHNwZG1lbS5jDQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09DQpSQ1Mg
 ZmlsZTogL2N2c3Jvb3Qvc3JjL3N5cy9kZXYvaTJjL3NwZG1lbS5jLHYNCnJl
 dHJpZXZpbmcgcmV2aXNpb24gMS4xNQ0KZGlmZiAtdSAtcCAtcjEuMTUgc3Bk
 bWVtLmMNCi0tLSBzcGRtZW0uYwk5IE1heSAyMDA5IDE3OjMyOjI3IC0wMDAw
 CTEuMTUNCisrKyBzcGRtZW0uYwkyMiBGZWIgMjAxMCAxNzo0MDoxNyAtMDAw
 MA0KQEAgLTU2LDE0ICs1NiwxNSBAQCBzdGF0aWMgdWludDhfdCBzcGRtZW1f
 cmVhZChzdHJ1Y3Qgc3BkbWVtDQogLyogUm91dGluZXMgZm9yIGRlY29kaW5n
 IHNwZCBkYXRhICovDQogc3RhdGljIHZvaWQgZGVjb2RlX2Vkb2ZwbShjb25z
 dCBzdHJ1Y3Qgc3lzY3Rsbm9kZSAqLCBkZXZpY2VfdCwgc3RydWN0IHNwZG1l
 bSAqKTsNCiBzdGF0aWMgdm9pZCBkZWNvZGVfcm9tKGNvbnN0IHN0cnVjdCBz
 eXNjdGxub2RlICosIGRldmljZV90LCBzdHJ1Y3Qgc3BkbWVtICopOw0KLXN0
 YXRpYyB2b2lkIGRlY29kZV9zZHJhbShjb25zdCBzdHJ1Y3Qgc3lzY3Rsbm9k
 ZSAqLCBkZXZpY2VfdCwgc3RydWN0IHNwZG1lbSAqKTsNCitzdGF0aWMgdm9p
 ZCBkZWNvZGVfc2RyYW0oY29uc3Qgc3RydWN0IHN5c2N0bG5vZGUgKiwgZGV2
 aWNlX3QsIHN0cnVjdCBzcGRtZW0gKiwNCisJaW50KTsNCiBzdGF0aWMgdm9p
 ZCBkZWNvZGVfZGRyKGNvbnN0IHN0cnVjdCBzeXNjdGxub2RlICosIGRldmlj
 ZV90LCBzdHJ1Y3Qgc3BkbWVtICopOw0KIHN0YXRpYyB2b2lkIGRlY29kZV9k
 ZHIyKGNvbnN0IHN0cnVjdCBzeXNjdGxub2RlICosIGRldmljZV90LCBzdHJ1
 Y3Qgc3BkbWVtICopOw0KIHN0YXRpYyB2b2lkIGRlY29kZV9kZHIzKGNvbnN0
 IHN0cnVjdCBzeXNjdGxub2RlICosIGRldmljZV90LCBzdHJ1Y3Qgc3BkbWVt
 ICopOw0KIHN0YXRpYyB2b2lkIGRlY29kZV9mYmRpbW0oY29uc3Qgc3RydWN0
 IHN5c2N0bG5vZGUgKiwgZGV2aWNlX3QsIHN0cnVjdCBzcGRtZW0gKik7DQog
 DQogc3RhdGljIHZvaWQgZGVjb2RlX3NpemVfc3BlZWQoY29uc3Qgc3RydWN0
 IHN5c2N0bG5vZGUgKiwgaW50LCBpbnQsIGludCwgaW50LA0KLQkJCSAgICAg
 IGJvb2wsIGNvbnN0IGNoYXIgKik7DQorCQkJICAgICAgYm9vbCwgY29uc3Qg
 Y2hhciAqLCBpbnQpOw0KIHN0YXRpYyB2b2lkIGRlY29kZV92b2x0YWdlX3Jl
 ZnJlc2goZGV2aWNlX3QsIHN0cnVjdCBzcGRtZW0gKik7DQogDQogQ0ZBVFRB
 Q0hfREVDTF9ORVcoc3BkbWVtLCBzaXplb2Yoc3RydWN0IHNwZG1lbV9zb2Z0
 YyksDQpAQCAtMzkwLDcgKzM5MSw4IEBAIHNwZG1lbV9hdHRhY2goZGV2aWNl
 X3QgcGFyZW50LCBkZXZpY2VfdCANCiAJCWRlY29kZV9yb20obm9kZSwgc2Vs
 Ziwgcyk7DQogCQlicmVhazsNCiAJY2FzZSBTUERNRU1fTUVNVFlQRV9TRFJB
 TToNCi0JCWRlY29kZV9zZHJhbShub2RlLCBzZWxmLCBzKTsNCisJCWRlY29k
 ZV9zZHJhbShub2RlLCBzZWxmLCBzLA0KKwkJCShzcGRfbGVuID49IDEyOCkg
 PyAoKHVpbnQ4X3QgKilzKVsxMjZdIDogMCk7DQogCQlicmVhazsNCiAJY2Fz
 ZSBTUERNRU1fTUVNVFlQRV9ERFJTRFJBTToNCiAJCWRlY29kZV9kZHIobm9k
 ZSwgc2VsZiwgcyk7DQpAQCAtNDM3LDcgKzQzOSw4IEBAIFNZU0NUTF9TRVRV
 UChzeXNjdGxfc3BkbWVtX3NldHVwLCAic3lzY3QNCiANCiBzdGF0aWMgdm9p
 ZA0KIGRlY29kZV9zaXplX3NwZWVkKGNvbnN0IHN0cnVjdCBzeXNjdGxub2Rl
 ICpub2RlLCBpbnQgZGltbV9zaXplLCBpbnQgY3ljbGVfdGltZSwNCi0JCSAg
 aW50IGRfY2xrLCBpbnQgYml0cywgYm9vbCByb3VuZCwgY29uc3QgY2hhciAq
 ZGRyX3R5cGVfc3RyaW5nKQ0KKwkJICBpbnQgZF9jbGssIGludCBiaXRzLCBi
 b29sIHJvdW5kLCBjb25zdCBjaGFyICpkZHJfdHlwZV9zdHJpbmcsDQorCQkg
 IGludCBzcGVlZCkNCiB7DQogCWludCBwX2NsazsNCiANCkBAIC00NzEsNiAr
 NDc0LDggQEAgZGVjb2RlX3NpemVfc3BlZWQoY29uc3Qgc3RydWN0IHN5c2N0
 bG5vZA0KIAkJCXBfY2xrICs9IDUwOw0KIAkJcF9jbGsgLT0gcF9jbGsgJSAx
 MDA7DQogCX0NCisJaWYgKHNwZWVkKQ0KKwkJcF9jbGsgPSBzcGVlZDsNCiAJ
 YXByaW50X25vcm1hbCgiLCAlZE1IeiAoJXMtJWQpXG4iLA0KIAkJICAgICAg
 ZF9jbGssIGRkcl90eXBlX3N0cmluZywgcF9jbGspOw0KIAlpZiAobm9kZSAh
 PSBOVUxMKQ0KQEAgLTUxOSw4ICs1MjQsOSBAQCBkZWNvZGVfcm9tKGNvbnN0
 IHN0cnVjdCBzeXNjdGxub2RlICpub2RlDQogfQ0KIA0KIHN0YXRpYyB2b2lk
 DQotZGVjb2RlX3NkcmFtKGNvbnN0IHN0cnVjdCBzeXNjdGxub2RlICpub2Rl
 LCBkZXZpY2VfdCBzZWxmLCBzdHJ1Y3Qgc3BkbWVtICpzKSB7DQotCWludCBk
 aW1tX3NpemUsIGN5Y2xlX3RpbWUsIGJpdHMsIHRBQSwgaTsNCitkZWNvZGVf
 c2RyYW0oY29uc3Qgc3RydWN0IHN5c2N0bG5vZGUgKm5vZGUsIGRldmljZV90
 IHNlbGYsIHN0cnVjdCBzcGRtZW0gKnMsDQorCSAgICAgaW50IGZyZXEpIHsN
 CisJaW50IGRpbW1fc2l6ZSwgY3ljbGVfdGltZSwgYml0cywgdEFBLCBpLCBz
 cGVlZDsNCiANCiAJYXByaW50X25vcm1hbCgiJXMsICVzLCAiLA0KIAkJKHMt
 PnNtX3Nkci5zZHJfbW9kX2F0dHJzICYgU1BETUVNX1NEUl9NQVNLX1JFRyk/
 DQpAQCAtNTM2LDcgKzU0MiwyMiBAQCBkZWNvZGVfc2RyYW0oY29uc3Qgc3Ry
 dWN0IHN5c2N0bG5vZGUgKm5vDQogCWJpdHMgPSBsZTE2dG9oKHMtPnNtX3Nk
 ci5zZHJfZGF0YXdpZHRoKTsNCiAJaWYgKHMtPnNtX2NvbmZpZyA9PSAxIHx8
 IHMtPnNtX2NvbmZpZyA9PSAyKQ0KIAkJYml0cyAtPSA4Ow0KLQlkZWNvZGVf
 c2l6ZV9zcGVlZChub2RlLCBkaW1tX3NpemUsIGN5Y2xlX3RpbWUsIDEsIGJp
 dHMsIFRSVUUsICJQQyIpOw0KKw0KKwkvKiBDYWxjdWxhdGUgc3BlZWQgaGVy
 ZSAtIGZyb20gT3BlbkJTRCAqLw0KKwlzd2l0Y2ggKGZyZXEpIHsNCisJICAg
 IGNhc2UgMTAwOg0KKwkgICAgY2FzZSAxMzM6DQorCQlpZiAoY3ljbGVfdGlt
 ZSA8IDgwMDApDQorCQkJc3BlZWQgPSAxMzM7DQorCQllbHNlDQorCQkJc3Bl
 ZWQgPSAxMDA7DQorCQlicmVhazsNCisJICAgIGNhc2UgNjY6DQorCSAgICBk
 ZWZhdWx0Og0KKwkJc3BlZWQgPSA2NjsNCisJfQ0KKwlkZWNvZGVfc2l6ZV9z
 cGVlZChub2RlLCBkaW1tX3NpemUsIGN5Y2xlX3RpbWUsIDEsIGJpdHMsIEZB
 TFNFLCAiUEMiLA0KKwkJCSAgc3BlZWQpOw0KIA0KIAlhcHJpbnRfdmVyYm9z
 ZV9kZXYoc2VsZiwNCiAJICAgICIlZCByb3dzLCAlZCBjb2xzLCAlZCBiYW5r
 cywgJWQgYmFua3MvY2hpcCwgJWQuJWRucyBjeWNsZSB0aW1lXG4iLA0KQEAg
 LTU3Myw3ICs1OTQsNyBAQCBkZWNvZGVfZGRyKGNvbnN0IHN0cnVjdCBzeXNj
 dGxub2RlICpub2RlDQogCWJpdHMgPSBsZTE2dG9oKHMtPnNtX2Rkci5kZHJf
 ZGF0YXdpZHRoKTsNCiAJaWYgKHMtPnNtX2NvbmZpZyA9PSAxIHx8IHMtPnNt
 X2NvbmZpZyA9PSAyKQ0KIAkJYml0cyAtPSA4Ow0KLQlkZWNvZGVfc2l6ZV9z
 cGVlZChub2RlLCBkaW1tX3NpemUsIGN5Y2xlX3RpbWUsIDIsIGJpdHMsIFRS
 VUUsICJQQyIpOw0KKwlkZWNvZGVfc2l6ZV9zcGVlZChub2RlLCBkaW1tX3Np
 emUsIGN5Y2xlX3RpbWUsIDIsIGJpdHMsIFRSVUUsICJQQyIsIDApOw0KIA0K
 IAlhcHJpbnRfdmVyYm9zZV9kZXYoc2VsZiwNCiAJICAgICIlZCByb3dzLCAl
 ZCBjb2xzLCAlZCByYW5rcywgJWQgYmFua3MvY2hpcCwgJWQuJWRucyBjeWNs
 ZSB0aW1lXG4iLA0KQEAgLTYxNyw3ICs2MzgsNyBAQCBkZWNvZGVfZGRyMihj
 b25zdCBzdHJ1Y3Qgc3lzY3Rsbm9kZSAqbm9kDQogCWJpdHMgPSBzLT5zbV9k
 ZHIyLmRkcjJfZGF0YXdpZHRoOw0KIAlpZiAoKHMtPnNtX2NvbmZpZyAmIDB4
 MDMpICE9IDApDQogCQliaXRzIC09IDg7DQotCWRlY29kZV9zaXplX3NwZWVk
 KG5vZGUsIGRpbW1fc2l6ZSwgY3ljbGVfdGltZSwgMiwgYml0cywgVFJVRSwg
 IlBDMiIpOw0KKwlkZWNvZGVfc2l6ZV9zcGVlZChub2RlLCBkaW1tX3NpemUs
 IGN5Y2xlX3RpbWUsIDIsIGJpdHMsIFRSVUUsICJQQzIiLCAwKTsNCiANCiAJ
 YXByaW50X3ZlcmJvc2VfZGV2KHNlbGYsDQogCSAgICAiJWQgcm93cywgJWQg
 Y29scywgJWQgcmFua3MsICVkIGJhbmtzL2NoaXAsICVkLiUwMmRucyBjeWNs
 ZSB0aW1lXG4iLA0KQEAgLTY3MSw3ICs2OTIsNyBAQCBkZWNvZGVfZGRyMyhj
 b25zdCBzdHJ1Y3Qgc3lzY3Rsbm9kZSAqbm9kDQogCQkgICAgIHMtPnNtX2Rk
 cjMuZGRyM19tdGJfZGl2aXNvcjsNCiAJY3ljbGVfdGltZSAqPSBzLT5zbV9k
 ZHIzLmRkcjNfdENLbWluOw0KIAliaXRzID0gMSA8PCAocy0+c21fZGRyMy5k
 ZHIzX2RhdGF3aWR0aCArIDMpOw0KLQlkZWNvZGVfc2l6ZV9zcGVlZChub2Rl
 LCBkaW1tX3NpemUsIGN5Y2xlX3RpbWUsIDIsIGJpdHMsIEZBTFNFLCAiUEMz
 Iik7DQorCWRlY29kZV9zaXplX3NwZWVkKG5vZGUsIGRpbW1fc2l6ZSwgY3lj
 bGVfdGltZSwgMiwgYml0cywgRkFMU0UsICJQQzMiLCAwKTsNCiANCiAJYXBy
 aW50X3ZlcmJvc2VfZGV2KHNlbGYsDQogCSAgICAiJWQgcm93cywgJWQgY29s
 cywgJWQgbG9nLiBiYW5rcywgJWQgcGh5cy4gYmFua3MsICINCkBAIC03MDYs
 NyArNzI3LDcgQEAgZGVjb2RlX2ZiZGltbShjb25zdCBzdHJ1Y3Qgc3lzY3Rs
 bm9kZSAqbg0KIAkJCSAgICAocy0+c21fZmJkLmZiZGltbV9tdGJfZGl2aXNv
 ciAvIDIpKSAvDQogCQkgICAgIHMtPnNtX2ZiZC5mYmRpbW1fbXRiX2Rpdmlz
 b3I7DQogCWJpdHMgPSAxIDw8IChzLT5zbV9mYmQuZmJkaW1tX2Rldl93aWR0
 aCArIDIpOw0KLQlkZWNvZGVfc2l6ZV9zcGVlZChub2RlLCBkaW1tX3NpemUs
 IGN5Y2xlX3RpbWUsIDIsIGJpdHMsIFRSVUUsICJQQzIiKTsNCisJZGVjb2Rl
 X3NpemVfc3BlZWQobm9kZSwgZGltbV9zaXplLCBjeWNsZV90aW1lLCAyLCBi
 aXRzLCBUUlVFLCAiUEMyIiwgMCk7DQogDQogCWFwcmludF92ZXJib3NlX2Rl
 dihzZWxmLA0KIAkgICAgIiVkIHJvd3MsICVkIGNvbHMsICVkIGJhbmtzLCAl
 ZC4lMDJkbnMgY3ljbGUgdGltZVxuIiwNCg==
 
 --0-2007850013-1266860520=:20770--