Re: kernel broken? (pcm)

1999-10-12 Thread A . Leidinger

On 12 Okt, [EMAIL PROTECTED] wrote:

>   You can try the patch in (my) PR 'kern/14278'.

Works.

Bye,
Alexander.

-- 
   What do you mean, my birth certificate expired?

http://netchild.home.pages.de   A.Leidinger+Home @ WJPServer.CS.Uni-SB.de



To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message



Re: kernel broken? (pcm)

1999-10-12 Thread nnd

In <[EMAIL PROTECTED]> Alexander Leidinger 
<[EMAIL PROTECTED]> wrote:
> On 11 Oct, Bill Fumerola wrote:
> 
>>> #makeoptionsCONF_CFLAGS=-fno-builtin
>> 
>> We have enough breakages with the _documented_ kernel options that we
> 
> As Bruce Evans already said, It's documented.
> 
>> don't need to go hunting down oddities. :>
> 
> Have you seen the '#' in "#makeoptions"? I assume a '#' in the config
> file means: "Hey config, don't look at this!". Please correct me if I'm
> wrong.
> 
> I've tested both variants with and without '#'. After editing the file I
> did a 'config -r CONFIG', 'cd ../../compile/CONFIG', 'make depend',
> 'make'. I get the error independently of CONF_FLAGS.
> Back at home I do it again if you want. 

Yes, in your case the errors are independent from
this "makeoptions ..." line. But you are linking kernel.debug,
so the (undefined in kernel) 'abs' function shows itself.

You can try the patch in (my) PR 'kern/14278'.

N.Dudorov


To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message



Re: kernel broken? (pcm)

1999-10-12 Thread Alexander Leidinger

On 11 Oct, Bill Fumerola wrote:

>> #makeoptions CONF_CFLAGS=-fno-builtin
> 
> We have enough breakages with the _documented_ kernel options that we

As Bruce Evans already said, It's documented.

> don't need to go hunting down oddities. :>

Have you seen the '#' in "#makeoptions"? I assume a '#' in the config
file means: "Hey config, don't look at this!". Please correct me if I'm
wrong.

I've tested both variants with and without '#'. After editing the file I
did a 'config -r CONFIG', 'cd ../../compile/CONFIG', 'make depend',
'make'. I get the error independently of CONF_FLAGS.
Back at home I do it again if you want. 

Bye,
Alexander.

-- 
http://netchild.home.pages.de   A.Leidinger @ WJPServer.CS.Uni-SB.de



To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message



Re: kernel broken? (pcm)

1999-10-11 Thread Bruce Evans

> >>Such an errors results from (uncommented) kernel option
 ^^^
> >> 
> >> #makeoptions   CONF_CFLAGS=-fno-builtin
> > 
> > We have enough breakages with the _documented_ kernel options that we 
  
> > don't need to go hunting down oddities. :>

It's well documented.  -fno-builtin is a standard gcc option, and
CONF_FLAGS and its use of -fno-builtin is documented in 4 lines in LINT.

>   Does it mean that I throw away my PR with patches to
> the 'newpcm' files which add 'abs' definition and therefore
> make it possible to make kernel with 
> 
> makeoptions CONF_CFLAGS=-fno-builtin   ?

No, some patches are needed.  I think pcm should use explicit code
like (x < 0 : -x : x).  It only calls abs() twice.

> P.S. It seems to me that this option (or its absent) can
> severely influence kernel run time efficience (not in the 'abs'
> case, of course ;-).

I used to use it for all kernels as a side effect of putting it in
/etc/make.conf so that it gets used for LINT.  Any efficiency effects
seem to be limited to the +-0.1% range.  The only place where it is
likely to make much difference is for bcopy() vs memcpy() on some
machines.  The builtin memcpy() can be pessimal.  This is supposed to
be handled by using bcopy() for large copies and memcpy() (only) for
small, fix-sized copies.  The builtin memcpy() tends to be better for
small copies.  Since using memcpy() in the kernel is correct in some
cases, it is backed up by a function to handle cases where there is
no inline memcpy().

Bruce



To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message



Re: kernel broken? (pcm)

1999-10-11 Thread Bruce Evans

On Tue, 12 Oct 1999 [EMAIL PROTECTED] wrote:

> In <[EMAIL PROTECTED]> [EMAIL PROTECTED] wrote:
>   Such an errors results from (uncommented) kernel option
> 
> #makeoptions  CONF_CFLAGS=-fno-builtin

This option is used in LINT to help expose such errors before they are
committed.

>   because there is NO 'abs' function/macro in this files
> (ac97.c, channel.c and mss.c) and they can be only compiled using
> gcc's builtin 'abs'

abs() doesn't exist in the kernel, so it shouldn't be used.

> > linking kernel.debug
> > ac97.o: In function `ac97_setmixer':
> > /usr/src/sys/compile/WORK/../../dev/pcm/ac97.c(.text+0x18d): undefined reference 
>to `abs'
> > channel.o: In function `chn_setblocksize':

Test coverage of pcm in LINT is broken by not configuring pcm because of
conflicts with the old sound drivers.

Bruce



To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message



Re: kernel broken? (pcm)

1999-10-11 Thread Bill Fumerola

On Tue, 12 Oct 1999 [EMAIL PROTECTED] wrote:

> > We have enough breakages with the _documented_ kernel options that we 
> > don't need to go hunting down oddities. :>
> 
>   Does it mean that I throw away my PR with patches to
> the 'newpcm' files which add 'abs' definition and therefore
> make it possible to make kernel with 
> 
> makeoptions CONF_CFLAGS=-fno-builtin   ?
> 
> P.S. It seems to me that this option (or its absent) can
> severely influence kernel run time efficience (not in the 'abs'
> case, of course ;-).

No. I was merely joking around. We welcome your patches.

-- 
- bill fumerola - [EMAIL PROTECTED] - BF1560 - computer horizons corp -
- ph:(800) 252-2421 - [EMAIL PROTECTED] - [EMAIL PROTECTED]  -






To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message



Re: kernel broken? (pcm)

1999-10-11 Thread nnd

In <[EMAIL PROTECTED]> Bill Fumerola 
<[EMAIL PROTECTED]> wrote:
> On Tue, 12 Oct 1999 [EMAIL PROTECTED] wrote:
> 
>> In <[EMAIL PROTECTED]> [EMAIL PROTECTED] wrote:
>>  Such an errors results from (uncommented) kernel option
>> 
>> #makeoptions CONF_CFLAGS=-fno-builtin
> 
> We have enough breakages with the _documented_ kernel options that we 
> don't need to go hunting down oddities. :>

Does it mean that I throw away my PR with patches to
the 'newpcm' files which add 'abs' definition and therefore
make it possible to make kernel with 

makeoptions CONF_CFLAGS=-fno-builtin   ?

P.S. It seems to me that this option (or its absent) can
severely influence kernel run time efficience (not in the 'abs'
case, of course ;-).

N.Dudorov


To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message



Re: kernel broken? (pcm)

1999-10-11 Thread Bill Fumerola

On Tue, 12 Oct 1999 [EMAIL PROTECTED] wrote:

> In <[EMAIL PROTECTED]> [EMAIL PROTECTED] wrote:
>   Such an errors results from (uncommented) kernel option
> 
> #makeoptions  CONF_CFLAGS=-fno-builtin

We have enough breakages with the _documented_ kernel options that we 
don't need to go hunting down oddities. :>

-- 
- bill fumerola - [EMAIL PROTECTED] - BF1560 - computer horizons corp -
- ph:(800) 252-2421 - [EMAIL PROTECTED] - [EMAIL PROTECTED]  -






To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message



Re: kernel broken? (pcm)

1999-10-11 Thread nnd

In <[EMAIL PROTECTED]> [EMAIL PROTECTED] wrote:
Such an errors results from (uncommented) kernel option

#makeoptionsCONF_CFLAGS=-fno-builtin

because there is NO 'abs' function/macro in this files
(ac97.c, channel.c and mss.c) and they can be only compiled using
gcc's builtin 'abs'

N.Dudorov

> Hi,
> 
> linking kernel.debug
> ac97.o: In function `ac97_setmixer':
> /usr/src/sys/compile/WORK/../../dev/pcm/ac97.c(.text+0x18d): undefined reference to 
>`abs'
> channel.o: In function `chn_setblocksize':
> /usr/src/sys/compile/WORK/../../dev/pcm/channel.c:712: undefined reference to `abs'
> mss.o: In function `mss_speed':
> /usr/src/sys/compile/WORK/../../dev/pcm/isa/mss.c(.text+0x182d): undefined reference 
>to `abs'
> /usr/src/sys/compile/WORK/../../dev/pcm/isa/mss.c(.text+0x183f): undefined reference 
>to `abs'
> /usr/src/sys/compile/WORK/../../dev/pcm/isa/mss.c(.text+0x1869): undefined reference 
>to `abs'
> mss.o(.text+0x187b):/usr/src/sys/compile/WORK/../../dev/pcm/isa/mss.c: more 
>undefined references to `abs' follow
> *** Error code 1
> 1 error
> 
> I tried several cvsups since the morning.
> 
> Kernelconfig attached.
> 


To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message



kernel broken? (pcm)

1999-10-11 Thread A . Leidinger

Hi,

linking kernel.debug
ac97.o: In function `ac97_setmixer':
/usr/src/sys/compile/WORK/../../dev/pcm/ac97.c(.text+0x18d): undefined reference to 
`abs'
channel.o: In function `chn_setblocksize':
/usr/src/sys/compile/WORK/../../dev/pcm/channel.c:712: undefined reference to `abs'
mss.o: In function `mss_speed':
/usr/src/sys/compile/WORK/../../dev/pcm/isa/mss.c(.text+0x182d): undefined reference 
to `abs'
/usr/src/sys/compile/WORK/../../dev/pcm/isa/mss.c(.text+0x183f): undefined reference 
to `abs'
/usr/src/sys/compile/WORK/../../dev/pcm/isa/mss.c(.text+0x1869): undefined reference 
to `abs'
mss.o(.text+0x187b):/usr/src/sys/compile/WORK/../../dev/pcm/isa/mss.c: more undefined 
references to `abs' follow
*** Error code 1
1 error

I tried several cvsups since the morning.

Kernelconfig attached.

Bye,
Alexander.

-- 
Am I supposed to be impressed?

http://netchild.home.pages.de   A.Leidinger+Home @ WJPServer.CS.Uni-SB.de


machine i386
ident   WORK
maxusers32

makeoptions DEBUG=-g
#makeoptionsCONF_CFLAGS=-fno-builtin

#
# Allow user-mode programs to manipulate their local descriptor tables.
# This option is required for the WINE Windows(tm) emulator, and is
# not used by anything else (that we know of).
# 
#optionsUSER_LDT#allow user-level control of i386 ldt

# Options for the VM subsystem
#optionsPQ_NOOPT# No coloring
options PQ_LARGECACHE   # color for 512k/16k cache
#optionsPQ_HUGECACHE# color for 1024k/16k cache

#cpuI586_CPU
cpu I686_CPU# aka Pentium Pro(tm)
options CPU_FASTER_5X86_FPU
options CPU_SUSP_HLT
options "NO_F00F_HACK"

options COMPAT_43

options SYSVSHM
options SYSVSEM
options SYSVMSG
options MD5
options DDB
options GDB_REMOTE_CHAT
options KTRACE  #kernel tracing
#optionsPERFMON
options UCONSOLE
options USERCONFIG  #boot -c editor
options VISUAL_USERCONFIG   #visual boot -c editor
options INET#Internet communications protocols
pseudo-device   ether   #Generic Ethernet
pseudo-device   sppp#Generic Synchronous PPP
pseudo-device   loop#Network loopback device
pseudo-device   bpf #Berkeley packet filter
pseudo-device   disc#Discard device
pseudo-device   streams
options PPP_BSDCOMP #PPP BSD-compress support
options PPP_DEFLATE #PPP zlib/deflate/gzip support
options PPP_FILTER  #enable bpf filtering (needs bpfilter)

options MROUTING# Multicast routing
options IPFIREWALL  #firewall
options IPFIREWALL_VERBOSE  #print information about
# dropped packets
options IPFIREWALL_FORWARD  #enable transparent proxy support
options IPFIREWALL_VERBOSE_LIMIT=100 #limit verbosity
options IPDIVERT#divert sockets
options IPSTEALTH   #support for stealth forwarding
options TCP_RESTRICT_RST#restrict emission of TCP RST
options ICMP_BANDLIM
options FFS #Fast filesystem
options CD9660  #ISO 9660 filesystem
options KERNFS  #Kernel filesystem
options MSDOSFS #MS DOS File System
options PROCFS  #Process filesystem
options MFS #Memory File System
options NFS #Network File System
#optionsNFS_NOSERVER#Disable the NFS-server code.
options FFS_ROOT#FFS usable as root device
options SOFTUPDATES
options P1003_1B
options _KPOSIX_PRIORITY_SCHEDULING
options _KPOSIX_VERSION=199309L

controller  scbus0  #base SCSI code
#device ch0 #SCSI media changers
device  da0 #SCSI direct access devices (aka disks)
#device sa0 #SCSI tapes
device  cd0 #SCSI CD-ROMs
device  pass0   #CAM passthrough driver
device pt0 at scbus?# SCSI processor type
options CAM_MAX_HIGHPOWER=4
options SCSI_DELAY=5000# Be pessimistic about Joe SCSI device

pseudo-device   pty #Pseudo ttys
pseudo-device   speaker #Play IBM BASIC-style noises out your speaker
pseudo-device   gzip#Exec gzipped a.out's
pseudo-device   vn  #Vnode driver (turns a file into a device)
#pseudo-device  snp 3   #Snoop device - to look at pty/vty/etc..

options MSGBUF_SIZE=40960

controller  isa0

# Enable support for the kernel PLL to use an external PPS signal,
# under supervision of [x]ntpd(8)
# More