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

devices: add per-device VFIO ids #721

Merged
merged 1 commit into from
Sep 13, 2018

Conversation

eguzman3
Copy link
Contributor

Backport of: #715
Adds per-device VFIO ids allowing IOMMU groups with
multiple devices to be passed to qemu.

Fixes #708

Signed-off-by: Edward Guzman eguzman@nvidia.com

Adds per-device VFIO ids allowing IOMMU groups with
multiple devices to be passed to qemu.

Fixes kata-containers#708

Signed-off-by: Edward Guzman <eguzman@nvidia.com>
@katacontainersbot
Copy link
Contributor

PSS Measurement:
Qemu: 167334 KB
Proxy: 4198 KB
Shim: 8858 KB

Memory inside container:
Total Memory: 2043460 KB
Free Memory: 2003564 KB

@opendev-zuul
Copy link

opendev-zuul bot commented Sep 12, 2018

Build failed (third-party-check pipeline) integration testing with
OpenStack. For information on how to proceed, see
http://docs.openstack.org/infra/manual/developers.html#automated-testing

@codecov
Copy link

codecov bot commented Sep 13, 2018

Codecov Report

Merging #721 into stable-1.2 will not change coverage.
The diff coverage is 0%.

@@             Coverage Diff             @@
##           stable-1.2     #721   +/-   ##
===========================================
  Coverage       66.89%   66.89%           
===========================================
  Files              85       85           
  Lines            9161     9161           
===========================================
  Hits             6128     6128           
  Misses           2394     2394           
  Partials          639      639

@WeiZhang555
Copy link
Member

WeiZhang555 commented Sep 13, 2018

LGTM

Approved with PullApprove

@bergwolf
Copy link
Member

bergwolf commented Sep 13, 2018

LGTM

Approved with PullApprove

@bergwolf bergwolf merged commit d52f426 into kata-containers:stable-1.2 Sep 13, 2018
@bergwolf
Copy link
Member

Is this needed in stable-1.1?

@eguzman3
Copy link
Contributor Author

Is this needed in stable-1.1?

It looks like this bug was introduced after stable-1.1, so it should be fine.

@sboeuf
Copy link

sboeuf commented Sep 13, 2018

Yes this was introduced as part of the whole device model rework, which was introduced in 1.2.

egernst pushed a commit to egernst/runtime that referenced this pull request Feb 9, 2021
Do not ignore ipv6 addresses and routes. These are not processes along
with ipv4 routes.
Add tests to verify ipv6 addresses and routes are processed
with `updateRoutes`.

Fixes kata-containers#721

Signed-off-by: Archana Shinde <archana.m.shinde@intel.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.

5 participants