Re: kern/50453

[NONAKA Kimihiro <nonakap%gmail.com@localhost>]

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

From: NONAKA Kimihiro <nonakap%gmail.com@localhost>
To: "gnats-bugs%netbsd.org@localhost" <gnats-bugs%netbsd.org@localhost>
Cc: kern-bug-people%netbsd.org@localhost, 
	"gnats-admin%netbsd.org@localhost" <gnats-admin%netbsd.org@localhost>, "netbsd-bugs%netbsd.org@localhost" <netbsd-bugs%netbsd.org@localhost>, 
	Patrick Welche <prlw1%cam.ac.uk@localhost>
Subject: Re: kern/50453
Date: Tue, 2 Feb 2016 22:49:44 +0900

 --001a1140f9882763cc052ac9c640
 Content-Type: text/plain; charset=UTF-8
 
 Please test the attached patch.
 
 2016-01-29 20:50 GMT+09:00 Patrick Welche <prlw1%cam.ac.uk@localhost>:
 > The following reply was made to PR kern/50453; it has been noted by GNATS.
 >
 > From: Patrick Welche <prlw1%cam.ac.uk@localhost>
 > To: gnats-bugs%netbsd.org@localhost
 > Cc:
 > Subject: Re: kern/50453
 > Date: Fri, 29 Jan 2016 11:46:20 +0000
 >
 >  With NO_PCI_EXTENDED_CONFIG:
 >
 >  agp0 at pchb0: i915-family chipset
 >  agp0: AGP_I915_IFPADDR = e0000001 (100000,ffffffff)
 >  agp0: detected 7932k stolen memory
 >  agp0: BAR size 100000 mismatches detected GTT size 40000; trusting BAR
 >  agp0: aperture at 0xd0000000, size 0x10000000
 >
 >  Without NO_PCI_EXTENDED_CONFIG:
 >
 >  acpi0: acpimcfg_map_bus done
 >  ...
 >  agp0 at pchb0: i915-family chipset
 >  agp0: AGP_I915_IFPADDR = e0000001 (100000,ffffffff)
 >  agp0: Try bus_space map again
 >  agp0: bus_space map returned error = 35 (addr = e0000001)
 >  agp0: can't set up chipset flush page: 35
 >
 >  From sys/arch/x86/pci/pci_machdep.c
 >
 >  #if NACPICA > 0 && !defined(NO_PCI_EXTENDED_CONFIG)
 >          acpimcfg_map_bus(self, pba->pba_pc, pba->pba_bus);
 >  #endif
 >
 >  the call to acpimcfg_map_bus causes the outcome to be different
 >  even though the same address was read.
 >
 >  Note that in both cases we see:
 >
 >  MCFG: bus 0-255, address 0x00000000e0000000: no valid region
 >  ...
 >  acpi0: MCFG: PNP0C01: Type=7
 >  acpi0: MCFG: PNP0C01: bus 0-255, address 0x00000000e0000000: no valid region
 >  ...
 >  acpi0: MCFG: PNP0C02: Type=7
 >  acpi0: MCFG: PNP0C02: bus 0-255, address 0x00000000e0000000: no valid region
 >  ...
 >  acpi0: MCFG: PNP0C02: Type=7
 >  acpi0: MCFG: PNP0C02: bus 0-255, address 0x00000000e0000000: no valid region
 >  acpi0: MCFG: PNP0C02: Type=10(FIXED_MEMORY32), Address=0x00000000e0000000, Lengt
 >  h=0x0000000010000000
 >  acpi0: MCFG: segment 0, bus 0-255, address 0x00000000e0000000
 >
 
 --001a1140f9882763cc052ac9c640
 Content-Type: application/octet-stream; 
 	name="0001-PR-50453-Check-overlaid-MCHBAR-region-on-PCIEXBAR-re.patch"
 Content-Disposition: attachment; 
 	filename="0001-PR-50453-Check-overlaid-MCHBAR-region-on-PCIEXBAR-re.patch"
 Content-Transfer-Encoding: base64
 X-Attachment-Id: f_ik5gohdm0
 
 RnJvbSAxYmE2OWI3NGZkZGFkZTM4MDk1Y2I4MjJmYTU5ZDYyY2Y1Y2YzMmNmIE1vbiBTZXAgMTcg
 MDA6MDA6MDAgMjAwMQpGcm9tOiBOT05BS0EgS2ltaWhpcm8gPG5vbmFrYXBAZ21haWwuY29tPgpE
 YXRlOiBUdWUsIDIgRmViIDIwMTYgMjA6NDU6MzEgKzA5MDAKU3ViamVjdDogW1BBVENIXSBQUi81
 MDQ1MzogQ2hlY2sgb3ZlcmxhaWQgTUNIQkFSIHJlZ2lvbiBvbiBQQ0lFWEJBUiByZWdpb24uCgot
 LS0KIHN5cy9hcmNoL3g4Ni9hY3BpL2FjcGlfbWFjaGRlcC5jIHwgNzIgKysrKysrKysrKysrKysr
 KysrKysrKysrKysrKysrKysrKysrKysrKwogc3lzL2Rldi9hY3BpL2FjcGlfbWNmZy5jICAgICAg
 ICAgfCAxOSArKysrKysrKysrKwogc3lzL2Rldi9hY3BpL2FjcGlfbWNmZy5oICAgICAgICAgfCAg
 NyArKystCiAzIGZpbGVzIGNoYW5nZWQsIDk3IGluc2VydGlvbnMoKyksIDEgZGVsZXRpb24oLSkK
 CmRpZmYgLS1naXQgYS9zeXMvYXJjaC94ODYvYWNwaS9hY3BpX21hY2hkZXAuYyBiL3N5cy9hcmNo
 L3g4Ni9hY3BpL2FjcGlfbWFjaGRlcC5jCmluZGV4IGU1Yjg5MmQuLmYyMjA2NWUgMTAwNjQ0Ci0t
 LSBhL3N5cy9hcmNoL3g4Ni9hY3BpL2FjcGlfbWFjaGRlcC5jCisrKyBiL3N5cy9hcmNoL3g4Ni9h
 Y3BpL2FjcGlfbWFjaGRlcC5jCkBAIC02Nyw2ICs2Nyw3IEBAIF9fS0VSTkVMX1JDU0lEKDAsICIk
 TmV0QlNEOiBhY3BpX21hY2hkZXAuYyx2IDEuMTIgMjAxNi8wMS8yOCAyMzo1MDowNCBodG9kZCBF
 eHAKICNpbmNsdWRlIDx4ODYvZWZpLmg+CiAKICNpbmNsdWRlIDxkZXYvcGNpL3BjaXZhci5oPgor
 I2luY2x1ZGUgPGRldi9wY2kvcGNpZGV2cy5oPgogCiAjaW5jbHVkZSA8ZGV2L2lzYS9pc2FyZWcu
 aD4KICNpbmNsdWRlIDxkZXYvaXNhL2lzYXZhci5oPgpAQCAtNDEwLDYgKzQxMSw3NiBAQCBhY3Bp
 X21kX21jZmdfdmFsaWRhdGUodWludDY0X3QgYWRkciwgaW50IGJ1c19zdGFydCwgaW50ICpidXNf
 ZW5kKQogCXJldHVybiBmYWxzZTsKIH0KIAorc3RhdGljIGJvb2wKK2FjcGlfbWRfbWNmZ192YWxp
 ZGF0ZV9kZXYocGNpX2NoaXBzZXRfdGFnX3QgcGMsIHBjaXRhZ190IHRhZykKK3sKKwl1aW50NjRf
 dCBwY2lleGJhciwgcGNpZXhiYXJfc2l6ZSwgcGNpZXhiYXJfbWFzazsKKwl1aW50NjRfdCBtY2hi
 YXIsIG1jaGJhcl9zaXplOworCXVpbnQ2NF90IGRldm9mZiwgYmFyb2ZmOworCXBjaXRhZ190IHBj
 aGJfdGFnOworCXBjaXJlZ190IGlkOworCWludCBidXMsIGRldjsKKworCXBjaV9kZWNvbXBvc2Vf
 dGFnKHBjLCB0YWcsICZidXMsICZkZXYsIE5VTEwpOworCisJcGNoYl90YWcgPSBwY2lfbWFrZV90
 YWcocGMsIDAsIDAsIDApOworCWlkID0gcGNpX2NvbmZfcmVhZChwYywgcGNoYl90YWcsIFBDSV9J
 RF9SRUcpOworCXN3aXRjaCAoUENJX1ZFTkRPUihpZCkpIHsKKwljYXNlIFBDSV9WRU5ET1JfSU5U
 RUw6CisJCXN3aXRjaCAoUENJX1BST0RVQ1QoaWQpKSB7CisJCWNhc2UgUENJX1BST0RVQ1RfSU5U
 RUxfUElORVZJRVdfSEI6CisJCQkvKiBDaGVjayBvdmVybGFpZCBNQ0hCQVIgcmVnaW9uICovCisJ
 CQlwY2lleGJhciA9IHBjaV9jb25mX3JlYWQocGMsIHBjaGJfdGFnLCAweDYwKTsKKwkJCXBjaWV4
 YmFyIHw9CisJCQkgICAgKHVpbnQ2NF90KXBjaV9jb25mX3JlYWQocGMsIHBjaGJfdGFnLCAweDY0
 KSA8PCAzMjsKKwkJCWlmICghKHBjaWV4YmFyICYgMSkpCisJCQkJcmV0dXJuIGZhbHNlOwkvKiBQ
 Q0lFWEJBUiBpcyBkaXNhYmxlZCAqLworCQkJc3dpdGNoICgocGNpZXhiYXIgPj4gMSkgJiAzKSB7
 CisJCQljYXNlIDA6CisJCQkJcGNpZXhiYXJfc2l6ZSA9IDI1NiAqIDEwMjQgKiAxMDI0OworCQkJ
 CXBjaWV4YmFyX21hc2sgPSAweGZmMDAwMDAwMFVMTDsKKwkJCQlicmVhazsKKwkJCWNhc2UgMToK
 KwkJCQlwY2lleGJhcl9zaXplID0gMTI4ICogMTAyNCAqIDEwMjQ7CisJCQkJcGNpZXhiYXJfbWFz
 ayA9IDB4ZmY4MDAwMDAwVUxMOworCQkJCWJyZWFrOworCQkJY2FzZSAyOgorCQkJCXBjaWV4YmFy
 X3NpemUgPSA2NCAqIDEwMjQgKiAxMDI0OworCQkJCXBjaWV4YmFyX21hc2sgPSAweGZmYzAwMDAw
 MFVMTDsKKwkJCQlicmVhazsKKwkJCWRlZmF1bHQ6CisJCQkJLyogcmVzZXJ2ZWQgKi8KKwkJCQly
 ZXR1cm4gZmFsc2U7CisJCQl9CisJCQlwY2lleGJhciAmPSBwY2lleGJhcl9tYXNrOworCisJCQlt
 Y2hiYXIgPSBwY2lfY29uZl9yZWFkKHBjLCBwY2hiX3RhZywgMHg0OCk7CisJCQltY2hiYXIgfD0K
 KwkJCSAgICAodWludDY0X3QpcGNpX2NvbmZfcmVhZChwYywgcGNoYl90YWcsIDB4NGMpIDw8IDMy
 OworCQkJaWYgKCEobWNoYmFyICYgMSkpCisJCQkJYnJlYWs7CS8qIE1DSEJBUiBpcyBkaXNhYmxl
 ZCAqLworCQkJbWNoYmFyX3NpemUgPSAxNiAqIDEwMjQ7CisJCQltY2hiYXIgJj0gfjE7CisKKwkJ
 CWlmIChtY2hiYXIgPCBwY2lleGJhciAmJgorCQkJICAgIG1jaGJhciArIG1jaGJhcl9zaXplIDwg
 cGNpZXhiYXIpCisJCQkJYnJlYWs7CisJCQlpZiAobWNoYmFyID49IHBjaWV4YmFyICsgcGNpZXhi
 YXJfc2l6ZSkKKwkJCQlicmVhazsKKworCQkJYmFyb2ZmID0gbWNoYmFyIC0gcGNpZXhiYXI7CisJ
 CQlkZXZvZmYgPSBidXMgKiBBQ1BJTUNGR19TSVpFX1BFUl9CVVMgKworCQkJICAgIGRldiAqIEFD
 UElNQ0ZHX1NJWkVfUEVSX0RFVjsKKwkJCWlmIChiYXJvZmYgPj0gZGV2b2ZmICYmCisJCQkgICAg
 YmFyb2ZmIDwgZGV2b2ZmICsgQUNQSU1DRkdfU0laRV9QRVJfREVWKQorCQkJCXJldHVybiBmYWxz
 ZTsJLyogb3ZlcmxhaWQgKi8KKwkJCWJyZWFrOworCQl9CisJCWJyZWFrOworCX0KKwlyZXR1cm4g
 dHJ1ZTsKK30KKwogc3RhdGljIHVpbnQzMl90CiBhY3BpX21kX21jZmdfcmVhZChidXNfc3BhY2Vf
 dGFnX3QgYnN0LCBidXNfc3BhY2VfaGFuZGxlX3QgYnNoLCBidXNfYWRkcl90IGFkZHIpCiB7CkBA
 IC00MzYsNiArNTA3LDcgQEAgYWNwaV9tZF9tY2ZnX3dyaXRlKGJ1c19zcGFjZV90YWdfdCBic3Qs
 IGJ1c19zcGFjZV9oYW5kbGVfdCBic2gsIGJ1c19hZGRyX3QgYWRkciwKIAogc3RhdGljIGNvbnN0
 IHN0cnVjdCBhY3BpbWNmZ19vcHMgYWNwaV9tZF9tY2ZnX29wcyA9IHsKIAkuYW9fdmFsaWRhdGUg
 PSBhY3BpX21kX21jZmdfdmFsaWRhdGUsCisJLmFvX3ZhbGlkYXRlX2RldiA9IGFjcGlfbWRfbWNm
 Z192YWxpZGF0ZV9kZXYsCiAKIAkuYW9fcmVhZCA9IGFjcGlfbWRfbWNmZ19yZWFkLAogCS5hb193
 cml0ZSA9IGFjcGlfbWRfbWNmZ193cml0ZSwKZGlmZiAtLWdpdCBhL3N5cy9kZXYvYWNwaS9hY3Bp
 X21jZmcuYyBiL3N5cy9kZXYvYWNwaS9hY3BpX21jZmcuYwppbmRleCA2MzM2YzUxLi5jODc2YWNm
 IDEwMDY0NAotLS0gYS9zeXMvZGV2L2FjcGkvYWNwaV9tY2ZnLmMKKysrIGIvc3lzL2Rldi9hY3Bp
 L2FjcGlfbWNmZy5jCkBAIC03NCw2ICs3NCw3IEBAIHN0YXRpYyBzdHJ1Y3QgYWNwaV9zb2Z0YyAq
 YWNwaV9zYzsKIAogc3RhdGljIGNvbnN0IHN0cnVjdCBhY3BpbWNmZ19vcHMgbWNmZ19kZWZhdWx0
 X29wcyA9IHsKIAkuYW9fdmFsaWRhdGUgPSBhY3BpbWNmZ19kZWZhdWx0X3ZhbGlkYXRlLAorCS5h
 b192YWxpZGF0ZV9kZXYgPSBhY3BpbWNmZ19kZWZhdWx0X3ZhbGlkYXRlX2RldiwKIAogCS5hb19y
 ZWFkID0gYWNwaW1jZmdfZGVmYXVsdF9yZWFkLAogCS5hb193cml0ZSA9IGFjcGltY2ZnX2RlZmF1
 bHRfd3JpdGUsCkBAIC05MSw2ICs5MiwxNCBAQCBhY3BpbWNmZ19kZWZhdWx0X3ZhbGlkYXRlKHVp
 bnQ2NF90IGFkZHJlc3MsIGludCBidXNfc3RhcnQsIGludCAqYnVzX2VuZCkKIAlyZXR1cm4gdHJ1
 ZTsKIH0KIAorYm9vbAorYWNwaW1jZmdfZGVmYXVsdF92YWxpZGF0ZV9kZXYocGNpX2NoaXBzZXRf
 dGFnX3QgcGMsIHBjaXRhZ190IHRhZykKK3sKKworCS8qIEFsd2F5cyBPayAqLworCXJldHVybiB0
 cnVlOworfQorCiB1aW50MzJfdAogYWNwaW1jZmdfZGVmYXVsdF9yZWFkKGJ1c19zcGFjZV90YWdf
 dCBic3QsIGJ1c19zcGFjZV9oYW5kbGVfdCBic2gsCiAgICAgYnVzX2FkZHJfdCBhZGRyKQpAQCAt
 NTY2LDYgKzU3NSwxNiBAQCBhY3BpbWNmZ19tYXBfYnVzKGRldmljZV90IHNlbGYsIHBjaV9jaGlw
 c2V0X3RhZ190IHBjLCBpbnQgYnVzKQogCiAJLyogUHJvYmUgZXh0ZW5kZWQgY29uZmlndXJhdGlv
 biBzcGFjZSBvZiBhbGwgZGV2aWNlcy4gKi8KIAltZW1zZXQobWItPnZhbGlkX2RldnMsIDB4ZmYs
 IHNpemVvZihtYi0+dmFsaWRfZGV2cykpOworCWlmIChtY2ZnX29wcy0+YW9fdmFsaWRhdGVfZGV2
 ICE9IE5VTEwpIHsKKwkJZm9yIChpID0gMDsgaSA8IDMyOyBpKyspIHsKKwkJCWlmICghbWNmZ19v
 cHMtPmFvX3ZhbGlkYXRlX2RldihwYywKKwkJCSAgICBwY2lfbWFrZV90YWcocGMsIGJ1cywgaSwg
 MCkpKSB7CisJCQkJZm9yIChqID0gMDsgaiA8IDg7IGorKykgeworCQkJCQlFWFRDT05GX1NFVF9J
 TlZBTElEKG1iLCBpLCBqKTsKKwkJCQl9CisJCQl9CisJCX0KKwl9CiAJbWItPnZhbGlkX25kZXZz
 ID0gMDsKIAltYi0+bGFzdF9wcm9iZWQgPSBwY2lfbWFrZV90YWcocGMsIGJ1cywgMCwgMCk7CiAK
 ZGlmZiAtLWdpdCBhL3N5cy9kZXYvYWNwaS9hY3BpX21jZmcuaCBiL3N5cy9kZXYvYWNwaS9hY3Bp
 X21jZmcuaAppbmRleCBiNjdjMzYwLi45OGM5MzUyIDEwMDY0NAotLS0gYS9zeXMvZGV2L2FjcGkv
 YWNwaV9tY2ZnLmgKKysrIGIvc3lzL2Rldi9hY3BpL2FjcGlfbWNmZy5oCkBAIC00MCw2ICs0MCw5
 IEBAIHN0cnVjdCBhY3BpbWNmZ19vcHMgewogCS8qIHZhbGlkYXRlIE1DRkcgbWVtb3J5IHJlZ2lv
 biAqLwogCWJvb2wJCSgqYW9fdmFsaWRhdGUpKHVpbnQ2NF90LCBpbnQsIGludCAqKTsKIAorCS8q
 IHZhbGlkYXRlIGRldmljZSAqLworCWJvb2wJCSgqYW9fdmFsaWRhdGVfZGV2KShwY2lfY2hpcHNl
 dF90YWdfdCwgcGNpdGFnX3QpOworCiAJLyogb3ZlcnJpZGUgZGVmYXVsdCBidXNfc3BhY2UoOSkg
 ZnVuY3Rpb24gKi8KIAl1aW50MzJfdAkoKmFvX3JlYWQpKGJ1c19zcGFjZV90YWdfdCwgYnVzX3Nw
 YWNlX2hhbmRsZV90LAogCQkJICAgIGJ1c19hZGRyX3QpOwpAQCAtNDcsOSArNTAsMTEgQEAgc3Ry
 dWN0IGFjcGltY2ZnX29wcyB7CiAJCQkgICAgYnVzX2FkZHJfdCwgdWludDMyX3QpOwogfTsKIAot
 I2RlZmluZQlBQ1BJTUNGR19TSVpFX1BFUl9CVVMJKFBDSV9FWFRDT05GX1NJWkUgKiAzMi8qZGV2
 Ki8gKiA4LypmdW5jKi8pCisjZGVmaW5lCUFDUElNQ0ZHX1NJWkVfUEVSX0RFVgkoUENJX0VYVENP
 TkZfU0laRSAqIDgvKmZ1bmMqLykKKyNkZWZpbmUJQUNQSU1DRkdfU0laRV9QRVJfQlVTCShBQ1BJ
 TUNGR19TSVpFX1BFUl9ERVYgKiAzMi8qZGV2Ki8pCiAKIGJvb2wJCWFjcGltY2ZnX2RlZmF1bHRf
 dmFsaWRhdGUodWludDY0X3QsIGludCwgaW50ICopOworYm9vbAkJYWNwaW1jZmdfZGVmYXVsdF92
 YWxpZGF0ZV9kZXYocGNpX2NoaXBzZXRfdGFnX3QsIHBjaXRhZ190KTsKIHVpbnQzMl90CWFjcGlt
 Y2ZnX2RlZmF1bHRfcmVhZChidXNfc3BhY2VfdGFnX3QsIGJ1c19zcGFjZV9oYW5kbGVfdCwKIAkJ
 ICAgIGJ1c19hZGRyX3QpOwogdm9pZAkJYWNwaW1jZmdfZGVmYXVsdF93cml0ZShidXNfc3BhY2Vf
 dGFnX3QsIGJ1c19zcGFjZV9oYW5kbGVfdCwKLS0gCjIuNy4wCgo=
 --001a1140f9882763cc052ac9c640--