From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758019AbcGZSyj (ORCPT ); Tue, 26 Jul 2016 14:54:39 -0400 Received: from mout.web.de ([212.227.17.11]:52222 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752416AbcGZSyf (ORCPT ); Tue, 26 Jul 2016 14:54:35 -0400 Subject: [PATCH 00/12] staging-Lustre: Fine-tuning for seven function implementations To: devel@driverdev.osuosl.org, lustre-devel@lists.lustre.org, Andreas Dilger , Greg Kroah-Hartman , Oleg Drokin References: <566ABCD9.1060404@users.sourceforge.net> <566D7733.1030102@users.sourceforge.net> <56784D83.7080108@users.sourceforge.net> <56784F0C.6040007@users.sourceforge.net> <20151221234857.GA27079@kroah.com> Cc: LKML , kernel-janitors@vger.kernel.org, Julia Lawall , Bhumika Goyal From: SF Markus Elfring Message-ID: <59d94e70-7476-728e-5f63-013557ec2db9@users.sourceforge.net> Date: Tue, 26 Jul 2016 20:54:17 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2 MIME-Version: 1.0 In-Reply-To: <20151221234857.GA27079@kroah.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:FgZXK0qyL3ZusHKP3aXGRpU0ZSpJi+/ed8zBfDMn8fcPLGz1u+G DzioKDRZmGNgTuKfFDj5rijpKMUn4Xagwsfi++SFpY7u9eqfJ14tpZxViV393rRO7TIjSCK 0ZT9S6qckpYFdXa2aozVr79LdXDXkgBSm78MOi6damh2jybfnRQwsMrcq/yksssKZfldYI2 kqclJsQPBbXlkTMp4N/6A== X-UI-Out-Filterresults: notjunk:1;V01:K0:GP30GfAi8pM=:oPkwZ6f/zIfbAtiz7CZrvy 3AaTDkkc9EnSGC6hNPXu4KtwzHLXK5gXvqSPsOAUWy2atZsrSgYUcu3ARXCOGZFtLHA9TpXbi qe9g5bLCL9Xb5tiO+ihV68zhUY+6OuPw2Mblb9w+GSlfiOpSDATad/0e28Bs8YKE744jYmTmw F80QofgWLUgxb8sCzDKvjPVQaFhaC0quRo6lz2wkVQ7wWx506cN2/yucF4lq21CYDfaTUc3Yd FvawbVqnhRcrvWxhYkKW+1xGAdl6C3zrT82wvgRMGN7Vxa6qvAhMeQxdh9X0hJcna7835rvve C1YjrBlK1Zow+RBwKogroAavE3vdLoA75QjuHFcqFzCAxFbptzRxRTRgCrFVQ9poQJh/FFL3Z egjizd8vC9S54+6akXFnjKpFfrOp1KVIFAmMEbm1oJrGdSkWtfgdadiVfj66c+fyh4DoxTJYE hQD0FePm6t8IL0fuQ1T8l/eRjNWcPGYEMeWJ6OU3s1kyIcqlyCNfyB8oWsWpzze+nJ5/zXq2/ O/1HBC1Eoxm7+D6nDv4WbII7j+1KyzFOm0fW/oMZg1ohktTYH6QSzESOo6emQWSsz8zhsjs0D ucaMIuyLNXfrVpoCWb+nECtceteJQi1zoEtlSghD7WPdhpBAw+IWIlJWOhY2MbGV7Fl2lPmMP z9ajbJ4bQ7RUvNSK5NNv9eZDR1Iu2i+8KYzU0ZVjmOsnEaUxEECskXCfEv1ta/IxcM3sGeERp iaYDownGpFs8itRJkVjnZ2krrmSwtImQraV+n9Yza3Oi01hvp7evK1/i1w/WyvtB4m6P0hFGQ N3seegI Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Further update suggestions were taken into account after a patch was applied from static source code analysis. Markus Elfring (12): ldlm: Delete unnecessary checks before the function call "kset_unregister" Delete unnecessary checks before the function call "kobject_put" One function call less in class_register_type() after error detection Split a condition check in class_register_type() Optimize error handling in class_register_type() Return directly after a failed kcalloc() in mgc_process_recover_log() Less checks after a failed alloc_page() in mgc_process_recover_log() Less checks after a failed ptlrpc_request_alloc() inmgc_process_recover_log() Delete a check for the variable "req" in mgc_process_recover_log() Rename jump labels in mgc_process_recover_log() Move an assignment for the variable "eof" in mgc_process_recover_log() Delete an unnecessary variable initialisation in mgc_process_recover_log() drivers/staging/lustre/lustre/ldlm/ldlm_lockd.c | 10 ++--- drivers/staging/lustre/lustre/lmv/lmv_obd.c | 5 +-- drivers/staging/lustre/lustre/lov/lov_obd.c | 4 +- drivers/staging/lustre/lustre/mgc/mgc_request.c | 50 ++++++++++++------------- drivers/staging/lustre/lustre/obdclass/genops.c | 41 ++++++++++++-------- 5 files changed, 54 insertions(+), 56 deletions(-) -- 2.9.2 From mboxrd@z Thu Jan 1 00:00:00 1970 From: SF Markus Elfring Date: Tue, 26 Jul 2016 18:54:17 +0000 Subject: [PATCH 00/12] staging-Lustre: Fine-tuning for seven function implementations Message-Id: <59d94e70-7476-728e-5f63-013557ec2db9@users.sourceforge.net> List-Id: References: <566ABCD9.1060404@users.sourceforge.net> <566D7733.1030102@users.sourceforge.net> <56784D83.7080108@users.sourceforge.net> <56784F0C.6040007@users.sourceforge.net> <20151221234857.GA27079@kroah.com> In-Reply-To: <20151221234857.GA27079@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: devel@driverdev.osuosl.org, lustre-devel@lists.lustre.org, Andreas Dilger , Greg Kroah-Hartman , Oleg Drokin Cc: LKML , kernel-janitors@vger.kernel.org, Julia Lawall , Bhumika Goyal From: Markus Elfring Further update suggestions were taken into account after a patch was applied from static source code analysis. Markus Elfring (12): ldlm: Delete unnecessary checks before the function call "kset_unregister" Delete unnecessary checks before the function call "kobject_put" One function call less in class_register_type() after error detection Split a condition check in class_register_type() Optimize error handling in class_register_type() Return directly after a failed kcalloc() in mgc_process_recover_log() Less checks after a failed alloc_page() in mgc_process_recover_log() Less checks after a failed ptlrpc_request_alloc() inmgc_process_recover_log() Delete a check for the variable "req" in mgc_process_recover_log() Rename jump labels in mgc_process_recover_log() Move an assignment for the variable "eof" in mgc_process_recover_log() Delete an unnecessary variable initialisation in mgc_process_recover_log() drivers/staging/lustre/lustre/ldlm/ldlm_lockd.c | 10 ++--- drivers/staging/lustre/lustre/lmv/lmv_obd.c | 5 +-- drivers/staging/lustre/lustre/lov/lov_obd.c | 4 +- drivers/staging/lustre/lustre/mgc/mgc_request.c | 50 ++++++++++++------------- drivers/staging/lustre/lustre/obdclass/genops.c | 41 ++++++++++++-------- 5 files changed, 54 insertions(+), 56 deletions(-) -- 2.9.2 From mboxrd@z Thu Jan 1 00:00:00 1970 From: SF Markus Elfring Date: Tue, 26 Jul 2016 20:54:17 +0200 Subject: [lustre-devel] [PATCH 00/12] staging-Lustre: Fine-tuning for seven function implementations In-Reply-To: <20151221234857.GA27079@kroah.com> References: <566ABCD9.1060404@users.sourceforge.net> <566D7733.1030102@users.sourceforge.net> <56784D83.7080108@users.sourceforge.net> <56784F0C.6040007@users.sourceforge.net> <20151221234857.GA27079@kroah.com> Message-ID: <59d94e70-7476-728e-5f63-013557ec2db9@users.sourceforge.net> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: devel@driverdev.osuosl.org, lustre-devel@lists.lustre.org, Andreas Dilger , Greg Kroah-Hartman , Oleg Drokin Cc: LKML , kernel-janitors@vger.kernel.org, Julia Lawall , Bhumika Goyal From: Markus Elfring Further update suggestions were taken into account after a patch was applied from static source code analysis. Markus Elfring (12): ldlm: Delete unnecessary checks before the function call "kset_unregister" Delete unnecessary checks before the function call "kobject_put" One function call less in class_register_type() after error detection Split a condition check in class_register_type() Optimize error handling in class_register_type() Return directly after a failed kcalloc() in mgc_process_recover_log() Less checks after a failed alloc_page() in mgc_process_recover_log() Less checks after a failed ptlrpc_request_alloc() inmgc_process_recover_log() Delete a check for the variable "req" in mgc_process_recover_log() Rename jump labels in mgc_process_recover_log() Move an assignment for the variable "eof" in mgc_process_recover_log() Delete an unnecessary variable initialisation in mgc_process_recover_log() drivers/staging/lustre/lustre/ldlm/ldlm_lockd.c | 10 ++--- drivers/staging/lustre/lustre/lmv/lmv_obd.c | 5 +-- drivers/staging/lustre/lustre/lov/lov_obd.c | 4 +- drivers/staging/lustre/lustre/mgc/mgc_request.c | 50 ++++++++++++------------- drivers/staging/lustre/lustre/obdclass/genops.c | 41 ++++++++++++-------- 5 files changed, 54 insertions(+), 56 deletions(-) -- 2.9.2