src/sys/fs/devfs, branch releng/11.0 FreeBSD source tree https://cgit-dev.freebsd.org/src/atom?h=releng%2F11.0 2016-06-15T15:55:14Z Another follow-up to r291460. Only access vp->v_rdev for VCHR vnodes 2016-06-15T15:55:14Z Konstantin Belousov kib@FreeBSD.org 2016-06-15T15:55:14Z urn:sha1:2d5bba3ae320422ed721d160fd59624ba449b6ad in devfs_reclaim(). Reported and tested by: pho Sponsored by: The FreeBSD Foundation Approved by: re (gjb) MFC after: 1 week Remove zero assignments in the cdev allocator. cdp memory is 2016-05-21T09:55:32Z Konstantin Belousov kib@FreeBSD.org 2016-05-21T09:55:32Z urn:sha1:ae402378745cc26ec43bf8d0725cc7be6a1e63b8 requested with M_ZERO. Sponsored by: The FreeBSD Foundation MFC after: 2 weeks sys/devfs: unsign an index to prevent signed integer overflow. 2016-04-28T02:39:43Z Pedro F. Giffuni pfg@FreeBSD.org 2016-04-28T02:39:43Z urn:sha1:b114da42afb21ef916c28e180b1848dc194eab8e cdp_maxdirent in struct:cdev_priv is of type u_int. Use the same type for the corresponding index in devfs_revoke(). MFC after: 1 week When devfs dirent is freed, a vnode might still keep a pointer to it, 2016-01-22T20:30:51Z Konstantin Belousov kib@FreeBSD.org 2016-01-22T20:30:51Z urn:sha1:1a2dd035fbfb7a46f72d47cf179e3d87e8b48484 apparently. Interlock and clear the pointer to avoid free memory dereference. Submitted by: bde (previous version) MFC after: 3 weeks Assert that the linkage between struct cdev_privdata and and struct 2016-01-17T08:34:35Z Konstantin Belousov kib@FreeBSD.org 2016-01-17T08:34:35Z urn:sha1:aeace3c33cc49227dc51892c0afa7023555f6ddf file is consistent. Tested by: pho Sponsored by: The FreeBSD Foundation MFC after: 2 weeks Make devfs_fpdrop() static. It was not a public KPI, and it has no 2016-01-13T14:03:06Z Konstantin Belousov kib@FreeBSD.org 2016-01-13T14:03:06Z urn:sha1:a53b7c692dcb812b3745f113e0d99d29cac58ff6 reason to remain exported for some time. Sponsored by: The FreeBSD Foundation MFC after: 1 week Hide transient EBADF errors caused by the parallel revoke(2) or forced 2016-01-02T20:29:28Z Konstantin Belousov kib@FreeBSD.org 2016-01-02T20:29:28Z urn:sha1:fb57d63e4760e74c3f410d3a3bfd7a67541ad67f unmount of devfs mounts, by restarting the failed syscall. When restarted, failing syscalls eventually either stop finding the node and returning ENOENT, or the vnode op vectors finally transition to the deadfs vop. The later return EIO or other error, more appropriate for the operation. Submitted by: bde Tested by: pho MFC after: 3 weeks Minor style cleanup. 2016-01-01T15:48:48Z Konstantin Belousov kib@FreeBSD.org 2016-01-01T15:48:48Z urn:sha1:d52aff3c7a1fe921e94c03cc4e2ef3b9b9458c92 Submitted by: bde MFC after: 1 week Make it possible for the cdevsw d_close() driver method to detect last 2015-12-22T20:37:34Z Konstantin Belousov kib@FreeBSD.org 2015-12-22T20:37:34Z urn:sha1:cccac8a1ef5b58f91623066680e3b59d477f3885 close and close due to revoke(2)-like operation. A new FLASTCLOSE flag indicates that this is last close. FREVOKE is set for revokes, and FNONBLOCK is also set, same as is already done for VOP_CLOSE() call from vgonel(). The flags reuse user open(2) flags which are never stored in f_flag, to not consume bit space in the ABI visible way. Assert this with the static check. Requested and reviewed by: bde Sponsored by: The FreeBSD Foundation MFC after: 2 weeks Keep devfs mount locked for the whole duration of the devfs_setattr(), 2015-12-22T20:22:17Z Konstantin Belousov kib@FreeBSD.org 2015-12-22T20:22:17Z urn:sha1:b63d070ad17e361cda7b769aac13ed87767f1958 and ensure that our dirent is instantiated. Reported and tested by: bde Sponsored by: The FreeBSD Foundation MFC after: 1 week