Re: [PATCH] Fix whitespace issues in multipath-tools

[Hannes Reinecke <hare@suse.de>]

Hi Christophe,

Christophe Varoqui wrote:
> Le mercredi 23 juillet 2008 à 10:02 +0200, Hannes Reinecke a écrit :
>> Hi Christophe,
>>
>> one day I got fed up with all of those whitespace issues.
>> Here's a patch to fix up (most) of them.
>> So no guarantee for completeness, but a good step in
>> that direction.
>>
> Sure, committed with 2 left-overs :
> 
> 1/ in uxsock.c, the patch unduly removed addr declaration
> 
Ah, that's where it went. I was merging / re-diffing my SLES
patch queue and suddenly I got this missing 'addr' declaration.
But then I got too lazy to chase it down :-)

> 2/ in config.c, there was traces of something I smelling like a good
> fix, but not fully fonctionnal (use of alloc_keyword where not
> available). As this fix should not be merged in a format patch anyway,
> I'll let you juge if you want to submit the complete fix separetely.
> 
Yes, another bugger-up on my side.
This is actually part of my 'multipath -t' patch,
which allows you to dump the internal hwtable to stdout.
I still prefer that approach over the 'multipathd -k xxx' option,
as this does _not_ require to have multipathing running, hence
you can even modify / create a multipath.conf in dry-run mode
and you wouldn't risk any service disruption.
Which you inevitably would have if you have to fire up multipathd
and have him claim all unused devices.

I'll see to prepare the patch.

Thanks for merging.

Cheers,

Hannes
-- 
Dr. Hannes Reinecke		      zSeries & Storage
hare@suse.de			      +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: Markus Rex, HRB 16746 (AG Nürnberg)