Skip to content
This repository was archived by the owner on May 12, 2021. It is now read-only.

kernel: disable pci shpc hotplug for arm64#498

Merged
jodh-intel merged 1 commit into
kata-containers:masterfrom
jongwu:nic
May 20, 2019
Merged

kernel: disable pci shpc hotplug for arm64#498
jodh-intel merged 1 commit into
kata-containers:masterfrom
jongwu:nic

Conversation

@jongwu
Copy link
Copy Markdown
Contributor

@jongwu jongwu commented May 9, 2019

As shpchp used for pci hotplug on arm64 initialized
its bottom half work as a delay work for 5 seconds, pci bus
rescan triggered between up half and bottom half of shpc interrupt
handling will fail. so disable shpc and let bus rescan
to do the device hotplug on arm64.

Fixes: #499
Signed-off-by: Jianyong Wu jianyong.wu@arm.com

@jodh-intel @Pennyzct @bergwolf @grahamwhaley

Copy link
Copy Markdown

@jodh-intel jodh-intel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@grahamwhaley, @ganeshmaharaj - this option is also set for x86_64 so we'd better double-check if it should be.

@grahamwhaley
Copy link
Copy Markdown
Contributor

@devimc for input around bus rescanning pci and stuff I think....

@jongwu
Copy link
Copy Markdown
Contributor Author

jongwu commented May 9, 2019

As far as I know, x86 use ACPI to do pci device hotplug, so it will no this issue I think.

@devimc
Copy link
Copy Markdown

devimc commented May 9, 2019

/test

Copy link
Copy Markdown

@devimc devimc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

jongwu added a commit to jongwu/packaging that referenced this pull request May 10, 2019
As shpchp used for pci hotplug on arm64 initialized
its bottom half work as a delay work for 5 seconds, pci bus
rescan triggered between up half and bottom half of shpc interrupt
handling will fail. so disable shpc and let bus rescan
to do the device hotplug on arm64.

Fixes: kata-containers#498
Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>
@jongwu
Copy link
Copy Markdown
Contributor Author

jongwu commented May 13, 2019

@devimc help retest, thank you.

@jodh-intel
Copy link
Copy Markdown

@jongwu - here you go...

/retest

As shpchp used for pci hotplug on arm64 initialized
its bottom half work as a delay work for 5 seconds, pci bus
rescan triggered between up half and bottom half of shpc interrupt
handling will fail. so disable shpc and let bus rescan
to do the device hotplug on arm64.

Fixes: kata-containers#498
Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>
@jongwu
Copy link
Copy Markdown
Contributor Author

jongwu commented May 13, 2019

@jodh-intel need retest, thank you!

@devimc
Copy link
Copy Markdown

devimc commented May 13, 2019

/test

@jodh-intel jodh-intel merged commit 4d8d841 into kata-containers:master May 20, 2019
Pennyzct added a commit to Pennyzct/packaging that referenced this pull request Apr 9, 2020
Since we disable pci shpc hotplug for arm64, see
kata-containers#498 for detailed
reason.
We need to move CONFIG_HOTPLUG_PCI_SHPC from common conf to
x86_64-specific.

Fixes: kata-containers#1004

Signed-off-by: Penny Zheng <penny.zheng@arm.com>
Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>
Pennyzct added a commit to Pennyzct/packaging that referenced this pull request Apr 15, 2020
Since we disable pci shpc hotplug for arm64, see
kata-containers#498 for detailed
reason.
We need to move CONFIG_HOTPLUG_PCI_SHPC from common conf to
x86_64-specific.

Fixes: kata-containers#1004

Signed-off-by: Penny Zheng <penny.zheng@arm.com>
Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>
Pennyzct added a commit to Pennyzct/packaging that referenced this pull request Apr 17, 2020
Since we disable pci shpc hotplug for arm64, see
kata-containers#498 for detailed
reason.
We need to move CONFIG_HOTPLUG_PCI_SHPC from common conf to
x86_64-specific.

Fixes: kata-containers#1004

Signed-off-by: Penny Zheng <penny.zheng@arm.com>
Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>
Pennyzct added a commit to Pennyzct/packaging that referenced this pull request Apr 21, 2020
Since we disable pci shpc hotplug for arm64, see
kata-containers#498 for detailed
reason.
We need to move CONFIG_HOTPLUG_PCI_SHPC from common conf to
x86_64-specific.

Fixes: kata-containers#1004

Signed-off-by: Penny Zheng <penny.zheng@arm.com>
Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

kernel: disable pci shpc hotplug for arm64

4 participants