Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

devices: fix pv_panic alignment #5956

Merged
merged 1 commit into from
Nov 16, 2023
Merged

devices: fix pv_panic alignment #5956

merged 1 commit into from
Nov 16, 2023

Conversation

thomasbarrett
Copy link
Contributor

The PvPanicDevice has a 2-byte BAR size, which is smaller than the 16-byte minimum legacy PCI-Express BAR size and the 128-byte minimum modern PCI-Express BAR size. The base address register masks off the lower four bits, so all BARs must be at least 16-byte aligned (although the PCI specification requires that devices are aligned to their size). Despite being smaller than the minimum BAR size, the PvPanic device appears to work as long as it's BAR is 16-byte aligned.

@thomasbarrett thomasbarrett requested a review from a team as a code owner November 15, 2023 17:30
devices: fix pv_panic alignment
820dc9f 
Signed-off-by: Thomas Barrett <tbarrett@crusoeenergy.com>
@thomasbarrett thomasbarrett mentioned this pull request Nov 15, 2023
Merged
@rbradford rbradford merged commit 5f3ff3c into cloud-hypervisor:main Nov 16, 2023
22 checks passed
@likebreath likebreath added the bug-fix Bug fix to include in release notes label Nov 27, 2023
This was referenced Jan 9, 2024
Merged
Merged
russell-islam pushed a commit to microsoft/cloud-hypervisor that referenced this pull request Apr 12, 2024
@weltling
Merge tag 'v37.0' into anbelski/msft-main-37.0
2300341 
v37.0

