Re: [PATCH] nested partitions

[Vladimir 'phcoder' Serbinenko]

>> +      grub_device_close (dev);
>
> I don't think we should allow chainload at all in partitions other than msdos
> ones.  There's no existing use of this for other labels "in the wild", and we
> shouldn't encourage it to appear IMO.
Actually it's something which happens with GPT. Suppose you have an OS
on your GPT which has no multiboot support. In this case you use
gptsync to create a fake msdos partition map and boot your OS. But
probably in this case this code should create a fake partition entry
at %es:%di too. Creating a fake entry may also be beneficial for
extended partitions and may make some targets which usually doesn't
support booting from extended partition to boot however
>
> If we have partitions and the "pc_partition_map" match fails, I would just
> abort.
>
> --
> Robert Millan
>
>  The DRM opt-in fallacy: "Your data belongs to us. We will decide when (and
>  how) you may access your data; but nobody's threatening your freedom: we
>  still allow you to remove your data and not access it at all."
>
>
> _______________________________________________
> Grub-devel mailing list
> address@hidden
> http://lists.gnu.org/mailman/listinfo/grub-devel
>



-- 
Regards
Vladimir 'phcoder' Serbinenko

Personal git repository: http://repo.or.cz/w/grub2/phcoder.git