[Milhouse]

Hi,

I did some research with clean LibreElec builds, only the stock addons enabled,

the builds #0330 to #0403 are absolutely fine, no problems with my 100% CPU issue,

with #0405, I'm getting a kernel panic,

with #0406 to #0519, I have the 100% CPU issue, but only on systems with a Nvidia GPU,

no issues with the builds, on systems with a Intel IGPU, expect #0405.

Right with " DEBUG: ------ Window Init (VideoFullScreen.xml) ------ " one CPU core is fully locked at 100%,

on Nvidia systems, init a other window like DEBUG: ------ Window Init (MyVideoNav.xml) ------, immediately

drops the CPU usage to ~10% - 12%.


Was there a major change in Kodi, between f6264876 and ce28cb6, that could cause this?


I can reproduce this on all Kodi 17 installations, Arch, Suse, Ubuntu ..., from ce28cb6 on,

VideoFullScreen is causing my issue on Nvidia system, and no problems with Intel system.

I also cross checked this two of my mates, with the same result.

Or maybe I'm absolutely stupid and doing something wrong.


I found a posting on a German Kodi forum, that chap

is also digging in that issue.



Best regards

OK, I'm able to reproduce this on my Revo 3700 with ION2.

I do remember now that LE #0405 was built incorrectly using popcornmix linux and newclock5 commits on top of the normal x86 linux and kodi commits - this likely explains the kernel panic.

I've also reproduced the high-CPU load behaviour in OE: OE #0402 is "normal", while OE #0410 shows increased CPU loading on one CPU core.

OE #0402 watching a video, full screen:

Code:

Mem: 1074716K used, 965672K free, 206484K shrd, 68904K buff, 593148K cached
CPU0:  2.6% usr  3.0% sys  1.6% nic 92.8% idle  0.0% io  0.0% irq  0.0% sirq
CPU1:  6.4% usr  6.6% sys  0.0% nic 86.5% idle  0.0% io  0.0% irq  0.4% sirq
CPU2:  1.6% usr  0.8% sys  0.0% nic 96.9% idle  0.4% io  0.0% irq  0.2% sirq
CPU3:  8.1% usr  9.7% sys  0.0% nic 81.7% idle  0.0% io  0.0% irq  0.4% sirq
Load average: 0.79 0.46 0.18 2/189 1067
  PID  PPID USER     STAT   VSZ %VSZ CPU %CPU COMMAND
  845   839 root     S    2671m133.5   1 10.1 /usr/lib/kodi/kodi.bin --standalone -fs --lircdev /run/lirc/lircd
1065  1060 root     R     7732  0.3   1  0.0 top
  289     2 root     SW       0  0.0   1  0.0 [kworker/1:4]
  249     1 root     S <   212m 10.6   1  0.0 /usr/bin/pulseaudio --system
  444   442 root     S      99m  4.9   1  0.0 /usr/bin/Xorg -nolisten tcp vt01 -s 0 -noreset -allowMouseOpenFail -config /storage/.config/xorg.conf -ignoreABI
  468     1 root     S    29148  1.4   0  0.0 /usr/bin/smbd --configfile=/run/samba/smb.conf
  418     1 root     S    27144  1.3   1  0.0 /usr/bin/nmbd --configfile=/run/samba/smb.conf
  700     1 root     S    23836  1.1   0  0.0 /usr/bin/fluxbox
  471     1 root     S    19604  0.9   3  0.0 /usr/bin/wpa_supplicant -u

OE #0410 watching the same video, full screen:

Code:

