<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
I'd be happy to continue work on the format, whitespace, and coding
style changes, on any areas that aren't already spoken for. <br>
<br>
Chris<br>
<br>
<div class="moz-cite-prefix">On 6/10/15 1:59 PM, Patrick Farrell
wrote:<br>
</div>
<blockquote cite="mid:55787B02.4080802@cray.com" type="cite">I
suggest breaking it up first by work type. As Julia noted,
coccinelle/checkpatch/etc fixes (coding style, straightforward
errors caught by the tools, sparse complaints, warnings, etc.)
rarely require awareness of the larger flow of the design.
<br>
<br>
Then there is some other types of cleanup that require more Lustre
knowledge to tackle. If possible, those would be better for
someone like Ben (or you, James) to work on. What do we have for
that kind of work?
<br>
<br>
- Patrick
<br>
<br>
On 06/10/2015 12:16 PM, Julia Lawall wrote:
<br>
<blockquote type="cite">
<br>
On Wed, 10 Jun 2015, Simmons, James A. wrote:
<br>
<br>
<blockquote type="cite">
<blockquote type="cite">Where would be a good place to get
started with helping merge the client
<br>
</blockquote>
into mainline?
<br>
<br>
<br>
Hi Ben
<br>
<br>
<br>
Thanks for helping out for this work. I added people in
the chain that
<br>
have contributed a great
<br>
<br>
deal to the upstream client so we can coordinate our work.
Also I like to
<br>
make people aware
<br>
<br>
a lustre IRC channel does exist. Andreas can supply the
details about the
<br>
IRC channel. Currently
<br>
<br>
my work has been focused on the libcfs/lnet layer. I had
discuss early with
<br>
Mike Shuey since he
<br>
<br>
was also working in that area but will be migrating to the
lustre core.
<br>
<br>
<br>
Since more people are getting involved we need to find a
way to break
<br>
up the work to avoid
<br>
<br>
duplicate work. Should the work be broken up by layer, i.e
lov, mdc, or by
<br>
task such as removing
<br>
<br>
specific checkpatch errors with files?
<br>
</blockquote>
For the work I have done with Coccinelle, it is more convenient
to do the
<br>
whole thing. Breaking things up by layer is not helpful.
<br>
<br>
This is just a data point, though. It doesn't have to imply
anything about
<br>
how others work on the code.
<br>
<br>
julia
<br>
<br>
<br>
_______________________________________________
<br>
lustre-devel mailing list
<br>
<a class="moz-txt-link-abbreviated" href="mailto:lustre-devel@lists.lustre.org">lustre-devel@lists.lustre.org</a>
<br>
<a class="moz-txt-link-freetext" href="http://lists.lustre.org/listinfo.cgi/lustre-devel-lustre.org">http://lists.lustre.org/listinfo.cgi/lustre-devel-lustre.org</a>
<br>
</blockquote>
<br>
<br>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
lustre-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:lustre-devel@lists.lustre.org">lustre-devel@lists.lustre.org</a>
<a class="moz-txt-link-freetext" href="http://lists.lustre.org/listinfo.cgi/lustre-devel-lustre.org">http://lists.lustre.org/listinfo.cgi/lustre-devel-lustre.org</a>
</pre>
</blockquote>
<br>
</body>
</html>