From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932524Ab3BKVGb (ORCPT ); Mon, 11 Feb 2013 16:06:31 -0500 Received: from mail-ia0-f175.google.com ([209.85.210.175]:35487 "EHLO mail-ia0-f175.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932078Ab3BKVG2 (ORCPT ); Mon, 11 Feb 2013 16:06:28 -0500 MIME-Version: 1.0 In-Reply-To: References: <1360466060.3703.15.camel@thor.lan> <1360506199.3461.5.camel@thor.lan> <1360587768.3454.2.camel@thor.lan> Date: Mon, 11 Feb 2013 13:06:27 -0800 X-Google-Sender-Auth: C4geX3WjakrJeh1Rp177Vt-NMRc Message-ID: Subject: Re: [Bisected] [-next-20130204] usb/hcd: irq 18: nobody cared From: Yinghai Lu To: Bjorn Helgaas , "Rafael J. Wysocki" Cc: Peter Hurley , Alan Stern , Lan Tianyu , Greg Kroah-Hartman , Jiri Kosina , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, linux-next@vger.kernel.org Content-Type: multipart/mixed; boundary=e89a8f3ba0374d87ee04d57948b5 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --e89a8f3ba0374d87ee04d57948b5 Content-Type: text/plain; charset=ISO-8859-1 On Mon, Feb 11, 2013 at 11:57 AM, Yinghai Lu wrote: >> >> Bjorn, Rafael, >> >> acpi_pci_irq_add_prt need to be called after pci bridge get scanned, >> so we can not call it from pci_acpi_setup, after we move dev_register >> for pci_dev early. >> >> The attached debug patch move down that calling into >> pci_bus_add_devices and that will make prt works again. >> >> Can acpi provide another hook after bridge get scanned? >> Rafael, Bjorn, Can you check attached patch? Thanks Yinghai --e89a8f3ba0374d87ee04d57948b5 Content-Type: application/octet-stream; name="move_add_irq_prt_down_v2.patch" Content-Disposition: attachment; filename="move_add_irq_prt_down_v2.patch" Content-Transfer-Encoding: base64 X-Attachment-Id: f_hd243ss70 U3ViamVjdDogW1BBVENIXSBBQ1BJLCBQQ0k6IGFkZCBhY3BpX3BsYXRmb3JtX25vdGlmeV9zY2Fu KCkKClBldGVyIEh1cmxleSBmb3VuZCBpcnEgbm9ib2R5IGNhcmVkLCBhbmQgZG1lc2cgaGFzCgpb ICAgIDguOTgzMjQ2XSBwY2kgMDAwMDowMDoxZS4wOiBjYW4ndCBkZXJpdmUgcm91dGluZyBmb3Ig UENJIElOVCBBClsgICAgOC45ODM2MDBdIHNuZF9jdHhmaSAwMDAwOjA5OjAyLjA6IFBDSSBJTlQg QTogbm8gR1NJIC0gdXNpbmcgSVNBIElSUSA1CgpiaXNlY3QgdG8KfCBjb21taXQgNGY1MzUwOTNj ZjhmNmRhOGNmZGE3YzM2YzJjMWVjZDJlOTU4NmVlNAp8ICAgICBQQ0k6IFB1dCBwY2lfZGV2IGlu IGRldmljZSB0cmVlIGFzIGVhcmx5IGFzIHBvc3NpYmxlCgpJdCB0dXJucyBvdXQgd2UgbmVlZCB0 byBjYWxsIGFjcGlfcGNpX2lycV9hZGRfcHJ0KCkgYWZ0ZXIgdGhlIHBjaSBicmlkZ2VzCmFyZSBz Y2FubmVkLgoKQWRkIGFjcGlfcGxhdGZvcm1fbm90aWZ5X3NjYW4oKSB0byBjYWxsIGFjcGlfcGNp X2lycV9hZGRfcHJ0IGxhdGVyLgoKUmVwb3J0ZWQtYW5kLXRlc3RlZC1ieTogUGV0ZXIgSHVybGV5 IDxwZXRlckBodXJsZXlzb2Z0d2FyZS5jb20+ClNpZ25lZC1vZmYtYnk6IFlpbmdoYWkgTHUgPHlp bmdoYWlAa2VybmVsLm9yZz4KCi0tLQogZHJpdmVycy9hY3BpL2dsdWUuYyAgICAgfCAgIDEyICsr KysrKysrKysrKwogZHJpdmVycy9iYXNlL2NvcmUuYyAgICAgfCAgICAxICsKIGRyaXZlcnMvcGNp L2J1cy5jICAgICAgIHwgICAgNCArKysrCiBkcml2ZXJzL3BjaS9wY2ktYWNwaS5jICB8ICAgMjcg KysrKysrKysrKysrKysrKystLS0tLS0tLS0tCiBpbmNsdWRlL2FjcGkvYWNwaV9idXMuaCB8ICAg IDEgKwogaW5jbHVkZS9saW51eC9kZXZpY2UuaCAgfCAgICAzICstLQogNiBmaWxlcyBjaGFuZ2Vk LCAzNiBpbnNlcnRpb25zKCspLCAxMiBkZWxldGlvbnMoLSkKCkluZGV4OiBsaW51eC0yLjYvZHJp dmVycy9hY3BpL2dsdWUuYwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBsaW51eC0yLjYub3JpZy9kcml2ZXJzL2Fj cGkvZ2x1ZS5jCisrKyBsaW51eC0yLjYvZHJpdmVycy9hY3BpL2dsdWUuYwpAQCAtMzEyLDYgKzMx MiwxNyBAQCBzdGF0aWMgaW50IGFjcGlfcGxhdGZvcm1fbm90aWZ5KHN0cnVjdCBkCiAJcmV0dXJu IHJldDsKIH0KIAorc3RhdGljIGludCBhY3BpX3BsYXRmb3JtX25vdGlmeV9zY2FuKHN0cnVjdCBk ZXZpY2UgKmRldikKK3sKKwlzdHJ1Y3QgYWNwaV9idXNfdHlwZSAqdHlwZTsKKworCXR5cGUgPSBh Y3BpX2dldF9idXNfdHlwZShkZXYtPmJ1cyk7CisJaWYgKHR5cGUgJiYgdHlwZS0+c2V0dXBfc2Nh bikKKwkJdHlwZS0+c2V0dXBfc2NhbihkZXYpOworCisJcmV0dXJuIDA7Cit9CisKIHN0YXRpYyBp bnQgYWNwaV9wbGF0Zm9ybV9ub3RpZnlfcmVtb3ZlKHN0cnVjdCBkZXZpY2UgKmRldikKIHsKIAlz dHJ1Y3QgYWNwaV9idXNfdHlwZSAqdHlwZTsKQEAgLTMzMSw2ICszNDIsNyBAQCBpbnQgX19pbml0 IGluaXRfYWNwaV9kZXZpY2Vfbm90aWZ5KHZvaWQpCiAJCXJldHVybiAwOwogCX0KIAlwbGF0Zm9y bV9ub3RpZnkgPSBhY3BpX3BsYXRmb3JtX25vdGlmeTsKKwlwbGF0Zm9ybV9ub3RpZnlfc2NhbiA9 IGFjcGlfcGxhdGZvcm1fbm90aWZ5X3NjYW47CiAJcGxhdGZvcm1fbm90aWZ5X3JlbW92ZSA9IGFj cGlfcGxhdGZvcm1fbm90aWZ5X3JlbW92ZTsKIAlyZXR1cm4gMDsKIH0KSW5kZXg6IGxpbnV4LTIu Ni9kcml2ZXJzL2Jhc2UvY29yZS5jCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIGxpbnV4LTIuNi5vcmlnL2RyaXZl cnMvYmFzZS9jb3JlLmMKKysrIGxpbnV4LTIuNi9kcml2ZXJzL2Jhc2UvY29yZS5jCkBAIC00NCw2 ICs0NCw3IEBAIGVhcmx5X3BhcmFtKCJzeXNmcy5kZXByZWNhdGVkIiwgc3lzZnNfZGUKICNlbmRp ZgogCiBpbnQgKCpwbGF0Zm9ybV9ub3RpZnkpKHN0cnVjdCBkZXZpY2UgKmRldikgPSBOVUxMOwor aW50ICgqcGxhdGZvcm1fbm90aWZ5X3NjYW4pKHN0cnVjdCBkZXZpY2UgKmRldikgPSBOVUxMOwog aW50ICgqcGxhdGZvcm1fbm90aWZ5X3JlbW92ZSkoc3RydWN0IGRldmljZSAqZGV2KSA9IE5VTEw7 CiBzdGF0aWMgc3RydWN0IGtvYmplY3QgKmRldl9rb2JqOwogc3RydWN0IGtvYmplY3QgKnN5c2Zz X2Rldl9jaGFyX2tvYmo7CkluZGV4OiBsaW51eC0yLjYvZHJpdmVycy9wY2kvYnVzLmMKPT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PQotLS0gbGludXgtMi42Lm9yaWcvZHJpdmVycy9wY2kvYnVzLmMKKysrIGxpbnV4LTIuNi9k cml2ZXJzL3BjaS9idXMuYwpAQCAtMTcwLDYgKzE3MCwxMCBAQCBpbnQgcGNpX2J1c19hZGRfZGV2 aWNlKHN0cnVjdCBwY2lfZGV2ICpkCiB7CiAJaW50IHJldHZhbDsKIAorCS8qIG5lZWQgdG8gYmUg Y2FsbGVkIGFmdGVyIGJyaWRnZSBpcyBzY2FubmVkICovCisJaWYgKHBsYXRmb3JtX25vdGlmeV9z Y2FuKQorCQlwbGF0Zm9ybV9ub3RpZnlfc2NhbigmZGV2LT5kZXYpOworCiAJLyoKIAkgKiBDYW4g bm90IHB1dCBpbiBwY2lfZGV2aWNlX2FkZCB5ZXQgYmVjYXVzZSByZXNvdXJjZXMKIAkgKiBhcmUg bm90IGFzc2lnbmVkIHlldCBmb3Igc29tZSBkZXZpY2VzLgpJbmRleDogbGludXgtMi42L2RyaXZl cnMvcGNpL3BjaS1hY3BpLmMKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gbGludXgtMi42Lm9yaWcvZHJpdmVycy9w Y2kvcGNpLWFjcGkuYworKysgbGludXgtMi42L2RyaXZlcnMvcGNpL3BjaS1hY3BpLmMKQEAgLTMw Nyw2ICszMDcsMjIgQEAgc3RhdGljIHZvaWQgcGNpX2FjcGlfc2V0dXAoc3RydWN0IGRldmljZQog CXN0cnVjdCBwY2lfZGV2ICpwY2lfZGV2ID0gdG9fcGNpX2RldihkZXYpOwogCWFjcGlfaGFuZGxl IGhhbmRsZSA9IEFDUElfSEFORExFKGRldik7CiAJc3RydWN0IGFjcGlfZGV2aWNlICphZGV2Owor CisJaWYgKGFjcGlfYnVzX2dldF9kZXZpY2UoaGFuZGxlLCAmYWRldikgfHwgIWFkZXYtPndha2V1 cC5mbGFncy52YWxpZCkKKwkJcmV0dXJuOworCisJZGV2aWNlX3NldF93YWtldXBfY2FwYWJsZShk ZXYsIHRydWUpOworCWFjcGlfcGNpX3NsZWVwX3dha2UocGNpX2RldiwgZmFsc2UpOworCisJcGNp X2FjcGlfYWRkX3BtX25vdGlmaWVyKGFkZXYsIHBjaV9kZXYpOworCWlmIChhZGV2LT53YWtldXAu ZmxhZ3MucnVuX3dha2UpCisJCWRldmljZV9zZXRfcnVuX3dha2UoZGV2LCB0cnVlKTsKK30KKwor c3RhdGljIHZvaWQgcGNpX2FjcGlfc2V0dXBfc2NhbihzdHJ1Y3QgZGV2aWNlICpkZXYpCit7CisJ c3RydWN0IHBjaV9kZXYgKnBjaV9kZXYgPSB0b19wY2lfZGV2KGRldik7CisJYWNwaV9oYW5kbGUg aGFuZGxlID0gQUNQSV9IQU5ETEUoZGV2KTsKIAlhY3BpX3N0YXR1cyBzdGF0dXM7CiAJYWNwaV9o YW5kbGUgZHVtbXk7CiAKQEAgLTMyNiwxNiArMzQyLDYgQEAgc3RhdGljIHZvaWQgcGNpX2FjcGlf c2V0dXAoc3RydWN0IGRldmljZQogCQkJcGNpX2Rldi0+c3Vib3JkaW5hdGUtPm51bWJlciA6IHBj aV9kZXYtPmJ1cy0+bnVtYmVyOwogCQlhY3BpX3BjaV9pcnFfYWRkX3BydChoYW5kbGUsIHBjaV9k b21haW5fbnIocGNpX2Rldi0+YnVzKSwgYnVzKTsKIAl9Ci0KLQlpZiAoYWNwaV9idXNfZ2V0X2Rl dmljZShoYW5kbGUsICZhZGV2KSB8fCAhYWRldi0+d2FrZXVwLmZsYWdzLnZhbGlkKQotCQlyZXR1 cm47Ci0KLQlkZXZpY2Vfc2V0X3dha2V1cF9jYXBhYmxlKGRldiwgdHJ1ZSk7Ci0JYWNwaV9wY2lf c2xlZXBfd2FrZShwY2lfZGV2LCBmYWxzZSk7Ci0KLQlwY2lfYWNwaV9hZGRfcG1fbm90aWZpZXIo YWRldiwgcGNpX2Rldik7Ci0JaWYgKGFkZXYtPndha2V1cC5mbGFncy5ydW5fd2FrZSkKLQkJZGV2 aWNlX3NldF9ydW5fd2FrZShkZXYsIHRydWUpOwogfQogCiBzdGF0aWMgdm9pZCBwY2lfYWNwaV9j bGVhbnVwKHN0cnVjdCBkZXZpY2UgKmRldikKQEAgLTM1OSw2ICszNjUsNyBAQCBzdGF0aWMgc3Ry dWN0IGFjcGlfYnVzX3R5cGUgYWNwaV9wY2lfYnVzCiAJLmJ1cyA9ICZwY2lfYnVzX3R5cGUsCiAJ LmZpbmRfZGV2aWNlID0gYWNwaV9wY2lfZmluZF9kZXZpY2UsCiAJLnNldHVwID0gcGNpX2FjcGlf c2V0dXAsCisJLnNldHVwX3NjYW4gPSBwY2lfYWNwaV9zZXR1cF9zY2FuLAogCS5jbGVhbnVwID0g cGNpX2FjcGlfY2xlYW51cCwKIH07CiAKSW5kZXg6IGxpbnV4LTIuNi9pbmNsdWRlL2FjcGkvYWNw aV9idXMuaAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09Ci0tLSBsaW51eC0yLjYub3JpZy9pbmNsdWRlL2FjcGkvYWNwaV9i dXMuaAorKysgbGludXgtMi42L2luY2x1ZGUvYWNwaS9hY3BpX2J1cy5oCkBAIC00NDAsNiArNDQw LDcgQEAgc3RydWN0IGFjcGlfYnVzX3R5cGUgewogCS8qIEZvciBicmlkZ2VzLCBzdWNoIGFzIFBD SSByb290IGJyaWRnZSwgSURFIGNvbnRyb2xsZXIgKi8KIAlpbnQgKCpmaW5kX2JyaWRnZSkgKHN0 cnVjdCBkZXZpY2UgKiwgYWNwaV9oYW5kbGUgKik7CiAJdm9pZCAoKnNldHVwKShzdHJ1Y3QgZGV2 aWNlICopOworCXZvaWQgKCpzZXR1cF9zY2FuKShzdHJ1Y3QgZGV2aWNlICopOwogCXZvaWQgKCpj bGVhbnVwKShzdHJ1Y3QgZGV2aWNlICopOwogfTsKIGludCByZWdpc3Rlcl9hY3BpX2J1c190eXBl KHN0cnVjdCBhY3BpX2J1c190eXBlICopOwpJbmRleDogbGludXgtMi42L2luY2x1ZGUvbGludXgv ZGV2aWNlLmgKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PQotLS0gbGludXgtMi42Lm9yaWcvaW5jbHVkZS9saW51eC9kZXZp Y2UuaAorKysgbGludXgtMi42L2luY2x1ZGUvbGludXgvZGV2aWNlLmgKQEAgLTg5NywxMCArODk3 LDkgQEAgZXh0ZXJuIHZvaWQgZGV2aWNlX2Rlc3Ryb3koc3RydWN0IGNsYXNzCiAgKi8KIC8qIE5v dGlmeSBwbGF0Zm9ybSBvZiBkZXZpY2UgZGlzY292ZXJ5ICovCiBleHRlcm4gaW50ICgqcGxhdGZv cm1fbm90aWZ5KShzdHJ1Y3QgZGV2aWNlICpkZXYpOwotCitleHRlcm4gaW50ICgqcGxhdGZvcm1f bm90aWZ5X3NjYW4pKHN0cnVjdCBkZXZpY2UgKmRldik7CiBleHRlcm4gaW50ICgqcGxhdGZvcm1f bm90aWZ5X3JlbW92ZSkoc3RydWN0IGRldmljZSAqZGV2KTsKIAotCiAvKgogICogZ2V0X2Rldmlj ZSAtIGF0b21pY2FsbHkgaW5jcmVtZW50IHRoZSByZWZlcmVuY2UgY291bnQgZm9yIHRoZSBkZXZp Y2UuCiAgKgo= --e89a8f3ba0374d87ee04d57948b5--