[openib-general][PATCH][kdapltest]: adding memtype options to server

James Lentini jlentini at netapp.com
Thu Jul 28 11:06:20 PDT 2005


Hi Guy,

This patch has been garbled too. See below:

On Wed, 27 Jul 2005, Guy German wrote:

> [kdapltest]: this patch adds the option of using memory registration
> types other then the default PHY on the server side.
>
> Signed-off-by: Guy German <guyg at voltaire.com>
>
> Index: dapltest/test/dapl_server.c
> ===================================================================
> --- dapltest/test/dapl_server.c	(revision 2899)
> +++ dapltest/test/dapl_server.c	(working copy)
> @@ -232,7 +232,14 @@ DT_cs_Server (Params_t * params_ptr)
>      * Create two buffers, large enough to hold ClientInfo and the largest
>      * command we'll use.
>      */
> -    ps_ptr->bpool = DT_BpoolAlloc (NULL,

Something is missing here. If you remove the line above you can't 
replace it with the line below...

> + +    if (!(pt_ptr = DT_Alloc_Per_Test_Data (phead)))

...which has a double +

> +	    goto server_exit;
> +    DT_MemListInit (pt_ptr);
> +    memcpy ( (void *) (uintptr_t) &pt_ptr->Params,
> +	    (const void *) params_ptr,
> +	    sizeof (Params_t));
> +    ps_ptr->bpool = DT_BpoolAlloc (pt_ptr,
> 	    		       phead,
> 			       ps_ptr->ia,
> 			       ps_ptr->pz,
> @@ -243,6 +250,7 @@ DT_cs_Server (Params_t * params_ptr)
> 			       256, /* FIXME query for this value */
> 			       FALSE,
> 			       FALSE);
> +    DT_Free_Per_Test_Data (pt_ptr);
>     if (ps_ptr->bpool == 0)
>     {
> 	DT_Tdep_PT_Printf (phead,
>



More information about the general mailing list