Re: [lustre-devel] [PATCH] staging : lustre : Remove braces from single-line body.

2016-12-16 Thread Xiong, Jinshan
> On Dec 16, 2016, at 8:05 AM, Joe Perches wrote: > > On Fri, 2016-12-16 at 19:59 +0530, Tabrez khan wrote: >> Remove unnecessary braces {} for single while statement. > > Your patch is fine Tabrez, but to the lustre folk: > >> diff --git a/drivers/staging/lustre/lustre/obdclass/cl_io.c >> b/

Re: [bug report] staging: lustre: clio: Revise read ahead implementation

2016-10-20 Thread Xiong, Jinshan
> On Oct 20, 2016, at 7:17 PM, Dan Carpenter wrote: > > Hello Jinshan Xiong, > > The patch 1e1db2a97be5: "staging: lustre: clio: Revise read ahead > implementation" from Oct 2, 2016, leads to the following static > checker warning: > > drivers/staging/lustre/lustre/lov/lov_io.c:611 lov_i

Re: [PATCH v3 1/2] Staging: lustre: Refactor the function interval_erase_color() in /lustre/ldlm/interval_tree.c

2014-01-14 Thread Xiong, Jinshan
On Jan 14, 2014, at 3:09 AM, Monam Agarwal mailto:monamagarwal...@gmail.com>> wrote: On Tue, Jan 14, 2014 at 1:31 PM, Xiong, Jinshan mailto:jinshan.xi...@intel.com>> wrote: On Jan 13, 2014, at 11:56 PM, Dilger, Andreas mailto:andreas.dil...@intel.com>> wrote: Begin

Re: [PATCH v3 1/2] Staging: lustre: Refactor the function interval_erase_color() in /lustre/ldlm/interval_tree.c

2014-01-14 Thread Xiong, Jinshan
On Jan 13, 2014, at 11:56 PM, Dilger, Andreas wrote: > > > Begin forwarded message: > >> From: Greg KH >> Subject: Re: [PATCH v3 1/2] Staging: lustre: Refactor the function >> interval_erase_color() in /lustre/ldlm/interval_tree.c >> Date: January 11, 2014 at 1:33:58 PM MST >> To: Monam Aga