From: Nicholas Piggin <npiggin@gmail.com> To: Haren Myneni <haren@linux.ibm.com>, herbert@gondor.apana.org.au, linux-crypto@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, mpe@ellerman.id.au Cc: haren@us.ibm.com, hbabu@us.ibm.com Subject: Re: [V3 PATCH 09/16] powerpc/pseries/vas: Implement to get all capabilities Date: Mon, 10 May 2021 16:13:08 +1000 [thread overview] Message-ID: <1620626553.7v3m168yl3.astroid@bobo.none> (raw) In-Reply-To: <f6cdf811a29c22056740d48fa3de010f4ea4b848.camel@linux.ibm.com> Excerpts from Haren Myneni's message of April 18, 2021 7:08 am: > > pHyp provides various VAS capabilities such as GZIP default and QoS > capabilities which are used to determine total number of credits > available in LPAR, maximum window credits, maximum LPAR credits, > whether usermode copy/paste is supported, and etc. > > So first retrieve overall vas capabilities using > H_QUERY_VAS_CAPABILITIES HCALL which tells the specific features that > are available. Then retrieve the specific capabilities by using the > feature type in H_QUERY_VAS_CAPABILITIES HCALL. > > pHyp supports only GZIP default and GZIP QoS capabilities right now. Changelog and title could use a bit of work. > > Signed-off-by: Haren Myneni <haren@linux.ibm.com> > --- > arch/powerpc/platforms/pseries/vas.c | 130 +++++++++++++++++++++++++++ > 1 file changed, 130 insertions(+) > > diff --git a/arch/powerpc/platforms/pseries/vas.c b/arch/powerpc/platforms/pseries/vas.c > index 06960151477c..35946fb02995 100644 > --- a/arch/powerpc/platforms/pseries/vas.c > +++ b/arch/powerpc/platforms/pseries/vas.c > @@ -30,6 +30,13 @@ > /* phyp allows one credit per window right now */ > #define DEF_WIN_CREDS 1 > > +static struct vas_all_capabs capabs_all; Does this name come from PAPR? If not, capabilities or caps are better for readability than capabs. > +static int copypaste_feat; Should be a bool? And what does it mean? copy-paste is a host core capability. > + > +struct vas_capabs vcapabs[VAS_MAX_FEAT_TYPE]; > + > +DEFINE_MUTEX(vas_pseries_mutex); Can these be made static if they're only used here, and export them if a future patch uses them (or add the header declaration now). > + > static int64_t hcall_return_busy_check(int64_t rc) > { > /* Check if we are stalled for some time */ > @@ -215,3 +222,126 @@ int plpar_vas_query_capabilities(const u64 hcall, u8 query_type, > return -EIO; > } > } > + > +/* > + * Get the specific capabilities based on the feature type. > + * Right now supports GZIP default and GZIP QoS capabilities. > + */ > +static int get_vas_capabilities(u8 feat, enum vas_cop_feat_type type, > + struct vas_ct_capabs_be *capab_be) > +{ > + struct vas_ct_capabs *capab; > + struct vas_capabs *vcapab; > + int rc = 0; > + > + vcapab = &vcapabs[type]; > + memset(vcapab, 0, sizeof(*vcapab)); > + INIT_LIST_HEAD(&vcapab->list); > + > + capab = &vcapab->capab; > + > + rc = plpar_vas_query_capabilities(H_QUERY_VAS_CAPABILITIES, feat, > + (u64)virt_to_phys(capab_be)); > + if (rc) > + return rc; > + > + capab->user_mode = capab_be->user_mode; > + if (!(capab->user_mode & VAS_COPY_PASTE_USER_MODE)) { > + pr_err("User space COPY/PASTE is not supported\n"); > + return -ENOTSUPP; > + } > + > + snprintf(capab->name, VAS_DESCR_LEN + 1, "%.8s", > + (char *)&capab_be->descriptor); > + capab->descriptor = be64_to_cpu(capab_be->descriptor); > + capab->win_type = capab_be->win_type; > + if (capab->win_type >= VAS_MAX_FEAT_TYPE) { > + pr_err("Unsupported window type %u\n", capab->win_type); > + return -EINVAL; > + } > + capab->max_lpar_creds = be16_to_cpu(capab_be->max_lpar_creds); > + capab->max_win_creds = be16_to_cpu(capab_be->max_win_creds); > + atomic_set(&capab->target_lpar_creds, > + be16_to_cpu(capab_be->target_lpar_creds)); > + if (feat == VAS_GZIP_DEF_FEAT) { > + capab->def_lpar_creds = be16_to_cpu(capab_be->def_lpar_creds); > + > + if (capab->max_win_creds < DEF_WIN_CREDS) { > + pr_err("Window creds(%u) > max allowed window creds(%u)\n", > + DEF_WIN_CREDS, capab->max_win_creds); > + return -EINVAL; > + } > + } > + > + copypaste_feat = 1; > + > + return 0; > +} > + > +static int __init pseries_vas_init(void) > +{ > + struct vas_ct_capabs_be *ct_capabs_be; > + struct vas_all_capabs_be *capabs_be; > + int rc; > + > + /* > + * Linux supports user space COPY/PASTE only with Radix > + */ > + if (!radix_enabled()) { > + pr_err("API is supported only with radix page tables\n"); > + return -ENOTSUPP; > + } > + > + capabs_be = kmalloc(sizeof(*capabs_be), GFP_KERNEL); > + if (!capabs_be) > + return -ENOMEM; > + /* > + * Get VAS overall capabilities by passing 0 to feature type. > + */ > + rc = plpar_vas_query_capabilities(H_QUERY_VAS_CAPABILITIES, 0, > + (u64)virt_to_phys(capabs_be)); > + if (rc) > + goto out; > + > + snprintf(capabs_all.name, VAS_DESCR_LEN, "%.7s", > + (char *)&capabs_be->descriptor); > + capabs_all.descriptor = be64_to_cpu(capabs_be->descriptor); > + capabs_all.feat_type = be64_to_cpu(capabs_be->feat_type); > + > + ct_capabs_be = kmalloc(sizeof(*ct_capabs_be), GFP_KERNEL); > + if (!ct_capabs_be) { > + rc = -ENOMEM; > + goto out; > + } > + /* > + * QOS capabilities available > + */ > + if (capabs_all.feat_type & VAS_GZIP_QOS_FEAT_BIT) { > + rc = get_vas_capabilities(VAS_GZIP_QOS_FEAT, > + VAS_GZIP_QOS_FEAT_TYPE, ct_capabs_be); > + > + if (rc) > + goto out_ct; > + } > + /* > + * Default capabilities available > + */ > + if (capabs_all.feat_type & VAS_GZIP_DEF_FEAT_BIT) { > + rc = get_vas_capabilities(VAS_GZIP_DEF_FEAT, > + VAS_GZIP_DEF_FEAT_TYPE, ct_capabs_be); > + if (rc) > + goto out_ct; > + } Using the same buffer for two hcalls? Do they fill in different parts of it? > + > + if (!copypaste_feat) > + pr_err("GZIP feature is not supported\n"); This is dead code AFAIKS, because errors will always branch to out. Thanks, Nick > + > + pr_info("GZIP feature is available\n"); > + > +out_ct: > + kfree(ct_capabs_be); > +out: > + kfree(capabs_be); > + return rc; > +} > +machine_device_initcall(pseries, pseries_vas_init); > -- > 2.18.2 > > >
WARNING: multiple messages have this Message-ID (diff)
From: Nicholas Piggin <npiggin@gmail.com> To: Haren Myneni <haren@linux.ibm.com>, herbert@gondor.apana.org.au, linux-crypto@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, mpe@ellerman.id.au Subject: Re: [V3 PATCH 09/16] powerpc/pseries/vas: Implement to get all capabilities Date: Mon, 10 May 2021 16:13:08 +1000 [thread overview] Message-ID: <1620626553.7v3m168yl3.astroid@bobo.none> (raw) In-Reply-To: <f6cdf811a29c22056740d48fa3de010f4ea4b848.camel@linux.ibm.com> Excerpts from Haren Myneni's message of April 18, 2021 7:08 am: > > pHyp provides various VAS capabilities such as GZIP default and QoS > capabilities which are used to determine total number of credits > available in LPAR, maximum window credits, maximum LPAR credits, > whether usermode copy/paste is supported, and etc. > > So first retrieve overall vas capabilities using > H_QUERY_VAS_CAPABILITIES HCALL which tells the specific features that > are available. Then retrieve the specific capabilities by using the > feature type in H_QUERY_VAS_CAPABILITIES HCALL. > > pHyp supports only GZIP default and GZIP QoS capabilities right now. Changelog and title could use a bit of work. > > Signed-off-by: Haren Myneni <haren@linux.ibm.com> > --- > arch/powerpc/platforms/pseries/vas.c | 130 +++++++++++++++++++++++++++ > 1 file changed, 130 insertions(+) > > diff --git a/arch/powerpc/platforms/pseries/vas.c b/arch/powerpc/platforms/pseries/vas.c > index 06960151477c..35946fb02995 100644 > --- a/arch/powerpc/platforms/pseries/vas.c > +++ b/arch/powerpc/platforms/pseries/vas.c > @@ -30,6 +30,13 @@ > /* phyp allows one credit per window right now */ > #define DEF_WIN_CREDS 1 > > +static struct vas_all_capabs capabs_all; Does this name come from PAPR? If not, capabilities or caps are better for readability than capabs. > +static int copypaste_feat; Should be a bool? And what does it mean? copy-paste is a host core capability. > + > +struct vas_capabs vcapabs[VAS_MAX_FEAT_TYPE]; > + > +DEFINE_MUTEX(vas_pseries_mutex); Can these be made static if they're only used here, and export them if a future patch uses them (or add the header declaration now). > + > static int64_t hcall_return_busy_check(int64_t rc) > { > /* Check if we are stalled for some time */ > @@ -215,3 +222,126 @@ int plpar_vas_query_capabilities(const u64 hcall, u8 query_type, > return -EIO; > } > } > + > +/* > + * Get the specific capabilities based on the feature type. > + * Right now supports GZIP default and GZIP QoS capabilities. > + */ > +static int get_vas_capabilities(u8 feat, enum vas_cop_feat_type type, > + struct vas_ct_capabs_be *capab_be) > +{ > + struct vas_ct_capabs *capab; > + struct vas_capabs *vcapab; > + int rc = 0; > + > + vcapab = &vcapabs[type]; > + memset(vcapab, 0, sizeof(*vcapab)); > + INIT_LIST_HEAD(&vcapab->list); > + > + capab = &vcapab->capab; > + > + rc = plpar_vas_query_capabilities(H_QUERY_VAS_CAPABILITIES, feat, > + (u64)virt_to_phys(capab_be)); > + if (rc) > + return rc; > + > + capab->user_mode = capab_be->user_mode; > + if (!(capab->user_mode & VAS_COPY_PASTE_USER_MODE)) { > + pr_err("User space COPY/PASTE is not supported\n"); > + return -ENOTSUPP; > + } > + > + snprintf(capab->name, VAS_DESCR_LEN + 1, "%.8s", > + (char *)&capab_be->descriptor); > + capab->descriptor = be64_to_cpu(capab_be->descriptor); > + capab->win_type = capab_be->win_type; > + if (capab->win_type >= VAS_MAX_FEAT_TYPE) { > + pr_err("Unsupported window type %u\n", capab->win_type); > + return -EINVAL; > + } > + capab->max_lpar_creds = be16_to_cpu(capab_be->max_lpar_creds); > + capab->max_win_creds = be16_to_cpu(capab_be->max_win_creds); > + atomic_set(&capab->target_lpar_creds, > + be16_to_cpu(capab_be->target_lpar_creds)); > + if (feat == VAS_GZIP_DEF_FEAT) { > + capab->def_lpar_creds = be16_to_cpu(capab_be->def_lpar_creds); > + > + if (capab->max_win_creds < DEF_WIN_CREDS) { > + pr_err("Window creds(%u) > max allowed window creds(%u)\n", > + DEF_WIN_CREDS, capab->max_win_creds); > + return -EINVAL; > + } > + } > + > + copypaste_feat = 1; > + > + return 0; > +} > + > +static int __init pseries_vas_init(void) > +{ > + struct vas_ct_capabs_be *ct_capabs_be; > + struct vas_all_capabs_be *capabs_be; > + int rc; > + > + /* > + * Linux supports user space COPY/PASTE only with Radix > + */ > + if (!radix_enabled()) { > + pr_err("API is supported only with radix page tables\n"); > + return -ENOTSUPP; > + } > + > + capabs_be = kmalloc(sizeof(*capabs_be), GFP_KERNEL); > + if (!capabs_be) > + return -ENOMEM; > + /* > + * Get VAS overall capabilities by passing 0 to feature type. > + */ > + rc = plpar_vas_query_capabilities(H_QUERY_VAS_CAPABILITIES, 0, > + (u64)virt_to_phys(capabs_be)); > + if (rc) > + goto out; > + > + snprintf(capabs_all.name, VAS_DESCR_LEN, "%.7s", > + (char *)&capabs_be->descriptor); > + capabs_all.descriptor = be64_to_cpu(capabs_be->descriptor); > + capabs_all.feat_type = be64_to_cpu(capabs_be->feat_type); > + > + ct_capabs_be = kmalloc(sizeof(*ct_capabs_be), GFP_KERNEL); > + if (!ct_capabs_be) { > + rc = -ENOMEM; > + goto out; > + } > + /* > + * QOS capabilities available > + */ > + if (capabs_all.feat_type & VAS_GZIP_QOS_FEAT_BIT) { > + rc = get_vas_capabilities(VAS_GZIP_QOS_FEAT, > + VAS_GZIP_QOS_FEAT_TYPE, ct_capabs_be); > + > + if (rc) > + goto out_ct; > + } > + /* > + * Default capabilities available > + */ > + if (capabs_all.feat_type & VAS_GZIP_DEF_FEAT_BIT) { > + rc = get_vas_capabilities(VAS_GZIP_DEF_FEAT, > + VAS_GZIP_DEF_FEAT_TYPE, ct_capabs_be); > + if (rc) > + goto out_ct; > + } Using the same buffer for two hcalls? Do they fill in different parts of it? > + > + if (!copypaste_feat) > + pr_err("GZIP feature is not supported\n"); This is dead code AFAIKS, because errors will always branch to out. Thanks, Nick > + > + pr_info("GZIP feature is available\n"); > + > +out_ct: > + kfree(ct_capabs_be); > +out: > + kfree(capabs_be); > + return rc; > +} > +machine_device_initcall(pseries, pseries_vas_init); > -- > 2.18.2 > > >
next prev parent reply other threads:[~2021-05-10 6:13 UTC|newest] Thread overview: 88+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-04-17 20:52 [V3 PATCH 00/16] Enable VAS and NX-GZIP support on powerVM Haren Myneni 2021-04-17 20:52 ` Haren Myneni 2021-04-17 21:00 ` [V3 PATCH 01/16] powerpc/powernv/vas: Rename register/unregister functions Haren Myneni 2021-04-17 21:00 ` Haren Myneni 2021-05-10 5:10 ` Nicholas Piggin 2021-05-10 5:10 ` Nicholas Piggin 2021-05-10 16:59 ` Haren Myneni 2021-05-10 16:59 ` Haren Myneni 2021-04-17 21:02 ` [PATCH V3 02/16] powerpc/vas: Move VAS API to common book3s platform Haren Myneni 2021-04-17 21:02 ` Haren Myneni 2021-05-10 5:19 ` Nicholas Piggin 2021-05-10 5:19 ` Nicholas Piggin 2021-05-10 17:12 ` Haren Myneni 2021-05-10 17:12 ` Haren Myneni 2021-05-11 1:50 ` Michael Ellerman 2021-05-11 1:50 ` Michael Ellerman 2021-04-17 21:03 ` [V3 PATCH 03/16] powerpc/vas: Create take/drop task reference functions Haren Myneni 2021-04-17 21:03 ` Haren Myneni 2021-05-10 5:28 ` Nicholas Piggin 2021-05-10 5:28 ` Nicholas Piggin 2021-05-10 20:32 ` Haren Myneni 2021-05-10 20:32 ` Haren Myneni 2021-04-17 21:03 ` [V3 PATCH 04/16] powerpc/vas: Move update_csb/dump_crb to common book3s platform Haren Myneni 2021-04-17 21:03 ` Haren Myneni 2021-05-10 5:37 ` Nicholas Piggin 2021-05-10 5:37 ` Nicholas Piggin 2021-04-17 21:04 ` [V3 PATCH 05/16] powerpc/vas: Define and use common vas_window struct Haren Myneni 2021-04-17 21:04 ` Haren Myneni 2021-05-10 5:37 ` Nicholas Piggin 2021-05-10 5:37 ` Nicholas Piggin 2021-05-10 17:02 ` Haren Myneni 2021-05-10 17:02 ` Haren Myneni 2021-04-17 21:05 ` [V3 PATCH 06/16] powerpc/pseries/vas: Define VAS/NXGZIP HCALLs and structs Haren Myneni 2021-04-17 21:05 ` Haren Myneni 2021-05-10 5:49 ` Nicholas Piggin 2021-05-10 5:49 ` Nicholas Piggin 2021-05-11 2:18 ` Haren Myneni 2021-05-11 2:18 ` Haren Myneni 2021-04-17 21:06 ` [V3 PATCH 07/16] powerpc/vas: Define QoS credit flag to allocate window Haren Myneni 2021-04-17 21:06 ` Haren Myneni 2021-05-10 5:54 ` Nicholas Piggin 2021-05-10 5:54 ` Nicholas Piggin 2021-05-11 2:12 ` Haren Myneni 2021-05-11 2:12 ` Haren Myneni 2021-04-17 21:07 ` [V3 PATCH 08/16] powerpc/pseries/VAS: Implement allocate/modify/deallocate HCALLS Haren Myneni 2021-04-17 21:07 ` Haren Myneni 2021-05-10 6:01 ` Nicholas Piggin 2021-05-10 6:01 ` Nicholas Piggin 2021-04-17 21:08 ` [V3 PATCH 09/16] powerpc/pseries/vas: Implement to get all capabilities Haren Myneni 2021-04-17 21:08 ` Haren Myneni 2021-05-10 6:13 ` Nicholas Piggin [this message] 2021-05-10 6:13 ` Nicholas Piggin 2021-05-11 2:08 ` Haren Myneni 2021-05-11 2:08 ` Haren Myneni 2021-04-17 21:08 ` [V3 PATCH 10/16] powerpc/pseries/vas: Integrate API with open/close windows Haren Myneni 2021-04-17 21:08 ` Haren Myneni 2021-05-10 6:18 ` Nicholas Piggin 2021-05-10 6:18 ` Nicholas Piggin 2021-05-10 6:28 ` Nicholas Piggin 2021-05-10 6:28 ` Nicholas Piggin 2021-04-17 21:09 ` [V3 PATCH 11/16] powerpc/pseries/vas: Setup IRQ and fault handling Haren Myneni 2021-04-17 21:09 ` Haren Myneni 2021-04-17 21:10 ` [V3 PATCH 12/16] powerpc/pseries/vas: sysfs interface to export capabilities Haren Myneni 2021-04-17 21:10 ` Haren Myneni 2021-05-10 6:34 ` Nicholas Piggin 2021-05-10 6:34 ` Nicholas Piggin 2021-05-10 20:52 ` Haren Myneni 2021-05-10 20:52 ` Haren Myneni 2021-04-17 21:11 ` [V3 PATCH 13/16] crypto/nx: Rename nx-842-pseries file name to nx-common-pseries Haren Myneni 2021-04-17 21:11 ` Haren Myneni 2021-04-22 6:55 ` Herbert Xu 2021-04-22 6:55 ` Herbert Xu 2021-04-17 21:12 ` [V3 PATCH 14/16] crypto/nx: Register and unregister VAS interface Haren Myneni 2021-04-17 21:12 ` Haren Myneni 2021-04-22 6:56 ` Herbert Xu 2021-04-22 6:56 ` Herbert Xu 2021-04-17 21:12 ` [V3 PATCH 15/16] crypto/nx: Get NX capabilities for GZIP coprocessor type Haren Myneni 2021-04-17 21:12 ` Haren Myneni 2021-04-22 6:56 ` Herbert Xu 2021-04-22 6:56 ` Herbert Xu 2021-05-10 6:38 ` Nicholas Piggin 2021-05-10 6:38 ` Nicholas Piggin 2021-05-10 20:44 ` Haren Myneni 2021-05-10 20:44 ` Haren Myneni 2021-04-17 21:13 ` [V3 PATCH 16/16] crypto/nx: Add sysfs interface to export NX capabilities Haren Myneni 2021-04-17 21:13 ` Haren Myneni 2021-04-22 6:56 ` Herbert Xu 2021-04-22 6:56 ` Herbert Xu
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=1620626553.7v3m168yl3.astroid@bobo.none \ --to=npiggin@gmail.com \ --cc=haren@linux.ibm.com \ --cc=haren@us.ibm.com \ --cc=hbabu@us.ibm.com \ --cc=herbert@gondor.apana.org.au \ --cc=linux-crypto@vger.kernel.org \ --cc=linuxppc-dev@lists.ozlabs.org \ --cc=mpe@ellerman.id.au \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.