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

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;

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

2015-05-21 Thread James Simmons
From: James Simmons uja.o...@yahoo.com 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