diff options
author | Neil Brown <neilb@suse.de> | 2006-12-14 07:33:14 +0100 |
---|---|---|
committer | Neil Brown <neilb@suse.de> | 2006-12-14 07:33:14 +0100 |
commit | 350f29f90d1f6bb3ddfafea368327911f9e8b27c (patch) | |
tree | eada62e3fadcadb4c3de3df72098687b0bd625f8 /super1.c | |
parent | Support --uuid= with --create to choose your own UUID. (diff) | |
download | mdadm-350f29f90d1f6bb3ddfafea368327911f9e8b27c.tar.xz mdadm-350f29f90d1f6bb3ddfafea368327911f9e8b27c.zip |
Centralise code for copying uuid
Rather than opencoding the byteswap all the time.
Diffstat (limited to 'super1.c')
-rw-r--r-- | super1.c | 30 |
1 files changed, 4 insertions, 26 deletions
@@ -538,18 +538,7 @@ static int update_super1(struct mdinfo *info, void *sbv, char *update, sb->resync_offset = 0ULL; } if (strcmp(update, "uuid") == 0) { - if (super1.swapuuid) { - unsigned char *ac = (unsigned char *)sb->set_uuid; - unsigned char *bc = (unsigned char *)info->uuid; - int i; - for (i=0; i<16; i+= 4) { - ac[i+0] = bc[i+3]; - ac[i+1] = bc[i+2]; - ac[i+2] = bc[i+1]; - ac[i+3] = bc[i+0]; - } - } else - memcpy(sb->set_uuid, info->uuid, 16); + copy_uuid(sb->set_uuid, info->uuid, super1.swapuuid); if (__le32_to_cpu(sb->feature_map)&MD_FEATURE_BITMAP_OFFSET) { struct bitmap_super_s *bm; @@ -627,20 +616,9 @@ static int init_super1(struct supertype *st, void **sbp, mdu_array_info_t *info, sb->feature_map = 0; sb->pad0 = 0; - if (uuid) { - if (super1.swapuuid) { - unsigned char *ac = (unsigned char *)sb->set_uuid; - unsigned char *bc = (unsigned char *)uuid; - int i; - for (i=0; i<16; i+= 4) { - ac[i+0] = bc[i+3]; - ac[i+1] = bc[i+2]; - ac[i+2] = bc[i+1]; - ac[i+3] = bc[i+0]; - } - } else - memcpy(sb->set_uuid, uuid, 16); - } else { + if (uuid) + copy_uuid(sb->set_uuid, uuid, super1.swapuuid); + else { if ((rfd = open("/dev/urandom", O_RDONLY)) < 0 || read(rfd, sb->set_uuid, 16) != 16) { *(__u32*)(sb->set_uuid) = random(); |