From: Martin Steigerwald <martin@lichtvoll.de>
To: Michael Schmitz <schmitzmic@gmail.com>
Cc: linux-block@vger.kernel.org, axboe@kernel.dk,
linux-m68k@vger.kernel.org, geert@linux-m68k.org
Subject: Re: [PATCH v5 0/2] Amiga RDB partition support fixes
Date: Fri, 12 Oct 2018 10:17:58 +0200 [thread overview]
Message-ID: <1886580.oI6ZJUcLCv@merkaba> (raw)
In-Reply-To: <1539304018-11786-1-git-send-email-schmitzmic@gmail.com>
Michael Schmitz - 12.10.18, 02:26:
> yet another new version of the Amiga RDB partition table patch.
> I've split off the part fixing the incorrect use of signed int
> for partition start address and size as separate patch. This change
> should be incontroversial (I hope). It does fix the bug that Martin
> Steigerwald reported six years ago for a 2 TB disk. I have tested this
> patch with the identical RDB partition data that Martin saw the bug
> with. I have incorporated Andreas' feedback on proper use of casts to
> avoid integer overflow in this version.
Just as a note for all the AmigaOS users out there: Hyperion
Entertainment released AmigaOS 3.1.4 for various Amiga models. This
should fix all (known) issues with large disks for good directly in the
Kickstart ROM instead of via SetPatch updating the ROM by replacing
modules or by using NSDPatch approach. Olaf Barthel was one of the
developers involved with the update. Digital downloads only so far,
physical releases with floppy disks and ROM chips are announced.
This would be another reason to support large disks in Linux as well.
Thank you, Michael.
(Omitting link as to not too directly advertise proprietary products
here, it is easy enough to find.)
Thanks,
--
Martin
next prev parent reply other threads:[~2018-10-12 8:17 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-10-12 0:26 [PATCH v5 0/2] Amiga RDB partition support fixes Michael Schmitz
2018-10-12 0:26 ` [PATCH v5 1/2] block: fix signed int overflow in Amiga partition support Michael Schmitz
2018-10-12 8:59 ` Geert Uytterhoeven
2018-10-13 2:46 ` Michael Schmitz
2018-10-13 7:04 ` Geert Uytterhoeven
2018-10-12 0:26 ` [PATCH v5 2/2] block: add overflow checks for " Michael Schmitz
2018-10-12 8:54 ` Geert Uytterhoeven
2018-10-13 2:23 ` Michael Schmitz
2018-10-13 7:02 ` Geert Uytterhoeven
2018-10-13 8:27 ` Michael Schmitz
2018-10-12 8:17 ` Martin Steigerwald [this message]
2018-10-12 8:41 ` [PATCH v5 0/2] Amiga RDB partition support fixes John Paul Adrian Glaubitz
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1886580.oI6ZJUcLCv@merkaba \
--to=martin@lichtvoll.de \
--cc=axboe@kernel.dk \
--cc=geert@linux-m68k.org \
--cc=linux-block@vger.kernel.org \
--cc=linux-m68k@vger.kernel.org \
--cc=schmitzmic@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox