On Mon, Oct 09, 2023 at 05:08:21PM +0200, Daniel Kiper wrote: > On Fri, Oct 06, 2023 at 08:10:11PM +0300, ValdikSS via Grub-devel wrote: > > GRUB2 limits number of sectors read at once in LBA mode to the > > number reported in CHS disk geometry by BIOS. > > This is unnecessary, as IBM/MS INT13 Extensions does not have > > such limit and it is safe to read up to 127 sectors in a single call. > > > > This fixes greatly increased boot times on WYSE C10LE x86 thin > > client with Phoenix bios. > > > > See: > > https://lore.kernel.org/grub-devel/d42a11fa-2a59-b5e7-08b1-d2c60444b...@valdikss.org.ru/ > > > > ValdikSS (2): > > disk: read up to 63 sectors in LBA mode > > disk: increase sector size up to 127 for LBA reads > > After reading whole discussion about this patch set I think it is quite > safe to take both patches into the release. If we realy want to be on the > safe side we can drop the second one and merge it after the release.
ValdikSS, Vladimir and I agreed we will take patch #1 into the release. The patch #2 will be merged after the release. Though please update both patches and post v3 now. Thank you for debugging and fixing this issue! Daniel _______________________________________________ Grub-devel mailing list Grub-devel@gnu.org https://lists.gnu.org/mailman/listinfo/grub-devel
