src-test2/sys/dev/gem, branch release/11.4.0 FreeBSD source tree https://cgit-dev.freebsd.org/src-test2/atom?h=release%2F11.4.0 2018-03-29T02:50:57Z Revert r330897: 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) Partial merge of the SPDX changes 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 Mechanically convert to if_inc_counter(). 2014-09-18T20:21:46Z Gleb Smirnoff glebius@FreeBSD.org 2014-09-18T20:21:46Z urn:sha1:8da56a6fec0849725d99dded113d344ea994282e Use define from if_var.h to access a field inside struct if_data, 2014-08-30T19:55:54Z Gleb Smirnoff glebius@FreeBSD.org 2014-08-30T19:55:54Z urn:sha1:1bffa9511fe94125820858b0ead796db54ac019d that resides in struct ifnet. Sponsored by: Nginx, Inc. The r48589 promised to remove implicit inclusion of if_var.h soon. Prepare 2013-10-26T17:58:36Z Gleb Smirnoff glebius@FreeBSD.org 2013-10-26T17:58:36Z urn:sha1:76039bc84fae9915788b54ff28fe0cc4876952d2 to this event, adding if_var.h to files that do need it. Also, include all includes that now are included due to implicit pollution via if_var.h Sponsored by: Netflix Sponsored by: Nginx, Inc. Mechanically substitute flags from historic mbuf allocator with 2012-12-04T09:32:43Z Gleb Smirnoff glebius@FreeBSD.org 2012-12-04T09:32:43Z urn:sha1:c6499eccad497913a5025fbde8ae76da70e08043 malloc(9) flags in sys/dev. Remove duplicate const specifiers in many drivers (I hope I got all of 2012-11-05T19:16:27Z Dimitry Andric dim@FreeBSD.org 2012-11-05T19:16:27Z urn:sha1:29658c96ce9914fdf4ff4e3519fbec696ef32118 them, please let me know if not). Most of these are of the form: static const struct bzzt_type { [...list of members...] } const bzzt_devs[] = { [...list of initializers...] }; The second const is unnecessary, as arrays cannot be modified anyway, and if the elements are const, the whole thing is const automatically (e.g. it is placed in .rodata). I have verified this does not change the binary output of a full kernel build (except for build timestamps embedded in the object files). Reviewed by: yongari, marius MFC after: 1 week - There's no need to overwrite the default device method with the default 2011-11-22T21:28:20Z Marius Strobl marius@FreeBSD.org 2011-11-22T21:28:20Z urn:sha1:4b7ec27007570e0ce4fe9dbb447e7afc131b487a one. Interestingly, these are actually the default for quite some time (bus_generic_driver_added(9) since r52045 and bus_generic_print_child(9) since r52045) but even recently added device drivers do this unnecessarily. Discussed with: jhb, marcel - While at it, use DEVMETHOD_END. Discussed with: jhb - Also while at it, use __FBSDID. Revert r224157, re-enabling r222135. The underlying problem keeping the 2011-08-19T19:13:31Z Marius Strobl marius@FreeBSD.org 2011-08-19T19:13:31Z urn:sha1:5ca7dfffdf925aec3238b53ca7034f7be4d37388 latter from working as expected was fixed in r225014. MFC after: 3 days Approved by: re (kib) Revert r222135 by allowing controller reinitialization. Due to 2011-07-17T21:54:51Z Pyun YongHyeon yongari@FreeBSD.org 2011-07-17T21:54:51Z urn:sha1:23b179f5d3b14f6a6164e0a2a988bbe558a1eb2e unknown reason Apple UniNorth2 gem(4) device required manual interface down/up operation after r222135. Even though this is not correct thing and I don't like to revert it but it would be better than breaking gem(4) on PPC. This should be revisited. PR: kern/157405