Date: Tue, 16 Jul 1996 03:19:48 -0700 (PDT) From: Doug Rabson <dfr> To: CVS-committers, cvs-all, cvs-sys Subject: cvs commit: src/sys/nfs nfs_bio.c nfs_subs.c nfs_vnops.c Message-ID: <199607161019.DAA06243@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
dfr 96/07/16 03:19:47 Modified: sys/nfs nfs_bio.c nfs_subs.c nfs_vnops.c Log: Various fixes from frank@fwi.uva.nl (Frank van der Linden) via rick@snowhite.cis.uoguelph.ca: 1. Clear B_NEEDCOMMIT in nfs_write to make sure that dirty data is correctly send to the server. If a buffer was dirtied when it was in the B_DELWRI+B_NEEDCOMMIT state, the state of the buffer was left unchanged and when the buffer was later cleaned, just a commit rpc was made to the server to complete the previous write. Clearing B_NEEDCOMMIT ensures that another write is made to the server. 2. If a server returned a server (for whatever reason) returned an answer to a write RPC that implied that fewer bytes than requested were written, bad things would happen. 3. The setattr operation passed on the atime in stead of the mtime to the server. The fix is trivial. 4. XIDs always started at 0, but this caused some servers (older DEC OSF/1 3.0 so I've been told) who had very long-lasting XID caches to get confused if, after a reboot of a BSD client, RPCs came in with a XID that had in the past been used before from that client. Patch is to use the current time in seconds as a starting point for XIDs. The patch below is not perfect, because it requires the root fs to be mounted first. This is because of the check BSD systems do, comparing FS time to system time. Reviewed by: Bruce Evans, Terry Lambert. Obtained from: frank@fwi.uva.nl (Frank van der Linden) via rick@snowhite.cis.uoguelph.ca Revision Changes Path 1.24 +8 -1 src/sys/nfs/nfs_bio.c 1.31 +24 -11 src/sys/nfs/nfs_subs.c 1.34 +2 -2 src/sys/nfs/nfs_vnops.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199607161019.DAA06243>