[Libosinfo] [libosinfo PATCH v2 00/15] Deal with "removed" devices

Fabiano Fidêncio fidencio at redhat.com
Mon Nov 12 10:30:47 UTC 2018


This patch series aim to have OSes properly dealing with removed
devices. The path I've taken is:
- Use OsinfoDeviceLink to store whether the device is removed;
- Filter out the removed devices from _get_devices(),
  _get_device_links(), _get_all_devices() and _get_all_device_links()
- A bunch of tests to ensure I'm not breaking anything, neither that
  something will end up broken in the future;

Now, questions ...
- Shall we do the same for platform and deployment?
- I've introduced and exposed osinfo_os_get_all_device_links(), and the
  rationale for exposing it is basically for coherence with the
  osinfo_os_get_all_devices() .... However, it could to

Changes since v1:
  https://www.redhat.com/archives/libosinfo/2018-November/msg00038.html
  - Dropped the patches dealing with "deprecated" devices;

* osinfo-db:
  Fabiano Fidêncio (1):
    schema: Add "removed" attribute to devices element

   data/schema/osinfo.rng.in | 5 +++++
   1 file changed, 5 insertions(+)

* libosinfo:
  Fabiano Fidêncio (14):
    devicelinklist: Fix _get_devices()
    tests: Add basic tests for devicelinklist
    test-devicelinklist: Add _get_devices() test
    tests-os: Add test_devices_inheritance_basic()
    loader: Deal with "removed" devices for an OS
    os: Add osinfo_os_get_all_device_links()
    os: Add osinfo_os_get_devices_internal()
    os: Add osinfo_os_get_device_links_internal()
    os: Add osinfo_os_get_all_device_links_internal()
    os: Use get_devices_internal in _get_all callback
    os: Use get_device_links_internal in _get_all callback
    os: Add add_entity_to_list_check()
    os: Deal with "removed" devices
    test-os: Add test_devices_inheritance_removal()

   osinfo/libosinfo.syms                         |   2 +
   osinfo/osinfo_devicelink.h                    |   1 +
   osinfo/osinfo_devicelinklist.c                |   6 +-
   osinfo/osinfo_loader.c                        |   9 +
   osinfo/osinfo_os.c                            | 222 +++++++++++--
   osinfo/osinfo_os.h                            |   1 +
   tests/Makefile.am                             |   5 +
   .../device/libosinfo.org/test-device-one.xml  |   7 +
   .../libosinfo.org/test-os-devices-basic-1.xml |  13 +
   .../test-os-devices-basic-2-clone.xml         |  10 +
   .../libosinfo.org/test-os-devices-basic-2.xml |  10 +
   .../test-os-devices-removed-1.xml             |  14 +
   .../test-os-devices-removed-2-clone.xml       |  10 +
   .../test-os-devices-removed-2.xml             |  10 +
   .../test-os-devices-removed-3.xml             |  14 +
   .../test-os-devices-removed-4-clone.xml       |  10 +
   .../test-os-devices-removed-4.xml             |  10 +
   tests/test-devicelinklist.c                   | 313 ++++++++++++++++++
   tests/test-os.c                               | 169 ++++++++++
   19 files changed, 809 insertions(+), 27 deletions(-)
   create mode 100644 tests/dbdata/device/libosinfo.org/test-device-one.xml
   create mode 100644 tests/dbdata/os/libosinfo.org/test-os-devices-basic-1.xml
   create mode 100644 tests/dbdata/os/libosinfo.org/test-os-devices-basic-2-clone.xml
   create mode 100644 tests/dbdata/os/libosinfo.org/test-os-devices-basic-2.xml
   create mode 100644 tests/dbdata/os/libosinfo.org/test-os-devices-removed-1.xml
   create mode 100644 tests/dbdata/os/libosinfo.org/test-os-devices-removed-2-clone.xml
   create mode 100644 tests/dbdata/os/libosinfo.org/test-os-devices-removed-2.xml
   create mode 100644 tests/dbdata/os/libosinfo.org/test-os-devices-removed-3.xml
   create mode 100644 tests/dbdata/os/libosinfo.org/test-os-devices-removed-4-clone.xml
   create mode 100644 tests/dbdata/os/libosinfo.org/test-os-devices-removed-4.xml
   create mode 100644 tests/test-devicelinklist.c
-- 
2.19.1




More information about the Libosinfo mailing list