FreeBSD source tree https://cgit-dev.freebsd.org/src-test2/atom?h=release%2F11.3.0 2018-03-29T02:50:57Z 2018-03-29T02:50:57Z Eitan Adler eadler@FreeBSD.org 2018-03-29T02:50:57Z urn:sha1:4ab2e064d7950be84256d671a7ae93f87cc6aa36 This was intended to be a non-functional change. It wasn't. The commit message was thus wrong. In addition it broke arm, and merged crypto related code. Revert with prejudice. This revert skips files touched in r316370 since that commit was since MFCed. This revert also skips files that require $FreeBSD$ property changes. Thank you to those who helped me get out of this mess including but not limited to gonzo, kevans, rgrimes. Requested by: gjb (re) 2018-03-14T03:19:51Z Eitan Adler eadler@FreeBSD.org 2018-03-14T03:19:51Z urn:sha1:be5d0b9566b13fdf8cabebb63334cbec12bfc409 These changes are incomplete but are making it difficult to determine what other changes can/should be merged. No objections from: pfg 2016-04-19T23:37:24Z Pedro F. Giffuni pfg@FreeBSD.org 2016-04-19T23:37:24Z urn:sha1:73a1170a8c41cb848f17cc0a8839e9dcee3d126e No functional change, only trivial cases are done in this sweep, Drivers that can get further enhancements will be done independently. Discussed in: freebsd-current 2013-09-07T13:45:44Z Davide Italiano davide@FreeBSD.org 2013-09-07T13:45:44Z urn:sha1:d56b4cd4acc825cf2a4d1150bb93dc75e6513dc8 dev_ref() in the clone handlers that still use it. - Don't set SI_CHEAPCLONE flag, it's not used anywhere neither in devfs (for anything real) Reviewed by: kib 2011-10-18T08:09:44Z Ed Schouten ed@FreeBSD.org 2011-10-18T08:09:44Z urn:sha1:a185bd12f3266f8b39d5756c37918d6edd3e5d56 It seems the D_PSEUDO flag was meant to allow make_dev() to return NULL. Nowadays we have a different interface for that; make_dev_p(). There's no need to keep it there. While there, remove an unneeded D_NEEDMINOR from the gpio driver. Discussed with: gonzo@ (gpio) 2011-07-17T08:19:19Z Ed Schouten ed@FreeBSD.org 2011-07-17T08:19:19Z urn:sha1:78d4d8eeb241bfc6f0a9487894945886b3ee2b41 Back in 2009 I changed the ABI of the GIO_KEYMAP and PIO_KEYMAP ioctls to support wide characters. I created a patch to add ABI compatibility for the old calls, but I didn't get any feedback to that. It seems now people are upgrading from 8 to 9 they experience this issue, so add it anyway. 2008-09-26T14:19:52Z Ed Schouten ed@FreeBSD.org 2008-09-26T14:19:52Z urn:sha1:d3ce8327194c0bcf742f2e03b5b0dc8f8cfe7f6b When I changed kern_conf.c three months ago I made device unit numbers equal to (unneeded) device minor numbers. We used to require bitshifting, because there were eight bits in the middle that were reserved for a device major number. Not very long after I turned dev2unit(), minor(), unit2minor() and minor2unit() into macro's. The unit2minor() and minor2unit() macro's were no-ops. We'd better not remove these four macro's from the kernel, because there is a lot of (external) code that may still depend on them. For now it's harmless to remove all invocations of unit2minor() and minor2unit(). Reviewed by: kib 2008-06-11T18:55:19Z Ed Schouten ed@FreeBSD.org 2008-06-11T18:55:19Z urn:sha1:29d4cb241b5b8d786221402075febdb832fea55a Except for the case where we use the cloner library (clone_create() and friends), there is no reason to enforce a unique device minor number policy. There are various drivers in the source tree that allocate unr pools and such to provide minor numbers, without using them themselves. Because we still need to support unique device minor numbers for the cloner library, introduce a new flag called D_NEEDMINOR. All cdevsw's that are used in combination with the cloner library should be marked with this flag to make the cloning work. This means drivers can now freely use si_drv0 to store their own flags and state, making it effectively the same as si_drv1 and si_drv2. We still keep the minor() and dev2unit() routines around to make drivers happy. The NTFS code also used the minor number in its hash table. We should not do this anymore. If the si_drv0 field would be changed, it would no longer end up in the same list. Approved by: philip (mentor) 2007-12-29T21:55:25Z Wojciech A. Koszek wkoszek@FreeBSD.org 2007-12-29T21:55:25Z urn:sha1:259699b294163f99205912e9e2ffcd59d1fbf9b8 implemented with macros. This patch improves code readability. Reasoning behind kbdd_* is a "keyboard discipline". List of macros is supposed to be complete--all methods of keyboard_switch should have their respective macros from now on. Functionally, this code should be no-op. My intention is to leave current behaviour of code as is. Glanced at by: rwatson Reviewed by: emax, marcel Approved by: cognet 2006-09-27T19:57:02Z Ruslan Ermilov ru@FreeBSD.org 2006-09-27T19:57:02Z urn:sha1:9fddcc6661d2cf7bb8e704308437d2b1674b50be ioctls passing integer arguments should use the _IOWINT() macro. This fixes a lot of ioctl's not working on sparc64, most notable being keyboard/syscons ioctls. Full ABI compatibility is provided, with the bonus of fixing the handling of old ioctls on sparc64. Reviewed by: bde (with contributions) Tested by: emax, marius MFC after: 1 week