Date: Sun, 19 May 2019 05:06:04 -0400 From: Cy Schubert <Cy.Schubert@cschubert.com> To: freebsd-current@freebsd.org, Konstantin Belousov <kostikbel@gmail.com>, Rick Macklem <rmacklem@uoguelph.ca> Cc: Alan Somers <asomers@freebsd.org>, "freebsd-current@FreeBSD.org" <freebsd-current@FreeBSD.org> Subject: Re: RFC w.r.t. toggling debugging on/off for mountd via a signal Message-ID: <564922E3-7BE4-498B-871A-6152A0DF9AD4@cschubert.com> In-Reply-To: <20190519085501.GL2748@kib.kiev.ua> References: <YQXPR0101MB2055B12D1AB31D4F70AF2480DD050@YQXPR0101MB2055.CANPRD01.PROD.OUTLOOK.COM> <CAOtMX2gXdjf2NhtU5Jt%2BMGg3zcONx_Gyun%2B2thy2rDYDt3FugQ@mail.gmail.com> <YQXPR0101MB205583575B42875DD298FA65DD050@YQXPR0101MB2055.CANPRD01.PROD.OUTLOOK.COM> <20190519085501.GL2748@kib.kiev.ua>
next in thread | previous in thread | raw e-mail | index | archive | help
On May 19, 2019 4:55:01 AM EDT, Konstantin Belousov <kostikbel@gmail=2Ecom>= wrote: >On Sun, May 19, 2019 at 02:47:10AM +0000, Rick Macklem wrote: >> Alan Somers wrote: >> >On Sat, May 18, 2019 at 7:59 PM Rick Macklem <rmacklem@uoguelph=2Eca> >wrote: >> >> >> >> Hi, >> >> >> >> I've been working with Peter Errikson on a patch for mountd that >adds a new option >> >> for incremental updating of exports=2E This seems to be helping a >lot w=2Er=2Et=2E performance >> >> on an NFS server with lots (10000+) of exported file systems=2E >> >> >> >> I have debug syslog() calls in the code, which I/Peter think would >be worth keeping >> >> in the production code in case someone runs into problems with >this new option=2E >> >> >> >> As such, I'd like to have the code compiled in by default (not >only if DEBUG is defined, >> >> as mountd=2Ec has now)=2E I also was thinking it would be nice if th= e >daemon didn't need >> >> to be restarted to enable/disable the debugging output, since that >breaks NFS >> >> mounting during the restart=2E >> >> >> >> So, I was thinking of having the debugging output toggled on/off >via SIGUSR1=2E >> >> >> >> What do you think of this idea? >> >> Any other/better ways to do this? >> >> Also, would LOG_DAEMON and LOG_DEBUG sound like the correct >facility and >> >> priority for theses syslog() calls? >> >> >> >> Thanks in advance for any comments, rick >> > >> >If the debug messages aren't so verbose that they'll slow down >> >syslogd, then you can just leave them enabled all the time=2E syslogd >> >will filter them=2E However, if they're super-verbose then SIGUSR1 >> >sounds reasonable=2E I can't think of another daemon with runtime >> >selectable logging verbosity like that=2E >> Yes, these are pretty chatty=2E 5-10 lines for each entry in an exports >file=2E >> Multiply that times the number of entries=2E (Peter's servers are >between 20000 >> to 72000+ file systems=2E Not sure if he has multiple entries/file >system=2E) >>=20 >> To give you a clue, without this patch, it can take 20sec->over 1min >to reload them >> when mountd gets a SIGHUP=2E >>=20 >> It's just that the export file handling code is pretty convoluted, so >I think the patch >> is ok, but I won't be too surprised if someone finds a problem=2E > >Instead of toggling the debugging by SIGUSR1, you can make the daemon >to try to open some file by receiving the normal reload signal, e=2Eg=2E >/var/run/mountd=2Edebug=2E If the file is present, the debugging is >enabled=2E >The advantage is that the user knows what is the debugging state, >instead >of requiring to count the number of sent SIGUSR1's=2E > >More, in traditions of malloc(3), /var/run/mountd=2Edebug could be a >symlink, >which is read and interpreted as some options controlling the debug=2E >_______________________________________________ >freebsd-current@freebsd=2Eorg mailing list >https://lists=2Efreebsd=2Eorg/mailman/listinfo/freebsd-current >To unsubscribe, send any mail to >"freebsd-current-unsubscribe@freebsd=2Eorg" Instead of syslog() calls, DTrace probes are designed for this type of ins= trumentation=2E=20 --=20 Pardon the typos and autocorrect, small keyboard in use=2E Cheers, Cy Schubert <Cy=2ESchubert@cschubert=2Ecom> FreeBSD UNIX: <cy@FreeBSD=2Eorg> Web: http://www=2EFreeBSD=2Eorg The need of the many outweighs the greed of the few=2E
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?564922E3-7BE4-498B-871A-6152A0DF9AD4>