lkml.org 
[lkml]   [2006]   [Jun]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH][RFC] Clean-up: TASK_UNMAPPED_BASE and mmap_base
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1



Arjan van de Ven wrote:
> On Tue, 2006-06-06 at 21:33 -0400, John Richard Moser wrote:
>> This patch applies to 2.6.17-rc6 to replace several occurrences of
>> TASK_UNMAPPED_BASE with current->mm->mmap_base, mm->mmap_base, or base,
>> as appropriate.
>>
>> I am not entirely sure what all of the code I messed with is doing, to
>> be quite honest. Code that seemed to be initializing a task and setting
>> up the mmap() base I left with TASK_UNMAPPED_BASE; code that seemed to
>> be trying to figure out what the mmap() base was I replaced with
>> mm->mmap_base.
>>
>> Because of this, I may have made a couple errors. Could I get some
>> comments on whether any of this is dirty and why? I'll make appropriate
>> changes and re-submit. This only took 2 hours anyway.
>>
>
> now if you put your patch inline you'd get comments in detail
>

Hmm. I hit "Attach" in Thunderbird. Viewing the message shows the
patch inline with the message for me. I wasn't aware I needed to do
something for that. (Any help here is appreciated.. I'm sure I can't
just copy-paste the diff because Thunderbird likes to wrap long lines
PHYSICALLY)

> but you missed that some of the places where this is used is the
> fallback in case the per-task value doesnt result in a good memory hole
> found.

So I replaced TASK_UNMAPPED_BASE somewhere where it needs to be
TASK_UNMAPPED_BASE?

Function? get_unmapped_area() and arch_get_unmapped_area()?

>
>
>

- --
All content of all messages exchanged herein are left in the
Public Domain, unless otherwise explicitly stated.

Creative brains are a valuable, limited resource. They shouldn't be
wasted on re-inventing the wheel when there are so many fascinating
new problems waiting out there.
-- Eric Steven Raymond

We will enslave their women, eat their children and rape their
cattle!
-- Bosc, Evil alien overlord from the fifth dimension
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2.2 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iQIVAwUBRIhDoQs1xW0HCTEFAQKMUhAAnZtNfzUKealy7VhEWHSTdPAvGRK1s/0U
u+Cr9j2uhGPeiYwwZIIpx6Ilr3E2EiFm9alOZfL0kRIiYUEYU1jFXWIi7HYPf4XG
JR/0JyN1X8DU6vdPhMSfLjvhKaTHPhhnuU5IV384gtGp8noTSS2RV0Zyry0r38Lp
AH6Q7yt237a30mv97lSHhjqT+RotiASlLqvTbwLKrDrNy0z1biM6q4cl2eMKLwEg
LtTCF5H0Mh8MPBQSwM1ESejGX83mwnMMfyslwzU3wzglx9uhK/dVoa2l5mGZe6vP
5p5/MweQb8S7T9jzcxiSF0vYJTA/IGMCR2DYmlpN6s5ww2t0VZRdEoNHCPx/3aPA
Ibjz70xl6hS6AjY9lvVcOc73vOvRgjHZl4jhMhJU8mL5i0yWSm6tKgBLG29hcZHI
OuI4hdH16drEf3keRTt6vNeYSV2RgiMj6Gb/bQfXaUt4doAwv6hJiyQERG3lgOl3
KhusWdIvPtR1BJoBHu6BxqmKXa43rGtoJF6I54pVQ3jbwj7HWtixT/XD1GrjZnM8
w/0elm80PSHINfvyC+jt7yELFd9xTVAtGaa2Cakg32Hx/XnpNKlSaxE6Fv6PxuHx
7+78mBYNXp1Mlr4CRsYGq09xanWfwgmMcOVq73+CbIP3r8mp4g5rRY0d6wKdN9AX
J/aLeoYIQvs=
=mQFP
-----END PGP SIGNATURE-----
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-06-08 17:43    [W:0.064 / U:0.352 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site