Re: kern/52409: uvm_fault when waking up after sleep (Let's Note CF-{N,S}-9, etc.)

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

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

From: Kimihiro Nonaka <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>
Subject: Re: kern/52409: uvm_fault when waking up after sleep (Let's Note
 CF-{N,S}-9, etc.)
Date: Wed, 19 Jul 2017 11:45:52 +0900

 --001a114dd5907218ed0554a2a1d1
 Content-Type: text/plain; charset="UTF-8"
 
 Hi,
 
 > (1) In file sys/external/bsd/drm2/pci/drm_pci.c,
 >    ---------------------
 >    234  static int
 >    235  drm_pci_irq_install(struct drm_device *dev, irqreturn_t (*handler)(void *),
 >    236      int flags, const char *name, void *arg, struct drm_bus_irq_cookie **cookiep)
 >
 >         ... (omit) ...
 >    253          intrstr = pci_intr_string(pa->pa_pc, irq_cookie->intr_handles[0],
 >    254              intrbuf, sizeof(intrbuf));
 >    ---------------------
 >    at the line 253, irq_cookie->intr_handles == 0,
 >    which causes following trap.
 
 Could you try the attached patch?
 
 Regards,
 -- 
 Kimihiro Nonaka
 
 --001a114dd5907218ed0554a2a1d1
 Content-Type: text/plain; charset="US-ASCII"; name="drm_pci.c.diff"
 Content-Disposition: attachment; filename="drm_pci.c.diff"
 Content-Transfer-Encoding: base64
 X-Attachment-Id: f_j5aeo7yu0
 
 ZGlmZiAtLWdpdCBhL3N5cy9leHRlcm5hbC9ic2QvZHJtMi9wY2kvZHJtX3BjaS5jIGIvc3lzL2V4
 dGVybmFsL2JzZC9kcm0yL3BjaS9kcm1fcGNpLmMKaW5kZXggOWYzNzI5ZTMzZGQuLmZjNzEzMmRh
 Yjk5IDEwMDY0NAotLS0gYS9zeXMvZXh0ZXJuYWwvYnNkL2RybTIvcGNpL2RybV9wY2kuYworKysg
 Yi9zeXMvZXh0ZXJuYWwvYnNkL2RybTIvcGNpL2RybV9wY2kuYwpAQCAtMjQzLDExICsyNDMsMTcg
 QEAgZHJtX3BjaV9pcnFfaW5zdGFsbChzdHJ1Y3QgZHJtX2RldmljZSAqZGV2LCBpcnFyZXR1cm5f
 dCAoKmhhbmRsZXIpKHZvaWQgKiksCiAJaXJxX2Nvb2tpZSA9IGttZW1fYWxsb2Moc2l6ZW9mKCpp
 cnFfY29va2llKSwgS01fU0xFRVApOwogCiAJaWYgKGRldi0+cGRldi0+bXNpX2VuYWJsZWQpIHsK
 LQkJaXJxX2Nvb2tpZS0+aW50cl9oYW5kbGVzID0gZGV2LT5wZGV2LT5pbnRyX2hhbmRsZXM7Ci0J
 CWRldi0+cGRldi0+aW50cl9oYW5kbGVzID0gTlVMTDsKKwkJaWYgKGRldi0+cGRldi0+aW50cl9o
 YW5kbGVzID09IE5VTEwpIHsKKwkJCWlmIChwY2lfbXNpX2FsbG9jX2V4YWN0KHBhLCAmaXJxX2Nv
 b2tpZS0+aW50cl9oYW5kbGVzLAorCQkJICAgIDEpKQorCQkJCWdvdG8gZXJyb3I7CisJCX0gZWxz
 ZSB7CisJCQlpcnFfY29va2llLT5pbnRyX2hhbmRsZXMgPSBkZXYtPnBkZXYtPmludHJfaGFuZGxl
 czsKKwkJCWRldi0+cGRldi0+aW50cl9oYW5kbGVzID0gTlVMTDsKKwkJfQogCX0gZWxzZSB7CiAJ
 CWlmIChwY2lfaW50eF9hbGxvYyhwYSwgJmlycV9jb29raWUtPmludHJfaGFuZGxlcykpCi0JCQly
 ZXR1cm4gLUVOT0VOVDsKKwkJCWdvdG8gZXJyb3I7CiAJfQogCiAJaW50cnN0ciA9IHBjaV9pbnRy
 X3N0cmluZyhwYS0+cGFfcGMsIGlycV9jb29raWUtPmludHJfaGFuZGxlc1swXSwKQEAgLTI1Nywx
 MiArMjYzLDE3IEBAIGRybV9wY2lfaXJxX2luc3RhbGwoc3RydWN0IGRybV9kZXZpY2UgKmRldiwg
 aXJxcmV0dXJuX3QgKCpoYW5kbGVyKSh2b2lkICopLAogCWlmIChpcnFfY29va2llLT5paF9jb29r
 aWUgPT0gTlVMTCkgewogCQlhcHJpbnRfZXJyb3JfZGV2KGRldi0+ZGV2LAogCQkgICAgImNvdWxk
 bid0IGVzdGFibGlzaCBpbnRlcnJ1cHQgYXQgJXMgKCVzKVxuIiwgaW50cnN0ciwgbmFtZSk7Ci0J
 CXJldHVybiAtRU5PRU5UOworCQlwY2lfaW50cl9yZWxlYXNlKHBhLT5wYV9wYywgaXJxX2Nvb2tp
 ZS0+aW50cl9oYW5kbGVzLCAxKTsKKwkJZ290byBlcnJvcjsKIAl9CiAKIAlhcHJpbnRfbm9ybWFs
 X2RldihkZXYtPmRldiwgImludGVycnVwdGluZyBhdCAlcyAoJXMpXG4iLCBpbnRyc3RyLCBuYW1l
 KTsKIAkqY29va2llcCA9IGlycV9jb29raWU7CiAJcmV0dXJuIDA7CisKK2Vycm9yOgorCWttZW1f
 ZnJlZShpcnFfY29va2llLCBzaXplb2YoKmlycV9jb29raWUpKTsKKwlyZXR1cm4gLUVOT0VOVDsK
 IH0KIAogc3RhdGljIHZvaWQK
 --001a114dd5907218ed0554a2a1d1--