Re: System V Shared Memory

2002-04-30 Thread Jeff Bailey
On Mon, Apr 15, 2002 at 05:11:17PM -0400, Neal H Walfield wrote: > 2002-04-15 Neal H Walfield <[EMAIL PROTECTED]> > > * sysdeps/mach/hurd/Makefile (sysdep_routines) [$(subdir) = > sysvipc]: Add sysvshm. I notice conversation has stalled on this. Is it at a point where you re

Re: System V Shared Memory

2002-04-16 Thread Jon Arney
Neal, > My problem is not so much with setting the owner and changing the file > mode, but rather, using the author field for the creator uid and the > flags for the pid of the last operation, etc. Please understand that > I was sloppy as it was not my intention to decide what to overload > with

Re: System V Shared Memory

2002-04-15 Thread Neal H Walfield
> I have a couple of questions on your implementation, however, that I > just > don't understand. You commented out the entire 'IPC_SET' case from the > shmctl function. I'm not clear on why you did this. How else can we > enforce permissions on shared memory segments? I'm not sure why you > o

Re: System V Shared Memory

2002-04-15 Thread Jon Arney
Neal, Thanks for doing the cleanup. I'll try to study your changes and improve next time. I'll concede that there was probably a logic error or two in there and I didn't throw any mutexes in for thread safety. My fault. The file 'list.h' comes from the Linux source and I like to use it becuase

Re: System V Shared Memory

2002-04-15 Thread Neal H Walfield
I have rewritten your patch. There were a variety of problems with it.: o Coding and logic errors. o Not robust. o Not thread safe. o You should use the libc functions, not hurd rpcs. o You need to use the GNU coding standards. o You need to follow the glibc conventions o Where did

Re: System V Shared Memory

2002-03-22 Thread Jon Arney
Marcus, Roland, > Neal did something on this before. Here's what I've done. I started with Neal's code and for the most part the ipcs and ipcrm utilities did not need any changes. I took the shmat/shmget/shmctl routines and re-worked them, replacing the need for a new translator with existing _