search

miketheprogrammer / go-thrust

Cross Platform UI Kit powered by Blink/V8/Chromium Content Lib
MIT License
445 stars 34 forks source link

Severe CPU Overuse #66

Open miketheprogrammer opened 8 years ago

miketheprogrammer commented 8 years ago

A while back I had switched from time.Sleep to runtime.GoSched to lock thread to keep application open. As GoLang has evolved it seems that now results in a huge amount of CPU Churn.

tehbilly commented 8 years ago

@miketheprogrammer Point out the spots where pprof shows churn, if you can? I'm scheduling some downtime later today to work on some issues, and will take a look at this one asap.

miketheprogrammer commented 8 years ago

I think i fixed this, must not have actually commented on it @tehbilly btw we should chat soon.

Ill mark this as closed once i review, but pretty sure i fixed it.