Mem: 1055664K used, 984724K free, 208084K shrd, 67384K buff, 600168K cached
CPU0: 26.8% usr 72.7% sys  0.0% nic  0.0% idle  0.0% io  0.0% irq  0.3% sirq
CPU1:  5.4% usr  4.6% sys  1.7% nic 87.7% idle  0.3% io  0.0% irq  0.0% sirq
CPU2:  0.8% usr  0.6% sys  0.0% nic 98.5% idle  0.0% io  0.0% irq  0.0% sirq
CPU3:  5.5% usr  2.1% sys  0.0% nic 92.3% idle  0.0% io  0.0% irq  0.0% sirq
Load average: 1.22 0.55 0.21 5/187 997
  PID  PPID USER     STAT   VSZ %VSZ CPU %CPU COMMAND
  837   832 root     R    2675m133.7   0 28.6 /usr/lib/kodi/kodi.bin --standalone -fs --lircdev /run/lirc/lircd
  412     1 avahi    S    19116  0.9   1  0.0 avahi-daemon: running [OpenELEC.local]
  990   987 root     R     7732  0.3   1  0.0 top
  149     2 root     SW       0  0.0   1  0.0 [jbd2/sda2-8]
  251     1 root     S <   212m 10.6   3  0.0 /usr/bin/pulseaudio --system
  424   422 root     S     113m  5.6   2  0.0 /usr/bin/Xorg -nolisten tcp vt01 -s 0 -noreset -allowMouseOpenFail -config /storage/.config/xorg.conf -ignoreABI
  598     1 root     S    29148  1.4   2  0.0 /usr/bin/smbd --configfile=/run/samba/smb.conf
  419     1 root     S    27144  1.3   2  0.0 /usr/bin/nmbd --configfile=/run/samba/smb.conf
  684     1 root     S    23836  1.1   2  0.0 /usr/bin/fluxbox
  493     1 root     S    19604  0.9   1  0.0 /usr/bin/wpa_supplicant -u

Build OE #0410 is also using the 340.96 nvidia driver, assuming this is even relevant.

More likely this is a fernetmenta VideoPlayer change, where an old nvidia "quirk" has been dropped which may now result in the CPU busy-waiting while rendering. The Render/Wait changes in LE #0404c and LE #0406 look likely candidates for this new behaviour - in OE #0410 this is merged as PR9542.

Since I'd already spent the time working out the changes in LE #0404c/#0405/#0406, here they are - perhaps FernetMenta can comment when he's available.

• #0404c (compared with #0403):
  • LibreELEC:
    • build: Include git hash in /etc/os-release for release builds too (PR:87, 1 commit, 1 file changed)
      
    • RPi/RPi2: Linux 4.4.6 and Kodi Jarvis updates (PR:78, 2 commits, 4 files changed)
      
    • RPi: Stop enabling serial console in devel images (PR:84, 1 commit, 2 files changed)
      
    • console: enable cursor for textmode and RPi serial consoles (PR:85, 1 commit, 3 files changed)
      
    • u-boot: update to u-boot-imx6-408544d for imx6 project (PR:80, 1 commit, 3 files changed)
      
    • imx6: add second kernel 4.4 from xbian (PR:81, 1 commit, 8 files changed)
      
    • RBP: Update firmware to be in sync with kernel (PR:92, 1 commit, 2 files changed)
      
    • rtmpdump: cleanup download url (PR:82, 1 commit, 1 file changed)
      
    • libbdplus: Fix download URL (PR:93, 1 commit, 1 file changed)
      
    • imx6: add irqbalanced daemon for linux 4.4 (PR:94, 1 commit, 3 files changed)
      
    • v86d: Fix download url (PR:95, 1 commit, 1 file changed)
      
  • Kodi:
    • [DVDVideoCodecOpenMax] update after d5f63a8 (PR:9539, 1 commit, 2 files changed)
      
    • VideoPlayer: add method for requesting desired video resolution to de… (PR:9531, 1 commit, 12 files changed)
      
    • [json] - add support for tvshow status (PR:9537, 1 commit, 2 files changed)
      
    • [cmake] Add cmake parameter to BuildSetup.bat (PR:9529, 5 commits, 5 files changed)
      
    • [python] add mediatype to setinfo for music (PR:9396, 1 commit, 4 files changed)
      
    • [cmake] Fix android build (PR:9540, 2 commits, 1 file changed)
      
    • [gui] - remove go-to-root context menu buttons (PR:9536, 1 commit, 6 files changed)
      
    • [buildsystem][win32] Updated some more build files to vs2015 that was forgotten about (PR:9541, 1 commit, 4 files changed)
      
    • [json] fix params order for SetTVShowDetails (PR:9544, 1 commit, 1 file changed)
      
    • [adsp] rework to new system who already on PVR (PR:9483, 4 commits, 26 files changed)
      
    • VideoPlayer: inputstream, fix cp error (e9a6d59f)
      
    • [lang][skin.estuary] updated language files from Transifex (da74385e)
      
    • [lang][skin.estouchy] updated language files from Transifex (02e2e021)
      
  • FernetMenta (new additions):
    • VideoPlayer: drop WaitPresentTime (6ba07493)
      
  • Changes not merged upstream:
    • PR:178: xf86-video-nvidia-legacy: Change driver from 304.x to 340.x
      
  
  
  
  
