From: Dan Carpenter <dan.carpenter@oracle.com>
To: Mike Shuey <shuey@purdue.edu>
Cc: oleg.drokin@intel.com, devel@driverdev.osuosl.org,
gregkh@linuxfoundation.org, kernel-janitors@vger.kernel.org,
linux-kernel@vger.kernel.org, HPDD-discuss@ml01.01.org,
lustre-deve@lists.lustre.org
Subject: Re: [PATCH v4 00/13] staging: lustre: lnet: code cleanups
Date: Fri, 22 May 2015 09:21:42 +0000 [thread overview]
Message-ID: <20150522092142.GJ4150@mwanda> (raw)
In-Reply-To: <1432237849-53947-1-git-send-email-shuey@purdue.edu>
On Thu, May 21, 2015 at 03:50:23PM -0400, Mike Shuey wrote:
> This patch series cleans up code in staging/lustre/lnet - mostly spacing
> and dead function cleanup. Should apply against linux-next 20150518.
Why are you sending a v4 of this? We don't actually enjoy reviewing the
same patchset over and over...
Update: I have investigated and it is because you add a few new patches
to this set which were not in the original. Just assume that the
original patchset will be merge and send the additional patches as a new
set.
Also since no one replied to the v3 patchset and Greg has a 5000 patches
in between v3 and v4 he will not seen v4 until days after he has applied
v3 so it causes confusion.
Also you sent a v5 of this patchset but without a v5 label...
regards,
dan carpenter
WARNING: multiple messages have this Message-ID (diff)
From: Dan Carpenter <dan.carpenter@oracle.com>
To: Mike Shuey <shuey@purdue.edu>
Cc: oleg.drokin@intel.com, devel@driverdev.osuosl.org,
gregkh@linuxfoundation.org, kernel-janitors@vger.kernel.org,
linux-kernel@vger.kernel.org, HPDD-discuss@ml01.01.org,
lustre-deve@lists.lustre.org
Subject: Re: [PATCH v4 00/13] staging: lustre: lnet: code cleanups
Date: Fri, 22 May 2015 12:21:42 +0300 [thread overview]
Message-ID: <20150522092142.GJ4150@mwanda> (raw)
In-Reply-To: <1432237849-53947-1-git-send-email-shuey@purdue.edu>
On Thu, May 21, 2015 at 03:50:23PM -0400, Mike Shuey wrote:
> This patch series cleans up code in staging/lustre/lnet - mostly spacing
> and dead function cleanup. Should apply against linux-next 20150518.
Why are you sending a v4 of this? We don't actually enjoy reviewing the
same patchset over and over...
Update: I have investigated and it is because you add a few new patches
to this set which were not in the original. Just assume that the
original patchset will be merge and send the additional patches as a new
set.
Also since no one replied to the v3 patchset and Greg has a 5000 patches
in between v3 and v4 he will not seen v4 until days after he has applied
v3 so it causes confusion.
Also you sent a v5 of this patchset but without a v5 label...
regards,
dan carpenter
next prev parent reply other threads:[~2015-05-22 9:21 UTC|newest]
Thread overview: 125+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-21 19:50 [PATCH v4 00/13] staging: lustre: lnet: code cleanups Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-21 19:50 ` [PATCH v4 01/13] staging: lustre: lnet: lnet: code cleanups - variable declarations Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-21 19:50 ` [PATCH v4 02/13] staging: lustre: lnet: dead code - remove lnet_fini_locks Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-21 19:50 ` [PATCH v4 03/13] staging: lustre: lnet: dead code - remove LNetSetAsync Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-21 19:50 ` [PATCH v4 04/13] staging: lustre: lnet: lnet: Module is LNet, not Portals Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-21 19:50 ` [PATCH v4 05/13] staging: lustre: lnet: o2iblnd: code cleanup - align whitespace Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-22 13:03 ` Dan Carpenter
2015-05-22 13:03 ` Dan Carpenter
2015-05-21 19:50 ` [PATCH v4 06/13] staging: lustre: lnet: socklnd: code cleanup - align spacing Mike Shuey
2015-05-21 19:50 ` [PATCH v4 07/13] staging: lustre: lnet: selftest: code cleanup - variable spacing, indentation Mike Shuey
2015-05-21 19:50 ` [PATCH v4 08/13] staging: lustre: lnet: remove LNET_MUTEX_LOCK macro Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-21 19:50 ` [PATCH v4 09/13] staging: lustre: lnet: lnet: remove dead code, fix checkpatch.pl issue Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-21 19:50 ` [PATCH v4 10/13] staging: lustre: lnet: lnet: checkpatch.pl fixes Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-21 21:00 ` Joe Perches
2015-05-21 21:00 ` Joe Perches
2015-05-21 21:29 ` Julia Lawall
2015-05-21 21:29 ` Julia Lawall
2015-05-21 22:04 ` [lustre-devel] " Michael Shuey
2015-05-22 2:46 ` Joe Perches
2015-05-22 2:46 ` Joe Perches
2015-05-22 2:46 ` [lustre-devel] " Joe Perches
2015-05-22 5:06 ` Julia Lawall
2015-05-22 5:06 ` Julia Lawall
2015-05-22 5:06 ` [lustre-devel] " Julia Lawall
2015-05-22 8:08 ` Drokin, Oleg
2015-05-22 8:08 ` [lustre-devel] " Drokin, Oleg
2015-05-22 15:42 ` Joe Perches
2015-05-22 15:42 ` Joe Perches
2015-05-22 15:42 ` [lustre-devel] " Joe Perches
2015-05-22 21:16 ` Drokin, Oleg
2015-05-22 21:16 ` [lustre-devel] " Drokin, Oleg
2015-05-22 23:57 ` Joe Perches
2015-05-22 23:57 ` Joe Perches
2015-05-22 23:57 ` [lustre-devel] " Joe Perches
2015-05-23 0:07 ` Drokin, Oleg
2015-05-23 0:07 ` [lustre-devel] " Drokin, Oleg
2015-05-23 0:18 ` Joe Perches
2015-05-23 0:18 ` Joe Perches
2015-05-23 0:18 ` [lustre-devel] " Joe Perches
2015-05-23 0:25 ` Drokin, Oleg
2015-05-23 0:25 ` [lustre-devel] " Drokin, Oleg
2015-05-23 0:36 ` Joe Perches
2015-05-23 0:36 ` Joe Perches
2015-05-23 0:36 ` [lustre-devel] " Joe Perches
2015-05-23 3:13 ` [HPDD-discuss] " Patrick Farrell
2015-05-23 3:13 ` [lustre-devel] " Patrick Farrell
[not found] ` <CABqvwjs=S1VrCHe7M4JnDikGBVN3yqjNMrtSA=UvftGmxSasRQ@mail.gmail.com>
2015-05-22 3:06 ` Joe Perches
2015-05-22 3:06 ` Joe Perches
2015-05-22 8:04 ` Drokin, Oleg
2015-05-22 10:00 ` Julia Lawall
2015-05-22 10:00 ` Julia Lawall
2015-05-23 6:28 ` [PATCH] checkpatch: Categorize some long line length checks Joe Perches
2015-05-23 6:28 ` Joe Perches
2015-05-23 11:32 ` Julia Lawall
2015-05-23 11:32 ` Julia Lawall
2015-05-23 17:26 ` Joe Perches
2015-05-23 17:26 ` Joe Perches
2015-05-23 18:07 ` Dan Carpenter
2015-05-23 18:07 ` Dan Carpenter
2015-05-23 18:13 ` Joe Perches
2015-05-23 18:13 ` Joe Perches
2015-05-23 18:21 ` Dan Carpenter
2015-05-23 18:21 ` Dan Carpenter
2015-05-24 2:49 ` [PATCH V2] " Joe Perches
2015-05-21 19:50 ` [PATCH v4 11/13] staging: lnet: o2iblnd: checkpatch.pl fixes Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-22 10:49 ` Dan Carpenter
2015-05-22 10:49 ` Dan Carpenter
2015-05-21 19:50 ` [PATCH v4 12/13] staging: lustre: lnet: socklnd: checkpatch.pl cleanups Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-21 19:50 ` [PATCH v4 13/13] staging: lustre: lnet: selftest: checkpatch.pl fixes Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-21 19:50 ` [PATCH 01/13] staging: lustre: lnet: lnet: code cleanups - variable declarations Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-21 19:50 ` [PATCH 02/13] staging: lustre: lnet: dead code - remove lnet_fini_locks Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-21 19:50 ` [PATCH 03/13] staging: lustre: lnet: dead code - remove LNetSetAsync Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-21 19:50 ` [PATCH 04/13] staging: lustre: lnet: lnet: Module is LNet, not Portals Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-21 19:50 ` [PATCH 05/13] staging: lustre: lnet: o2iblnd: code cleanup - align whitespace Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-21 19:50 ` [PATCH 06/13] staging: lustre: lnet: socklnd: code cleanup - align spacing Mike Shuey
2015-05-21 19:50 ` [PATCH 07/13] staging: lustre: lnet: selftest: code cleanup - variable spacing, indentation Mike Shuey
2015-05-21 19:50 ` [PATCH 08/13] staging: lustre: lnet: remove LNET_MUTEX_LOCK macro Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-21 19:50 ` [PATCH 09/13] staging: lustre: lnet: lnet: remove dead code, fix checkpatch.pl issue Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-21 19:50 ` [PATCH 10/13] staging: lustre: lnet: lnet: checkpatch.pl fixes Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-21 19:50 ` [PATCH 11/13] staging: lnet: o2iblnd: " Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-21 19:50 ` [PATCH 12/13] staging: lustre: lnet: socklnd: checkpatch.pl cleanups Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-21 19:50 ` [PATCH 13/13] staging: lustre: lnet: selftest: checkpatch.pl fixes Mike Shuey
2015-05-21 19:50 ` Mike Shuey
2015-05-22 10:55 ` Dan Carpenter
2015-05-22 10:55 ` Dan Carpenter
2015-05-22 9:21 ` Dan Carpenter [this message]
2015-05-22 9:21 ` [PATCH v4 00/13] staging: lustre: lnet: code cleanups Dan Carpenter
2015-05-22 22:48 ` [lustre-devel] " Michael Shuey
2015-05-23 10:14 ` Dan Carpenter
2015-05-23 10:14 ` Dan Carpenter
2015-05-23 10:14 ` [lustre-devel] " Dan Carpenter
2015-05-23 12:09 ` Michael Shuey
2015-05-23 12:09 ` Michael Shuey
2015-05-23 12:09 ` [lustre-devel] " Michael Shuey
2015-05-23 12:39 ` Sudip Mukherjee
2015-05-23 12:51 ` Sudip Mukherjee
2015-05-23 12:39 ` Sudip Mukherjee
2015-05-23 14:05 ` Michael Shuey
2015-05-23 14:05 ` Michael Shuey
2015-05-23 14:05 ` [lustre-devel] " Michael Shuey
2015-05-31 2:24 ` Greg KH
2015-05-31 2:24 ` Greg KH
2015-05-31 2:24 ` [lustre-devel] " Greg KH
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150522092142.GJ4150@mwanda \
--to=dan.carpenter@oracle.com \
--cc=HPDD-discuss@ml01.01.org \
--cc=devel@driverdev.osuosl.org \
--cc=gregkh@linuxfoundation.org \
--cc=kernel-janitors@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lustre-deve@lists.lustre.org \
--cc=oleg.drokin@intel.com \
--cc=shuey@purdue.edu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.