lkml.org 
[lkml]   [2005]   [Dec]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] fs: remove s_old_blocksize from struct super_block
From
Date
Hi Anton,

On Fri, 2 Dec 2005, Pekka Enberg wrote:
> > The s_old_blocksize field of struct super_block is only used as a temporary
> > variable in get_sb_bdev(). This patch changes the function to use a local
> > variable instead so we can kill the field from struct super_block.

On Sat, 2005-12-03 at 11:02 +0000, Anton Altaparmakov wrote:
> s_old_blocksize used to be used to restore the blocksize after the
> filesystem had failed to mount or had unmounted. Not restoring this leads
> to all sorts of problems since the blocksize may be set for example to 4k
> but some userspace app may need it to be set to 1k or whatever. There
> used to be applications that failed which is why s_old_blocksize was
> introduced and it used to restore the blocksize.
>
> I have no idea why/when the restoring has been removed but chances are the
> removal was wrong. Now every file system will need to restore the
> blocksize itself (as it used to be before s_old_blocksize and blocksize
> restoral was introduced). Except whoever removed the restoration failed
> to fix up all file systems. )-:

It was removed in this commit, I think:

http://www.kernel.org/git/?p=linux/kernel/git/torvalds/old-2.6-bkcvs.git;a=commit;h=294c42046966e927ef86c0d4ce71cff32d9b458c

Pekka

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-12-03 12:40    [W:0.052 / U:0.268 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site