Theodore Ts'o wrote:
Here's an initial code review of the mballoc-core.patch. As far as I
can tell, the patch sent by Aneesh doesn't address any of these.
Aneesh, assuming that you'll be fixing these, could you please combine
your last set of fixes with fixes to address these, and send out an
updated mballoc-core.patch?
Will do that
Patches on top of patches are useful for
seeing what has changed, but for the patch queue I'd like to get the
patches folded into a single patch, since that's what we'd want to have
placed into the git tree.
That is what i did with my last update. The diff of the source was sent to
the mailing list and and updated mballoc-core.patch was put at www.radian.org
so that we can use the same to update the patch queue. The diff against mballoc.c
helps in reviewing the changes made.
Thanks!!
- Ted
[.. snip...]
There are a bunch of FIXME's in this which I assume will get addressed
before we consider this ready to push to mainline?
I have updated the code with your suggestion. I haven't resolved all the
questions. The rest are marked with FIXME in the source. The idea is this
will help Alex to find easily the portion of the code that needs his immediate
attention.
-aneesh
-
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
