similar to: [nbdkit PATCH] nbd: Properly mop up stranded requests

In preparation for allowing interleaved response, refactor the nbd forwarder so that writes are still done from the thread handling the original request from the client, but all reads are done by a dedicated reader thread. Control between the two flags is gated by a mutex for storing the transaction information, coupled with a pipe for the reader thread to send the final status back to the

Now that we properly clean up 'trans' in the reader thread, we must not dereference 'trans' from the write thread at any point after trans has been added to the list unless we have grabbed it back off the list ourselves, or we risk an occasional use-after-free or even double free that valgrind can detect. Reported-by: Richard W.M. Jones <rjones@redhat.com> Fixes:

Time to enhance the nbd plugin to request structured replies from the server. For now, deal only with structured reads. The server can now return sparse reads, even though we need nbdkit version 3 before we can in turn return sparse reads back to the client. In general, we have to assume the server is malicious, and so we must sanity check that it sends replies we expect. Thus, we have a choice

With this, I am finally able to get the nbd plugin to do out-of-order responses to the client. Once this series goes in, we should be ready for Rich to cut a release. Eric Blake (4): nbd: Split reading into separate thread nbd: Protect writes with mutex nbd: Enable parallel handling tests: Test parallel nbd behavior plugins/nbd/nbd.c | 217

I'm still working on the interleaving (and Rich reminded me on IRC that we still don't have THREAD_MODEL_PARALLEL working anywhere yet, anyways). Since nbdkit doesn't really have a parallel plugin yet, my testing on that front will have to use qemu-nbd as the original server, as well as qemu-io as the driver (qemu-io's aio_read and aio_write commands can be used to trigger

I've rebuilt libmateweather for EL7 with the aforementioned patch and it seems to have fixed the issue. Feel free to use it until EPEL package the fix. http://li.nux.ro/download/nux//tmp/libmateweather7/ -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro ----- Original Message ----- > From: "Nux!" <nux at li.nux.ro> > To: "CentOS mailing

When converting from a single transaction to a linked list, I forgot to free the storage for each member of the list. Reported-by: Richard W.M. Jones <rjones@redhat.com> Fixes: 7f5bb9bf13f041ea7702bda557d9dd668bc3423a Signed-off-by: Eric Blake <eblake@redhat.com> --- plugins/nbd/nbd.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/plugins/nbd/nbd.c

On Thu, Aug 25, 2016 at 08:33:46AM +0100, Nux! wrote: > I've rebuilt libmateweather for EL7 with the aforementioned patch and it seems to have fixed the issue. > Feel free to use it until EPEL package the fix. > > http://li.nux.ro/download/nux//tmp/libmateweather7/ thanks Nux! I installed it and it now gets current conditions just fine. but I tried several different locations

Hm, this is bad. I was relying on this feature heavily. Let's see if there is a fix. I don't imagine RH will care about this on EL6, but maybe MATE guys could help. -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro ----- Original Message ----- > From: "isdtor" <isdtor at gmail.com> > To: "CentOS mailing list" <centos at

I've patched libgweather with the same MATE patch and it seems to be working fine. Give it a try http://li.nux.ro/download/nux/tmp/libgweather6/ -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro ----- Original Message ----- > From: "isdtor" <isdtor at gmail.com> > To: "CentOS mailing list" <centos at centos.org> > Sent:

When we first used libnbd (nbdkit 1.14), it was brand new, so we left in the ability to build a limited nbd plugin without the use of libnbd for comparison. But now that libnbd has had more time under its belt, and we have learned that our nbd-standalone code is not getting much testing, it's time to follow through with our plan to finish off the old code, and now build the nbd plugin only

Fred, Check here https://git.gnome.org/browse/libgweather/tree/libgweather/weather-metar.c?h=gnome-2-30 -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro ----- Original Message ----- > From: "Fred Smith" <fredex at fcshome.stoneham.ma.us> > To: centos at centos.org > Sent: Thursday, 25 August, 2016 02:17:27 > Subject: Re: [CentOS] Gnome weather

I ran some local testing on my Fedora 29 system via: $ ./nbdkit memory 1m $ for i in `seq 10`; do ./nbdkit -U - --filter=stats nbd \ statsappend=true statsfile=$file hostname=localhost port=10809 \ --run '~/libnbd/examples/threaded-reads-and-writes $unixsocket' done Pre-patch, the runs averaged 1.266s, 1.30E+08 bits/s Post-patch, the runs averaged 1.154s, 1.42E+08

isdtor wrote: > The Gnome weather report applet has ceased working on CentOS5/6. > http://weather.noaa.gov/ says "This service is no longer available". Are > there any good alternatives? I liked the applet as it allowed the use of a > custom radar map. > > As I just discovered, the weather functionality of the CentOS6 Gnome clock > applet was using the same service.

On Wed, 24 Aug 2016, m.roth at 5-cent.us wrote: > isdtor wrote: >> The Gnome weather report applet has ceased working on CentOS5/6. >> http://weather.noaa.gov/ says "This service is no longer available". Are >> there any good alternatives? I liked the applet as it allowed the use of a >> custom radar map. >> >> As I just discovered, the weather

Nux! writes: > Hm, this is bad. I was relying on this feature heavily. > Let's see if there is a fix. I don't imagine RH will care about this on EL6, but maybe MATE guys could help. The code for the respective applets is quite different. Mate uses weather.com, but I bet the API is different and a fix requires actual coding.

On Wed, August 24, 2016 10:46 am, m.roth at 5-cent.us wrote: > isdtor wrote: >> The Gnome weather report applet has ceased working on CentOS5/6. >> http://weather.noaa.gov/ says "This service is no longer available". Are >> there any good alternatives? I liked the applet as it allowed the use of >> a >> custom radar map. >> >> As I just

On Wed, 24 Aug 2016 16:13:51 +0100 isdtor wrote: > Nux! writes: > > Hm, this is bad. I was relying on this feature heavily. > > Let's see if there is a fix. I don't imagine RH will care about this on > > EL6, but maybe MATE guys could help. > > The code for the respective applets is quite different. Mate uses > weather.com, but I bet the API is different

On 8/24/2016 8:46 AM, m.roth at 5-cent.us wrote: > Had you gone to noaa.gov, and clicked on climate, it appears that they > want to now charge you $3/yr for the service. I'm getting DNS not found errors from anything in NOAA.GOV. anyways, 'climate' refers to historical averages by location/month/week/date. presumably the API involved here was for current weather conditions

On 8/24/2016 2:44 PM, John R Pierce wrote: > On 8/24/2016 8:46 AM, m.roth at 5-cent.us wrote: >> Had you gone to noaa.gov, and clicked on climate, it appears that they >> want to now charge you $3/yr for the service. > > I'm getting DNS not found errors from anything in NOAA.GOV. flushed my dns cache, now its working, whack. > > anyways, 'climate' refers to