good catch. thank you.
--
cinap
Yoann Padioleau p...@fb.com once said:
in the newseg() function there is:
[...]
I think it should be
if(mapsize (SEGMAPSIZE))
mapsize = SEGMAPSIZE;
Yes, you're correct.
The code allows the creation of a segment VM map with more
than SEGMAPSIZE Ptes.
Personally, I'd remove
I have finally got around to porting p9p's stats(1),
I renamed it nstats(1) for now.
It is split into two parts, nstats(1) the GUI, and auxstats(1)
which collects the performance data.
backends supplied for plan9, linux, and windows (at present).
You will need my dos(1) command line access to
I think it should be
if(mapsize (SEGMAPSIZE))
mapsize = SEGMAPSIZE;
hmm. i think this code is correct. ssegmap is a static map
to handle small segments. small segments fit in ssegmap.
the point must have been to avoid malloc.
this test is a little more questionable
i don't know where a history of stuff older than sources (2002) is.
http://swtch.com/plan9history/
too bad, i don't see anthony's diff, and i get this error
(perhaps unrelated)
Too many diffs (26 25). Stopping.
- erik
On 3 June 2014 02:14, Yoann Padioleau p...@fb.com wrote:
if(mapsize (SEGMAPSIZE*PTEPERTAB))
mapsize = (SEGMAPSIZE*PTEPERTAB);
...
I made the change you suggest in the PAE kernel but perhaps Erik missed it
during his merge:
if(mapsize nelem(s-ssegmap)){
mapsize *= 2;
I made the change you suggest in the PAE kernel but perhaps Erik missed it
during his merge:
if(mapsize nelem(s-ssegmap)){
mapsize *= 2;
if(mapsize SEGMAPSIZE)
mapsize = SEGMAPSIZE;
s-map = smalloc(mapsize*sizeof(Pte*));
s-mapsize = mapsize;
}
ok. good. that's what happened. perhaps
On 3 June 2014 18:08, erik quanstrom quans...@quanstro.net wrote:
ok. good. that's what happened. perhaps some change needs
to be made to segattach to mirror this change.
I think this is independent. In ibrk I changed the test from using newsize
to calculating mapsize first and
checking
Looking at the diffs I can see how it might easily be missed, since there
are many other changes to allow for different page sizes.
It's different again in the 64-bit version, where I really went over the
top (nice 1914 centenary reference there) on changes.
I'd better put that code somewhere
03 июня 2014 г., в 18:09, Steve Simon st...@quintile.net написал(а):
I have finally got around to porting p9p's stats(1),
I renamed it nstats(1) for now.
http://plan9.bell-labs.com/contrib/steve/nstats.tbz
/sources between .com and /contrib.
Indeed, my bad.
11 matches
Mail list logo