From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
To: "Jan Kundrát" <jan.kundrat@cesnet.cz>
Cc: Baruch Siach <baruch@tkos.co.il>,
Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>,
Jason Cooper <jason@lakedaemon.net>,
linux-pci@vger.kernel.org,
Russell King - ARM Linux <linux@armlinux.org.uk>,
linux-kernel@vger.kernel.org, Bjorn Helgaas <bhelgaas@google.com>,
linux-arm-kernel@lists.infradead.org
Subject: Re: [BISECTED] Regression: Solidrun Clearfog Base won't boot since "PCI: mvebu: Only remap I/O space if configured"
Date: Thu, 13 Sep 2018 10:42:41 +0200 [thread overview]
Message-ID: <20180913104241.65db8243@windsurf> (raw)
In-Reply-To: <5ad46fec-a71a-477a-b23f-d20aacfb481d@cesnet.cz>
SGVsbG8sCgpPbiBUaHUsIDEzIFNlcCAyMDE4IDEwOjIwOjQ1ICswMjAwLCBKYW4gS3VuZHLDoXQg
d3JvdGU6Cj4gT24gxI10dnJ0ZWsgMTMuIHrDocWZw60gMjAxOCA5OjQ1OjE1IENFU1QsIFRob21h
cyBQZXRhenpvbmkgd3JvdGU6Cj4gPiBXaGF0IGFib3V0IHNvbWV0aGluZyBsaWtlIHRoZSBiZWxv
dy4gSSB0ZXN0ZWQgaXQsIGluY2x1ZGluZyB0aGUgZXJyb3IKPiA+IGNhc2UgYnkgZm9yY2luZyBh
biAtRVBST0JFX0RFRkVSLiBUaGUgbmV3IHBjaV91bm1hcF9pbygpIGlzIG1vZGVsZWQKPiA+IGFm
dGVyIHBjaV91bm1hcF9pb3NwYWNlKCkuIEFjdHVhbGx5LCBJIHdvdWxkIHByZWZlciB0byB1c2UK
PiA+IHBjaV9yZW1hcF9pb3NwYWNlKCkgYW5kIHBjaV91bm1hcF9pb3NwYWNlKCkgYnV0IGZvciBu
b3cgdGhpcyBBUEkKPiA+IGRvZXNuJ3QgYWxsb3cgb3ZlcmxvYWRpbmcgdGhlIG1lbW9yeSB0eXBl
IHVzZWQgZm9yIHRoZSBtYXBwaW5nLiAgCj4gCj4gVGhhbmtzIGZvciBwcm92aWRpbmcgdGhpcyBm
aXggc28gcXVpY2tseSwgVGhvbWFzLiBJIGNhbiBjb25maXJtIHRoYXQgdGhpcyAKPiBwYXRjaCAt
LSB0ZXN0ZWQgb24gdG9wIG9mIDU0ZWRhOWRmMTdmMzIxNWI5ZWQxNjYyOWVlNzFlYTA3NDEzZWZk
YWYgKCJNZXJnZSAKPiB0YWcgJ3BjaS12NC4xOS1maXhlcy0xJyBvZiAKPiBnaXQ6Ly9naXQua2Vy
bmVsLm9yZy9wdWIvc2NtL2xpbnV4L2tlcm5lbC9naXQvaGVsZ2Fhcy9wY2kiKS4gRGlzY2xhaW1l
cjogSSAKPiBoYXZlIHplcm8gZmFtaWxpYXJpdHkgd2l0aCBMaW51eCcgUENJIGNvZGUuCj4gCj4g
VGVzdGVkLWJ5OiBKYW4gS3VuZHLDoXQgPGphbi5rdW5kcmF0QGNlc25ldC5jej4KClRoYW5rcyBm
b3IgdGhlIHRlc3RpbmcuIEknbGwgd2FpdCBmb3IgUnVzc2VsbCB0byBzYXkgaWYgaGUgaXMgaGFw
cHkKKG9yIG5vdCkgd2l0aCB0aGUgYWRkaXRpb24gb2YgcGNpX3VubWFwX2lvKCkgaW4gdGhlIEFS
TSBjb2RlLCBpZiB0aGF0J3MKdGhlIGNhc2UsIEknbGwgc2VuZCBhIHByb3BlciBwYXRjaCB0byBm
aXggdGhlIGlzc3VlLgoKQmVzdCByZWdhcmRzLAoKVGhvbWFzCi0tIApUaG9tYXMgUGV0YXp6b25p
LCBDVE8sIEJvb3RsaW4KRW1iZWRkZWQgTGludXggYW5kIEtlcm5lbCBlbmdpbmVlcmluZwpodHRw
czovL2Jvb3RsaW4uY29tCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f
X19fX19fXwpsaW51eC1hcm0ta2VybmVsIG1haWxpbmcgbGlzdApsaW51eC1hcm0ta2VybmVsQGxp
c3RzLmluZnJhZGVhZC5vcmcKaHR0cDovL2xpc3RzLmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0
aW5mby9saW51eC1hcm0ta2VybmVsCg==
WARNING: multiple messages have this Message-ID (diff)
From: thomas.petazzoni@bootlin.com (Thomas Petazzoni)
To: linux-arm-kernel@lists.infradead.org
Subject: [BISECTED] Regression: Solidrun Clearfog Base won't boot since "PCI: mvebu: Only remap I/O space if configured"
Date: Thu, 13 Sep 2018 10:42:41 +0200 [thread overview]
Message-ID: <20180913104241.65db8243@windsurf> (raw)
In-Reply-To: <5ad46fec-a71a-477a-b23f-d20aacfb481d@cesnet.cz>
Hello,
On Thu, 13 Sep 2018 10:20:45 +0200, Jan Kundr?t wrote:
> On ?tvrtek 13. z??? 2018 9:45:15 CEST, Thomas Petazzoni wrote:
> > What about something like the below. I tested it, including the error
> > case by forcing an -EPROBE_DEFER. The new pci_unmap_io() is modeled
> > after pci_unmap_iospace(). Actually, I would prefer to use
> > pci_remap_iospace() and pci_unmap_iospace() but for now this API
> > doesn't allow overloading the memory type used for the mapping.
>
> Thanks for providing this fix so quickly, Thomas. I can confirm that this
> patch -- tested on top of 54eda9df17f3215b9ed16629ee71ea07413efdaf ("Merge
> tag 'pci-v4.19-fixes-1' of
> git://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci"). Disclaimer: I
> have zero familiarity with Linux' PCI code.
>
> Tested-by: Jan Kundr?t <jan.kundrat@cesnet.cz>
Thanks for the testing. I'll wait for Russell to say if he is happy
(or not) with the addition of pci_unmap_io() in the ARM code, if that's
the case, I'll send a proper patch to fix the issue.
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
WARNING: multiple messages have this Message-ID (diff)
From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
To: "Jan Kundrát" <jan.kundrat@cesnet.cz>
Cc: Russell King - ARM Linux <linux@armlinux.org.uk>,
Baruch Siach <baruch@tkos.co.il>,
Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>,
Jason Cooper <jason@lakedaemon.net>, <linux-pci@vger.kernel.org>,
<linux-kernel@vger.kernel.org>,
Bjorn Helgaas <bhelgaas@google.com>,
<linux-arm-kernel@lists.infradead.org>
Subject: Re: [BISECTED] Regression: Solidrun Clearfog Base won't boot since "PCI: mvebu: Only remap I/O space if configured"
Date: Thu, 13 Sep 2018 10:42:41 +0200 [thread overview]
Message-ID: <20180913104241.65db8243@windsurf> (raw)
In-Reply-To: <5ad46fec-a71a-477a-b23f-d20aacfb481d@cesnet.cz>
Hello,
On Thu, 13 Sep 2018 10:20:45 +0200, Jan Kundrát wrote:
> On čtvrtek 13. září 2018 9:45:15 CEST, Thomas Petazzoni wrote:
> > What about something like the below. I tested it, including the error
> > case by forcing an -EPROBE_DEFER. The new pci_unmap_io() is modeled
> > after pci_unmap_iospace(). Actually, I would prefer to use
> > pci_remap_iospace() and pci_unmap_iospace() but for now this API
> > doesn't allow overloading the memory type used for the mapping.
>
> Thanks for providing this fix so quickly, Thomas. I can confirm that this
> patch -- tested on top of 54eda9df17f3215b9ed16629ee71ea07413efdaf ("Merge
> tag 'pci-v4.19-fixes-1' of
> git://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci"). Disclaimer: I
> have zero familiarity with Linux' PCI code.
>
> Tested-by: Jan Kundrát <jan.kundrat@cesnet.cz>
Thanks for the testing. I'll wait for Russell to say if he is happy
(or not) with the addition of pci_unmap_io() in the ARM code, if that's
the case, I'll send a proper patch to fix the issue.
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
next prev parent reply other threads:[~2018-09-13 8:42 UTC|newest]
Thread overview: 57+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-09-12 16:11 [BISECTED] Regression: Solidrun Clearfog Base won't boot since "PCI: mvebu: Only remap I/O space if configured" Jan Kundrát
2018-09-12 16:11 ` Jan Kundrát
2018-09-12 16:11 ` Jan Kundrát
2018-09-12 18:49 ` Baruch Siach
2018-09-12 18:49 ` Baruch Siach
2018-09-12 18:49 ` Baruch Siach
2018-09-12 18:50 ` Thomas Petazzoni
2018-09-12 18:50 ` Thomas Petazzoni
2018-09-12 18:50 ` Thomas Petazzoni
2018-09-12 19:00 ` Jan Kundrát
2018-09-12 19:00 ` Jan Kundrát
2018-09-12 19:00 ` Jan Kundrát
2018-09-12 23:10 ` Russell King - ARM Linux
2018-09-12 23:10 ` Russell King - ARM Linux
2018-09-12 23:10 ` Russell King - ARM Linux
2018-09-13 3:19 ` Baruch Siach
2018-09-13 3:19 ` Baruch Siach
2018-09-13 3:19 ` Baruch Siach
2018-09-13 7:45 ` Thomas Petazzoni
2018-09-13 7:45 ` Thomas Petazzoni
2018-09-13 7:45 ` Thomas Petazzoni
2018-09-13 8:20 ` Jan Kundrát
2018-09-13 8:20 ` Jan Kundrát
2018-09-13 8:20 ` Jan Kundrát
2018-09-13 8:42 ` Thomas Petazzoni [this message]
2018-09-13 8:42 ` Thomas Petazzoni
2018-09-13 8:42 ` Thomas Petazzoni
2018-09-24 10:02 ` Jan Kundrát
2018-09-24 10:02 ` Jan Kundrát
2018-09-24 10:10 ` Thomas Petazzoni
2018-09-24 10:10 ` Thomas Petazzoni
2018-09-24 10:12 ` Russell King - ARM Linux
2018-09-24 10:12 ` Russell King - ARM Linux
2018-09-24 10:26 ` Thomas Petazzoni
2018-09-24 10:26 ` Thomas Petazzoni
2018-09-24 11:13 ` Russell King - ARM Linux
2018-09-24 11:13 ` Russell King - ARM Linux
2018-09-24 12:12 ` Thomas Petazzoni
2018-09-24 12:12 ` Thomas Petazzoni
2018-09-24 12:46 ` Lorenzo Pieralisi
2018-09-24 12:46 ` Lorenzo Pieralisi
2018-09-24 13:10 ` Thomas Petazzoni
2018-09-24 13:10 ` Thomas Petazzoni
2018-09-24 14:15 ` Lorenzo Pieralisi
2018-09-24 14:15 ` Lorenzo Pieralisi
2018-09-24 14:52 ` Thomas Petazzoni
2018-09-24 14:52 ` Thomas Petazzoni
2018-09-24 16:42 ` Lorenzo Pieralisi
2018-09-24 16:42 ` Lorenzo Pieralisi
2018-10-01 10:56 ` Jan Kundrát
2018-10-01 10:56 ` Jan Kundrát
2018-10-01 12:51 ` Thomas Petazzoni
2018-10-01 12:51 ` Thomas Petazzoni
2018-10-01 21:01 ` Bjorn Helgaas
2018-10-01 21:01 ` Bjorn Helgaas
2018-09-25 8:18 ` Andrew Murray
2018-09-25 8:18 ` Andrew Murray
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180913104241.65db8243@windsurf \
--to=thomas.petazzoni@bootlin.com \
--cc=baruch@tkos.co.il \
--cc=bhelgaas@google.com \
--cc=jan.kundrat@cesnet.cz \
--cc=jason@lakedaemon.net \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=lorenzo.pieralisi@arm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.