Re: [PATCH v5 3/5] hyperv: Enable the hypercall output page for the VTL mode

From: Roman Kisel
Date: Wed Jan 08 2025 - 16:22:23 EST




On 1/8/2025 1:08 PM, Nuno Das Neves wrote:
On 12/30/2024 10:09 AM, Roman Kisel wrote:

[...]

Replying in a new thread since I don't have all the context about the different
VTL code paths that may need to converge. To my perspective, the approach in this
patch seems perfectly reasonable to fix the current issue.

One small improvement might be to put this check into a helper function. That would
make it easier to amend later when/if a clearly better solution is proposed.

Something like:

static inline bool hv_output_page_exists()
{
return hv_root_partition || IS_ENABLED(CONFIG_HYPERV_VTL_MODE);
}

Will apply, thank you for the suggestion, Nuno!


Thanks
Nuno

--
Thank you,
Roman