That code is useful for debugging with something short, but normally you shouldn't call printf(3) from a signal handler. Yes, it makes sense. I agree about no printfs in signal handler. But this is a demo to prove that SIGTERM is called and put an entry to prove the point. Also so we leave this as a comment and no need to update shutdown man pages then? Sorry to bug just trying to understand the process more : ) On Fri, 22 Dec 2023 at 19:50, Alejandro Colomar <alx@xxxxxxxxxx> wrote: > > Hi rajesh, > > On Fri, Dec 22, 2023 at 07:40:19PM +0530, rajesh wrote: > > Thanks, I'll check them out Alejandro. Yes I m ok with git (but > > looking forward to learn more) > > Ok. > > > I also made a comment on https://bugzilla.kernel.org/show_bug.cgi?id=218266 > > > > Does my comment looks ok? The language, > > Yes, the language looks good. > > > code etc,.? If it's ok then > > putting this code in shutdown makes sense? > > That code is useful for debugging with something short, but normally you > shouldn't call printf(3) from a signal handler. > <https://stackoverflow.com/questions/16891019/how-to-avoid-using-printf-in-a-signal-handler> > > I can't give any other response without actually investigating myself, > since I don't use signal handlers often. > > Have a lovely day, > Alex > > -- > <https://www.alejandro-colomar.es/> > Looking for a remote C programming job at the moment.
