Re: [RFH] Partition table recovery

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Rene Herman wrote:
On 07/23/2007 10:41 AM, Jan-Benedict Glaw wrote:

As multibyte on-disk variables, these will need LE/BE conversion.

Indeed, thanks -- has been updated in the version that is attached.

Also fixes a bug that snuck in (failed to add offset to entry->start).

struct entry {
    uint8_t flags;
    uint8_t type;
    uint16_t __1;
    uint64_t start;
    uint32_t size;

Dito.

This can stay for now. The partition table backup would indeed need some defined byte-order but it might be "whatever order the filesystem it's backed up onto uses". Since it's not directly written to any filesystem for now, host order will do currently.

Looks like a useful program, but you'd definively fix the LE/BE issues.
If you do that, it'll be able to even run on BE machines, too.

I might disagree with the usefulness. I believe that preferably a backup should be made with help from the kernel (if only by walking sysfs) to avoid things getting out of sync between kernel and backup program.

(this program largely does the same as the kernel does but even now there's already a difference in so far that I didn't bother to de-garbage the 3rd and 4th entries in the second level extendeds).

How can I politely say this code really needs comments?
To quote the late R. W. Benway, "If it was hard to write it should be hard to understand."
(regarding code in FORTRAN II on punched cards, ca 1965)
Rene.

--
bill davidsen <davidsen@xxxxxxx>
 CTO TMR Associates, Inc
 Doing interesting things with small computers since 1979

-
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [Samba]     [Device Mapper]     [CEPH Development]
  Powered by Linux