On 07/15/2013 04:12 PM, Hannes Reinecke wrote: > On 07/15/2013 03:53 PM, Sebastian Riemer wrote: >> On 15.07.2013 15:00, Hannes Reinecke wrote: >> [...] >>> - When using libudev to read attribute variables we'd be getting >>> an automated caching from libudev. Which is nice, unless the >>> data changed in between. So I'd have to go back an roll our >>> own accessors, skipping libudev here. >> [...] >> >> Hi Hannes, >> >> nice, that's what I've already reported. Could you point me to the fix >> commit(s) for this please? >> > > Gnaa. Merge error. > For some reason the patch _should_ be 0014, but then it contains > random merge errors. > > I'll be posting an additional patch for this. > Christophe, I'll be probably have to rebase the branch, so wait a > bit with pulling. > Sigh. Sebastian, you've got me confused. The patch you're talking about is 0004 (Increase dev_loss_tmo prior to fast_io_fail), which contains the code for reading from sysfs. 0014 is only related to that as it uses it in some more places. Only the 'state' attribute itself wasn't modified, so for that we'd still be using libudev. So my patchset was in fact okay, but your point is valid, too. Hence I've done a new patchset, identical to the first one, but with one additional patch reading the 'state' attribute directly from sysfs. Cheers, Hannes -- Dr. Hannes Reinecke zSeries & Storage hare@xxxxxxx +49 911 74053 688 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: J. Hawn, J. Guild, F. Imendörffer, HRB 16746 (AG Nürnberg) -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel
