Age | Commit message (Collapse) | Author |
|
and return EOPNOTSUPP directly from the VOP_* functions.
Filesystems should, at some point fill in every function
in the vop_default struct so we can get rid of the 'if'
statements in VOP_*.
|
|
Diff from Antti Harri.
|
|
unrelated, and his alpha is much happier now.
OK deraadt@
|
|
(in case eventually we change their type from u_long to something else),
and do not truncate them to 32 bits in the ntfs code.
ok tedu@
|
|
have been resolved.
|
|
|
|
Small cleanup while here, nuke the ntfs_bypass function and just
use eopnotsupp like every other filesystem, it makes no sense to
return ENOTTY for unimplemented VOPs.
|
|
|
|
|
|
|
|
fix a bonus off by one bug. ok matthew
|
|
ok deraadt krw
|
|
$ADORABLE_FELINE.
ok deraadt@ matthew@
|
|
around vinvalbuf(). vrele() -> vput() after VOP_CLOSE().
ok tedu@
|
|
|
|
and remove the function. ok thib
|
|
|
|
replace ntfs_nthash_slock usage with comments prefixed XXXLOCKING (for
grepability).
This lock looks to be correct, but it could well be the bad way to do it
(having a rwlock for inserts to avoid races inserting the same inode but
then simple locking on list accesses).
approach discussed with deraadt@
|
|
sysctl.h was reliant on this particular include, and many drivers included
sysctl.h unnecessarily. remove sysctl.h or add proc.h as needed.
ok deraadt
|
|
- properly spell 'usefulness'
|
|
- remove #ifdef someos blocks, makes this a tad easier to read
agreed by art@ and thib@
|
|
which is exactly what the macro does.
Macro's that are nothing more then:
#define FUNCTION(arg) function(arg)
are almost always pointless and should go away.
OK blambert@
Agreed by many.
|
|
OK thib@
|
|
Tested by a couple of people, no regressions.
|
|
This requires that structures defined within __packed structures must
independently request that they themselves become __packed, too.
worked on with toby
CVS: ----------------------------------------------------------------------
|
|
Not sure what's more surprising: how long it took for NetBSD to
catch up to the rest of the BSDs (including UCB), or the amount of
code that NetBSD has claimed for itself without attributing to the
actual authors.
OK deraadt@
|
|
ok dlg@
|
|
ok gilles
|
|
|
|
In ip_esp.c all allocated memory is now zero'd in the
"malloc(sizeof(*tc) + alen ..." case. The +alen memory was not
initialized by the bzero() call. Noticed by chl@.
"Looks good" art@ "seems ok" chl@
|
|
MALLOC/FREE, etc. Just adding M_ZERO to malloc() and deleting an
immediately adjacent bzero().
|
|
"this code sucks -- pedro", "just put it in -- tom".
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Inspiration from miod@, okay deraadt@. Tested on i386, macppc and amd64.
|
|
of panics and bugfixes. Access curproc directly, do not expect a process
pointer as an argument. Should fix many "process context required" bugs.
Incentive and okay millert@, okay marc@. Various testing, thanks.
|
|
|
|
|
|
To check a directory's in-use bitmap bit by bit, we use
a pointer to an 8 bit wide unsigned value.
The index used to dereference this pointer is calculated
by shifting the bit index right 3 bits. Then we do a
logical AND with the bit# represented by the lower 3
bits of the bit index.
This is an idiomatic way of iterating through a bit map
with simple bitwise operations.
This commit fixes the bug that we only checked bits
3:0 of each 8 bit chunk, because we only used bits 1:0
of the bit index for the bit# in the current 8 bit value.
This resulted in files not being returned by getdirentries(2).
Change the type of the bit map pointer from `char *' to
`u_int8_t *'.
From FreeBSD
ok pedro
|
|
front of the pointer value. Therefore, remove the "0x"
from the format string.
From FreeBSD
ok millert@
|
|
|
|
of 64bit values to 'int'
- read always single cluster of data, to avoid confusing the buffer
cache; this appears to fix the random file content corruption which
happens when more than one cluster is read at the same time, i.e. for
files > 3*cluster_size
- fix some questionable pointer arithmetics; gcc-3.4.x didn't like it
- g/c redundant debug printf
- remove all traces of non-working quota support
add quota support to TODO - makes sense only once writing support
would be implemented, and only once NTFS would support notion of file 'owner'
- use 64bit local variable for free space computation
From NetBSD
- Comment fixes from joris@
ok millert@ joris@ pedro@
|
|
|
|
to the fact ntfs uses the generic locking functions which
do not lock anything right now.
ok pedro@
|
|
from FreeBSD
ok pedro tedu miod, tested by millert and myself
|