On Sat, Feb 8, 2014 at 1:43 AM, Greg Kroah-Hartman <gre...@linuxfoundation.org> wrote: > On Wed, Jan 22, 2014 at 09:36:17PM +0800, Peng Tao wrote: >> From: Artem Blagodarenko <artem_blagodare...@xyratex.com> >> >> This is only part of the original Lustre commit, splitted to do the >> cleanup work. >> >> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3155 >> Lustre-change: http://review.whamcloud.com/6025 >> Signed-off-by: Artem Blagodarenko <artem_blagodare...@xyratex.com> >> Reviewed-by: Andreas Dilger <andreas.dil...@intel.com> >> Reviewed-by: Emoly Liu <emoly....@intel.com> >> Signed-off-by: Peng Tao <bergw...@gmail.com> >> Signed-off-by: Andreas Dilger <andreas.dil...@intel.com> >> --- >> .../staging/lustre/lustre/obdclass/obd_config.c | 4 ---- >> 1 file changed, 4 deletions(-) >> >> diff --git a/drivers/staging/lustre/lustre/obdclass/obd_config.c >> b/drivers/staging/lustre/lustre/obdclass/obd_config.c >> index 27f56c0..fd08f1d 100644 >> --- a/drivers/staging/lustre/lustre/obdclass/obd_config.c >> +++ b/drivers/staging/lustre/lustre/obdclass/obd_config.c >> @@ -1226,24 +1226,20 @@ int class_process_config(struct lustre_cfg *lcfg) >> case LCFG_POOL_NEW: { >> err = obd_pool_new(obd, lustre_cfg_string(lcfg, 2)); >> GOTO(out, err = 0); >> - break; > > This confuses the static checkers. Just fix up the GOTO logic (hint, > remove it!) and then it will be "obvious" that the break can go away. > But for now, please leave it, unless you want to have someone just put > it back... Got it.
Thanks, Tao -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
