Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1209614

Re: [PATCH 2/2] cxl: add set/get private data to context struct

Show all headers | View raw

On Wed, 2015-08-19 at 14:19 +1000, Ian Munsie wrote:
> From: Michael Neuling <mikey@neuling.org>
> 
> This provides AFU drivers a means to associate private data with a cxl
> context. This is particularly intended for make the new callbacks for
> driver specific events easier for AFU drivers to use, as they can easily
> get back to any private data structures they may use.
> 
> Signed-off-by: Michael Neuling <mikey@neuling.org>
> Signed-off-by: Ian Munsie <imunsie@au1.ibm.com>
> ---
>  drivers/misc/cxl/api.c | 21 +++++++++++++++++++++
>  drivers/misc/cxl/cxl.h |  3 +++
>  include/misc/cxl.h     |  7 +++++++
>  3 files changed, 31 insertions(+)
> 
> diff --git a/drivers/misc/cxl/api.c b/drivers/misc/cxl/api.c
> index e0f0c78..5f0b22e 100644
> --- a/drivers/misc/cxl/api.c
> +++ b/drivers/misc/cxl/api.c
> @@ -70,6 +70,27 @@ int cxl_release_context(struct cxl_context *ctx)
>  }
>  EXPORT_SYMBOL_GPL(cxl_release_context);
>  
> +
> +int cxl_set_priv(struct cxl_context *ctx, void *priv)
> +{
> +	if (!ctx)
> +		return -EINVAL;
> +
> +	ctx->priv = priv;
> +
> +	return 0;
> +}
> +EXPORT_SYMBOL_GPL(cxl_set_priv);
> +
> +void *cxl_get_priv(struct cxl_context *ctx)
> +{
> +	if (!ctx)
> +		return ERR_PTR(-EINVAL);
> +
> +	return ctx->priv;
> +}
> +EXPORT_SYMBOL_GPL(cxl_get_priv);


Do we really need the accessors? They don't buy anything I can see over just
using ctx->priv directly.

cheers


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 1/2] cxl: Add mechanism for delivering AFU driver specific events "Ian Munsie" <imunsie@au1.ibm.com> - 2015-08-19 06:30 +0200
  [PATCH 2/2] cxl: add set/get private data to context struct "Ian Munsie" <imunsie@au1.ibm.com> - 2015-08-19 06:30 +0200
    Re: [PATCH 2/2] cxl: add set/get private data to context struct Michael Ellerman <mpe@ellerman.id.au> - 2015-08-19 06:50 +0200
      Re: [PATCH 2/2] cxl: add set/get private data to context struct Ian Munsie <imunsie@au1.ibm.com> - 2015-08-19 07:20 +0200
        Re: [PATCH 2/2] cxl: add set/get private data to context struct Michael Ellerman <mpe@ellerman.id.au> - 2015-08-19 07:40 +0200
  Re: [PATCH 1/2] cxl: Add mechanism for delivering AFU driver  specific events Michael Ellerman <mpe@ellerman.id.au> - 2015-08-19 07:40 +0200
    Re: [PATCH 1/2] cxl: Add mechanism for delivering AFU driver specific events Ian Munsie <imunsie@au1.ibm.com> - 2015-08-19 09:10 +0200

csiph-web