From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755266AbdFWUZc (ORCPT ); Fri, 23 Jun 2017 16:25:32 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:46560 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754964AbdFWUZa (ORCPT ); Fri, 23 Jun 2017 16:25:30 -0400 Date: Sat, 24 Jun 2017 00:15:13 +0800 From: Greg Kroah-Hartman To: James Simmons Cc: devel@driverdev.osuosl.org, Andreas Dilger , Oleg Drokin , Linux Kernel Mailing List , Lustre Development List Subject: Re: [PATCH 5/6] staging: lustre: lustre: several over 80 characters cleanups Message-ID: <20170623161513.GB20104@kroah.com> References: <1497452481-31124-1-git-send-email-jsimmons@infradead.org> <1497452481-31124-6-git-send-email-jsimmons@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1497452481-31124-6-git-send-email-jsimmons@infradead.org> User-Agent: Mutt/1.8.3 (2017-05-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jun 14, 2017 at 11:01:20AM -0400, James Simmons wrote: > -void ldlm_extent_policy_wire_to_local(const union ldlm_wire_policy_data *wpolicy, > - union ldlm_policy_data *lpolicy) > +void > +ldlm_extent_policy_wire_to_local(const union ldlm_wire_policy_data *wpolicy, > + union ldlm_policy_data *lpolicy) Stuff like this is not really needed. But you could work on making your function names smaller and more concise, that is one major reason why you have so many line-length issues in your codebase. > - LDLM_LRU_FLAG_AGED = BIT(0), /* Cancel aged locks (non lru resize). */ > + LDLM_LRU_FLAG_AGED = BIT(0), /* Cancel aged locks (non lru resize). > + */ Not needed to be changed. > --- a/drivers/staging/lustre/lustre/ldlm/ldlm_lock.c > +++ b/drivers/staging/lustre/lustre/ldlm/ldlm_lock.c > @@ -1035,7 +1035,8 @@ void ldlm_grant_lock(struct ldlm_lock *lock, struct list_head *work_list) > ldlm_extent_add_lock(res, lock); > else if (res->lr_type == LDLM_FLOCK) { > /* > - * We should not add locks to granted list in the following cases: > + * We should not add locks to granted list in the following > + * cases: That's a good fix, as is: > * - this is an UNLOCK but not a real lock; > * - this is a TEST lock; > * - this is a F_CANCELLK lock (async flock has req_mode == 0) > @@ -2050,13 +2051,16 @@ void _ldlm_lock_debug(struct ldlm_lock *lock, > libcfs_debug_vmsg2(msgdata, fmt, args, > " ns: \?\? lock: %p/%#llx lrc: %d/%d,%d mode: %s/%s res: \?\? rrc=\?\? type: \?\?\? flags: %#llx nid: %s remote: %#llx expref: %d pid: %u timeout: %lu lvb_type: %d\n", > lock, > - lock->l_handle.h_cookie, atomic_read(&lock->l_refc), > + lock->l_handle.h_cookie, > + atomic_read(&lock->l_refc), That one. So do it where it maes the code better looking and easier to understand. Not just where you have to work around crazy limitations. Please fix this patch up and resend. thanks, greg k-h