src/sys/fs/ext2fs, branch main

ext2fs: Use __sdt_used for variables only used by SDT probes

2026-02-09T16:27:44+00:00

Reviewed by:	imp, markj
Differential Revision:	https://reviews.freebsd.org/D55164

vfs: retire the VREF macro

2025-09-27T04:00:59+00:00

It is defined as a plain use of vref.

Churn generated with coccinelle:
@@
expression vp;
@@

- VREF(vp)
+ vref(vp)

vfs: retire the NULLVP macro

2025-09-27T04:00:59+00:00

The kernel was already mostly using plain NULL, just whack it and be
doen with the legacy.

Churn generated with coccinelle:
@@
@@

- NULLVP
+ NULL

ext2fs: Set st_rdev to NODEV when not a device

2025-08-17T17:29:27+00:00

Reviewed by:	kib
Differential Revision:	https://reviews.freebsd.org/D51955

vfs: Introduce VN_ISDEV() macro

2025-08-17T17:29:27+00:00

We frequently need to check if a vnode refers to either a character or
block special, so we might as well have a macro for it.

We somewhat less frequently need to perform similar checks on things
that aren't vnodes (usually a struct vattr *), so add VATTR_ISDEV()
and a generic VTYPE_ISDEV() as well.

Sponsored by:	Klara, Inc.
Sponsored by:	NetApp, Inc.
Reviewed by:	kib
Differential Revision:	https://reviews.freebsd.org/D51947

ext2fs: 'struct ufid': Re-order fields and unpack

2024-12-13T14:04:50+00:00

Re-ordering the fields suppresses the trailing padding which was causing
the structure to overflow 'struct fid'.

While here, re-indent in a more visually pleasing way.

Reviewed by:    rmacklem, emaste, markj
Approved by:    markj (mentor)
MFC after:      5 days
Differential Revision:  https://reviews.freebsd.org/D47955

ext2fs: Fix the size of struct ufid and add a static assert

2024-12-06T02:05:06+00:00

File system specific *fid structures are copied into the generic
struct fid defined in sys/mount.h.
As such, they cannot be larger than struct fid.

This patch packed the structure and checks via a __Static_assert().

Reviewed by:	markj
MFC after:	2 weeks

ext4_ext_tree_init: correct memset initialization

2024-06-25T17:41:11+00:00

gcc -Wmemset-elt-size diagnosed this.  The code was only initializing
1/4 of the array.  However, it was actually harmless, as the only caller
had done an M_ZERO allocation anyway.

Reviewed by:	markj
Differential Revision:	https://reviews.freebsd.org/D45701

Stop treating size 0 as unknown size in vnode_create_vobject().

2024-05-23T06:08:14+00:00

Whenever file is created, the vnode_create_vobject() function will
try to determine its size by calling vn_getsize_locked() as size 0
is ambigious: it means either the file size is 0 or the file size
is unknown.

Introduce special value for the size argument: VNODE_NO_SIZE.
Only when it is given, the vnode_create_vobject() will try to obtain
file's size on its own.

Introduce dedicated vnode_disk_create_vobject() for use by
g_vfs_open(), so we don't have to call vn_isdisk() in the common case
(for regular files).

Handle the case of mediasize==0 in g_vfs_open().

Reviewed by: alc, kib, markj, olce
Approved by: oshogbo (mentor), allanjude (mentor)
Differential Revision: https://reviews.freebsd.org/D45244

sys: Remove ancient SCCS tags.

2023-11-27T05:23:30+00:00

Remove ancient SCCS tags from the tree, automated scripting, with two
minor fixup to keep things compiling. All the common forms in the tree
were removed with a perl script.

Sponsored by:		Netflix