svn commit: r327379 - head/sys/isa

[Warner Losh]

On Sat, Dec 30, 2017 at 8:27 AM, Rodney W. Grimes <
freebsd at pdx.rh.cn85.dnsmgr.net> wrote:

> [ Charset UTF-8 unsupported, converting... ]
> > Author: imp
> > Date: Sat Dec 30 08:16:31 2017
> > New Revision: 327379
> > URL: https://svnweb.freebsd.org/changeset/base/327379
> >
> > Log:
> >   On further testing on actual machines with this hardware, we should
> >   only warn for devices that are attached. Add missing \n.
> >
> > Modified:
> >   head/sys/isa/isa_common.c
> >
> > Modified: head/sys/isa/isa_common.c
> > ============================================================
> ==================
> > --- head/sys/isa/isa_common.c Sat Dec 30 07:59:32 2017        (r327378)
> > +++ head/sys/isa/isa_common.c Sat Dec 30 08:16:31 2017        (r327379)
> > @@ -573,9 +573,10 @@ isa_probe_children(device_t dev)
> >
> >               err = device_probe_and_attach(child);
> >               if (err == 0 && idev->id_vendorid == 0 &&
> > -                 strcmp(kern_ident, "GENERIC") == 0)
> > +                 strcmp(kern_ident, "GENERIC") == 0 &&
> > +                 device_is_attached(child))
> >                       device_printf(child,
> > -                         "non-PNP ISA device will be removed from
> GENERIC in FreeBSD 12.");
> > +                         "non-PNP ISA device will be removed from
> GENERIC in FreeBSD 12.\n");
>
> Hat:  RE
>
> Do you plan to give this notice in 11.x?  (Technically a bit late,
> it should of been in 11.0).
>

Yes. It will be MFC'd soon. Of course it should have happened in 11.0,  but
it didn't. But this notice is just about things in GENERIC. Not their
actual removal (though most of them will be removed). The notice is here
because GENERIC is going to be cut to the bone once the PNP stuff I'm
working on makes it viable to have all drivers (well, most) kld loaded. The
only ones that will affect users, though, are the ones that are for hinted
devices which will have no autoloading. Hence this warning.


> As giving it in 12.0 wont happen if you remove it in 12.  Ie, as you
> have stated you want to remove this before we branch 12.0, if you do
> that this notice well never be seen by the users.  Or well, they might
> get the notice, but the drivers well have already been removed.
>
> Or do you want to give notice in 12.* that it is being removed in 13.0.
> And that is not supporting these for 10 years, as 12.x should be EOL
> in a bit over 5 years from now if we cut in early 2018 which I believe
> to be the current target.
>

No. All non-PNP ISA devices will be removed from GENERIC in 12. That is, if
they are still in the tree. A number will simply be gone, however. I'll be
putting warnings in those drivers soon. It would have been ideal had we
done so earlier, but we didn't. Simple lack of notice, however, isn't going
to keep things in the tree. The following drivers will be removed in 12:
aha, adv, adw, bt, aic, ct, dpt, ncv, nsp, stg, mse, joy, cm and likely cx
and rx. arcnet too will be removed. Still need to check on dangling
references old TTY code as well that hasn't built in FreeBSD 8. This list
has circulated before, and I'll be codifying a run-time warning into all
these drivers on attach, plus getting a head-start on what to remove in 13.

FreeBSD has grown large. While you can still boot it on that old 32MB
machine, you really want 128MB or more to do anything useful. You need at
least 2GB or a heck of a lot of swap to self-host. We should simply be
removing the really old stuff that wouldn't be on a machine that size.
Expect more announcements like this to be coming soon from the people that
have been taking are of and feeding this old stuff for too long. If we cut
with the 128MB criteria, the list would be much, much longer, and the cuts
much deeper.

Warner


> >       }
> >
> >       /*
> >
> >
>
> --
> Rod Grimes
> rgrimes at freebsd.org
>