Skip to content

libsystemd: device_has_info: return true for usec_initialized#6551

Closed
sfanxiang wants to merge 1 commit intosystemd:masterfrom
sfanxiang:device-has-info
Closed

libsystemd: device_has_info: return true for usec_initialized#6551
sfanxiang wants to merge 1 commit intosystemd:masterfrom
sfanxiang:device-has-info

Conversation

@sfanxiang
Copy link
Contributor

Without this patch, device_update_db() will not create the database
file for an initialized device, which makes device_read_db() fail, and
sd_device_get_is_initialized() will say that the device is not
initialized.

Without this patch, device_update_db() will not create the database
file for an initialized device, which makes device_read_db() fail, and
sd_device_get_is_initialized() will say that the device is not
initialized.
@poettering poettering added the udev label Aug 7, 2017
@poettering
Copy link
Member

@teg @kaysievers please say something

@kaysievers
Copy link
Contributor

device_has_info() means "will this device need to write a database?", usec_initialized does not belong into this check. Can you please explain what exactly goes wrong here?

It is (so far at least) expected behavior, that devices which do not store anything defined by rules, will skip a database entry and not return "initialized".

@sfanxiang sfanxiang closed this Aug 8, 2017
@sfanxiang sfanxiang deleted the device-has-info branch August 8, 2017 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants