Re: CVS commit: src/sys

2013-11-26 Thread Dennis Ferguson
On 26 Nov, 2013, at 01:55 , Alan Barrett wrote: >> -for (mp = mountlist.cqh_first; mp != (void*)&mountlist; mp = nmp) { >> -nmp = mp->mnt_list.cqe_next; >> +for (mp = mountlist.tqh_first; mp != (void*)&mountlist; mp = nmp) { >> +nmp = mp->mnt_list.tqe_next; > > ... could be rewri

Re: CVS commit: src/sys

2013-11-26 Thread Christos Zoulas
In article <20131126095505.gj1...@apb-laptoy.apb.alt.za>, Alan Barrett wrote: >On Sat, 23 Nov 2013, Jeff Rizzo wrote: >>Log Message: >>Since mountlist is now a TAILQ, convert some missed usages >>so things build again. > >It would be better if the callers were rewritten to use the >queue.h macro

Re: CVS commit: src/sys

2013-11-26 Thread Alan Barrett
On Sat, 23 Nov 2013, Jeff Rizzo wrote: Log Message: Since mountlist is now a TAILQ, convert some missed usages so things build again. It would be better if the callers were rewritten to use the queue.h macros instead of direct access to the struct members. (Especially since the names of the s

Re: CVS commit: src/sys/arch/i386/conf

2013-11-26 Thread Alan Barrett
On Fri, 22 Nov 2013, Jeff Rizzo wrote: Modified Files: src/sys/arch/i386/conf: ALL Log Message: Comment out npf for now, as we can't have both NPF and PF in the same kernel It used to be possible to have npf and pf in the same kernel. The kernel I am running now (built a few weeks ag