diff options
author | green <green@FreeBSD.org> | 2000-07-07 22:11:37 +0000 |
---|---|---|
committer | green <green@FreeBSD.org> | 2000-07-07 22:11:37 +0000 |
commit | efb639324d91add439f24a49689fdc12cdecd182 (patch) | |
tree | 10560f9c7d2fa01957faa783a7d91d49179dc818 /sys/kern/sys_generic.c | |
parent | f3eed69a97bb9038ef6dc3b654c5d2bf55b3f81f (diff) | |
download | FreeBSD-src-efb639324d91add439f24a49689fdc12cdecd182.zip FreeBSD-src-efb639324d91add439f24a49689fdc12cdecd182.tar.gz |
Remove two micro-pessimizations I made. Bruce is teaching me well :)
KTRPOINT(p, KTR_GENIO) is more uncommon than error == 0, so it should
be first in the && statement.
Diffstat (limited to 'sys/kern/sys_generic.c')
-rw-r--r-- | sys/kern/sys_generic.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/kern/sys_generic.c b/sys/kern/sys_generic.c index 562536a..7489e0e 100644 --- a/sys/kern/sys_generic.c +++ b/sys/kern/sys_generic.c @@ -184,7 +184,7 @@ dofileread(p, fp, fd, buf, nbyte, offset, flags) error = 0; cnt -= auio.uio_resid; #ifdef KTRACE - if (error == 0 && KTRPOINT(p, KTR_GENIO)) { + if (KTRPOINT(p, KTR_GENIO) && error == 0) { ktruio.uio_iov = &ktriov; ktruio.uio_resid = cnt; ktrgenio(p->p_tracep, fd, UIO_READ, &ktruio, error); @@ -382,7 +382,7 @@ dofilewrite(p, fp, fd, buf, nbyte, offset, flags) } cnt -= auio.uio_resid; #ifdef KTRACE - if (error == 0 && KTRPOINT(p, KTR_GENIO)) { + if (KTRPOINT(p, KTR_GENIO) && error == 0) { ktruio.uio_iov = &ktriov; ktruio.uio_resid = cnt; ktrgenio(p->p_tracep, fd, UIO_WRITE, &ktruio, error); |