On Sun, Nov 1, 2020 at 12:37 AM Xie He <xie.he.0...@gmail.com> wrote: > > On Sat, Oct 31, 2020 at 2:41 PM Arnd Bergmann <a...@kernel.org> wrote: > > > > I think it can just go in the bin directly. I actually submitted a couple of > > patches to clean up drivers/net/wan last year but didn't follow up > > with a new version after we decided that x.25 is still needed, see > > https://lore.kernel.org/netdev/20191209151256.2497534-1-a...@arndb.de/ > > > > I can resubmit if you like. > > Should we also remove the two macro definitions in > "include/uapi/linux/sockios.h" (SIOCADDDLCI / SIOCDELDLCI), too? It > seems to be not included in your original patch.
Not sure, it should probably at least be marked as 'obsolete' in the header like SIOCGIFDIVERT, but removing the definitions might risk that someone later reuses the numbers for a new command. I don't know if there is an official policy for this. I see a couple of other definitions in the same file that have no apparent implementation: SIOCGIFCOUNT, SIOCDRARP, SIOCGRARP and SIOCSRARP. These were still referenced in 2.6.12, but only in dead code that has since been removed. arnd