Re: CVS commit: src/usr.sbin/rpc.lockd

2009-11-27 Thread Matthias Scheler
On Fri, Nov 27, 2009 at 06:22:30AM +, YAMAMOTO Takashi wrote:
 hi,
 
  Module Name:src
  Committed By:   christos
  Date:   Thu Nov 19 22:27:26 UTC 2009
  
  Modified Files:
  src/usr.sbin/rpc.lockd: lockd_lock.c
  
  Log Message:
  allow comparing v2 and v3 filehandles.
 
 this seems loose to me.
 how about normalizing file handles?
 
 (ideally lockd should be in kernel)

If I understood the Isilon presentation during EuroBSDCon 2009 correctly
FreeBSD 8.0 has an in-kernel lockd. Porting their code might be
an option to achieve that.

Kind regards

-- 
Matthias Scheler  http://zhadum.org.uk/


Re: CVS commit: src/gnu/dist/gdb6/gdb

2009-11-27 Thread David Laight
On Fri, Nov 27, 2009 at 12:45:18AM -0800, Matt Thomas wrote:
 
  i wonder if we can re-add proc0paddr, defined to be = lwp0.l_addr
  at some point in main, to help this work with older gdb?
  
  this seems like a worth-while change since it's part of bsd-kvm.c.
 
 would need to be in md code since L_ADDR needs to come from assym.h.
 
   .globl  _C_LABEL(proc0paddr)
 _C_LABEL(proc0paddr) = _C_LABEL(lwp0) + L_ADDR

Actually I think it might be possibly to do it in a MI .c file!

Some __asm pattern being passed offsetof(struct lwp, l_addr) generating
a global symbol.
One of the forms for defining a symbol is MI.

David

-- 
David Laight: da...@l8s.co.uk


Re: CVS commit: src/usr.sbin/rpc.lockd

2009-11-27 Thread Manuel Bouyer
On Fri, Nov 27, 2009 at 10:30:32AM +, Matthias Scheler wrote:
 If I understood the Isilon presentation during EuroBSDCon 2009 correctly
 FreeBSD 8.0 has an in-kernel lockd. Porting their code might be
 an option to achieve that.

One nice thing about the userland lockd is that if you restart it,
all the locks are dropped. It's an easy way to clean up locks that have
been forgotten by clients ...

-- 
Manuel Bouyer, LIP6, Universite Paris VI.   manuel.bou...@lip6.fr
 NetBSD: 26 ans d'experience feront toujours la difference
--


Re: CVS commit: src/usr.sbin/rpc.lockd

2009-11-27 Thread Manuel Bouyer
On Fri, Nov 27, 2009 at 03:24:51PM +, Matthias Scheler wrote:
 On Fri, Nov 27, 2009 at 03:39:51PM +0100, Manuel Bouyer wrote:
  On Fri, Nov 27, 2009 at 10:30:32AM +, Matthias Scheler wrote:
   If I understood the Isilon presentation during EuroBSDCon 2009 correctly
   FreeBSD 8.0 has an in-kernel lockd. Porting their code might be
   an option to achieve that.
  
  One nice thing about the userland lockd is that if you restart it,
  all the locks are dropped.
 
 Does it really? I have to remove /var/db/statd.status to achieve that.

Yes, the locks (held by lockd) are really dropped. statd.status is used
by statd; statd is there to tell the clients that the server dropped the
locks (e.g. because it rebooted). So with a nfslocking restart, statd
will inform clients tha locks got lost, and the clients will relock
their files. But if a client has forgotten the lock (because it's
down for example, or just because it's buggy) it won't relock it again.

-- 
Manuel Bouyer, LIP6, Universite Paris VI.   manuel.bou...@lip6.fr
 NetBSD: 26 ans d'experience feront toujours la difference
--