Comment 7 for bug 1046391

Revision history for this message
Phillip Susi (psusi) wrote :

I'm not sure why that commit was put in as it appears to be redundant. The old code punted to device.c, which had a 1 MiB default already.

From ped_device_get_optimum_alignment:

                        /* Align to a grain of 1MiB (like vista / win7) */
                        align = ped_alignment_new(0,
                                                  1048576 / dev->sector_size);

In fact, I have had a patch pending for some time now to correct that commit, which wrongly uses a 1 MiB alignment when the underlying device requests a larger alignment that is not an even power of two.