Ensure composite devices all share the same acquiesce delay

We go to great pains to set the parent to the MAX of the child delays, but we
don't actually set the children to have the new parent value.

This means if you update the child device without the parent in the transaction
we don't always wait for all the USB devices to come back -- which is what the
acquiesce delay is designed to do.

Fixes an intermittant device test for the Wacom USB tablet.

trivial: Ignore device requests when emulated

ccgx: Add support for the CalDigit Element Hub

ccgx: Detect factory mode and set a non-zero version

The Element Hub (and several other docks that use the CCGX5 DMC reference
design) is in "factory mode" out of the box, displaying a version of `0.0.0.0`

Detect factory mode and set a non-zero version -- using the "real" version if
possible by matching the CCG5 devx base firmware firmware.

trivial: Disable the automatic snap stable promotion

The snap is converted into a confined version, but we need to make
sure that the snapd has promoted into a stable release so that fwupd
snap also works.

We should revert this after that happens.

Fix fwupdtool firmware-convert to work with image-less formats

Converting 'simple' non-nested formats like Intel Hex into nested formats like
DfuSe needs us to create a binary child firmware to avoid missing data.

Fixes https://github.com/fwupd/fwupd/issues/5493

trivial: Do not return the internal libxmlb error back to the user

Print errors as JSON objects when using --json

For example:

$ fwupdmgr local-install --json foo.zip
{
  "Error" : {
    "Domain" : "FwupdError",
    "Code" : 7,
    "Message" : "failed to open foo.zip"
  }
}

Fixes https://github.com/fwupd/fwupd/issues/5499

snap: read the host os-release instead of base snap

This should fix #5485. Depends on https://github.com/snapcore/snapd/pull/12545

Optimise getting the list of upgrades using the legacy API

We use GetReleases for all the *possible* releases that could be installed
(although the device might be in UPDATABLE_HIDDEN), and the legacy GetUpgrades
API just wants to know what can be done right now.

Speed up regenerating the MOTD

This is regenerated on every device hotplug event, and although infrequent
there is no point doing all that CPU-heavy requirement checking for no reason.

wistron-dock: Use only-wait-for-replug to speed up device replug

Also, set the progress steps correctly.

wistron-dock: Fix critical warning during update

trivial: Only use clang-format on C source files

Add support for Quit() in fwupdmgr

This allows us to only shut down the P2P daemon in the installed tests.
The easy fix would have been to use something like:

    gdbus call --address unix:path=/run/fwupd.sock --object-path / --method org.freedesktop.fwupd.Quit'

...but the daemon does not implement an ObjectManager, which gdbus requires.

This worked correctly for so long because the p2p tests were being run after
the bus tests, so shutting down the system daemon had no effect.

Fedora 37 seems to have flipped the order for some reason -- and now it
*sometimes* matters that we were doing the worng thing...

trivial: Fix Fedora CI when it cannot autostart udisks

cros-ec: Clear the bulk transfer buffer to zero before using

This allow us to emulate the device as the buffer contents are now 100%
predictable and replayable.

trivial: Fix a tiny coverity-spotted memory leak on error

Move the Fedora CI job to F37

Use the correct range check when parsing SMBIOS

The SMBIOS specification says:

    Maximum size of SMBIOS Structure Table, pointed to by the
    Structure Table Address, in bytes. The actual size is guaranteed
    to be less or equal to the maximum size.

So, the firmware is actually allowed to return a DMI blob smaller than the
specified size.

Fixes https://github.com/fwupd/fwupd/issues/5486

nvme: Detect and warn users with the broken firmware 3B2QGXA7

trivial: Fix fu-self-test with libgusb from the main branch

trivial: Fix critical warning when loading an emulated DFU device

Revert "trivial: Fix clang-format of JSON files for newer versions of clang"

This reverts commit 2eb4357a70c6b1b28bc117e518d197b47df7b66c.

trivial: Fix clang-format of JSON files for newer versions of clang

This explains why Mario and I had different precommit settings.

ti-tps6598x: Ignore the 'wrong' USB device class before calling ->setup()

No functional change, but a few less i/o operations when inserting the dock.

snap: Use common directory for vendor path

snap: Use strict confinement

snap: Fix a meson parameter

snap: Remove some unused/redundant dependencies

snap: Remove dconf

snap: Add missing libraries

Optionally disable Polkit check with an environment variable

When running in a confined snap, it is not possible to check whether a
policies have been installed.

Allow overriding lock directory with an environment variable

dfu: Use more const buffer data when writing to AVR chips

dfu: Clear the buffer before getting the status

This makes emulation of DFU devices work as the control transfer data is then
predictable.

trivial: Enable USB debugging when backend debugging is enabled

trivial: Use new API in libgusb to better detect USB replug

Add API to wait for a device

This allows us to ignore all the delays when the device is emulated, with the
idea being to do dozens of device emulations in the CI tests.

Also, do not call fu_progress_sleep() when the device is emulated.

Do not invalidate the probe for emulated devices

If the device does not replug, and we change the GUsbDevice because the phase
changed we don't want reprobe the hardware.

If we reprobe then the emulated device will request extra data compared to the
real device, which will cause emulation to fail.

trivial: Require --force to overwrite files with 'fwupdmgr download'

Return an error when the pending db could not be updated

build(deps): bump docker/build-push-action from 3 to 4

Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 3 to 4.
- [Release notes](https://github.com/docker/build-push-action/releases)
- [Commits](https://github.com/docker/build-push-action/compare/v3...v4)

---
updated-dependencies:
- dependency-name: docker/build-push-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

trivial: Do not hardcode fwupdmgr as a binary name

trivial: Record the install phase for future functionality

trivial: Do not inherit history from emulated devices

trivial: Remove some dead comments left in after debugging

Use indexes when building the quirk prepared queries

This can save 250us on each query (of which we do a *lot*), and reduces the
number of queries by about half if the user is using a libxmlb version that can
convert from `_BOUND_TEXT` to `_BOUND_INDEX_TEXT`.

The more devices detected, the bigger the difference, and without this patch
fixing libxmlb would break fwupd as we'e been searching with an index, but not
actually providing one.

trivial: Add FuProgress to fu_context_load_hwinfo()

TIL: fu_bios_settings_setup() takes over 50ms (10%!) at startup.

trivial: Remove duplicate call to fu_engine_ensure_device_supported()

This is moderately expensive to call as it processes each requirement on each
release on each device, so the last thing we want to do is call it *twice*.

trivial: Update all fixes using markdownlint

trivial: Update all fixes using codespell

Use a more compact JSON format using precommit

Remove unused API for setting backend tags

It turns out we do not want to know the current phase of the device, and we can
achieve the same thing tagging the GUsbDevice manually and clearing the events
at exactly the correct time.

ti-tps6598x: Assign the DMC as the proxy rather than just the parent

We 'write through' the parent, and by telling fwupd this we can make better
decisions when emulating.

Copy some flags from the proxy to the logical device

trivial: update fwupd-efi to 1.4

fwupd-efi 1.4 enforces that NX bit is set by default.
This won't affect most users, it should only affect those that compile
by hand and don't have fwupd-efi installed already.

trivial: Do not ask for polkit auth if the device is emulated

trivial: Do not wait for the acquiesce delay if emulated

Propagate device tag to parent device

trivial: Correctly compress zip files in FuArchive

Ensure the new GUsbDevice is assigned in subclasses of FuUsbDevice

Re-incorporate the emulated USB devices when the backend device changes.

trivial: Fix critical warning in future code

Both types need to be FuUdevDevice to compare the sysfs paths.

Do not return emulated devices from GetHistory

trivial: Add fu_device_has_problem() to be consistent and hide details

Remove duplicate calls to fu_device_list_wait_for_replug()

The fu_engine_get_device() function is called at the start of each phase,
which calls fu_device_list_wait_for_replug() a second time.

Duplicate calls are at best confusing, and at worst could cause a 'device
failed to come back' failure.

Add a device problem for 'update-in-progress'

We already had this as an inhibit, but this was not translated client-side.

We also need to propagate the problem to the bootloader device if the device
replugs during firmware update.

colorhug: Show the flash success value in debugging

Add python3-pefile/python-pefile to CI build deps

This is so that it can be used by subproject fwupd-efi

Fix hotplug events when emulating devices

modem-manager: remove improper use of assert

FuSaharaLoader being NULL is normal for devices that only
support Firehose and don't use Sahara QDL port.

wacom-usb: Retry set_report on failure

Sometimes the flash process will randomly hang and time-out when sending
data to the device. We currently do not use any retry logic, so if this
happens the flash attempt is treated as a failure. This can be a source
of worry or frustration, especially if subsequent manual retries fail
in a similar way.

Adding FU_HID_DEVICE_FLAG_RETRY_FAILURE to the list of flags used when
calling fu_hid_device_set_report allows fwupd to try sending a block
multiple times if such a time-out (or other error) occurs. This makes
the flash process less prone to failure.

Signed-off-by: Jason Gerecke <jason.gerecke@wacom.com>

trivial: Fix tests to include 'emulated'

Add fu_device_remove_backend_tag() for future use

trivial: Proxy the _SAVE_EVENTS flag even if set at runtime

Use the new GUsb API for loading by tag

Based on a patch by frederic.danis@collabora.com, many thanks.

Do not make any of the HWIDs setup failures fatal

It's perfectly okay to have no HWIDs defined.

Should help with https://github.com/fwupd/fwupd/issues/5402

trivial: Add a device flag specifying it is emulated

trivial: Fix fuzzing with libxmlb from main, harder

trivial: Force enums to be uint64, harder

trivial: Fix fuzzing with libxmlb from main

Save all the FwupdDeviceFlags in the pending database

Force the compiler to treat them as uint64_t so that bitwise operations work.
At the moment flags greater than 1<<32 are just being silently ignored...

Support unsetting specific device flags

This makes it possible to 'undo' the notification, for example.

trivial: Fix build fix when using ppc64le system

trivial: Use %autochangelog in reference spec file

Allow desktop software to inhibit the system to prevent all  updates

On edge hardware a process may want to disable firmware updates as it might be
a bad time to allow an upgrade.

trivial: post release version bump

Release fwupd 1.8.10

emmc: correct legacy GUID generation

It used to be generated with decimal, not hexadecimal values.

Change-Id: I2a6a5bdea50e2d08a7de9f65a1f9a03de29e1591

trivial: Fix bitwise operators in last commit

Add fu_context_remove_flag() for future use

Speed up startup by a huge amount when using directory remotes

By using libxmlb correctly we can avoid rebuilding the metadata if the directory
is unchanged. This means the initial 'generation' is unchanged in speed, but
subsequent loads are almost instant as the archives are not decompressed.

Show some debugging when we ignore a release due to missing checksums

Fallback to the <checksum> if the <artifact> is invalid

In 62a62f8f0f707d815e65ca9d0634459edb6eed97 we started ignoring the artifact
when the type was 'binary' -- however, we need to fall back to the older tag
schema if the data was not set, rather than only if the <artifact> tag was not
found.

wacom-usb: prefix errors that occur when writing to the device

Make it easier to locate errors that occur when writing to the device.

Parse the DT chassis-type when parsing the FDT

uefi-capsule: Do not call grub2-probe without arguments

commit 684bc0381 ("trivial: fix various shellcheck warnings") adds call
to ${grub_probe} without any argument which causes grub2-probe to return
an error, and generating grub configuration fails.

Add --version argument to avoid the error, and redierct the output to
/dev/null so that it's not included in the grub configuration file.

The commit also adds superfluous single quotes causing another error:

/usr/sbin/grub2-probe: error: cannot find a GRUB drive for `${grub_probe} --target=device \${ESP}` | sed -e "s/^/\t/".  Check your device.map.

Fix the quoting to work at least for single device.

Fixes: #5424
Fixes: 684bc0381 ("trivial: fix various shellcheck warnings")
Signed-off-by: Michal Suchanek <msuchanek@suse.de>

Update documentation to indicate that loading "OS Optimized Defaults"
may enable security processor rollback protection on Lenovo systems.

fixes: #5394

Improve FDT parsing compatibility

Newer versions of dtc agressively pack the string table, using *any* matching
suffix rather than whole strings. This is specification compliant, but breaks
if you use the offset as a hash key as a [premature] optimization.

Just use the string table as a flat block of memory rather than pre-parsing it.

This fixes reading the system FDT on new OpenBMC builds.

Refactor the hwids functionality

This refactors the code as it was getting very confusing; before FuSmbios was
reading both SMBIOS and the kernel-provided DT -- and various things were
injecting overrides in three different place. To properly support FDT remove
one layer of indirection.

This also lets us use the compatible strings to enable plugins specifying the
flag _REQUIRE_HWID -- which means we only load the plugin if it's got a chance
of working. e.g.

    [aspeed,ast2500]