Skip to content

native: increase buffer size, set defaults conditionally#1904

Merged
OlegHahm merged 2 commits intoRIOT-OS:masterfrom
LudwigKnuepfer:native_bufferbloat
Jan 7, 2015
Merged

native: increase buffer size, set defaults conditionally#1904
OlegHahm merged 2 commits intoRIOT-OS:masterfrom
LudwigKnuepfer:native_bufferbloat

Conversation

@LudwigKnuepfer
Copy link
Copy Markdown
Member

Closes #1747

@LudwigKnuepfer LudwigKnuepfer added Platform: native Platform: This PR/issue effects the native platform Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation labels Oct 30, 2014
@LudwigKnuepfer LudwigKnuepfer added the Area: network Area: Networking label Oct 30, 2014
@LudwigKnuepfer
Copy link
Copy Markdown
Member Author

@Lotterleben does this default work for you?

@LudwigKnuepfer
Copy link
Copy Markdown
Member Author

@Lotterleben ping

@OlegHahm
Copy link
Copy Markdown
Member

@Lotterleben - ping

@LudwigKnuepfer
Copy link
Copy Markdown
Member Author

I conversed with @Lotterleben the other day, she's on it.

@OlegHahm
Copy link
Copy Markdown
Member

Okay, thanks for the info.

@OlegHahm
Copy link
Copy Markdown
Member

@Lotterleben, you're still on it?

@OlegHahm
Copy link
Copy Markdown
Member

@Lotterleben, did #2212 solve your problem and make AODVv2 usable again? If yes, could you test again, please?

@Lotterleben
Copy link
Copy Markdown
Member

Solved the problem, testing unveiled some other fuckups, I'm on it.. Sorry :(

@OlegHahm OlegHahm added the Community: Hack'n'ACK candidate This PR is a candidate for review and discussion during one of RIOT's monthly Hack'n'ACK parties label Jan 6, 2015
@cgundogan
Copy link
Copy Markdown
Member

I am not sure if this PR fixes any known problems - but I am in favor of your changes to have these parameters customizable => ACK

@OlegHahm
Copy link
Copy Markdown
Member

OlegHahm commented Jan 6, 2015

@Lotterleben, are you happy, too?

@Lotterleben
Copy link
Copy Markdown
Member

With my current testing setup, yes. ACK

OlegHahm added a commit that referenced this pull request Jan 7, 2015
native: increase buffer size, set defaults conditionally
@OlegHahm OlegHahm merged commit 6dac96a into RIOT-OS:master Jan 7, 2015
@LudwigKnuepfer LudwigKnuepfer deleted the native_bufferbloat branch January 7, 2015 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: network Area: Networking Community: Hack'n'ACK candidate This PR is a candidate for review and discussion during one of RIOT's monthly Hack'n'ACK parties Platform: native Platform: This PR/issue effects the native platform Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

transceiver buffer too small

4 participants