[linux-lvm] lvm-1.0.1 not working with DAC960/2.2.20
Adrian Phillips
a.phillips at dnmi.no
Wed Nov 28 00:37:02 UTC 2001
>>>>> "Andreas" == Andreas Dilger <adilger at turbolabs.com> writes:
<snip>
Andreas> Note that this will break other things, because you also
Andreas> need to update the LVM_DEVICE_TYPE_<foo> defines when you
Andreas> change this array (it was Joe or Alistair that wrote it
Andreas> this way, not me). It would also break to add a new enum
Andreas> value for DAC960, because it means we need to look for
Andreas> two values elsewhere in the code
Andreas> (e.g. lvm_partition_count).
Thanks for noticing this. My patch "seemed" to work but I have almost
no experience with LVM code so missed the further reaching
consequences of my change.
Andreas> I just checked in a fix, which cleans up this code a bit
Andreas> and makes it impossible to get wrong in the future.
It would be useful if people could post patches to this list so that
individual changes could be tested by individuals on this list without
checking out the whole CVS which may include other patches which may
possibly break things.
Sincerely,
Adrian Phillips
--
Your mouse has moved.
Windows NT must be restarted for the change to take effect.
Reboot now? [OK]
More information about the linux-lvm
mailing list