linux-toradex.git/fs/ufs, branch v4.4.28

fix ufs write vs readpage race when writing into a hole

2015-09-09T17:43:12+00:00

Followup to the UFS series - with the way we clear the new blocks (via
buffer cache, possibly on more than a page worth of file) we really
should not insert a reference to new block into inode block tree until
after we'd cleared it.

Signed-off-by: Al Viro 
Signed-off-by: Linus Torvalds

ufs_inode_get{frag,block}(): get rid of 'phys' argument

2015-07-06T21:40:05+00:00

Just pass NULL as locked_page in case of first block in the indirect
chain.  Old calling conventions aside, a reason for having 'phys'
was that ufs_inode_getfrag() used to be able to do _two_ allocations
- indirect block and extending/reallocating a tail.  We needed
locked_page for the latter (it's a data), but we also needed to
figure out that indirect block is metadata.  So we used to pass
non-NULL locked_page in all cases *and* used NULL phys as
indication of being asked to allocate an indirect.

With tail unpacking taken into a separate function we don't need
those convolutions anymore.

Signed-off-by: Al Viro

ufs_getfrag_block(): tidy up a bit

2015-07-06T21:40:04+00:00

Signed-off-by: Al Viro

ufs_inode_getblock(): failure to read an indirect block is -EIO

2015-07-06T21:40:03+00:00

... and not "write to beginning of the disk", TYVM...

Signed-off-by: Al Viro

ufs_getfrag_block(): turn following indirects into a loop

2015-07-06T21:40:02+00:00

Signed-off-by: Al Viro

ufs_inode_getfrag(): pass index instead of 'fragment'

2015-07-06T21:40:01+00:00

same story as with ufs_inode_getblock()

Signed-off-by: Al Viro

ufs_inode_getfrag(): split extending the partial blocks off

2015-07-06T21:40:00+00:00

ufs_extend_tail() is handling that now.

Signed-off-by: Al Viro

ufs_inode_getblock(): pass indirect block number and full index

2015-07-06T21:39:59+00:00

... instead of messing with buffer_head.  We can bloody well do
sb_bread() in there.

Signed-off-by: Al Viro

ufs_inode_getblock(): pass index instead of 'fragment'

2015-07-06T21:39:58+00:00

The value passed to ufs_inode_getblock() as the 3rd argument
had lower bits ignored; the upper bits were shifted down
and used and they actually make sense - those are _lower_ bits
of index in indirect block (i.e. they form the index within
a fragment within an indirect block).

Pass those as argument.  Upper bits of index (i.e. the number
of fragment within indirect block) will join them shortly.

Signed-off-by: Al Viro

ufs_inode_get{frag,block}(): leave sb_getblk() to caller

2015-07-06T21:39:57+00:00

just return the damn block number

Signed-off-by: Al Viro