obmc-ikvm, branch master

systemd: Don't start without configfs

2019-11-14T20:33:17+00:00

We cannot run without configfs. Don't attempt to start if we find ourselves in that situation. Change-Id: I67515d2eaaf982f6750aae6e90a939873338a77c Signed-off-by: Brad Bishop

build: drop autotools support

2019-11-14T20:05:52+00:00

It made sense to support both build systems while OpenBMC transitioned but that has been complete for some time - No need to support multiple build systems any longer. Change-Id: Ib698163f53a4f493f20491c54851247c5700f13e Signed-off-by: Brad Bishop

Prevent excessive journal logging by wakeup signal

2019-09-30T20:05:16+00:00

When the application is waiting for timings, it restarts the video to try and get a signal. Part of this process sends a wake-up signal on the virtual keyboard and pointer in case host VGA has slept. But if the host is shut down, this causes repeated errors in the journal as the HID gadgets return errors. Fix this by consolidating the keyboard/pointer write functions and therefore closing the HID gadget file handles when the host is shut down, preventing further wake-up attempts until host returns. Change-Id: Id5fe484c976eccefa6a72393d4d6b2b301d28a1a Signed-off-by: Eddie James

Remove frame rate limit

2019-09-27T16:57:28+00:00

Since we have flow control logic, better remove the frame rate limit to improve frame update speed as faster as it can provide. So this commit removes the limit parameter so that it can use the default 30fps setting. Tested: It didn't make bmcweb crash which is caused by frame data flooding. Frame update speed little bit faster than before. Change-Id: I30ec8f2c4124e27c20261b87042fb094160d27f8 Signed-off-by: Jae Hyun Yoo

Add flow control to prevent buffer over run

2019-09-12T18:13:50+00:00

This service uses direct frame update with bypassing image compression and invalidating logic in libvncserver to achieve better performance by using of H/W compressed JPEG frames as those come from the video engine driver. This behavior helps quick frame update using very small amount of CPU resources but it causes a side effect which crashes bmcweb by OOM killer due to a buffer over run issue. Usually, this issue happens often in a slow speed connection because this service keeps sending all frames without any handshaking with clients so a session buffer in the bmcweb gets bigger and bigger since the low speed connection can't send all stream data on time. To fix this issue, this commit adds flow control logic to make frame updating handshakes with client so that it'll send frames only when it recieved client frame update messages. All other frames when the client doesn't request will be dropped out to prevent the buffer over run issue. Tested: bmcweb didn't keep increasing its KVM session buffer. KVM worked well with showing good refresh speed. resolves https://github.com/openbmc/bmcweb/issues/80 Change-Id: I6b09a711137d15a38fce59adada9bf3d00afde86 Signed-off-by: Jae Hyun Yoo

Add keypad input support

2019-08-20T18:26:53+00:00

This commit makes keypad inputs available. Change-Id: I95a9268cd930c85846947b6bd49c97087121ca8a Signed-off-by: Jae Hyun Yoo

Add a maintainer

2019-08-09T21:34:02+00:00

Add Jae as a maintainer. Change-Id: I520b8da04d64ccd186ec22a68e0e38929c7bed68 Signed-off-by: Jae Hyun Yoo

eliminate excessive journal errors

2019-08-09T20:41:37+00:00

When the USB gadget is shut down, the application needs to avoid writes to it in order to avoid lots of errors in the kernel log. Close the file handle and re-open it when timings are detected again. Also, prevent logging the failed timings query more than once. Change-Id: Ic126828fe26ef44ebb0a5cb65cc47b23bb84f7f3 Signed-off-by: Eddie James

Reduce framerate to prevent lag

2019-04-19T14:33:20+00:00

Reduce the framerate to 16 from the default 30. In testing with high latency or low bandwidth situations, 30 fps is unusable. Signed-off-by: Eddie James Change-Id: Idf6ce9da4556a53500507dcf9d02a4425cba9a1b

Switch pointer gadget to non-blocking

2019-04-18T17:51:47+00:00

Writes to the pointer gadget can hang if the host has no cursor. Using non-blocking mode prevents this hanging. Also prevent repeated error logging due to pointer write failures in this case. Change-Id: I9c423a4a0af4b37a69ae20650b86a6266e5b4b41 Signed-off-by: Eddie James