Re: [PATCH v2 00/20] node_dev_udev: use workerpool and improve nodedev events
On Tue, Jun 18, 2024 at 08:59 AM -0500, Jonathon Jongsma wrote: > On 6/18/24 6:33 AM, Boris Fiuczynski wrote: >> On 6/13/24 11:14 PM, Jonathon Jongsma wrote: >>> On 6/13/24 6:13 AM, Marc Hartmayer wrote: On Tue, Apr 23, 2024 at 08:08 PM +0200, Marc Hartmayer wrote: > When an udev event occurs for a mediated device (mdev) the mdev > config data > requires an update via mdevctl as the udev event does not contain > all config > data. This update needs to occur immediately and to be finished > before the > libvirt nodedev event is issued to keep the API usage reliable. > […snip…] > src/test/test_driver.c | 11 +- > src/util/virmdev.c | 20 ++ > src/libvirt_private.syms | 2 + > 8 files changed, 398 insertions(+), 198 deletions(-) > > > base-commit: c38720b337f74337ec94c0fe2e97a7c2c57188ae > -- > 2.34.1 > ___ > Devel mailing list -- devel@lists.libvirt.org > To unsubscribe send an email to devel-le...@lists.libvirt.org Polite ping. Should I rebase or how do we want to proceed with this series? >>> >>> >>> My apologies. I thought I remembered reviewing this series and, >>> looking back at the archives, it seems that I did. Do you just need >>> somebody to push it upstream? >>> >>> Thanks, >>> Jonathon >>> >> >> Hi Jonathon, >> I think Marc is looking for someone to push his series. >> Seems like patches 2, 3 and 20 are missing your rb or did Marc forget to >> add them? >> @Marc, could you send a rebased v3 since it seems the series no longer >> cleanly applies on master. >> > > For the v2 patch series I acked the cover letter for the series. So I'll > just rebase and add my r-b to all of them and push. Sorry for the > delay. Thanks and no problems. > > Jonathon > -- Kind regards / Beste Grüße Marc Hartmayer IBM Deutschland Research & Development GmbH Vorsitzender des Aufsichtsrats: Wolfgang Wendt Geschäftsführung: David Faller Sitz der Gesellschaft: Böblingen Registergericht: Amtsgericht Stuttgart, HRB 243294
Re: [PATCH v2 00/20] node_dev_udev: use workerpool and improve nodedev events
On 6/18/24 6:33 AM, Boris Fiuczynski wrote: On 6/13/24 11:14 PM, Jonathon Jongsma wrote: On 6/13/24 6:13 AM, Marc Hartmayer wrote: On Tue, Apr 23, 2024 at 08:08 PM +0200, Marc Hartmayer wrote: When an udev event occurs for a mediated device (mdev) the mdev config data requires an update via mdevctl as the udev event does not contain all config data. This update needs to occur immediately and to be finished before the libvirt nodedev event is issued to keep the API usage reliable. […snip…] src/test/test_driver.c | 11 +- src/util/virmdev.c | 20 ++ src/libvirt_private.syms | 2 + 8 files changed, 398 insertions(+), 198 deletions(-) base-commit: c38720b337f74337ec94c0fe2e97a7c2c57188ae -- 2.34.1 ___ Devel mailing list -- devel@lists.libvirt.org To unsubscribe send an email to devel-le...@lists.libvirt.org Polite ping. Should I rebase or how do we want to proceed with this series? My apologies. I thought I remembered reviewing this series and, looking back at the archives, it seems that I did. Do you just need somebody to push it upstream? Thanks, Jonathon Hi Jonathon, I think Marc is looking for someone to push his series. Seems like patches 2, 3 and 20 are missing your rb or did Marc forget to add them? @Marc, could you send a rebased v3 since it seems the series no longer cleanly applies on master. For the v2 patch series I acked the cover letter for the series. So I'll just rebase and add my r-b to all of them and push. Sorry for the delay. Jonathon
Re: [PATCH v2 00/20] node_dev_udev: use workerpool and improve nodedev events
On 6/13/24 11:14 PM, Jonathon Jongsma wrote: On 6/13/24 6:13 AM, Marc Hartmayer wrote: On Tue, Apr 23, 2024 at 08:08 PM +0200, Marc Hartmayer wrote: When an udev event occurs for a mediated device (mdev) the mdev config data requires an update via mdevctl as the udev event does not contain all config data. This update needs to occur immediately and to be finished before the libvirt nodedev event is issued to keep the API usage reliable. […snip…] src/test/test_driver.c | 11 +- src/util/virmdev.c | 20 ++ src/libvirt_private.syms | 2 + 8 files changed, 398 insertions(+), 198 deletions(-) base-commit: c38720b337f74337ec94c0fe2e97a7c2c57188ae -- 2.34.1 ___ Devel mailing list -- devel@lists.libvirt.org To unsubscribe send an email to devel-le...@lists.libvirt.org Polite ping. Should I rebase or how do we want to proceed with this series? My apologies. I thought I remembered reviewing this series and, looking back at the archives, it seems that I did. Do you just need somebody to push it upstream? Thanks, Jonathon Hi Jonathon, I think Marc is looking for someone to push his series. Seems like patches 2, 3 and 20 are missing your rb or did Marc forget to add them? @Marc, could you send a rebased v3 since it seems the series no longer cleanly applies on master. -- Mit freundlichen Grüßen/Kind regards Boris Fiuczynski IBM Deutschland Research & Development GmbH Vorsitzender des Aufsichtsrats: Wolfgang Wendt Geschäftsführung: David Faller Sitz der Gesellschaft: Böblingen Registergericht: Amtsgericht Stuttgart, HRB 243294
Re: [PATCH v2 00/20] node_dev_udev: use workerpool and improve nodedev events
On 6/13/24 6:13 AM, Marc Hartmayer wrote: On Tue, Apr 23, 2024 at 08:08 PM +0200, Marc Hartmayer wrote: When an udev event occurs for a mediated device (mdev) the mdev config data requires an update via mdevctl as the udev event does not contain all config data. This update needs to occur immediately and to be finished before the libvirt nodedev event is issued to keep the API usage reliable. […snip…] src/test/test_driver.c | 11 +- src/util/virmdev.c | 20 ++ src/libvirt_private.syms | 2 + 8 files changed, 398 insertions(+), 198 deletions(-) base-commit: c38720b337f74337ec94c0fe2e97a7c2c57188ae -- 2.34.1 ___ Devel mailing list -- devel@lists.libvirt.org To unsubscribe send an email to devel-le...@lists.libvirt.org Polite ping. Should I rebase or how do we want to proceed with this series? My apologies. I thought I remembered reviewing this series and, looking back at the archives, it seems that I did. Do you just need somebody to push it upstream? Thanks, Jonathon
Re: [PATCH v2 00/20] node_dev_udev: use workerpool and improve nodedev events
On Tue, Apr 23, 2024 at 08:08 PM +0200, Marc Hartmayer wrote: > When an udev event occurs for a mediated device (mdev) the mdev config data > requires an update via mdevctl as the udev event does not contain all config > data. This update needs to occur immediately and to be finished before the > libvirt nodedev event is issued to keep the API usage reliable. > […snip…] > src/test/test_driver.c | 11 +- > src/util/virmdev.c | 20 ++ > src/libvirt_private.syms | 2 + > 8 files changed, 398 insertions(+), 198 deletions(-) > > > base-commit: c38720b337f74337ec94c0fe2e97a7c2c57188ae > -- > 2.34.1 > ___ > Devel mailing list -- devel@lists.libvirt.org > To unsubscribe send an email to devel-le...@lists.libvirt.org Polite ping. Should I rebase or how do we want to proceed with this series? -- Kind regards / Beste Grüße Marc Hartmayer IBM Deutschland Research & Development GmbH Vorsitzender des Aufsichtsrats: Wolfgang Wendt Geschäftsführung: David Faller Sitz der Gesellschaft: Böblingen Registergericht: Amtsgericht Stuttgart, HRB 243294