Skip to content

[CI-NO-BUILD] [vioscsi] Align memory allocations#1313

Draft
benyamin-codez wants to merge 1 commit intovirtio-win:masterfrom
benyamin-codez:vioscsi-mem-align
Draft

[CI-NO-BUILD] [vioscsi] Align memory allocations#1313
benyamin-codez wants to merge 1 commit intovirtio-win:masterfrom
benyamin-codez:vioscsi-mem-align

Conversation

@benyamin-codez
Copy link
Copy Markdown
Contributor

Refactors VioScsiFindAdapter() to:

  1. Pad the uncached extension to ensure it is page-aligned
  2. Update memory layout art
  3. Improve relevant instrumentation
  4. Adds VIRTIO_SCSI_MSI_CONTROL_Q_OFFSET and updates the QUEUE_TO_MESSAGE(QueueId) and MESSAGE_TO_QUEUE(MessageId) macros to use it (vioscsi.h)

Refactors VioScsiFindAdapter() to:

1. Pad the uncached extension to ensure it is page-aligned
2. Update memory layout art
3. Improve relevant instrumentation
4. Adds VIRTIO_SCSI_MSI_CONTROL_Q_OFFSET and updates the
   QUEUE_TO_MESSAGE(QueueId) and MESSAGE_TO_QUEUE(MessageId)
   macros to use it (vioscsi.h)

Signed-off-by: benyamin-codez <115509179+benyamin-codez@users.noreply.github.com>
@benyamin-codez
Copy link
Copy Markdown
Contributor Author

Shares some definitions with PR #1306. Waiting for that to merge before rebasing.

PR 1313 : I pray the devil isn't in the detail.... 8^O

@YanVugenfirer
Copy link
Copy Markdown
Collaborator

[Jenkins CI]: Can one of the admins verify this patch?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants