Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 6 Jan 2001 14:12:58 -0800
From:      Alfred Perlstein <bright@wintelcom.net>
To:        Poul-Henning Kamp <phk@critter.freebsd.dk>
Cc:        Bosko Milekic <bmilekic@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/dev/musycc musycc.c
Message-ID:  <20010106141257.F15744@fw.wintelcom.net>
In-Reply-To: <12666.978817510@critter>; from phk@critter.freebsd.dk on Sat, Jan 06, 2001 at 10:45:10PM %2B0100
References:  <20010106132943.E15744@fw.wintelcom.net> <12666.978817510@critter>

next in thread | previous in thread | raw e-mail | index | archive | help
* Poul-Henning Kamp <phk@critter.freebsd.dk> [010106 13:45] wrote:
> In message <20010106132943.E15744@fw.wintelcom.net>, Alfred Perlstein writes:
> >* Bosko Milekic <bmilekic@FreeBSD.org> [010106 12:44] wrote:
> >> bmilekic    2001/01/06 12:44:40 PST
> >> 
> >>   Modified files:
> >>     sys/dev/musycc       musycc.c 
> >
> >Grr, This isn't how it was supposed to be done.  Would you mind terribly
> >if I cleaned it up to what I intended it to be:
> 
> 
> 
> >! 		if ((m->m_flags M_EXT) == 0) {
> 
> How is this supposed to compile in either of the committed or proposed
> versions ?

How the hell should I know? :)

not only that but we have one of my favorite bugs:

    while (csc->state != C_RUNNING)
        tsleep(&csc->state, PZERO + PCATCH, "crun", hz/10);

should have no PCATCH, or somehow deal with the signal instead of
looping forever.

You're right, this file is pretty broken. :)

-- 
-Alfred Perlstein - [bright@wintelcom.net|alfred@freebsd.org]
"I have the heart of a child; I keep it in a jar on my desk."


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010106141257.F15744>