Re: [lustre-devel] [PATCH 2/3] staging:lustre: remove kernel defines in userland headers

2015-05-22 Thread Joe Perches
On Fri, 2015-05-22 at 15:12 +, Simmons, James A. wrote: > >> typedef struct lnet_peer { > >> - struct list_headlp_hashlist; /* chain on peer hash */ > >> - struct list_headlp_txq;/* messages blocking for tx > >> credits */ > >> - struct list_headlp_rtrq

RE: [lustre-devel] [PATCH 2/3] staging:lustre: remove kernel defines in userland headers

2015-05-22 Thread Simmons, James A.
>> typedef struct lnet_peer { >> -struct list_headlp_hashlist; /* chain on peer hash */ >> -struct list_headlp_txq;/* messages blocking for tx >> credits */ >> -struct list_headlp_rtrq; /* messages blocking for >> router credits */ >

Re: [PATCH 2/3] staging:lustre: remove kernel defines in userland headers

2015-05-22 Thread Dan Carpenter
This patch seems fine but it would also be better if it were broken up into easy to review patches. [patch 1] delete LNET_USE_LIB_FREELIST code This is theortetically what this patch does except it does tons of other things as well. Patches which delete whole functions, #defines o

[PATCH 2/3] staging:lustre: remove kernel defines in userland headers

2015-05-21 Thread James Simmons
From: James Simmons Currently the lnet headers used for user land applications contain various kernel definations. This is due to the fact libcfs contains kernel wrappers for user land which will be going away. This patch sorted the header data so all kernel containing structures are moved out of