Skip to content

cpu: native: work around shared errno in _native_lpm_sleep#3340

Merged
kaspar030 merged 1 commit intoRIOT-OS:masterfrom
kaspar030:workaround_for_native_eagain
Jul 9, 2015
Merged

cpu: native: work around shared errno in _native_lpm_sleep#3340
kaspar030 merged 1 commit intoRIOT-OS:masterfrom
kaspar030:workaround_for_native_eagain

Conversation

@kaspar030
Copy link
Copy Markdown
Contributor

Seems to fix #3222.

@kaspar030 kaspar030 added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Platform: native Platform: This PR/issue effects the native platform labels Jul 8, 2015
@PeterKietzmann
Copy link
Copy Markdown
Member

I did several experiments and can confirm that the error described in #3222 didn't occur any more with this PR. I don't think I'm in the right position to decide if this workaround is legit.

@PeterKietzmann PeterKietzmann added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Jul 9, 2015
@PeterKietzmann
Copy link
Copy Markdown
Member

Anyone dares to merge this?

@kaspar030
Copy link
Copy Markdown
Contributor Author

Yes, it makes native a lot more stable for me.

kaspar030 added a commit that referenced this pull request Jul 9, 2015
cpu: native: work around shared errno in _native_lpm_sleep
@kaspar030 kaspar030 merged commit 70c08b8 into RIOT-OS:master Jul 9, 2015
@kaspar030 kaspar030 deleted the workaround_for_native_eagain branch July 9, 2015 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: native Platform: This PR/issue effects the native platform Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ng_nativenet crashes when hammered

2 participants