Skip to content

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

Open
@docbender

Description

@docbender

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.

Activity

facchinm

facchinm commented on Feb 15, 2016

@facchinm
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

docbender commented on Feb 15, 2016

@docbender
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

facchinm commented on Feb 15, 2016

@facchinm
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

docbender commented on Feb 15, 2016

@docbender
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

sandeepmistry commented on Jul 18, 2016

@sandeepmistry
Contributor

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

facchinm

facchinm commented on Jul 19, 2016

@facchinm
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

tobozo commented on Oct 10, 2018

@tobozo

bump

gergelytakacs

gergelytakacs commented on Jul 11, 2019

@gergelytakacs

bump...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    feature requestA request to make an enhancement (not a bug fix)

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      No branches or pull requests

        Participants

        @sandeepmistry@tobozo@facchinm@docbender@gergelytakacs

        Issue actions

          [Improvement] - Add option to enable/disable network/serial discovery · Issue #4569 · arduino/Arduino