Drivers: hv: hv_common: use meaningful errnos for hypercall statuses

Several HV_STATUS_* codes were collapsed to -EIO in commit 3817854
("hyperv: Log hypercall status codes as strings") when hv_result_to_errno()
was converted from a switch statement to a table-driven lookup. The change
was a side-effect of consolidating string/errno mappings into one place;
the commit message focused on adding hv_status_printk() for debugging.

The previous OOT helper hv_status_to_errno() (in v6.12) preserved more
meaningful errnos for several codes that have natural POSIX counterparts.
The collapse-to-EIO loses information at call sites that key behavior on
the converted errno (for example mshv_vtl_configure_reg_page() whitelists
-EINVAL/-EACCES to fall back gracefully when the host refuses to install
the register-page overlay; with -EIO that call site now hits a BUG()).

Restore the semantically meaningful mappings for the affected statuses:

  HV_STATUS_ACCESS_DENIED                    -EIO  -> -EACCES
  HV_STATUS_OPERATION_DENIED                 -EIO  -> -EACCES
  HV_STATUS_UNKNOWN_PROPERTY                 -EIO  -> -EINVAL
  HV_STATUS_PROPERTY_VALUE_OUT_OF_RANGE      -EIO  -> -EINVAL
  HV_STATUS_PROCESSOR_FEATURE_NOT_SUPPORTED  -EIO  -> -EOPNOTSUPP

EACCES is the canonical errno for permission denied, EINVAL for invalid
argument, and EOPNOTSUPP for unsupported operations. These mappings match
both their <errno.h> definitions and the established meanings in the OOT
v6.12 helper.

Also drop two duplicate _STATUS_INFO() rows for HV_STATUS_INVALID_LP_INDEX
and HV_STATUS_INVALID_REGISTER_VALUE that were leftover from the original
table conversion. find_hv_status_info() is first-match-wins so the dead
rows were not reachable, but they are confusing on review.

Impact analysis
---------------

The change only matters at sites that distinguish between specific errnos
(e.g. `ret == -EIO` vs `ret == -EACCES`). To confirm safety, the entire
kernel tree was audited for equality comparisons against any of the
affected errnos in Hyper-V-touching code:

  $ git grep -nE '(ret|status|err|rc|r)[[:space:]]*==[[:space:]]*\
        -(EACCES|EBADFD|ENOTRECOVERABLE|EOPNOTSUPP|EIO|EINVAL|ENOMEM)' \
    -- drivers/hv/ arch/x86/hyperv/ arch/arm64/hyperv/ \
       drivers/iommu/hyperv-iommu.c

Results in Hyper-V code:

  drivers/hv/hv_balloon.c:    ret == -EEXIST | ret == -EAGAIN  (not in table)
  drivers/hv/mshv_eventfd.c:  ret == -EAGAIN                    (not in table)
  drivers/hv/connection.c:    ret == -ETIMEDOUT                 (not in table)
  drivers/hv/mshv_vtl_main.c: ret == -EINVAL || ret == -EACCES  (the consumer
                              this patch fixes)

No other in-tree code path inspects -EOPNOTSUPP, -EACCES, -EINVAL or -EIO
from a hypercall result to make a behavioral decision. The remaining
HV_STATUS_* codes that 3817854 mapped to -EIO are therefore left as
-EIO; nothing keys behavior off the distinction today.

Signed-off-by: Hardik Garg <hargar@microsoft.com>

Author: hargar19

drivers/hv/hv_common.c

@@ -869,11 +869,11 @@ static const struct hv_status_info hv_status_infos[] = {
 	_STATUS_INFO(HV_STATUS_INVALID_HYPERCALL_INPUT,		-EINVAL),
 	_STATUS_INFO(HV_STATUS_INVALID_ALIGNMENT,		-EIO),
 	_STATUS_INFO(HV_STATUS_INVALID_PARAMETER,		-EINVAL),
-	_STATUS_INFO(HV_STATUS_ACCESS_DENIED,			-EIO),
+	_STATUS_INFO(HV_STATUS_ACCESS_DENIED,			-EACCES),
 	_STATUS_INFO(HV_STATUS_INVALID_PARTITION_STATE,		-EIO),
-	_STATUS_INFO(HV_STATUS_OPERATION_DENIED,		-EIO),
-	_STATUS_INFO(HV_STATUS_UNKNOWN_PROPERTY,		-EIO),
-	_STATUS_INFO(HV_STATUS_PROPERTY_VALUE_OUT_OF_RANGE,	-EIO),
+	_STATUS_INFO(HV_STATUS_OPERATION_DENIED,		-EACCES),
+	_STATUS_INFO(HV_STATUS_UNKNOWN_PROPERTY,		-EINVAL),
+	_STATUS_INFO(HV_STATUS_PROPERTY_VALUE_OUT_OF_RANGE,	-EINVAL),
 	_STATUS_INFO(HV_STATUS_INSUFFICIENT_MEMORY,		-ENOMEM),
 	_STATUS_INFO(HV_STATUS_INVALID_PARTITION_ID,		-EINVAL),
 	_STATUS_INFO(HV_STATUS_INVALID_VP_INDEX,		-EINVAL),
@@ -884,11 +884,9 @@ static const struct hv_status_info hv_status_infos[] = {
 	_STATUS_INFO(HV_STATUS_NOT_ACKNOWLEDGED,		-EIO),
 	_STATUS_INFO(HV_STATUS_INVALID_VP_STATE,		-EIO),
 	_STATUS_INFO(HV_STATUS_NO_RESOURCES,			-EIO),
-	_STATUS_INFO(HV_STATUS_PROCESSOR_FEATURE_NOT_SUPPORTED,	-EIO),
+	_STATUS_INFO(HV_STATUS_PROCESSOR_FEATURE_NOT_SUPPORTED,	-EOPNOTSUPP),
 	_STATUS_INFO(HV_STATUS_INVALID_LP_INDEX,		-EINVAL),
 	_STATUS_INFO(HV_STATUS_INVALID_REGISTER_VALUE,		-EINVAL),
-	_STATUS_INFO(HV_STATUS_INVALID_LP_INDEX,		-EIO),
-	_STATUS_INFO(HV_STATUS_INVALID_REGISTER_VALUE,		-EIO),
 	_STATUS_INFO(HV_STATUS_OPERATION_FAILED,		-EIO),
 	_STATUS_INFO(HV_STATUS_TIME_OUT,			-EIO),
 	_STATUS_INFO(HV_STATUS_CALL_PENDING,			-EIO),