Re: vm_map.h

2014-03-18 Thread John Baldwin
On Thursday, March 13, 2014 6:46:13 pm Bruno Lauzé wrote: I suggesting this because it is used more ahead in the same file, but the other way around is fine for me No need to move them around: Index: vm_map.h === --- vm_map.h

Re: vm_map.h

2014-03-13 Thread John Baldwin
On Wednesday, March 12, 2014 5:13:28 pm Bruno Lauzé wrote: The two defines in vm/vm_map.h #define min_offset header.start/* (c) */ #define max_offset header.end /* (c) */ are really getting in the way because those words are most likely to be used downstream. I would

RE: vm_map.h

2014-03-13 Thread Bruno Lauzé
I suggesting this because it is used more ahead in the same file, but the other way around is fine for me From: j...@freebsd.org To: freebsd-current@freebsd.org Subject: Re: vm_map.h Date: Thu, 13 Mar 2014 14:00:59 -0400 CC: brunola...@msn.com

vm_map.h

2014-03-12 Thread Bruno Lauzé
The two defines in vm/vm_map.h  #define min_offset   header.start    /* (c) */ #define max_offset   header.end      /* (c) */ are really getting in the way because those words are most likely to be used downstream. I would suggest renaming those defines to: #define vm_min_offset

Re: cvs commit: src/sys/vm vm_map.c vm_map.h vm_pageout.c

2003-03-13 Thread Garrett Wollman
On Wed, 12 Mar 2003 16:51:15 -0800, David Schultz [EMAIL PROTECTED] said: A real problem is that a swapped out process' uarea has to be paged back in, even when no memory is available. I don't think there's an easy way around that, given that you need the uarea and kernel stack to handle the

Re: cvs commit: src/sys/vm vm_map.c vm_map.h vm_pageout.c

2003-03-13 Thread David Schultz
Thus spake Garrett Wollman [EMAIL PROTECTED]: On Wed, 12 Mar 2003 16:51:15 -0800, David Schultz [EMAIL PROTECTED] said: A real problem is that a swapped out process' uarea has to be paged back in, even when no memory is available. I don't think there's an easy way around that, given that

Re: cvs commit: src/sys/kern kern_mtxpool.c src/sys/sys kernel.h src/sys/vm vm_fault.c vm_glue.c vm_map.c vm_map.h vm_pageout.c vm_zone.c

2002-03-18 Thread Brian F. Feldman
Matthew Dillon [EMAIL PROTECTED] wrote: : :Hi, : :Sorry to unwantedly butt in, but the patch supplied by Seigo actually :solved the vm_map.c locking problems which used to come up on my system. :Just some info. :) : :Regards, : : -- Hiten Pandya : -- [EMAIL PROTECTED] It

Re: cvs commit: src/sys/kern kern_mtxpool.c src/sys/sys kernel.hsrc/sys/vm vm_fault.c vm_glue.c vm_map.c vm_map.h vm_pageout.c vm_zone.c

2002-03-17 Thread Matthew Dillon
I have no idea what the frig you guys are doing in vm_map.c. I would recommend that all the changes be backed out. Then I would recommend that the following be done: * change the lockmgr vm_map lock to be exclusive-only * test commit * change the lockmgr

Re: cvs commit: src/sys/kern kern_mtxpool.c src/sys/sys kernel.h src/sys/vm vm_fault.c vm_glue.c vm_map.c vm_map.h vm_pageout.c vm_zone.c

2002-03-17 Thread Hiten Pandya
Hi, Sorry to unwantedly butt in, but the patch supplied by Seigo actually solved the vm_map.c locking problems which used to come up on my system. Just some info. :) Regards, -- Hiten Pandya -- [EMAIL PROTECTED] --- Matthew Dillon [EMAIL PROTECTED] wrote: I have no idea what the frig

Re: cvs commit: src/sys/kern kern_mtxpool.c src/sys/sys kernel.h src/sys/vm vm_fault.c vm_glue.c vm_map.c vm_map.h vm_pageout.c vm_zone.c

2002-03-17 Thread Matthew Dillon
: :Hi, : :Sorry to unwantedly butt in, but the patch supplied by Seigo actually :solved the vm_map.c locking problems which used to come up on my system. :Just some info. :) : :Regards, : : -- Hiten Pandya : -- [EMAIL PROTECTED] It fixed some things, it broke some things. Pretty much

Re: cvs commit: src/sys/kern kern_mtxpool.c src/sys/sys kernel.h src/sys/vm vm_fault.c vm_glue.c vm_map.c vm_map.h vm_pageout.c vm_zone.c

2002-03-17 Thread Robert Watson
On Sun, 17 Mar 2002, Matthew Dillon wrote: It fixed some things, it broke some things. Pretty much standard fare for anyone who has ever done work on the vm_map lock, including yours truely. John Dyson couldn't get it right, David Greenman couldn't get it right, I couldn't

Re: cvs commit: src/sys/kern kern_mtxpool.c src/sys/sys kernel.h src/sys/vm vm_fault.c vm_glue.c vm_map.c vm_map.h vm_pageout.c vm_zone.c

2002-03-17 Thread Matthew Dillon
:Hmm. Ok, so right now the code that has been branched for DP1 makes use :of Brian's recent locking commits. My original thought was we'd simply :back it out of that branch to make sure that the DP is reliable. How hard :would it be for us to switch to what you propose (just convert all slocks

sx_upgrade() (was: Re: cvs commit: src/sys/kern kern_mtxpool.c src/sys/sys kernel.h src/sys/vm vm_fault.c vm_glue.c vm_map.c vm_map.h vm_pageout.c vm_zone.c)

2002-03-16 Thread Seigo Tanimura
[Add jhb and move to -current] On Fri, 15 Mar 2002 10:53:20 -0800, Alfred Perlstein [EMAIL PROTECTED] said: Alfred * Brian F. Feldman [EMAIL PROTECTED] [020315 03:22] wrote: Alfred Perlstein [EMAIL PROTECTED] wrote: What is the problem? Damn good question. Are the tracebacks

Re: sx_upgrade() (was: Re: cvs commit: src/sys/kern kern_mtxpool.c src/sys/sys kernel.h src/sys/vm vm_fault.c vm_glue.c vm_map.c vm_map.h vm_pageout.c vm_zone.c)

2002-03-16 Thread Seigo Tanimura
On Sat, 16 Mar 2002 19:08:53 +0900, Seigo Tanimura [EMAIL PROTECTED] said: Seigo Attached patch implements sx_upgrade() which should work as you said Seigo above. This compiles fine, but is not tested yet. The last patch breaks INVARIANTS. This one compiles and seems to work.