Proper way to remove never used ioctls
Vladimir Kondratyev
vladimir at kondratyev.su
Sat Jun 2 18:30:42 UTC 2018
On 2018-06-02 20:36, Allan Jude wrote:
> On 2018-06-02 11:03, Warner Losh wrote:
>> On Sat, Jun 2, 2018 at 7:33 AM, Vladimir Kondratyev
>> <vladimir at kondratyev.su>
>> wrote:
>>
>>> Hi,
>>>
>>> Our sys/mouse.h header has a definition of MOUSE_GETVARS and
>>> MOUSE_SETVARS
>>> ioctls which are not documented and only stubbed in a few drivers:
>>> mse(4),
>>> psm(4) and syscon's sysmouse(4). The only exception is MOUSE_GETVARS
>>> implemented in psm(4)
>>>
>>> Given the fact that they were introduced 20 years ago, implementation
>>> was
>>> never completed and googling on them shows no traces of usage in
>>> indexed
>>> universe, is it acceptable to just drop both defines and
>>> implementation
>>> w.o. leaving any COMPAT_FREEBSD shims?
>>
>>
>> I'd prepare a patch just removing them. I'd then send that patch to
>> the
>> ports mgr team and request a exp-run. Details for that can be found in
>> https://www.freebsd.org/doc/en_US.ISO8859-1/articles/committers-guide/ports.html
>>
>> Once that's done, submit a Phabricator review and send me email. I'll
>> make
>> sure it gets pushed in if there's no objections.
>>
>> Warner
>
> This indeed seems to be the correct approach. The exp-run will compile
> the entire ports tree against the patched base system, and identify any
> 3rd party software that fails to compile because of the change.
Thank you, Allan and Warner! I'll follow yours advice.
--
WBR
Vladimir Kondratyev
More information about the freebsd-current
mailing list