linux-um archives
 help / color / mirror / Atom feed
From: Andi Kleen <ak@suse.de>
To: Lars Ellenberg <Lars.Ellenberg@linbit.com>,
	user-mode-linux-devel@lists.sourceforge.net
Subject: Re: [uml-devel] SOLVED: uml "ip header error" for large (fragmented) udp packets
Date: Tue, 12 Oct 2004 02:11:40 +0200	[thread overview]
Message-ID: <20041012001140.GB17340@wotan.suse.de> (raw)
In-Reply-To: <xneXIovOv53ZtptO+AU2uyI=lge@web.de>

>  	   will assume they contain their original values. */
>  	: "=r" (sum), "=r" (iph), "=r" (ihl)
> -	: "1" (iph), "2" (ihl)
> -	: "memory");
> +	: "1" (iph), "2" (ihl));
>  	return(sum);
>  }
>  
> =======================

That's reverted, right? 
> 
> that's all, folks. only a missing memory barrier.
> 
> WTF :-/

This was fixed in mainline some time ago (several months probably more) 
The problem only started with newer gccs that optimize more aggressively.

> original (in include/asm-i386/checksum.h) right away??
> there may be similar bugs hiding in various areas of uml...

Sounds like a good idea. 

> 
> Thanks for now,
> keep it going...
> 
> btw,
> anyone wants to give me a hint how to tune it best?
> maybe how to up the mtu of the UML "nics"?

Don't go over 4K because the VM doesn't like >order 0 allocations
very much. But in general bigger is better.

-Andi


-------------------------------------------------------
This SF.net email is sponsored by: IT Product Guide on ITManagersJournal
Use IT products in your business? Tell us what you think of them. Give us
Your Opinions, Get Free ThinkGeek Gift Certificates! Click to find out more
http://productguide.itmanagersjournal.com/guidepromo.tmpl
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel

  reply	other threads:[~2004-10-12  0:18 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-10-03  2:36 [uml-devel] uml "ip header error" for large (fragmented) udp packets Lars Ellenberg
2004-10-03 15:11 ` BlaisorBlade
2004-10-03 19:29   ` Lars Ellenberg
2004-10-04 11:07   ` Gerd Knorr
2004-10-04 11:56     ` Henrik Nordstrom
2004-10-06 17:50 ` BlaisorBlade
2004-10-06 18:48   ` Lars Ellenberg
2004-10-06 20:35     ` Andi Kleen
2004-10-06 21:48       ` Lars Ellenberg
2004-10-07 18:41         ` BlaisorBlade
2004-10-07 18:41       ` BlaisorBlade
2004-10-07 20:24         ` Lars Ellenberg
2004-10-11 17:55           ` Lars Ellenberg
2004-10-12  0:03             ` [uml-devel] SOLVED: " Lars Ellenberg
2004-10-12  0:11               ` Andi Kleen [this message]
2004-10-12  1:10                 ` BlaisorBlade
2004-10-12  1:53                   ` Michael Richardson
2004-10-15 15:25                   ` [uml-devel] larger MTU _does_ work Lars Ellenberg
2004-10-12  0:27               ` [uml-devel] SOLVED: uml "ip header error" for large (fragmented) udp packets BlaisorBlade
2004-10-12 14:00                 ` Lars Ellenberg

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20041012001140.GB17340@wotan.suse.de \
    --to=ak@suse.de \
    --cc=Lars.Ellenberg@linbit.com \
    --cc=user-mode-linux-devel@lists.sourceforge.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox