Skip to content

riscv: rawfile also publishes its SR as sm-data/mem-pool - #11

Open
gounthar wants to merge 1 commit into
baptleduc:xapi-riscvfrom
gounthar:fix/rawfile-sm-data-mem-pool
Open

gounthar wants to merge 1 commit into
baptleduc:xapi-riscvfrom
gounthar:fix/rawfile-sm-data-mem-pool

Conversation

@gounthar

Copy link
Copy Markdown

Stock SM puts a disk's SR uuid under the VBD backend's sm-data/mem-pool; every vbd3 node on two x86_64 XCP-ng lab hosts has it. rawfile only writes sm-data/sr-uuid, so a tool that follows the stock key finds the VDI but not the SR. xentop-ng is one: it files the guest disk under an SR named /dev and shows the rawfile SR with no disk.

This writes mem-pool too, same value. sr-uuid stays, because the xcp-rrdd-iostat change on the riscv xen-api branch reads it.

Tested on the RISC-V dom0 with riscv1 running: the VBD node has both keys, xentop-ng (with olivierlambert/xentop-ng#21) shows the disk under SR Disk as VDI trixie-root, and the iostat SR data sources still move between two reads.

This work was assisted by an LLM.

Stock SM puts a disk's SR uuid under the VBD backend's
sm-data/mem-pool: every vbd3 node on two x86_64 XCP-ng lab hosts has
it. rawfile writes the same uuid as sm-data/sr-uuid only, so a tool
that follows the stock key finds the disk's VDI but not its SR.
xentop-ng is one: it files the guest disk under an SR named "/dev"
(the directory of /dev/loop0) and shows the rawfile SR with no disk.

Write mem-pool as well, with the same value. sr-uuid stays: the
xcp-rrdd-iostat change on the riscv xen-api branch reads it, and
renaming it would break disk stats until that changes too.

Tested on the RISC-V dom0 with this rawfileSR installed and riscv1
started: the VBD node has mem-pool and sr-uuid, both the SR uuid.
xentop-ng (with its sm-data change for blkback backends) shows the
disk under SR "Disk" (rawfile) as VDI trixie-root, and the SR with one
VBD. xcp-rrdd-iostat still feeds the SR data sources
(io_throughput_write_<sr>, iops_write_<sr> change between two reads).

Signed-off-by: Bruno Verachten <gounthar@gmail.com>
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.

1 participant