Re: rpctrace / libpager / signal preemptor

2016-12-01 Thread Brent W. Baccala
On Wed, Nov 30, 2016 at 11:12 PM, Samuel Thibault wrote: > > Err, I'm sorry, did you perhaps miss the fix I made after: > > commit 406b031c996ec4cd8c76d251de8b7bf462d8b975 > Author: Samuel Thibault > Date: Sun Nov 20 16:16:24 2016 +0100 >

Re: rpctrace / libpager / signal preemptor

2016-12-01 Thread Samuel Thibault
Hello, Brent W. Baccala, on Wed 30 Nov 2016 22:47:09 -1000, wrote: > The patch works, but is incomplete.  Samuel's test programs attempt to access > unmapped memory addresses, which generate KERN_MEMORY_FAILURE, but ext2fs > attempts to access mapped addresses back by a memory manager returning

Re: rpctrace / libpager / signal preemptor

2016-12-01 Thread Brent W. Baccala
On Wed, Nov 16, 2016 at 9:05 AM, Samuel Thibault wrote: > Samuel Thibault, on Wed 16 Nov 2016 19:50:07 +0100, wrote: > > Samuel Thibault, on Wed 16 Nov 2016 19:46:52 +0100, wrote: > > > The attached testcase does get the faulting address. > > > > And the attached

Re: rpctrace / libpager / signal preemptor

2016-11-21 Thread Samuel Thibault
Samuel Thibault, on Mon 21 Nov 2016 09:16:19 +0100, wrote: > Brent W. Baccala, on Sun 20 Nov 2016 21:50:04 -1000, wrote: > > I suppose I could test your patch just by building the > > library itself, but the Debian package build calls "make check" and it's > > causing me all kinds of grief. > >

Re: rpctrace / libpager / signal preemptor

2016-11-21 Thread Samuel Thibault
Hello, Brent W. Baccala, on Sun 20 Nov 2016 21:50:04 -1000, wrote: > I suppose I could test your patch just by building the > library itself, but the Debian package build calls "make check" and it's > causing me all kinds of grief. You can use export DEB_BUILD_OPTIONS=nocheck to avoid that,

Re: rpctrace / libpager / signal preemptor

2016-11-21 Thread Svante Signell
On Sun, 2016-11-20 at 21:50 -1000, Brent W. Baccala wrote: > On Sun, Nov 20, 2016 at 5:37 AM, Samuel Thibault > wrote: > > Samuel Thibault, on Sun 20 Nov 2016 14:50:50 +0100, wrote: > >  > Sorry I haven't answered for a few days.  I've been trying to test your patch > by

Re: rpctrace / libpager / signal preemptor

2016-11-20 Thread Brent W. Baccala
On Sun, Nov 20, 2016 at 5:37 AM, Samuel Thibault wrote: > Samuel Thibault, on Sun 20 Nov 2016 14:50:50 +0100, wrote: > > Samuel Thibault, on Wed 16 Nov 2016 20:05:49 +0100, wrote: > > > Samuel Thibault, on Wed 16 Nov 2016 19:50:07 +0100, wrote: > > > > And is fixed by

Re: rpctrace / libpager / signal preemptor

2016-11-20 Thread Samuel Thibault
Samuel Thibault, on Sun 20 Nov 2016 14:50:50 +0100, wrote: > Samuel Thibault, on Wed 16 Nov 2016 20:05:49 +0100, wrote: > > Samuel Thibault, on Wed 16 Nov 2016 19:50:07 +0100, wrote: > > > Samuel Thibault, on Wed 16 Nov 2016 19:46:52 +0100, wrote: > > > > The attached testcase does get the

Re: rpctrace / libpager / signal preemptor

2016-11-20 Thread Samuel Thibault
Samuel Thibault, on Wed 16 Nov 2016 20:05:49 +0100, wrote: > Samuel Thibault, on Wed 16 Nov 2016 19:50:07 +0100, wrote: > > Samuel Thibault, on Wed 16 Nov 2016 19:46:52 +0100, wrote: > > > The attached testcase does get the faulting address. > > > > And the attached testcase doesn't. > > And is

Re: rpctrace / libpager / signal preemptor

2016-11-16 Thread Samuel Thibault
Samuel Thibault, on Wed 16 Nov 2016 19:50:07 +0100, wrote: > Samuel Thibault, on Wed 16 Nov 2016 19:46:52 +0100, wrote: > > The attached testcase does get the faulting address. > > And the attached testcase doesn't. And is fixed by the attached patch, could you try it? Samuel Index:

Re: rpctrace / libpager / signal preemptor

2016-11-16 Thread Samuel Thibault
Samuel Thibault, on Wed 16 Nov 2016 19:46:52 +0100, wrote: > The attached testcase does get the faulting address. And the attached testcase doesn't. > I really believe the issue is related to this: Confirmed :) > > Note that there is a > > > > /* XXX what if handler != action->handler (for

Re: rpctrace / libpager / signal preemptor

2016-11-16 Thread Samuel Thibault
Samuel Thibault, on Mon 14 Nov 2016 01:07:40 +0100, wrote: > > Once that's been resolved, then we're back to the problem with signal > > preemptors!  libpager/pager-memcpy.c includes the following code: > > > >   void fault (int signo, long int sigcode, struct sigcontext *scp) > >     { > >  

Re: rpctrace / libpager / signal preemptor

2016-11-13 Thread Samuel Thibault
Hello, Brent W. Baccala, on Tue 08 Nov 2016 20:43:29 -1000, wrote: >    _pager_lock_object (p, offset, length, MEMORY_OBJECT_RETURN_NONE, 1, > - VM_PROT_WRITE, 1); > +   VM_PROT_WRITE, 0); Applied, thanks! > Once that's been resolved, then we're back to