This release has been tracked in our [roadmap
project](https://github.com/orgs/cloud-hypervisor/projects/6) as iteration
v37.0. The following user visible changes have been made:

Long Term Support (LTS) Release
-------------------------------

This release is a LTS release. Point releases for bug fixes will be made
for the next 18 months; live migration and live upgrade will be
supported between the point releases of the LTS.

Multiple PCI segments Support for 32-bit VFIO devices
-----------------------------------------------------

Now VFIO devices with 32-bit memory BARs can be attached to non-zero PCI
segments on the guest, allowing users to have more 32-bit devices and
assign such devices to appropriate NUMA nodes for better performance.

Configurable Named TAP Devices
------------------------------

Named TAP devices now accepts IP configuration from users, such as IP
and MAC address, as long as the named TAP device is created by Cloud
Hypervisor (e.g. not existing TAP devices).

TTY Output from Both Serial Device and Virtio Console
-----------------------------------------------------

Now legacy serial device and virtio console can be set as TTY mode as
the same time. This allows users to capture early boot logs with the
legacy serial device without losing performance benefits of using
virtio-console, when appropriate kernel configuration is used (such as
using kernel command-line `console=hvc0 earlyprintk=ttyS0` on x86).

Faster VM Restoration from Snapshots
------------------------------------

The speed of VM restoration from snapshots is improved with a better
implementation of deserializing JSON files.

Notable Bug Fixes
-----------------

* Fix aio backend behavior for block devices when writeback cache
  disabled (cloud-hypervisor#5930)
* Fix PvPanic device PCI BAR alignment (cloud-hypervisor#5956)
* Bug fix to OpenAPI specification file (cloud-hypervisor#5967)
* Error out early for live migration when TDX is enabled (cloud-hypervisor#6025)

Contributors
------------

Many thanks to everyone who has contributed to our release:

* Bo Chen <chen.bo@intel.com>
* Jinank Jain <jinankjain@microsoft.com>
* Markus Sütter <markus.suetter@secunet.com>
* Michael Zhao <michael.zhao@arm.com>
* Muminul Islam <muislam@microsoft.com>
* Rob Bradford <rbradford@rivosinc.com>
* Rui Chang <rui.chang@arm.com>
* Ruslan Mstoi <ruslan.mstoi@intel.com>
* Thomas Barrett <tbarrett@crusoeenergy.com>
* Wei Liu <liuwe@microsoft.com>
* Yi Wang <foxywang@tencent.com>
* Yong He <alexyonghe@tencent.com>

* tag 'v37.0': (540 commits)
  build: Release v37.0
  tests: Stabilize 'test_vfio_user' with retries to run host commands
  arch: x86_64: Refactor the way to generate e820 RAM maps
  build: Bump async-executor from 1.5.1 to 1.8.0
  build: Bump once_cell from 1.18.0 to 1.19.0 in /fuzz
  build: Bump mshv-ioctls from `6901f9c` to `0dd4d34`
  build: Bump anstyle-query from 1.0.0 to 1.0.2 in /fuzz
  vmm: tdx: Error out early for TD migration
  vmm: Fix a typo from send_migration()
  hypervisor: fix few typos and cosmetic issues
  build: Bump pin-project from 1.1.2 to 1.1.3
  build: Bump ryu from 1.0.15 to 1.0.16 in /fuzz
  docs: Add a missing backslash
  vmm: igvm: Remove redundant copy_from_slice
  github: ci: Move to action/checkout v4
  hypervisor: vmm: Fix warnings in Cargo.toml
  build: Bump futures-core from 0.3.28 to 0.3.29
  build: Bump anstream from 0.6.4 to 0.6.5 in /fuzz
  vmm: Pass IGVM file to the loader
  vmm: Add igvm module and loader module
  ...
russell-islam pushed a commit to microsoft/cloud-hypervisor that referenced this pull request Apr 12, 2024
@weltling
Merge tag 'v37.0' into anbelski/msft-main-37.0_2
e9b5753 
v37.0

This release has been tracked in our [roadmap
project](https://github.com/orgs/cloud-hypervisor/projects/6) as iteration
v37.0. The following user visible changes have been made:

Long Term Support (LTS) Release
-------------------------------

This release is a LTS release. Point releases for bug fixes will be made
for the next 18 months; live migration and live upgrade will be
supported between the point releases of the LTS.

Multiple PCI segments Support for 32-bit VFIO devices
-----------------------------------------------------

Now VFIO devices with 32-bit memory BARs can be attached to non-zero PCI
segments on the guest, allowing users to have more 32-bit devices and
assign such devices to appropriate NUMA nodes for better performance.

Configurable Named TAP Devices
------------------------------

Named TAP devices now accepts IP configuration from users, such as IP
and MAC address, as long as the named TAP device is created by Cloud
Hypervisor (e.g. not existing TAP devices).

TTY Output from Both Serial Device and Virtio Console
-----------------------------------------------------

Now legacy serial device and virtio console can be set as TTY mode as
the same time. This allows users to capture early boot logs with the
legacy serial device without losing performance benefits of using
virtio-console, when appropriate kernel configuration is used (such as
using kernel command-line `console=hvc0 earlyprintk=ttyS0` on x86).

Faster VM Restoration from Snapshots
------------------------------------

The speed of VM restoration from snapshots is improved with a better
implementation of deserializing JSON files.

Notable Bug Fixes
-----------------

* Fix aio backend behavior for block devices when writeback cache
  disabled (cloud-hypervisor#5930)
* Fix PvPanic device PCI BAR alignment (cloud-hypervisor#5956)
* Bug fix to OpenAPI specification file (cloud-hypervisor#5967)
* Error out early for live migration when TDX is enabled (cloud-hypervisor#6025)

Contributors
------------

Many thanks to everyone who has contributed to our release:

* Bo Chen <chen.bo@intel.com>
* Jinank Jain <jinankjain@microsoft.com>
* Markus Sütter <markus.suetter@secunet.com>
* Michael Zhao <michael.zhao@arm.com>
* Muminul Islam <muislam@microsoft.com>
* Rob Bradford <rbradford@rivosinc.com>
* Rui Chang <rui.chang@arm.com>
* Ruslan Mstoi <ruslan.mstoi@intel.com>
* Thomas Barrett <tbarrett@crusoeenergy.com>
* Wei Liu <liuwe@microsoft.com>
* Yi Wang <foxywang@tencent.com>
* Yong He <alexyonghe@tencent.com>

* tag 'v37.0': (540 commits)
  build: Release v37.0
  tests: Stabilize 'test_vfio_user' with retries to run host commands
  arch: x86_64: Refactor the way to generate e820 RAM maps
  build: Bump async-executor from 1.5.1 to 1.8.0
  build: Bump once_cell from 1.18.0 to 1.19.0 in /fuzz
  build: Bump mshv-ioctls from `6901f9c` to `0dd4d34`
  build: Bump anstyle-query from 1.0.0 to 1.0.2 in /fuzz
  vmm: tdx: Error out early for TD migration
  vmm: Fix a typo from send_migration()
  hypervisor: fix few typos and cosmetic issues
  build: Bump pin-project from 1.1.2 to 1.1.3
  build: Bump ryu from 1.0.15 to 1.0.16 in /fuzz
  docs: Add a missing backslash
  vmm: igvm: Remove redundant copy_from_slice
  github: ci: Move to action/checkout v4
  hypervisor: vmm: Fix warnings in Cargo.toml
  build: Bump futures-core from 0.3.28 to 0.3.29
  build: Bump anstream from 0.6.4 to 0.6.5 in /fuzz
  vmm: Pass IGVM file to the loader
  vmm: Add igvm module and loader module
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@liuw liuw liuw approved these changes

@rbradford rbradford rbradford approved these changes

@jinankjain jinankjain jinankjain approved these changes

@likebreath likebreath likebreath approved these changes

Assignees
No one assigned
Labels
bug-fix Bug fix to include in release notes
Projects
Cloud Hypervisor Roadmap
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@thomasbarrett @liuw @rbradford @jinankjain @likebreath