Received: by 2002:a05:6a10:206:0:0:0:0 with SMTP id 6csp48260pxj; Wed, 23 Jun 2021 15:18:55 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzXYntyoSX+1wU8+y10sYAfSP5krDUJ7UIzNmQ2QdY61NbJwAQg2y0K+94gI9AEHeMSzcSX X-Received: by 2002:aa7:c902:: with SMTP id b2mr2602277edt.287.1624486735458; Wed, 23 Jun 2021 15:18:55 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1624486735; cv=none; d=google.com; s=arc-20160816; b=BlsXVpt+bIgRH/tSvTRZi+C100qWLTL8GUTFhqNmyp8T1vqVkcZD8yxJTcXmZN5Fnz EBeM1fHiKibhg6RMhMtTI0vBRTbEcaVzx0cQgcJ1ZiMe88VkdBDQtLtQ+CGOxwY+0XGC iRzEMSGc0hvSaVdF55I907ZTC+UgzM09eOVhwci5HPZnKuhrAf+bgMqog6+yw2XWKKSK ENqoEN34S1v3GPQ1wl/NmDkHiNYLjEdIDeofdEjqTiFZTgHVPBZMe2bIejYGoftaaTfl B3qnckpXpOvK7heJ35as07tBFY+xcJLYi+loaYACtDWKRIZrTGSVnEGJUQJpwBnbAUe7 hbdA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:from:references :cc:to:subject:dkim-signature:dkim-filter; bh=fcz0zmv3fb9TLhXJgjPYUIlkJMTZb0BSgbSLG1PJe/c=; b=j3v1dknMp0VK6tIdzv8IQPYi2jzX0STTloCEX0Z6YgKCZeDWZ5Sv57j1Yt68cnIlli +kv9F4Ao29+gnd9ooZoGbEns/ezLWmKJKpc036bI06ZdSCAgz8DcL7N82jAUKzYCXEHY gCeEjPszsUKVLnNwLGDn5FQoQKiMN0OUJ6h7XKPQVseNRuaMVlMicKBsKcY7HKp4T6UE 6UOmR+EYCVGnh5m4lXx/6Fwoo8TibcqHpRzGDIX0HMuy5+nA66Pfs+Oh2bbszP27LYI1 ICgGu2PCjsrCNvoAMH4oprJHqCQ5b9tlDufTKyZKxidMPvI/yFFKNoJoskwapz9Xm0Cb KM/A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linux.microsoft.com header.s=default header.b="K/zy6V8t"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linux.microsoft.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id hz10si791067ejc.378.2021.06.23.15.18.28; Wed, 23 Jun 2021 15:18:55 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linux.microsoft.com header.s=default header.b="K/zy6V8t"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linux.microsoft.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230123AbhFWWR5 (ORCPT + 99 others); Wed, 23 Jun 2021 18:17:57 -0400 Received: from linux.microsoft.com ([13.77.154.182]:60830 "EHLO linux.microsoft.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229996AbhFWWRp (ORCPT ); Wed, 23 Jun 2021 18:17:45 -0400 Received: from [10.0.0.178] (c-67-168-106-253.hsd1.wa.comcast.net [67.168.106.253]) by linux.microsoft.com (Postfix) with ESMTPSA id F296220B7188; Wed, 23 Jun 2021 15:15:26 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com F296220B7188 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com; s=default; t=1624486527; bh=fcz0zmv3fb9TLhXJgjPYUIlkJMTZb0BSgbSLG1PJe/c=; h=Subject:To:Cc:References:From:Date:In-Reply-To:From; b=K/zy6V8tjn5/uf1v4mmYfqxhlY3QHJBwxGON+7R1mbtv2BoFN0Dno4newrA6ewY09 vhZBpeyipeDNpQPrzfkHW24O8R3Z9cflAZ3HHGx+UJb+NqVE4VwU5JCwo4BSQUwErM uvmHUnYIvJgOW2BC/ocUtfEEaj7lfOgsj24ZaMgM= Subject: Re: [PATCH 02/19] asm-generic/hyperv: convert hyperv statuses to strings To: Sunil Muthuswamy , "linux-hyperv@vger.kernel.org" , "linux-kernel@vger.kernel.org" Cc: "virtualization@lists.linux-foundation.org" , Michael Kelley , "viremana@linux.microsoft.com" , "wei.liu@kernel.org" , vkuznets , Lillian Grassin-Drake , KY Srinivasan References: <1622241819-21155-1-git-send-email-nunodasneves@linux.microsoft.com> <1622241819-21155-3-git-send-email-nunodasneves@linux.microsoft.com> From: Nuno Das Neves Message-ID: <0c8d9808-3bde-da95-a7fb-cd294b77b6f3@linux.microsoft.com> Date: Wed, 23 Jun 2021 15:15:26 -0700 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 6/10/2021 11:42 AM, Sunil Muthuswamy wrote: > > >> -----Original Message----- >> From: Nuno Das Neves >> Sent: Friday, May 28, 2021 3:43 PM >> To: linux-hyperv@vger.kernel.org; linux-kernel@vger.kernel.org >> Cc: virtualization@lists.linux-foundation.org; Michael Kelley ; viremana@linux.microsoft.com; Sunil >> Muthuswamy ; wei.liu@kernel.org; vkuznets ; Lillian Grassin-Drake >> ; KY Srinivasan >> Subject: [PATCH 02/19] asm-generic/hyperv: convert hyperv statuses to strings >> >> Allow hyperv hypercall failures to be debugged more easily with dmesg. >> This will be used in the mshv module. >> >> Signed-off-by: Nuno Das Neves >> --- >> arch/x86/hyperv/hv_init.c | 2 +- >> arch/x86/hyperv/hv_proc.c | 10 +++--- >> include/asm-generic/hyperv-tlfs.h | 52 ++++++++++++++++++------------- >> include/asm-generic/mshyperv.h | 8 +++++ >> 4 files changed, 44 insertions(+), 28 deletions(-) >> >> diff --git a/arch/x86/hyperv/hv_init.c b/arch/x86/hyperv/hv_init.c >> index bb0ae4b5c00f..722bafdb2225 100644 >> --- a/arch/x86/hyperv/hv_init.c >> +++ b/arch/x86/hyperv/hv_init.c >> @@ -349,7 +349,7 @@ static void __init hv_get_partition_id(void) >> status = hv_do_hypercall(HVCALL_GET_PARTITION_ID, NULL, output_page); >> if (!hv_result_success(status)) { >> /* No point in proceeding if this failed */ >> - pr_err("Failed to get partition ID: %lld\n", status); >> + pr_err("Failed to get partition ID: %s\n", hv_status_to_string(status)); >> BUG(); >> } >> hv_current_partition_id = output_page->partition_id; >> diff --git a/arch/x86/hyperv/hv_proc.c b/arch/x86/hyperv/hv_proc.c >> index 59cf9a9e0975..30951e778577 100644 >> --- a/arch/x86/hyperv/hv_proc.c >> +++ b/arch/x86/hyperv/hv_proc.c >> @@ -117,7 +117,7 @@ int hv_call_deposit_pages(int node, u64 partition_id, u32 num_pages) >> page_count, 0, input_page, NULL); >> local_irq_restore(flags); >> if (!hv_result_success(status)) { >> - pr_err("Failed to deposit pages: %lld\n", status); >> + pr_err("Failed to deposit pages: %s\n", hv_status_to_string(status)); >> ret = hv_status_to_errno(status); >> goto err_free_allocations; >> } >> @@ -172,8 +172,8 @@ int hv_call_add_logical_proc(int node, u32 lp_index, u32 apic_id) >> >> if (hv_result(status) != HV_STATUS_INSUFFICIENT_MEMORY) { >> if (!hv_result_success(status)) { >> - pr_err("%s: cpu %u apic ID %u, %lld\n", __func__, >> - lp_index, apic_id, status); >> + pr_err("%s: cpu %u apic ID %u, %s\n", __func__, >> + lp_index, apic_id, hv_status_to_string(status)); >> ret = hv_status_to_errno(status); >> } >> break; >> @@ -222,8 +222,8 @@ int hv_call_create_vp(int node, u64 partition_id, u32 vp_index, u32 flags) >> >> if (hv_result(status) != HV_STATUS_INSUFFICIENT_MEMORY) { >> if (!hv_result_success(status)) { >> - pr_err("%s: vcpu %u, lp %u, %lld\n", __func__, >> - vp_index, flags, status); >> + pr_err("%s: vcpu %u, lp %u, %s\n", __func__, >> + vp_index, flags, hv_status_to_string(status)); >> ret = hv_status_to_errno(status); >> } >> break; >> diff --git a/include/asm-generic/hyperv-tlfs.h b/include/asm-generic/hyperv-tlfs.h >> index fe6d41d0b114..40ff7cdd4a2b 100644 >> --- a/include/asm-generic/hyperv-tlfs.h >> +++ b/include/asm-generic/hyperv-tlfs.h >> @@ -189,28 +189,36 @@ enum HV_GENERIC_SET_FORMAT { >> #define HV_HYPERCALL_REP_START_MASK GENMASK_ULL(59, 48) >> >> /* hypercall status code */ >> -#define HV_STATUS_SUCCESS 0x0 >> -#define HV_STATUS_INVALID_HYPERCALL_CODE 0x2 >> -#define HV_STATUS_INVALID_HYPERCALL_INPUT 0x3 >> -#define HV_STATUS_INVALID_ALIGNMENT 0x4 >> -#define HV_STATUS_INVALID_PARAMETER 0x5 >> -#define HV_STATUS_ACCESS_DENIED 0x6 >> -#define HV_STATUS_INVALID_PARTITION_STATE 0x7 >> -#define HV_STATUS_OPERATION_DENIED 0x8 >> -#define HV_STATUS_UNKNOWN_PROPERTY 0x9 >> -#define HV_STATUS_PROPERTY_VALUE_OUT_OF_RANGE 0xA >> -#define HV_STATUS_INSUFFICIENT_MEMORY 0xB >> -#define HV_STATUS_INVALID_PARTITION_ID 0xD >> -#define HV_STATUS_INVALID_VP_INDEX 0xE >> -#define HV_STATUS_NOT_FOUND 0x10 >> -#define HV_STATUS_INVALID_PORT_ID 0x11 >> -#define HV_STATUS_INVALID_CONNECTION_ID 0x12 >> -#define HV_STATUS_INSUFFICIENT_BUFFERS 0x13 >> -#define HV_STATUS_NOT_ACKNOWLEDGED 0x14 >> -#define HV_STATUS_INVALID_VP_STATE 0x15 >> -#define HV_STATUS_NO_RESOURCES 0x1D >> -#define HV_STATUS_INVALID_LP_INDEX 0x41 >> -#define HV_STATUS_INVALID_REGISTER_VALUE 0x50 >> +#define __HV_STATUS_DEF(OP) \ >> + OP(HV_STATUS_SUCCESS, 0x0) \ >> + OP(HV_STATUS_INVALID_HYPERCALL_CODE, 0x2) \ >> + OP(HV_STATUS_INVALID_HYPERCALL_INPUT, 0x3) \ >> + OP(HV_STATUS_INVALID_ALIGNMENT, 0x4) \ >> + OP(HV_STATUS_INVALID_PARAMETER, 0x5) \ >> + OP(HV_STATUS_ACCESS_DENIED, 0x6) \ >> + OP(HV_STATUS_INVALID_PARTITION_STATE, 0x7) \ >> + OP(HV_STATUS_OPERATION_DENIED, 0x8) \ >> + OP(HV_STATUS_UNKNOWN_PROPERTY, 0x9) \ >> + OP(HV_STATUS_PROPERTY_VALUE_OUT_OF_RANGE, 0xA) \ >> + OP(HV_STATUS_INSUFFICIENT_MEMORY, 0xB) \ >> + OP(HV_STATUS_INVALID_PARTITION_ID, 0xD) \ >> + OP(HV_STATUS_INVALID_VP_INDEX, 0xE) \ >> + OP(HV_STATUS_NOT_FOUND, 0x10) \ >> + OP(HV_STATUS_INVALID_PORT_ID, 0x11) \ >> + OP(HV_STATUS_INVALID_CONNECTION_ID, 0x12) \ >> + OP(HV_STATUS_INSUFFICIENT_BUFFERS, 0x13) \ >> + OP(HV_STATUS_NOT_ACKNOWLEDGED, 0x14) \ >> + OP(HV_STATUS_INVALID_VP_STATE, 0x15) \ >> + OP(HV_STATUS_NO_RESOURCES, 0x1D) \ >> + OP(HV_STATUS_INVALID_LP_INDEX, 0x41) \ >> + OP(HV_STATUS_INVALID_REGISTER_VALUE, 0x50) >> + >> +#define __HV_MAKE_HV_STATUS_ENUM(NAME, VAL) NAME = (VAL), >> +#define __HV_MAKE_HV_STATUS_CASE(NAME, VAL) case (NAME): return (#NAME); >> + >> +enum hv_status { >> + __HV_STATUS_DEF(__HV_MAKE_HV_STATUS_ENUM) >> +}; >> >> /* >> * The Hyper-V TimeRefCount register and the TSC >> diff --git a/include/asm-generic/mshyperv.h b/include/asm-generic/mshyperv.h >> index 9a000ba2bb75..21fb71ca1ba9 100644 >> --- a/include/asm-generic/mshyperv.h >> +++ b/include/asm-generic/mshyperv.h >> @@ -59,6 +59,14 @@ static inline unsigned int hv_repcomp(u64 status) >> HV_HYPERCALL_REP_COMP_OFFSET; >> } >> >> +static inline const char *hv_status_to_string(u64 hv_status) >> +{ >> + switch (hv_result(hv_status)) { >> + __HV_STATUS_DEF(__HV_MAKE_HV_STATUS_CASE) >> + default : return "Unknown"; >> + } >> +} > Wouldn't this be a big switch statement that will get duplicated all over the place > in the code because of the inline (and also the strings within)? > I'm not totally sure. If so, I guess it should not be inline! > - Sunil >