Re: svn commit: r323518 - head/sys/conf
On Wednesday, September 13, 2017 10:37:55 AM Ian Lepore wrote: > On Wed, 2017-09-13 at 09:26 -0700, John Baldwin wrote: > > On Wednesday, September 13, 2017 09:09:42 AM Sean Bruno wrote: > > > > > > > > > On 09/13/17 08:35, John Baldwin wrote: > > > > > > > > On Wednesday, September 13, 2017 03:56:04 AM Sean Bruno wrote: > > > > > > > > > > Author: sbruno > > > > > Date: Wed Sep 13 03:56:03 2017 > > > > > New Revision: 323518 > > > > > URL: https://svnweb.freebsd.org/changeset/base/323518 > > > > > > > > > > Log: > > > > > Jenkins i386 LINT build uses NOTES to generate its LINT kernel > > > > > config. > > > > > > > > > > ixl(4) isn't in here either, so I'll remove lio(4) too. > > > > ixl missing is a bug, please put it in sys/amd64/conf/NOTES (or better > > > > yet, > > > > just fix the build on !amd64) > > > > > > > > > > In the case of lio(4), Cavium is explicitly not compiling for 32 bit > > > architectures. I'm inquiring to find out if they want to build on the > > > other 64bit architectures we have. > > > > > > In the case of ixl(4), Intel has said that this is not supported on > > > 32bit architectures. > > > > > > I'm unsure what the best course of action is. > > The bug is more if drivers aren't in NOTES at all. Any thing listed in > > sys/conf/files.* should be in some NOTES file, either sys/conf/NOTES for > > things that are MI or sys/${arch}/conf/NOTES for things that are MD. > > > > Didn't somebody start a task list of things for junior hackers to work > on? Writing a script/tool to verify the assertion in that last > sentence seems like a good candidate for such a list. tools/tools/notescheck/notescheck.py (I haven't run it myself in a long while though) -- John Baldwin ___ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
Re: svn commit: r323518 - head/sys/conf
On Wed, 13 Sep 2017, John Baldwin wrote: On Wednesday, September 13, 2017 09:09:42 AM Sean Bruno wrote: On 09/13/17 08:35, John Baldwin wrote: On Wednesday, September 13, 2017 03:56:04 AM Sean Bruno wrote: Author: sbruno Date: Wed Sep 13 03:56:03 2017 New Revision: 323518 URL: https://svnweb.freebsd.org/changeset/base/323518 Log: Jenkins i386 LINT build uses NOTES to generate its LINT kernel config. ixl(4) isn't in here either, so I'll remove lio(4) too. ixl missing is a bug, please put it in sys/amd64/conf/NOTES (or better yet, just fix the build on !amd64) In the case of lio(4), Cavium is explicitly not compiling for 32 bit architectures. I'm inquiring to find out if they want to build on the other 64bit architectures we have. In the case of ixl(4), Intel has said that this is not supported on 32bit architectures. I'm unsure what the best course of action is. Put it in NOTES for arches that support it. Strictly it should be in MD NOTES, but MI notes already has lots of MD buses like pci and drivers like bge that depend on pci. Some MD NOTES have many nodevice lines to kill non-MI things in MI NOTES, but apparently none needs to kill pci or the hundreds of drivers that depend on it yet. The bug is more if drivers aren't in NOTES at all. Any thing listed in sys/conf/files.* should be in some NOTES file, either sys/conf/NOTES for things that are MI or sys/${arch}/conf/NOTES for things that are MD. The most broken drivers aren't even listed in sys/conf/files.*. Bruce ___ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
Re: svn commit: r323518 - head/sys/conf
On Wed, 2017-09-13 at 09:26 -0700, John Baldwin wrote: > On Wednesday, September 13, 2017 09:09:42 AM Sean Bruno wrote: > > > > > > On 09/13/17 08:35, John Baldwin wrote: > > > > > > On Wednesday, September 13, 2017 03:56:04 AM Sean Bruno wrote: > > > > > > > > Author: sbruno > > > > Date: Wed Sep 13 03:56:03 2017 > > > > New Revision: 323518 > > > > URL: https://svnweb.freebsd.org/changeset/base/323518 > > > > > > > > Log: > > > > Jenkins i386 LINT build uses NOTES to generate its LINT kernel config. > > > > > > > > ixl(4) isn't in here either, so I'll remove lio(4) too. > > > ixl missing is a bug, please put it in sys/amd64/conf/NOTES (or better > > > yet, > > > just fix the build on !amd64) > > > > > > > In the case of lio(4), Cavium is explicitly not compiling for 32 bit > > architectures. I'm inquiring to find out if they want to build on the > > other 64bit architectures we have. > > > > In the case of ixl(4), Intel has said that this is not supported on > > 32bit architectures. > > > > I'm unsure what the best course of action is. > The bug is more if drivers aren't in NOTES at all. Any thing listed in > sys/conf/files.* should be in some NOTES file, either sys/conf/NOTES for > things that are MI or sys/${arch}/conf/NOTES for things that are MD. > Didn't somebody start a task list of things for junior hackers to work on? Writing a script/tool to verify the assertion in that last sentence seems like a good candidate for such a list. -- Ian ___ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
Re: svn commit: r323518 - head/sys/conf
On Wednesday, September 13, 2017 09:09:42 AM Sean Bruno wrote: > > On 09/13/17 08:35, John Baldwin wrote: > > On Wednesday, September 13, 2017 03:56:04 AM Sean Bruno wrote: > >> Author: sbruno > >> Date: Wed Sep 13 03:56:03 2017 > >> New Revision: 323518 > >> URL: https://svnweb.freebsd.org/changeset/base/323518 > >> > >> Log: > >> Jenkins i386 LINT build uses NOTES to generate its LINT kernel config. > >> > >> ixl(4) isn't in here either, so I'll remove lio(4) too. > > > > ixl missing is a bug, please put it in sys/amd64/conf/NOTES (or better yet, > > just fix the build on !amd64) > > > > > In the case of lio(4), Cavium is explicitly not compiling for 32 bit > architectures. I'm inquiring to find out if they want to build on the > other 64bit architectures we have. > > In the case of ixl(4), Intel has said that this is not supported on > 32bit architectures. > > I'm unsure what the best course of action is. The bug is more if drivers aren't in NOTES at all. Any thing listed in sys/conf/files.* should be in some NOTES file, either sys/conf/NOTES for things that are MI or sys/${arch}/conf/NOTES for things that are MD. -- John Baldwin ___ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
Re: svn commit: r323518 - head/sys/conf
On 09/13/17 08:35, John Baldwin wrote: > On Wednesday, September 13, 2017 03:56:04 AM Sean Bruno wrote: >> Author: sbruno >> Date: Wed Sep 13 03:56:03 2017 >> New Revision: 323518 >> URL: https://svnweb.freebsd.org/changeset/base/323518 >> >> Log: >> Jenkins i386 LINT build uses NOTES to generate its LINT kernel config. >> >> ixl(4) isn't in here either, so I'll remove lio(4) too. > > ixl missing is a bug, please put it in sys/amd64/conf/NOTES (or better yet, > just fix the build on !amd64) > In the case of lio(4), Cavium is explicitly not compiling for 32 bit architectures. I'm inquiring to find out if they want to build on the other 64bit architectures we have. In the case of ixl(4), Intel has said that this is not supported on 32bit architectures. I'm unsure what the best course of action is. sean signature.asc Description: OpenPGP digital signature
Re: svn commit: r323518 - head/sys/conf
On Wednesday, September 13, 2017 03:56:04 AM Sean Bruno wrote: > Author: sbruno > Date: Wed Sep 13 03:56:03 2017 > New Revision: 323518 > URL: https://svnweb.freebsd.org/changeset/base/323518 > > Log: > Jenkins i386 LINT build uses NOTES to generate its LINT kernel config. > > ixl(4) isn't in here either, so I'll remove lio(4) too. ixl missing is a bug, please put it in sys/amd64/conf/NOTES (or better yet, just fix the build on !amd64) -- John Baldwin ___ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
svn commit: r323518 - head/sys/conf
Author: sbruno Date: Wed Sep 13 03:56:03 2017 New Revision: 323518 URL: https://svnweb.freebsd.org/changeset/base/323518 Log: Jenkins i386 LINT build uses NOTES to generate its LINT kernel config. ixl(4) isn't in here either, so I'll remove lio(4) too. Modified: head/sys/conf/NOTES Modified: head/sys/conf/NOTES == --- head/sys/conf/NOTES Wed Sep 13 02:44:50 2017(r323517) +++ head/sys/conf/NOTES Wed Sep 13 03:56:03 2017(r323518) @@ -2144,7 +2144,6 @@ deviceixgb# Intel Pro/10Gbe PCI-X Ethernet device ix # Intel Pro/10Gbe PCIE Ethernet device ixv # Intel Pro/10Gbe PCIE Ethernet VF device le # AMD Am7900 LANCE and Am79C9xx PCnet -device lio # Cavium 23XX Ethernet device mxge# Myricom Myri-10G 10GbE NIC device nxge# Neterion Xframe 10GbE Server/Storage Adapter device oce # Emulex 10 GbE (OneConnect Ethernet) ___ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"