Re: svn commit: r323518 - head/sys/conf

2017-09-13 Thread John Baldwin
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

2017-09-13 Thread Bruce Evans

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

2017-09-13 Thread Ian Lepore
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

2017-09-13 Thread John Baldwin
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

2017-09-13 Thread Sean Bruno


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

2017-09-13 Thread John Baldwin
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

2017-09-12 Thread Sean Bruno
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"