From: Al Viro <viro@ZenIV.linux.org.uk>
To: Guenter Roeck <linux@roeck-us.net>
Cc: Evgeniy Dushistov <dushistov@mail.ru>,
Mark Brown <broonie@kernel.org>,
linux-kernel@vger.kernel.org, Arnd Bergmann <arnd@arndb.de>
Subject: Re: [PATCH] ufs: Fix build errors on 32 bit machines
Date: Sat, 17 Jun 2017 19:51:40 +0100 [thread overview]
Message-ID: <20170617185140.GW31671@ZenIV.linux.org.uk> (raw)
In-Reply-To: <20170617182328.GV31671@ZenIV.linux.org.uk>
On Sat, Jun 17, 2017 at 07:23:28PM +0100, Al Viro wrote:
> On Sat, Jun 17, 2017 at 10:35:13AM -0700, Guenter Roeck wrote:
> > Various 32 builds fail with error messages such as
> >
> > ERROR: "__udivdi3" [fs/ufs/ufs.ko] undefined!
> >
> > due to a variable type change from 32 bit to 64 bit.
>
> Actually, that's not the only problem in that place. The breakage
> came in 2.4.14.7; the critical part was this:
[snip]
Some background: there are two possible allocation policies for the
bad case of tail unpacking. If a small (less than 192K, on typical
ufs1) file has the last block packed along with those of other files
and we can't just expand that tail in place, we need to find a place
for longer tail somewhere and copy the old data over there.
First policy: try and put it into a block already containing tail(s).
Kinder on space, higher odds of having to do relocation the next time
that tail needs to grow. That's what OPTSPACE is.
Second policy: pick a free block and put the expanded tail there.
Better chance of being able to expand in place when/if the tail needs
to grow again, harsher on space. That's OPTTIME.
The choice between those is controlled by the amount of space left
in partially filled blocks. If it's high, we need to go for OPTSPACE,
if it's low - OPTTIME. Cutoff values depend upon the amount of
space reserved for root; for 5% (default) it's "go for OPTSPACE when
more than 3% of total space are taken by free space in partially
filled blocks, go for OPTTIME when fragmentation is less than 2.5%,
stay with the previous state when it's between 2.5% and 3%".
(Christoph's?) typo in 2.4.14.7 has broken the switch from OPTTIME to
OPTSPACE; once in OPTTIME it's stuck in OPTTIME.
next prev parent reply other threads:[~2017-06-17 18:51 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-17 17:35 [PATCH] ufs: Fix build errors on 32 bit machines Guenter Roeck
2017-06-17 18:23 ` Al Viro
2017-06-17 18:51 ` Al Viro [this message]
2017-06-18 9:30 ` Guenter Roeck
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=20170617185140.GW31671@ZenIV.linux.org.uk \
--to=viro@zeniv.linux.org.uk \
--cc=arnd@arndb.de \
--cc=broonie@kernel.org \
--cc=dushistov@mail.ru \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@roeck-us.net \
/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