lkml.org 
[lkml]   [1998]   [Oct]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: SB and 2.1.126 [NOT THE COMPILE PROBLEM]

On Mon, 26 Oct 1998, Alex Buell wrote:

> On Mon, 26 Oct 1998, Mike wrote:
>
> > -#define SOUND_MIXER_OUTSRC 0xfa /* Arg contains a bit for each input source to output */
> > -#define SOUND_MIXER_OUTMASK 0xf9 /* Arg contains a bit for each supported input source to output */
>
> > - sb = acct_file->f_dentry->d_inode->i_sb;
> > + sb = acct_file->f_dentry->f_inode->i_sb;
> > if (!sb->s_op || !sb->s_op->statfs)
> > return;
>
> NO NO NO NO NO!!! This patch is WRONG! WRONG WRONG WRONG. It's the other
> way around!!!!!! ARGHHHHHHHHHHHHHHHHHHHHHHHHHHHHHH!
>
> This is the second wrong patch I've seen sent to the mailing list in the
> last 2 days, hence my reaction.

1) AFAIR, the author said he wasn't sure the patch was right.
2) Most of patches that seem right are in fact wrong.

For these reasons, this patch had every chance to be wrong. Providing a
right patch would have been a lot more relevant than yelling out this way
at the kernel list.

If you have nothing interesting to post, then just donnot post.

Gerard.



-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:45    [W:0.327 / U:0.092 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site