• #0405:
  • LibreELEC:
    • LibCEC patches update (PR:88, 2 commits, 4 files changed)
      
    • projects/imx6/patches/u-boot: set delay of 1 second to speed up the boot (PR:97, 1 commit, 1 file changed)
      
    • projects/imx6/linux: disable PCI MSI interrupts for kernel 4.4 (PR:96, 1 commit, 1 file changed)
      
    • screensaver.shadertoy: Add package (PR:73, 1 commit, 1 file changed)
      
    • image: BUG_REPORT_URL should reflect origin repo used for build (PR:91, 1 commit, 1 file changed)
      
    • preset directories and paths for picons (PR:89, 2 commits, 2 files changed)
      
    • readme: update README.md text (PR:76, 1 commit, 1 file changed)
      
    • visualization.shadertoy: fix build for OpenGLES (PR:100, 1 commit, 1 file changed)
      
    • Package updates (PR:103, 6 commits, 7 files changed)
      • mesa: update to 11.2.0
        
      • bluez: update to 5.38
        
      • setuptools: update to 20.6.7
        
      • Pillow: update to 3.2.0
        
      • fontconfig: update to 2.11.94
        
      • inputproto: update to 2.3.2
        
    • libiconv: add libiconv package (PR:104, 1 commit, 3 files changed)
      
    • addons: add OSCam (PR:99, 1 commit, 15 files changed)
      
    • addons: version bump for LE 7.0 Beta 1 (PR:90, 1 commit, 21 files changed)
      
    • Hyperion! (PR:39, 3 commits, 10 files changed)
      
  • Kodi:
    • Videoplayer code cleanup & race condition fix (PR:9442, 4 commits, 2 files changed)
      
    • Cleanup dll loader / posix win32 emulation functions. (PR:8798, 13 commits, 66 files changed)
      
    • [EGLIMX] Added category D framebuffer modes (PR:9551, 1 commit, 1 file changed)
      
    • [guilib][fix] Fix "vector iterator not incrementable". dialog->Close … (PR:9547, 1 commit, 1 file changed)
      
    • [Builtins/Cec] add missing registrations (PR:9535, 1 commit, 1 file changed)
      
    • [EGLIMX] Added category D framebuffer modes (c70dfe76)
      
  • FernetMenta:
    • Not present - #0405 incorrectly built using Raspberry Pi linux and kodi (newclock5) patches
      
  
  
  
• #0406:
  • LibreELEC:
    • scripts/clean: allow cleaning package for all projects (PR:108, 1 commit, 1 file changed)
      
    • scripts/clean: fixup previous commit (PR:109, 1 commit, 1 file changed)
      
    • boblightd: bump version (PR:110, 1 commit, 1 file changed)
      
    • hyperion: updated description and bump version (PR:111, 1 commit, 2 files changed)
      
    • Upstream changes (PR:101, 5 commits, 5 files changed)
      
  • Kodi:
    • small syntax typo (PR:9554, 1 commit, 1 file changed)
      
    • VideoPlayer: set speed on demuxer when speed of player is changed (PR:9555, 1 commit, 1 file changed)
      
    • remove some unused/duplicate context menus (PR:9548, 4 commits, 8 files changed)
      
    • videoinfotag: fix typo introduced in e6e2d0ace5cfe6191b8f7727ced1747854254ce4 (PR:9558, 1 commit, 1 file changed)
      
    • [coverity] Some cleanup/fixes (PR:9512, 14 commits, 23 files changed)
      
    • [Estouchy] change include syntax (PR:9562, 1 commit, 1 file changed)
      
  • FernetMenta (new additions):
    • VideoPlayer: simplify video rednering (c7bb0a59)
      
    • simplify render loop (002a4df8)
      
    • VidoePlayer: do not block renderthread if no gui was rendered and video is separate layer (6b453df8)
      
    • move FinishPipeline from application to gfx context (06e797f3)
      
    • squash: fix build after WaitPresentTime (3e2e4f2a)