Re: updates?

[Charles Cui <charles.cui1984%gmail.com@localhost>]

Hi Christos,

   Thanks for providing these information!
I will revisit my current code base and think carefully
how those two bugs are generated (by keeping your hints in mind).
Hopefully I can find the correct place and realize my mistakes. :)

Thanks Charles

2016-07-31 5:54 GMT-07:00 Christos Zoulas <christos%zoulas.com@localhost>:

> On Jul 30, 11:42am, charles.cui1984%gmail.com@localhost (Charles Cui) wrote:
> -- Subject: Re: updates?
>
> | Yes, it works for me, each time I send you a new patch, I will compile my
> | kernel and userland,
> | and testing using the program we have written.
> |
> | Here is a short video that I created from my environment which shows the
> | real time signal works for me
> | https://www.dropbox.com/s/amds3jmpfev8jz9/realtime_signal.mov?dl=0
> |
> | Also, I looked at all files needed in the real time signal patches,
> | and attached all files of them, you can replace using these files and see
> | whether there are any
> | differences with the current settings.
> |
> | /usr/src/sys/sys/signal.h
> |
> | /usr/src/include/limits.h
> |
> | /usr/src/lib/libc/gen/sysconf.c
> |
> | /usr/src/sys/kern/kern_sig.c
> |
> | /usr/src/sys/kern/sys_sig.c
> |
> | /usr/src/sys/sys/signalvar.h
> |
> | /usr/src/sys/sys/unistd.h
> |
> | 2016-07-30 9:36 GMT-07:00 Christos Zoulas <christos%zoulas.com@localhost>:
>
> There are two issues with the code that I found and I have everything
> working now. It is not enough to test with the test program (which
> works); for example as I mentioned before, booting with a kernel
> that has all your patches fails on sshd (you can't login remotely)
> and an interrupted make does not work properly. If you want I can
> show you what's wrong with the changes, but I think it is better
> if you find the problems yourself. The idea here is to make sure
> that the code works exactly like before in the absense of real-time
> signals, which is not the case currently. Both bugs are in kern_sig.c.
> One of them is the critical one that breaks signal delivery, the other
> is more of a clarification of what happens when we want to maintain
> the pending signal mask.
>
> Best,
>
> christos
>