Opened 8 years ago

Closed 8 years ago

#7507 closed enhancement (fixed)

Do not overwrite the disk ID

Reported by: starsseed Owned by: bonefish
Priority: normal Milestone: R1
Component: Partitioning Systems/Intel Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: yes Platform: All

Description

Do not overwrite the disk ID in the MBR since Linux, Windows (and some BIOSes) may need it.

Attachments (1)

PreserveMBRdiskID.patch (1.7 KB) - added by starsseed 8 years ago.
coding style fixes

Download all attachments as: .zip

Change History (4)

comment:1 Changed 8 years ago by starsseed

Has a Patch: set

comment:2 Changed 8 years ago by bonefish

Looks OK save for the introduction of coding style issues. BTW, there are STATIC_ASSERT() macros defined in the kernel private <debug.h> and in the public <Debug.h>.

Changed 8 years ago by starsseed

Attachment: PreserveMBRdiskID.patch added

coding style fixes

comment:3 Changed 8 years ago by bonefish

Resolution: fixed
Status: newclosed

Applied in hrev41565 with the following coding style fixes:

  • debug.h is private kernel header and thus does not belong in the public C headers section.
  • Joined the status_t error declaration with its initialization.
  • Space after "//".
  • Spaces around non-unary operators.
Note: See TracTickets for help on using tickets.