Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 17 Feb 2017 12:54:29 -0700
From:      Sean Bruno <sbruno@freebsd.org>
To:        John Baldwin <jhb@freebsd.org>
Cc:        freebsd-arch@freebsd.org, Warner Losh <imp@bsdimp.com>
Subject:   Re: Thinning out GENERIC
Message-ID:  <ff6d9a9d-36f2-459f-78c5-a7269c2caae9@freebsd.org>
In-Reply-To: <1907688.dXvDEfTesD@ralph.baldwin.cx>
References:  <34e4e9bd-c7a8-e3e2-4cef-98a691ea7944@freebsd.org> <2553875.ilSvFZYQZ6@ralph.baldwin.cx> <bf4931ec-2c05-2b90-8bc6-affac684bd23@freebsd.org> <1907688.dXvDEfTesD@ralph.baldwin.cx>

next in thread | previous in thread | raw e-mail | index | archive | help
This is an OpenPGP/MIME signed message (RFC 4880 and 3156)
--gVlRJn8fEbCSXRWHAvVis5TjnfVGoMNN0
Content-Type: multipart/mixed; boundary="1gh5NKWQ4EsCTklft8sNmaAWbXrf9Agne";
 protected-headers="v1"
From: Sean Bruno <sbruno@freebsd.org>
To: John Baldwin <jhb@freebsd.org>
Cc: freebsd-arch@freebsd.org, Warner Losh <imp@bsdimp.com>
Message-ID: <ff6d9a9d-36f2-459f-78c5-a7269c2caae9@freebsd.org>
Subject: Re: Thinning out GENERIC
References: <34e4e9bd-c7a8-e3e2-4cef-98a691ea7944@freebsd.org>
 <2553875.ilSvFZYQZ6@ralph.baldwin.cx>
 <bf4931ec-2c05-2b90-8bc6-affac684bd23@freebsd.org>
 <1907688.dXvDEfTesD@ralph.baldwin.cx>
In-Reply-To: <1907688.dXvDEfTesD@ralph.baldwin.cx>

--1gh5NKWQ4EsCTklft8sNmaAWbXrf9Agne
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: quoted-printable



On 02/17/17 12:49, John Baldwin wrote:
> On Friday, February 17, 2017 11:43:45 AM Sean Bruno wrote:
>>
>> On 02/13/17 11:33, John Baldwin wrote:
>>> On Monday, February 13, 2017 10:51:36 AM Warner Losh wrote:
>>>> [[ It isn't just GENERIC, it's in every single kernel ]]
>>>>
>>>> On Sun, Feb 12, 2017 at 7:59 AM, Sean Bruno <sbruno@freebsd.org> wro=
te:
>>>>> Index: sys/conf/files
>>>>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
>>>>> --- sys/conf/files      (revision 313683)
>>>>> +++ sys/conf/files      (working copy)
>>>>> @@ -2419,7 +2419,7 @@
>>>>>  dev/pci/pci_user.c             optional pci
>>>>>  dev/pci/pcib_if.m              standard
>>>>>  dev/pci/pcib_support.c         standard
>>>>> -dev/pci/vga_pci.c              optional pci
>>>>> +#dev/pci/vga_pci.c             optional pci
>>>>>  dev/pcn/if_pcn.c               optional pcn pci
>>>>>  dev/pdq/if_fea.c               optional fea eisa
>>>>>  dev/pdq/if_fpa.c               optional fpa pci
>>>>>
>>>>> I'm playing around with small x86 builds and wanted to come up with=
 a
>>>>> knob that disables the vga_pci driver.  Obviously I still want pci(=
4) to
>>>>> be enabled, but I definitely do not need the vga interfaces for
>>>>> anything.  Any suggestions?
>>>>
>>>> You could make it dependent on vt or sc as well as pci, no?
>>>
>>> This is a fairly small driver:
>>>
>>>      6139      340         8      6487     0x1957   /usr/obj/usr/src/=
sys/GENERIC
>>> /vga_pci.o
>>>
>>> (From an 11.0-stable build)
>>>
>>> device ppc and friends is a lot more for example:
>>>
>>>   text   data   bss    dec      hex   filename
>>>   6238      0     0   6238   0x185e   /usr/obj/usr/src/sys/GENERIC/pp=
b_1284.o
>>>   1675     28     0   1703    0x6a7   /usr/obj/usr/src/sys/GENERIC/pp=
b_base.o
>>>   2042      0     0   2042    0x7fa   /usr/obj/usr/src/sys/GENERIC/pp=
b_msq.o
>>>   6025    296     8   6329   0x18b9   /usr/obj/usr/src/sys/GENERIC/pp=
bconf.o
>>>      0    168     0    168     0xa8   /usr/obj/usr/src/sys/GENERIC/pp=
bus_if.o
>>>   9099     36     0   9135   0x23af   /usr/obj/usr/src/sys/GENERIC/pp=
c.o
>>>    495    256     0    751    0x2ef   /usr/obj/usr/src/sys/GENERIC/pp=
c_acpi.o
>>>   1671    256     0   1927    0x787   /usr/obj/usr/src/sys/GENERIC/pp=
c_isa.o
>>>   1124    208     0   1332    0x534   /usr/obj/usr/src/sys/GENERIC/pp=
c_pci.o
>>>    451    208     0    659    0x293   /usr/obj/usr/src/sys/GENERIC/pp=
c_puc.o
>>>   2751    428     8   3187    0xc73   /usr/obj/usr/src/sys/GENERIC/pp=
i.o
>>>
>>> It is needed for drm.
>>>
>>
>> Ok, so ... make it dependant on drm, vt, sc and pci?
>=20
> You can kldload drm at runtime, so a static dependency isn't sufficient=
=2E
> It's not clear why removing this from GENERIC is that important.  ppc i=
s on far
> fewer machines than vga_pci and takes up 10x more space.
>=20

I don't want to remove it from GENERIC.  I want to add the capability to
remove it.  Sorry if that wasn't clear from my subject line, because it
doesn't look clear now that I re-read it.

sean


--1gh5NKWQ4EsCTklft8sNmaAWbXrf9Agne--

--gVlRJn8fEbCSXRWHAvVis5TjnfVGoMNN0
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: OpenPGP digital signature
Content-Disposition: attachment; filename="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQGTBAEBCgB9FiEEuq1GMucSHejSCZfdEgHvyh5yfmQFAlinVPZfFIAAAAAALgAo
aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldEJB
QUQ0NjMyRTcxMjFERThEMjA5OTdERDEyMDFFRkNBMUU3MjdFNjQACgkQEgHvyh5y
fmTm8gf/QQxkM1BXTW19vS5jQr9nFWlwJ2LMdTxF23k6WlNRdMzWlWPd0a0iDyse
lbVNnxaCsVHs3r1w3nia27RubpAZD6HDnuXLM7ACkijUb5D94TN5KjCJkfMrdob+
+wZe6elf/DgyNJsAYt0OdkHc6QNpGkdBvV/UYEV8JX1TrZm0CC0F7wGSyZ54goZd
3vDlaMUgpvWnn4hJdo96xhbWNxqLZoLvabfgNJxQDjO9tn1UIJMZVTozu/Ft50K1
yEDvOvuvrU5hxM8sC6hjPGu/A9q+cu7MSUACay9pN2qhur0VEW2+rOnoGWnW2fYy
I3czkL/nIKETENrz+Zyya25de5mvdA==
=xfYC
-----END PGP SIGNATURE-----

--gVlRJn8fEbCSXRWHAvVis5TjnfVGoMNN0--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?ff6d9a9d-36f2-459f-78c5-a7269c2caae9>