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

Guy German guyg at voltaire.com
Thu Jul 28 15:04:44 PDT 2005


Hi James,

>>Hi Guy,
>>This patch has been garbled too. See below:

I will resend all the garbled patches sunday morning

>> -    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 +

Why not ?

I replaced the first line :

-    ps_ptr->bpool = DT_BpoolAlloc (NULL,

with :

+ if (!(pt_ptr = DT_Alloc_Per_Test_Data (phead)))
+    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,

What I basically did was allocating pt_ptr and 
copying the params to the ds before calling DT_BpoolAlloc.
This time I call it with pt_ptr instead of NULL (that leads
to default use of physical memory registration).

(The + + has probably to do with mailer/diff problems)



More information about the general mailing list