<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/libvirt.git/src/node_device, branch master</title>
<subtitle>libvirt.org: git/libvirt.git
</subtitle>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libvirt.git/'/>
<entry>
<title>meson: Use initconfdir</title>
<updated>2023-05-05T13:08:25+00:00</updated>
<author>
<name>Andrea Bolognani</name>
<email>abologna@redhat.com</email>
</author>
<published>2023-04-29T16:23:12+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libvirt.git/commit/?id=32f772e98659eba8b93e940698102464c4b91209'/>
<id>32f772e98659eba8b93e940698102464c4b91209</id>
<content type='text'>
Signed-off-by: Andrea Bolognani &lt;abologna@redhat.com&gt;
Reviewed-by: Martin Kletzander &lt;mkletzan@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Andrea Bolognani &lt;abologna@redhat.com&gt;
Reviewed-by: Martin Kletzander &lt;mkletzan@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>node_device: Update format strings in translated messages</title>
<updated>2023-04-01T09:40:33+00:00</updated>
<author>
<name>Jiri Denemark</name>
<email>jdenemar@redhat.com</email>
</author>
<published>2023-03-09T11:27:53+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libvirt.git/commit/?id=6b8763675dc6c7c34d04c6388c959e4eff40ec5a'/>
<id>6b8763675dc6c7c34d04c6388c959e4eff40ec5a</id>
<content type='text'>
Signed-off-by: Jiri Denemark &lt;jdenemar@redhat.com&gt;
Reviewed-by: Daniel P. Berrangé &lt;berrange@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Jiri Denemark &lt;jdenemar@redhat.com&gt;
Reviewed-by: Daniel P. Berrangé &lt;berrange@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>util: remove waitForLock from virPidFileAcquire</title>
<updated>2023-03-08T11:16:55+00:00</updated>
<author>
<name>Ján Tomko</name>
<email>jtomko@redhat.com</email>
</author>
<published>2023-03-07T15:07:20+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libvirt.git/commit/?id=1cc783bc4431e6c546337413934bfc4fdc4169bd'/>
<id>1cc783bc4431e6c546337413934bfc4fdc4169bd</id>
<content type='text'>
The parameter was added for consistency with virPidFileAcquirePath.
However, all callers of virPidFileAcquire pass false.

Remove the argument.

Partially-reverts: 2250a2b5d21c3b3529727f38a99cba22f84024f7
Signed-off-by: Ján Tomko &lt;jtomko@redhat.com&gt;
Reviewed-by: Martin Kletzander &lt;mkletzan@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The parameter was added for consistency with virPidFileAcquirePath.
However, all callers of virPidFileAcquire pass false.

Remove the argument.

Partially-reverts: 2250a2b5d21c3b3529727f38a99cba22f84024f7
Signed-off-by: Ján Tomko &lt;jtomko@redhat.com&gt;
Reviewed-by: Martin Kletzander &lt;mkletzan@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>nodedev: ignore EINVAL from libudev in udevEventHandleThread</title>
<updated>2022-11-10T10:50:22+00:00</updated>
<author>
<name>Christian Ehrhardt</name>
<email>christian.ehrhardt@canonical.com</email>
</author>
<published>2022-11-10T09:36:28+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libvirt.git/commit/?id=33a38492b75acb7dbec9b64c41a5dba4acde4240'/>
<id>33a38492b75acb7dbec9b64c41a5dba4acde4240</id>
<content type='text'>
Certain udev entries might be of a size that makes libudev emit EINVAL
which right now leads to udevEventHandleThread exiting. Due to no more
handling events other elements of libvirt will start pushing for events
to be consumed which never happens causing a busy loop burning a cpu
without any gain.

After evaluation of the example case discussed in in #245 and a test
run ignoring EINVAL it was considered safe to add EINVAL to the ignored
errnos to not exit udevEventHandleThread giving it more resilience.

The root cause is in systemd and by now was discussed and fixed via
https://github.com/systemd/systemd/issues/24987, but hardening libvirt
to be able to better deal with EINVAL returned still is the right thing
to avoid the reported busy loops on systemd with older systemd versions.

Fixes: https://gitlab.com/libvirt/libvirt/-/issues/245

Signed-off-by: Christian Ehrhardt &lt;christian.ehrhardt@canonical.com&gt;
Reviewed-by: Daniel P. Berrangé &lt;berrange@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Certain udev entries might be of a size that makes libudev emit EINVAL
which right now leads to udevEventHandleThread exiting. Due to no more
handling events other elements of libvirt will start pushing for events
to be consumed which never happens causing a busy loop burning a cpu
without any gain.

After evaluation of the example case discussed in in #245 and a test
run ignoring EINVAL it was considered safe to add EINVAL to the ignored
errnos to not exit udevEventHandleThread giving it more resilience.

The root cause is in systemd and by now was discussed and fixed via
https://github.com/systemd/systemd/issues/24987, but hardening libvirt
to be able to better deal with EINVAL returned still is the right thing
to avoid the reported busy loops on systemd with older systemd versions.

Fixes: https://gitlab.com/libvirt/libvirt/-/issues/245

