[Bug 592100] New: reiserfs file size limit is 2 TiB but expected to be 8 TiB
http://bugzilla.novell.com/show_bug.cgi?id=592100 http://bugzilla.novell.com/show_bug.cgi?id=592100#c0 Summary: reiserfs file size limit is 2 TiB but expected to be 8 TiB Classification: openSUSE Product: openSUSE 11.3 Version: Factory Platform: All OS/Version: openSUSE 11.3 Status: NEW Severity: Normal Priority: P5 - None Component: Kernel AssignedTo: jeffm@novell.com ReportedBy: jeffm@novell.com QAContact: qa@suse.de Found By: Development Blocker: --- The reiserfs file size is expected to be 8 TiB as documented at http://en.wikipedia.org/wiki/Comparison_of_file_systems but is limited to 2 TiB in practice. Actual results: % dd if=/dev/zero of=fs.img bs=1M count=1 seek=8388607 dd: truncating at 8796091973632 bytes in output file `fs.img': File too large %ls fs.img -rw-r--r-- 1 root root 0 2010-03-29 18:24 fs.img Expected results: Successfully create the file. -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=592100 http://bugzilla.novell.com/show_bug.cgi?id=592100#c1 --- Comment #1 from Jeff Mahoney <jeffm@novell.com> 2010-03-29 22:27:49 UTC --- Created an attachment (id=351258) --> (http://bugzilla.novell.com/attachment.cgi?id=351258) [PATCH] reiserfs: Remove 2 TB file size limit In its early life, reiserfs had an evolving s_max_bytes. It started out at 4 GB, then was raised to MAX_LFS_FILESIZE, then dropped to 2 TiB when it was observed that struct stat only had a 32-bit st_blocks field. Since then, both the kernel and glibc have evolved as well and now both support 64-bit st_blocks. Applications that can't deal with these ranges are assumed to be "legacy" or "broken." File systems now routinely support file sizes much larger than can be represented by 2^32 * 512. But we never revisited that limitation. ReiserFS has always been able to support larger file sizes (up to 16 TiB, in fact), but the s_max_bytes limitation has prevented that. This patch adds a max_file_offset helper to set s_max_bytes to a more appropriate value. I noticed that XFS adjusts the limit based on the CPU but I'd prefer to err on the side of compatibility and place the limit at the smaller of the 32-bit MAX_LFS_FILESIZE and the maximum supported by the file system. At a 4k block size, this is conveniently also the advertised maximum file size of reiserfs. -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=592100 http://bugzilla.novell.com/show_bug.cgi?id=592100#c6 --- Comment #6 from Jeff Mahoney <jeffm@novell.com> 2010-04-09 16:29:54 UTC --- Created an attachment (id=353508) --> (http://bugzilla.novell.com/attachment.cgi?id=353508) [PATCH #2] reiserfs: Remove 2 TB file size limit This version appropriately casts sb->s_blocksize to avoid wrapping on 32-bit systems. The previous patch would result in an incorrect value for sb->s_max_bytes, which would cause anything over 0 bytes to fail. -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=592100 http://bugzilla.novell.com/show_bug.cgi?id=592100#c Jeff Mahoney <jeffm@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #351258|0 |1 is obsolete| | -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=592100 https://bugzilla.novell.com/show_bug.cgi?id=592100#c7 Jeff Mahoney <jeffm@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED --- Comment #7 from Jeff Mahoney <jeffm@novell.com> 2011-01-18 15:28:21 UTC --- This patch is wrong and has problems with sd_blocks. A real version of the patch will need to be approved upstream first since it involves a format change. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=592100 https://bugzilla.novell.com/show_bug.cgi?id=592100#c8 Jeff Mahoney <jeffm@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution| |WONTFIX --- Comment #8 from Jeff Mahoney <jeffm@suse.com> 2011-11-11 14:16:39 UTC --- Closing as WONTFIX. We'll just need to update the release notes. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply＠novell.com