Skip to content
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

[Improvement] - Add option to enable/disable network/serial discovery #4569

Open
docbender opened this issue Feb 12, 2016 · 8 comments
Open
Labels
feature request A request to make an enhancement (not a bug fix)

Comments

@docbender
Copy link

Would be nice to have option in preferences to control network and serial port discovery (option to disable it).

In my case (Win7 64bit and 5 network cards) discovery extremely stress CPU. One core usage is up 100% in 3-second intervals. This load is dependent on the number of allowed network interfaces a lot.

@facchinm facchinm added the feature request A request to make an enhancement (not a bug fix) label Feb 15, 2016
@facchinm
Copy link
Member

Hi @docbender ,
the high cpu usage for Serial discovery has been solved in mainline (you can test using the latest hourly version). Network discovery code is still untouched but I never saw it causing a huge performance drop (I must admit I've never used a PC with 5 network interfaces 😄 )
If you could report if the hourly solves your problems (at least partially) it would be great!

@docbender
Copy link
Author

Hi, I tested hourly version from wednesday. I can try latest one and put here some CPU usage screenshot.

Probably I am not only one who saw this kind of issue:
https://forum.arduino.cc/index.php?topic=347870.0

@facchinm
Copy link
Member

@docbender , could you test this version of the IDE?
I've stripped down the network discovery code to leave the barebone functionalities only (with a 5 second timer task to check for the board reachability)

@docbender
Copy link
Author

@facchinm , I tested your IDE version and CPU usage rapidly dropped down.

I attached two images - CPU usage screenshots. First belong to latest nighly build and second to your version. (my CPU is 4 core so 25% is one thread fully utilization)

ide_latestnighlybuild
ide_pr4482_build_509

@sandeepmistry
Copy link
Contributor

Closing this, now the the high CPU issue has been resolved, I don't think this request is needed any more.

@facchinm
Copy link
Member

The mdns changes were not merged at the end since there were problems with multiple network ports. Leaving this open for reference if sooner or later we want to tackle that issue

@tobozo
Copy link

tobozo commented Oct 10, 2018

bump

@gergelytakacs
Copy link

bump...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request A request to make an enhancement (not a bug fix)
Projects
None yet
Development

No branches or pull requests

5 participants