Date: Thu, 18 Jan 2018 22:15:16 +0000 From: Rick Macklem <rmacklem@uoguelph.ca> To: Emmanuel Vadot <manu@bidouilliste.com>, Emmanuel Vadot <manu@FreeBSD.org> Cc: "src-committers@freebsd.org" <src-committers@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "svn-src-head@freebsd.org" <svn-src-head@freebsd.org> Subject: Re: svn commit: r328129 - head/sys/fs/nfsserver Message-ID: <YTXPR0101MB21741F3254BD5F9381B42DBADDE80@YTXPR0101MB2174.CANPRD01.PROD.OUTLOOK.COM> In-Reply-To: <20180118163855.b0a55427709c52d0ec2482c9@bidouilliste.com> References: <201801181528.w0IFSnWm053535@repo.freebsd.org>, <20180118163855.b0a55427709c52d0ec2482c9@bidouilliste.com>
next in thread | previous in thread | raw e-mail | index | archive | help
Emmanuel Vadot wrote: >> Author: manu >> Date: Thu Jan 18 15:28:49 2018 >> New Revision: 328129 >> URL: https://svnweb.freebsd.org/changeset/base/328129 >> >> Log: >> nfs: Do not printf each time a lock structure is freed during module u= nload >> >> There can be a lot of those structures and printing a line each time w= e free >> one on module unload. > > Finishing the sentence : > > There can be a lot of those structures and printing a line each time >we free one on module unload is really time consuming, especially when >you have a serial console plugged in. I will note that when I put a printf() like this in the code, it is becaus= e I consider it an abnormal occurrence. It might be better to at least print the message once. For this case, normally all lock structures should go away when clients unmount and unloading the nfsd module while there are active mounts is not a safe practice. (Again NFSv4 isn't like NFSv3, there is server stat= e for NFSv4.) rick
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?YTXPR0101MB21741F3254BD5F9381B42DBADDE80>