Re: [PATCH 0/4] new statd [take 2]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Aug 5, 2009, at 1:15 PM, Jeff Layton wrote:
On Wed, 05 Aug 2009 10:45:30 -0400
Chuck Lever <chuck.lever@xxxxxxxxxx> wrote:

Apparently there is a 100KB per-email size limit imposed by
vger.kernel.org.  I've broken up the large statd patch, and
postponed the nfs(8) man page patches to another day.

Yesterday it was decided that:

 [PATCH 3/5] statd: change default state dir to /var/lib/nfs/statd

was overkill. That one has been dropped from this series.

The first two of the present patches introduce the .c files under
utils/new-statd/ .  The next patch adds actools infrastructure for
building the new statd and sm-notify executables.  The last patch
adds two new man pages for statd and sm-notify.

The old statd remains in utils/statd.  If sqlite3 and TI-RPC is
present on the build system, and TI-RPC is enabled during
configuration, the new statd is built and installed instead of the
old one.

See the description in the first patch for more details on the
features of the new implementation.


Nice work, Chuck.

Rewriting statd from scratch is a pretty bold step, but I think it's
warranted.

I've been testing this new implementation over several iterations and
the recent ones have worked well for me so far. I've also walked
through the code and it all seems to make sense.

Acked-by: Jeff Layton <jlayton@xxxxxxxxxx>

Independent testing was an important piece of this work. Thanks to Jeff and Sankar Bose at NetApp for their assistance so far.

--
Chuck Lever
chuck[dot]lever[at]oracle[dot]com
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux