All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Jones <pjones@whamcloud.com>
To: lustre-devel@lists.lustre.org
Subject: [Lustre-devel] [PATCH] Fix task IO accounting on reads
Date: Tue, 19 Jul 2011 05:34:03 -0700	[thread overview]
Message-ID: <4E2579BB.7070202@whamcloud.com> (raw)
In-Reply-To: <alpine.LFD.2.01.1107191135050.22747@sys880.ldn.framestore.com>

Mark

Sorry if Andreas was not clear enough about contributing patches. 
Hopefully the more detailed information at 
http://wiki.whamcloud.com/display/PUB/Submitting+Changes will help. If 
you are only able to work with 1.8.x then by all means upload the patch 
into gerrit and another engineer could handle porting it to master 
(subject to other priorities of course). IIRC Framestore did not sign a 
Sun\Oracle contributor agreement so I suspect Oracle will not land your 
patch.

Regards

Peter

On 11-07-19 3:41 AM, Mark Hills wrote:
> On Wed, 27 Apr 2011, Andreas Dilger wrote:
>
> [...]
>> for inclusion into the next 1.8 release the updated patch should be
>> attached to a new bug at bugzilla.lustre.org, and a review requested on
>> the patch (I can be one of the reviewers).  For inclusion into the 2.1
>> release, please create a new bug at jira.whamcloud.com with the updated
>> patch.  The patch should include the standard kernel "Signed-off-by:
>> {your name}<your email>" line.
>>
>> Sorry for the complexity in submitting patches while development
>> transitions away from Oracle for newer releases.
> Hi Andreas, I created an entry in Bugzilla with the revised patch attached
> to it:
>
>    https://bugzilla.lustre.org/show_bug.cgi?id=24536
>
> I wasn't able to add you as reviewer, because I could not clearly see a
> field to do this.
>
> As for the 2.1 release, I took a quick look and it seems the code has
> changed substantially and the patch does not apply.
>
> I don't even have a 2.1 setup to verify the bug exists. And as I'm
> unfamiliar with the code, I don't think I can reasonably make an
> equivalent patch to 2.1 right now.
>
> Thanks
>

-- 
Peter Jones
Whamcloud, Inc.
www.whamcloud.com

  reply	other threads:[~2011-07-19 12:34 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-26 16:06 [Lustre-devel] [PATCH] Fix task IO accounting on reads Mark Hills
2011-04-26 16:38 ` Andreas Dilger
2011-04-27 11:23   ` Mark Hills
2011-04-27 15:56     ` Peter Kjellström
2011-04-27 16:43     ` Andreas Dilger
2011-04-27 17:14       ` Mark Hills
2011-04-27 21:15         ` Andreas Dilger
2011-04-27 21:19           ` Peter Jones
2011-07-19 10:41           ` Mark Hills
2011-07-19 12:34             ` Peter Jones [this message]
2011-07-19 14:31 ` Richard Henwood
2011-07-19 16:08   ` Mark Hills
2011-07-19 17:03     ` Richard Henwood

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=4E2579BB.7070202@whamcloud.com \
    --to=pjones@whamcloud.com \
    --cc=lustre-devel@lists.lustre.org \
    /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.