[dm-devel] [PATCH 00/29] SLES resync

Hannes Reinecke hare at suse.de
Tue Jul 16 07:10:39 UTC 2013


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 at suse.de			      +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)




More information about the dm-devel mailing list