nodedev: cleanup some historical baggage

[libvirt PATCH 0/4] nodedev: cleanup some historical baggage

Daniel P. Berrangé posted 4 patches 3 years, 5 months ago

Download series mbox

Test syntax-check failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20201117125608.177278-1-berrange@redhat.com

src/node_device/node_device_udev.c | 70 +++++++++++++-----------------
1 file changed, 30 insertions(+), 40 deletions(-)

Expand all Fold all

[libvirt PATCH 0/4] nodedev: cleanup some historical baggage

Posted by Daniel P. Berrangé 3 years, 5 months ago


Daniel P. Berrangé (4):
  nodedev: improve debugging logs from udev device/event processing
  nodedev: dont rely on ignoring errors on missing properties
  nodedev: drop DKD_MEDIA_AVAILABLE property check
  nodedev: report errors about missing integer properties

 src/node_device/node_device_udev.c | 70 +++++++++++++-----------------
 1 file changed, 30 insertions(+), 40 deletions(-)

-- 
2.28.0

Re: [libvirt PATCH 0/4] nodedev: cleanup some historical baggage

Posted by Laine Stump 3 years, 5 months ago

On 11/17/20 7:56 AM, Daniel P. Berrangé wrote:
>
> Daniel P. Berrangé (4):
>    nodedev: improve debugging logs from udev device/event processing
>    nodedev: dont rely on ignoring errors on missing properties
>    nodedev: drop DKD_MEDIA_AVAILABLE property check
>    nodedev: report errors about missing integer properties
>
>   src/node_device/node_device_udev.c | 70 +++++++++++++-----------------
>   1 file changed, 30 insertions(+), 40 deletions(-)
>

Series

Reviewed-by: Laine Stump <laine@redhat.com>


(I made one comment on 2/4, but it's nothing that needs any action - 
just pointing out the level of trust you have in everybody always doing 
what is "sensible" :-)