On Sun, Mar 04, 2007 at 08:13:39PM +0100, Blaisorblade wrote:
> Have you checked if other calls require that register_netdev was already
> called?
Just did, nothing else depends on it.
> Hmm, I'm really not sure about what to do about this. However I'd leave that
> for a future cleanup and just add a comment about this.
> And the reason for which init now returns a value is that pcap_init _can_
> fail, so the switch transport is not the only odd one.
Yeah, that slipped my mind.
> struct uml_net_private {
> - int user[1];
> + char user[0];
> };
Neat, that used to be illegal.
> Another note: before of this patch, you should please apply the attached two
> ones.
>
> *) The first (net-mac-check-cleanup.diff) checks the validity of assigned MAC
> address, but to print a meaningful error message requires adding a local
> buffer.
> *) The second (net_kern-eth_configure...) allows avoiding this local buffer
> by
> moving code around.
>
> I think I'm excessively paranoid about these two patches and about not yet
> merging them, so please give a look and merge them.
I just dropped them in - they look OK at a first glance.
Jeff
--
Work email - jdike at linux dot intel dot com
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel
