Skip to content
This repository has been archived by the owner on May 6, 2020. It is now read-only.

kernel: add support for native hot plug #133

Merged
merged 1 commit into from
Oct 4, 2017

Conversation

devimc
Copy link

@devimc devimc commented Oct 3, 2017

Next kernel configs are needed to support native hot plug:

  • CONFIG_HOTPLUG_PCI=y
  • CONFIG_PCIEASPM_DEFAULT=y
  • CONFIG_PCIEASPM=y
  • CONFIG_HOTPLUG_PCI_PCIE=y
  • CONFIG_PCIEAER=y

enabling above configs is not enough, pci probe patches
must be removed

Fixes #132

Signed-off-by: Julio Montes [email protected]

@devimc devimc force-pushed the native_hotplug branch 2 times, most recently from 022d28f to 9196d75 Compare October 3, 2017 18:42
@devimc devimc changed the title Add support for native hot plug kernel: add support for native hot plug Oct 3, 2017
@miguelinux miguelinux self-requested a review October 3, 2017 22:08
Copy link
Contributor

@miguelinux miguelinux left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link

@gorozco1 gorozco1 left a comment

Choose a reason for hiding this comment

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

@devimc Could you add a short description of the change and do the same in the issue so we have in the history why this change was made please.

Next kernel configs are needed to support native hot plug:
* CONFIG_HOTPLUG_PCI=y
* CONFIG_PCIEASPM_DEFAULT=y
* CONFIG_PCIEASPM=y
* CONFIG_HOTPLUG_PCI_PCIE=y
* CONFIG_PCIEAER=y

enabling above configs is not enough, pci probe patches
must be removed

Fixes clearcontainers#132

Signed-off-by: Julio Montes <[email protected]>
@devimc
Copy link
Author

devimc commented Oct 4, 2017

@gorozco1 done

@gorozco1 gorozco1 merged commit d6fe783 into clearcontainers:master Oct 4, 2017
gorozco1 added a commit to gorozco1/packaging that referenced this pull request Oct 10, 2017
This update includes patches added in clearcontainers#110 and clearcontainers#133

Fixes: clearcontainers#143

Signed-off-by: Geronimo Orozco <[email protected]>
gorozco1 added a commit to gorozco1/packaging that referenced this pull request Oct 10, 2017
This update includes patches added in clearcontainers#110 and clearcontainers#133

Fixes: clearcontainers#143

Signed-off-by: Geronimo Orozco <[email protected]>
gorozco1 added a commit to gorozco1/packaging that referenced this pull request Oct 10, 2017
This update includes patches added in clearcontainers#110 and clearcontainers#133

Fixes: clearcontainers#143

Signed-off-by: Geronimo Orozco <[email protected]>
gorozco1 added a commit to gorozco1/packaging that referenced this pull request Oct 10, 2017
This update includes patches added in clearcontainers#110 and clearcontainers#133

Fixes: clearcontainers#143

Signed-off-by: Geronimo Orozco <[email protected]>
gorozco1 added a commit to gorozco1/tests that referenced this pull request Oct 23, 2017
We need to update clear-containers rootfs image and the kernel that
is being used in testing.

  * Image:
    Updated agent with a fix for deadlock
    clearcontainers/packaging#158

  * Kernel:
    Add support for native hot plug
    clearcontainers/packaging#133

    Fix 9p fs unlink-ftruncate
    clearcontainers/packaging#110

Fixes clearcontainers#631

Signed-off-by: Geronimo Orozco <[email protected]>
egernst pushed a commit to egernst/cc-packaging that referenced this pull request Aug 14, 2018
The Integration with docker link in
README is broken. Fix it.

Fixes: clearcontainers#133

Signed-off-by: Nitesh Konkar [email protected]
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.

3 participants