netdev - Re: SiS 191 failure with MTU

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <1243952084.19053.111.camel@plop>
Date:	Tue, 02 Jun 2009 16:14:44 +0200
From:	Pascal Terjan <pterjan@...driva.com>
To:	Francois Romieu <romieu@...zoreil.com>
Cc:	netdev@...r.kernel.org
Subject: Re: SiS 191 failure with MTU > 1496

Le mardi 02 juin 2009 à 13:50 +0200, Pascal Terjan a écrit :
> Le samedi 30 mai 2009 à 00:30 +0200, Francois Romieu a écrit :
> > Pascal Terjan <pterjan@...driva.com> :
> > [...]
> > > Oh and even
> > > http://bugzilla.kernel.org/show_bug.cgi?id=10694
> > 
> > Yes, it sucks.
> > 
> > > Should I add some info there ?
> > 
> > Not yet but could you retry with RX_BUF_SIZE set to 1544,
> > 1528 and 1520 ?
> > 
> 
> 1544, 1528 and 1520 fail the same way

By the way there is some strange code:

tp->rx_buf_sz = (mtu > RX_BUF_SIZE) ? mtu + ETH_HLEN + 8 : RX_BUF_SIZE;

Isn't there a (different from the one I'm interested in :) ) bug when
1500 < mtu <= RX_BUF_SIZE

Shouldn't the test be (mtu > ETH_DATA_LEN) ?

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html