On 11/20/2023 5:26 PM, David Hildenbrand wrote:

... did you shamelessly copy that from hw/virtio/virtio-mem.c ? ;)

Get caught.

This should be factored out into a common helper.

Sure, will do it in next version.

Factor it out in a separate patch. Then, this patch is get small that you can just squash it into #2.

A silly question. What file should the factored function be put in?

And my comment regarding "flags = 0" to patch #2 does no longer apply :)



Reply via email to