kata-containers/s390-tools
1
0
Fork 0
mirror of https://github.com/ibm-s390-linux/s390-tools.git synced 2026-08-05 02:14:52 +00:00
Code Issues Packages Projects Releases Wiki Activity

zdev: Introduce read-only attributes

Browse Source 
Add support for defining read-only attributes in the chzdev/lszdev
tools. These attributes can be used to provide online documentation
for specific attributes via the --help-attribute and --list-attributes
tool functions.

Signed-off-by: Peter Oberparleiter <oberpar@linux.ibm.com>
Reviewed-by: Jens Remus <jremus@linux.ibm.com>
Reviewed-by: Fedor Loshakov <loshakov@linux.ibm.com>
Reviewed-by: Benjamin Block <bblock@linux.ibm.com>
Signed-off-by: Jan Höppner <hoeppner@linux.ibm.com>
This commit is contained in:
Peter Oberparleiter
2019-02-05 15:51:26 +01:00
committed by Jan Höppner
parent d2f8f972cf
commit c063273b14
4 changed files with 23 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
zdev/src/device.c
View File

@@ -215,6 +215,9 @@ static exit_code_t apply_setting(struct device *dev, config_t config,
/* Check for known attribute. */
a = subtype_find_dev_attrib(st, key);
if (a) {
/* Check for read-only attribute. */
if (a->readonly)
goto err_readonly;
/* Check for acceptable value of known attribute. */
if (!force && !attrib_check_value(a, value))
goto err_invalid_forceable;
@@ -307,6 +310,10 @@ err_int_noactive:
delayed_err("Internal attribute '%s' cannot be set in the active config\n",
key);
return EXIT_INVALID_SETTING;
err_readonly:
delayed_err("Cannot modify read-only attribute: %s\n", key);
return EXIT_INVALID_SETTING;
}
/* Apply device settings from strlist to device. */
@@ -521,7 +528,7 @@ void device_read_active_settings(struct device *dev, read_scope_t scope)
a = attrib_find(st->dev_attribs, name);
s = setting_list_apply_actual(dev->active.settings, a, name,
value);
if (link || (scope == scope_all &&
if ((a && a->readonly) || link || (scope == scope_all &&
util_path_is_readonly_file("%s", path)))
s->readonly = 1;
if (link)