Signed-off-by: Christian Ehrhardt &lt;christian.ehrhardt@canonical.com&gt;
Reviewed-by: Daniel P. Berrangé &lt;berrange@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>nodedev|test: Implement support for validating node device XMLs</title>
<updated>2022-11-01T12:07:20+00:00</updated>
<author>
<name>Peter Krempa</name>
<email>pkrempa@redhat.com</email>
</author>
<published>2022-10-18T11:41:04+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libvirt.git/commit/?id=295908688d00d27279522f701b8d0795ea62ae76'/>
<id>295908688d00d27279522f701b8d0795ea62ae76</id>
<content type='text'>
Signed-off-by: Peter Krempa &lt;pkrempa@redhat.com&gt;
Reviewed-by: Jonathon Jongsma &lt;jjongsma@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Peter Krempa &lt;pkrempa@redhat.com&gt;
Reviewed-by: Jonathon Jongsma &lt;jjongsma@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>conf: node_device: Add 'validate' argument to virNodeDeviceDefParse</title>
<updated>2022-11-01T12:07:20+00:00</updated>
<author>
<name>Peter Krempa</name>
<email>pkrempa@redhat.com</email>
</author>
<published>2022-10-18T10:34:07+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libvirt.git/commit/?id=0268270b0f097cfca4219624daa31270a7cedf0f'/>
<id>0268270b0f097cfca4219624daa31270a7cedf0f</id>
<content type='text'>
Allow callers to request XML validation against the schema. All callers
for now pass 'false'.

Signed-off-by: Peter Krempa &lt;pkrempa@redhat.com&gt;
Reviewed-by: Jonathon Jongsma &lt;jjongsma@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Allow callers to request XML validation against the schema. All callers
for now pass 'false'.

Signed-off-by: Peter Krempa &lt;pkrempa@redhat.com&gt;
Reviewed-by: Jonathon Jongsma &lt;jjongsma@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>node_device_driver.h: Drop nodeDeviceLock() and nodeDeviceUnlock() fwd declarations</title>
<updated>2022-10-26T06:49:01+00:00</updated>
<author>
<name>Michal Privoznik</name>
<email>mprivozn@redhat.com</email>
</author>
<published>2022-10-25T13:56:33+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libvirt.git/commit/?id=a6474cfe4806ef90a4621b509d202b7efd8fd09e'/>
<id>a6474cfe4806ef90a4621b509d202b7efd8fd09e</id>
<content type='text'>
The node_device_driver.h declares nodeDeviceLock() and
nodeDeviceUnlock() functions which used to exist, but after
rework to automatic mutex management they exist no more. Their
last use was removed in v8.1.0-rc1~122.

Signed-off-by: Michal Privoznik &lt;mprivozn@redhat.com&gt;
Reviewed-by: Jonathon Jongsma &lt;jjongsma@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The node_device_driver.h declares nodeDeviceLock() and
nodeDeviceUnlock() functions which used to exist, but after
rework to automatic mutex management they exist no more. Their
last use was removed in v8.1.0-rc1~122.

Signed-off-by: Michal Privoznik &lt;mprivozn@redhat.com&gt;
Reviewed-by: Jonathon Jongsma &lt;jjongsma@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>node_device: Move fwd declaration of udevNodeRegister() into correct header file</title>
<updated>2022-10-26T06:48:23+00:00</updated>
<author>
<name>Michal Privoznik</name>
<email>mprivozn@redhat.com</email>
</author>
<published>2022-10-25T14:27:14+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libvirt.git/commit/?id=6f45d2c18139064e3bcc30821f8b918a0efb0bda'/>
<id>6f45d2c18139064e3bcc30821f8b918a0efb0bda</id>
<content type='text'>
Currently, udevNodeRegister() is forward declared in
node_device_driver.h even though the function is implemented in
node_device_udev.c which warrants node_device_udev.h header file.
Move the declaration into the correct file.

Signed-off-by: Michal Privoznik &lt;mprivozn@redhat.com&gt;
Reviewed-by: Jonathon Jongsma &lt;jjongsma@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Currently, udevNodeRegister() is forward declared in
node_device_driver.h even though the function is implemented in
node_device_udev.c which warrants node_device_udev.h header file.
Move the declaration into the correct file.

Signed-off-by: Michal Privoznik &lt;mprivozn@redhat.com&gt;
Reviewed-by: Jonathon Jongsma &lt;jjongsma@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>node_device_udev.h: Drop include of libudev.h</title>
<updated>2022-10-26T06:47:59+00:00</updated>
<author>
<name>Michal Privoznik</name>
<email>mprivozn@redhat.com</email>
</author>
<published>2022-10-25T08:20:10+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libvirt.git/commit/?id=6283e100628b48f0263492dc96232049a94360c6'/>
<id>6283e100628b48f0263492dc96232049a94360c6</id>
<content type='text'>
Nothing in the header file requires the include of libudev.h, as
the former header file is now empty.

Signed-off-by: Michal Privoznik &lt;mprivozn@redhat.com&gt;
Reviewed-by: Jonathon Jongsma &lt;jjongsma@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Nothing in the header file requires the include of libudev.h, as
the former header file is now empty.

Signed-off-by: Michal Privoznik &lt;mprivozn@redhat.com&gt;
Reviewed-by: Jonathon Jongsma &lt;jjongsma@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>node_device: Move DMI_DEVPATH into node_device_udev.c</title>
<updated>2022-10-26T06:47:32+00:00</updated>
<author>
<name>Michal Privoznik</name>
<email>mprivozn@redhat.com</email>
</author>
<published>2022-10-25T08:19:46+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/libvirt.git/commit/?id=a9db75ad3a04ddb3515c8df1bf818e662d673b14'/>
<id>a9db75ad3a04ddb3515c8df1bf818e662d673b14</id>
<content type='text'>
The DMI_DEVPATH macro is used exclusively within
node_device_udev.c. There's no need to expose it.

Signed-off-by: Michal Privoznik &lt;mprivozn@redhat.com&gt;
Reviewed-by: Jonathon Jongsma &lt;jjongsma@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The DMI_DEVPATH macro is used exclusively within
node_device_udev.c. There's no need to expose it.

Signed-off-by: Michal Privoznik &lt;mprivozn@redhat.com&gt;
Reviewed-by: Jonathon Jongsma &lt;jjongsma@redhat.com&gt;
</pre>
</div>
</content>
</entry>
</feed>
