Re: Proper way to remove never used ioctls

From: Allan Jude <allanjude_at_freebsd.org>
Date: Sat, 2 Jun 2018 13:36:49 -0400
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
> _______________________________________________
> freebsd-current_at_freebsd.org mailing list
> https://lists.freebsd.org/mailman/listinfo/freebsd-current
> To unsubscribe, send any mail to "freebsd-current-unsubscribe_at_freebsd.org"
> 

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.

-- 
Allan Jude


Received on Sat Jun 02 2018 - 15:36:50 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:41:16 UTC