Skip to content

Release candidate for mbed-os-5.5.0-rc3 #4568

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Jun 17, 2017
Merged

Conversation

adbridge
Copy link
Contributor

No description provided.

c1728p9 and others added 5 commits June 15, 2017 14:58
Remove HAL_Init and related code from SystemInit and move it to
mbed_sdk_init. The function SystemInit is called early in the boot
sequence before RAM is initialized or the VTOR is setup, so it should
not be used to perform the HAL initialization.

This fixes crashes due the vector table being used before it has been
relocated.
This fixes an issue as Timer.h is mbed file, thus if this file gets included first,
causes failures.
cmsis 5 does not include it but as mbed 2 requires this, we will bring this file
back. This brings back few other dependencies, that we add only for cortex-a.

Once cortex-a gets cmsis5 and rtx2 support it will be updated.
@adbridge
Copy link
Contributor Author

/morph mbed2-release

@adbridge
Copy link
Contributor Author

/morph export-build

@mbed-bot
Copy link

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph mbed2-release

@deepikabhavnani deepikabhavnani requested review from jeromecoutant and removed request for jeromecoutant June 15, 2017 14:44
@mbed-bot
Copy link

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph export-build

Output

mbed Build Number: 66

All exports and builds passed!

@ARMmbed ARMmbed deleted a comment from mbed-bot Jun 15, 2017
@ARMmbed ARMmbed deleted a comment from adbridge Jun 15, 2017
@sg-
Copy link
Contributor

sg- commented Jun 15, 2017

/morph test-nightly

@mbed-bot
Copy link

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test-nightly

Output

mbed Build Number: 568

Test failed!

@adbridge
Copy link
Contributor Author

/morph test-nightly

@mbed-bot
Copy link

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test-nightly

Output

mbed Build Number: 570

Test failed!

@studavekar
Copy link
Contributor

/morph test-nightly

@geky
Copy link
Contributor

geky commented Jun 16, 2017

So we figured out what happened. It's kinda interesting.

In Austin we have a /22 network. That is, our netmask is 255.255.252.0. If you're familiar with IPv4, this will tell you that IP addresses with the lower 10 bits set (such as 10.118.15.255) are broadcast addresses.

Now in this case, the device was being given the IP address 10.118.13.255, which looks like a broadcast address, but it's off by one bit. Sure enough, this IP address prevented the device from resolving DNS queries. And since DHCP leases are cached by MAC address, the same board would usually get the same IP address, adding to the confusion.

In trying to debug the issue, I set up a NATed /22 network to log the network traffic, but strangely enough the device worked perfectly fine with the 10.118.13.255 address. Weird. So I set up my laptop to use the 10.118.13.255 on the Austin network, and sure enough Windows was unable to resolve DNS queries either.

So we're pretty sure this is an IT issue at the moment and we've forwarded the issue. We have a device running locally that's just sitting on the 10.118.13.255 address in the meantime, so we should be able to continue testing.

@mbed-bot
Copy link

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test-nightly

Output

mbed Build Number: 572

Test failed!

@studavekar
Copy link
Contributor

/morph test-nightly

@mbed-bot
Copy link

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test-nightly

Output

mbed Build Number: 573

Test failed!

@studavekar
Copy link
Contributor

@studavekar
Copy link
Contributor

/morph test-nightly

@mbed-bot
Copy link

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test-nightly

Output

mbed Build Number: 574

All builds and test passed!

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 17, 2017

LGTM !

@0xc0170 0xc0170 merged commit 3179b1c into mbed-os-5.5 Jun 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants