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
next prev parent 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.