getvcp returns DDCRC_ALL_RESPONSES_NULL since 1.3.0 about ddcutil HOT 5 CLOSED

Some monitors misuse the DDC Null Response to indicate that a feature is unsupported. You've encountered a bug in the code that handles that situation. To help me understand the problem properly and and assess its severity, please submit the output from the following commands as attachments:

ddcutuil detect --verbose for both 1.4.1 and 0.9.9 (or some earlier version that doesn't have the problem)
ddcutil interrogate for 1.4.1 and, if that fails for 0.9.9 (or some earlier version that doesn't have the problem)
a ddcutil command exhibiting the problem

Thank you.

from ddcutil.

Hi, sorry for the late answer, I don't always have this monitor at hand.
Anyways, here are the logs: https://gist.github.com/koko-ng/a681a792206137b8bbc94d8b5a580aa7
Let me know if there is anything else I can do to help! :)

Thanks

from ddcutil.

I have just uploaded an extensive set of changes to branch 1.5.0-dev. Included in them are changes that are intended to address the problems you've encountered.

The root problem you hit is that the detection phase of ddcutil (apparently) incorrectly determined that your monitor misuses a DDC Null Message to indicate unsupported features. There was special recovery code in the retry logic (recall that DDC communication is unreliable) for DDC Null Messages, and it was no longer working properly.

Please try using the development branch. It may be that, with the default settings, things just work. However, the monitor may return a DDC Null Message during probing, leading to the monitor being marked invalid and a message that monitor communication failed. In that case, first try using option --sleep-multiplier 2.0 to give the monitor more time to formulate proper responses. Second, try using option --dsa2. This is a new dynamic sleep algorithm that adjusts the sleep multiplier up (as needed) and down (insofar as possible) on a per monitor basis. (Note: if you specify both --sleep-multiplier and --dsa2 on the same command, ddcutil discards what it knows about the monitor and starts its adjustment from the sleep multiplier specified.)

If you encounter problems, as a first step, please submit the output of for failed program executions. Thank you.

from ddcutil.

Thanks for the update, it works well with the latest version (commit 0b6971c) without any special parameters.

from ddcutil.

@koko-ng Thank you for letting me know that the recent changes have addressed your problem.

from ddcutil.