Thread (66 messages) 66 messages, 5 authors, 2018-07-03

Re: Subject: [PATCH RFC] block: fix Amiga RDB partition support for disks >= 2 TB

From: Martin Steigerwald <hidden>
Date: 2018-06-27 08:24:30

Thanks a lot again for your patch.

schmitzmic@gmail.com - 27.06.18, 03:24:
+               if (start_sect > INT_MAX || nr_sects > INT_MAX
+                       || (start_sect + nr_sects) > INT_MAX) {
+                       pr_err("%s: Warning: RDB partition
overflow!\n", +                               bdevname(state->bdev,
I=B4d word this:

Warning: RDB partition 32-bit overflow

AmigaOS developers can do 64 bit math on a 32 bit operating system. Just=20
like Linux can.
b));
+                       pr_err("%s: start 0x%llX size 0x%llX\n",
+                               bdevname(state->bdev, b), start_sect,
+                               nr_sects);
+                       pr_err("%s: partition incompatible with 32 bit
OS\n", +                               bdevname(state->bdev, b));
+               }
And as stated in my other reply to the patch:

partition needs 64 bit disk device support in AmigaOS or AmigaOS like=20
operating systems (NSD64, TD64 or SCSI direct)

see my other reply to the patch and my other mails in the
"Re: moving affs + RDB partition support to staging?" thread as to why.=20
And for references.

Thanks,
=2D-=20
Martin
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help