Commits · 6d0dd7684cc8d010ab3082db572534dffd2ad42d · nexedi / linux

13 Nov, 2015 21 commits

Orangefs: Remove unused #defines from signal blocking code. · 6d0dd768
Martin Brandenburg authored Oct 07, 2015
```
Signed-off-by: Martin Brandenburg <martin@omnibond.com>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>
```
6d0dd768

Orangefs: set pos after generic_write_checks · 3f1b6947

Mike Marshall authored Nov 13, 2015

if we are appending, generic_write_checks would have updated
pos to the end of the file...
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

3f1b6947

orangefs: validate the response in decode_dirents() · 8092895f

Al Viro authored Oct 09, 2015

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

8092895f

orangefs: don't leave uninitialized data in ->trailer_buf · ade1d48b

Al Viro authored Oct 09, 2015

minimal fix; it would be better to reject such requests outright.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

ade1d48b

orangefs: get rid of dec_string and enc_string · 9be68b08

Al Viro authored Oct 09, 2015

The latter is never used, the former has one user and would be
better off spelled out right there.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

9be68b08

orangefs: switch decode_dirents() to use of kcalloc() · ef4af94e

Al Viro authored Oct 09, 2015

gets rid of multiplication overflow
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

ef4af94e

orangefs: sanitize pvfs2_convert_time_field() · 5714156b

Al Viro authored Oct 08, 2015

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

5714156b

orangefs: kill pointless ->link() and ->mknod() · aada5c58

Al Viro authored Oct 08, 2015

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

aada5c58

pvfs2_fill_sb(): use kzalloc() · 75992b0f

Al Viro authored Oct 08, 2015

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

75992b0f

orangefs: kill struct pvfs2_mount_sb_info_s · 5c0dbbc6

Al Viro authored Oct 08, 2015

The only reason for that thing used to be the API of mount_nodev()
callback;  since we are calling pvfs2_fill_sb() ourselves now,
we don't have to shove everything into a single structure.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

5c0dbbc6

orangefs: double iput() in case of d_make_root() failure · b05a7851

Al Viro authored Oct 08, 2015

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

b05a7851

orangefs: use get_user_pages_fast(), not get_user_pages() · 16742f2d

Al Viro authored Oct 08, 2015

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

16742f2d

orangefs: kill kmap/kunmap wrappers · a0435ca1

Al Viro authored Oct 08, 2015

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

a0435ca1

orangefs: make pvfs2_inode_read() take iov_iter · 74f68fce

Al Viro authored Oct 08, 2015

... and make the only caller use page-backed iov_iter,
getting rid of kmap/kunmap *and* of the bug with
attempted use of iovec-backed copy_page_to_iter()
on a kernel pointer.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

74f68fce

orangefs: make do_readv_writev() take iov_iter · 0071ed1e

Al Viro authored Oct 08, 2015

no need to build a copy of what the caller already has;
what's more, we want the one given to caller properly
advanced *and* we shouldn't depend upon it being an
iovec-backed one.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

0071ed1e

orangefs: don't bother with splitting iovecs · dc4067f6

Al Viro authored Oct 08, 2015

copy_page_{to,from}_iter() advances it just fine *and* it has no
problem with partially consumed segments.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

dc4067f6

orangefs: make wait_for_direct_io() take iov_iter · 3c2fcfcb

Al Viro authored Oct 08, 2015

incidentally, insane or compromised server returning *more* than
requested on read should not oops the kernel - initialize the
iov_iter for read according to the iovec we've got.  That's why
pvfs_bufmap_copy_to_iovec() needed a separate size argument - we
shouldn't abuse iov_iter_count(iter) for passing that.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

3c2fcfcb

orangefs: make precopy_buffers() take iov_iter · a5c126a5

Al Viro authored Oct 08, 2015

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

a5c126a5

orangefs: make postcopy_buffers() take iov_iter · 5f0e3c95

Al Viro authored Oct 08, 2015

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

5f0e3c95

pvfs_bufmap_copy_from_iovec(): don't rely upon size being equal to iov_iter_count(iter) · 34204fde
Al Viro authored Oct 08, 2015
```
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>
```
34204fde
orangefs: explicitly pass the size to pvfs_bufmap_copy_to_iovec() · 5c278228
Al Viro authored Oct 08, 2015
```
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>
```
5c278228

05 Oct, 2015 1 commit
- Orangefs: fix some checkpatch.pl complaints that had creeped in. · 54804949
  Mike Marshall authored Oct 05, 2015
```
Signed-off-by: Mike Marshall <hubcap@omnibond.com>
```
  54804949
03 Oct, 2015 18 commits

Orangefs: Clean up error decoding. · 894ac432

Martin Brandenburg authored Oct 02, 2015

Errors from the server need to be decoded. A bunch of code was imported from
the server to do this but much of it is convoluted and not even needed. The
result is better but still as convoluted as required by the protocol.
Signed-off-by: Martin Brandenburg <martin@omnibond.com>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

894ac432

Orangefs: Use readonly mmap since writepage is not implemented. · 35390803

Martin Brandenburg authored Sep 30, 2015

Previously the code silently failed to update the disk. Now it will not
allow writable and shared mmaps.
Signed-off-by: Martin Brandenburg <martin@omnibond.com>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

35390803

Orangefs: don't use mount_nodev, use sget directly. · 1be21f86
Mike Marshall authored Sep 29, 2015
```
Signed-off-by: Mike Marshall <hubcap@omnibond.com>
```
1be21f86
Orangefs: update signal blocking code before Oleg sees it. · 8c3905ad
Mike Marshall authored Sep 29, 2015
```
Signed-off-by: Mike Marshall <hubcap@omnibond.com>
```
8c3905ad
Orangefs: Don't opencode memcpy. · 50e01586
Mike Marshall authored Sep 29, 2015
```
Signed-off-by: Mike Marshall <hubcap@omnibond.com>
```
50e01586
Orangefs: choose return codes from among the expected ones. · f957ae2d
Mike Marshall authored Sep 24, 2015
```
Signed-off-by: Mike Marshall <hubcap@omnibond.com>
```
f957ae2d
Orangefs: put PVFS_util_min out of its misery. · d6fe654b
Mike Marshall authored Sep 24, 2015
```
Signed-off-by: Mike Marshall <hubcap@omnibond.com>
```
d6fe654b

Orangefs: fix dir_emit code in pvfs2_readdir. · 88309aae

Mike Marshall authored Sep 23, 2015

Al Viro glanced at readdir and surmised that getdents
would misbehave the way it was written... and sure enough.
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

88309aae

Orangefs: use iov_iter interface · 4d1c4404

Mike Marshall authored Sep 04, 2015

replace opencoded pvfs_bufmap_copy_to_kernel_iovec,
pvfs_bufmap_copy_to_user_iovec, pvfs_bufmap_copy_iovec_from_kernel,
and pvfs_bufmap_copy_iovec_from_user with pvfs_bufmap_copy_to_iovec
and pvfs_bufmap_copy_from_iovec, which both use the iov_iter
interface.
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

4d1c4404

Orangefs: Swap order of include files · 81b784b1

Guenter Roeck authored Aug 01, 2015

spinlock_types.h requires types from linux/types.h.
Including spinlock_types.h first may result in the following build errors,
as seen with arm:allmodconfig.

arch/arm/include/asm/spinlock_types.h:12:3: error: unknown type name 'u32'
arch/arm/include/asm/spinlock_types.h:16:4: error: unknown type name 'u16'

Fixes: deb4fb58ff73 ("Orangefs: kernel client part 2")
Cc: Mark Brown <broonie@kernel.org>
Cc: Mike Marshall <hubcap@omnibond.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

81b784b1

fs: orangefs: remove execute priviliges from module params · cb987f3c

Sasha Levin authored Aug 03, 2015

This makes no sense and causes warnings on boot.
Signed-off-by: Sasha Levin <sasha.levin@oracle.com>
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

cb987f3c

Orangefs: use inode_set_bytes for directories · f0566532
Mike Marshall authored Aug 05, 2015
```
Signed-off-by: Mike Marshall <hubcap@omnibond.com>
```
f0566532

Orangefs: large integer implicitly truncated to unsigned type · c36316b7

Mike Marshall authored Jul 30, 2015

make.cross ARCH=tile doesn't like "inode->i_bytes = PAGE_CACHE_SIZE;",
so cast PAGE_CACHE_SIZE to unsigned short.
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

c36316b7

Orangefs: address problems found by static checker · eeaa3d44

Mike Marshall authored Jul 29, 2015

  Don't check for negative rc from boolean.

  Don't pointlessly initialize variables, it short-circuits
  gcc's uninitialized variable warnings. And max_new_nr_segs
  can never be zero, so don't check for it.

  Preserve original kstrdup pointer for freeing later.

  Don't check for negative value in unsigned variable.
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

eeaa3d44

Orangefs: sooth most sparse complaints · 84d02150
Mike Marshall authored Jul 28, 2015
```
Signed-off-by: Mike Marshall <hubcap@omnibond.com>
```
84d02150

Orangefs: kernel client update 1. · 2c590d5f

Mike Marshall authored Jul 24, 2015

Stephen Rothwell noticed that orangefs would not compile
on powerpc...
Signed-off-by: Mike Marshall <hubcap@omnibond.com>

2c590d5f

Orangefs: kernel client part 7 · 07f01962
Mike Marshall authored Jul 17, 2015
```
Signed-off-by: Mike Marshall <hubcap@omnibond.com>
```
07f01962
Orangefs: kernel client part 6 · 74a552a1
Mike Marshall authored Jul 17, 2015
```
Signed-off-by: Mike Marshall <hubcap@omnibond.com>
```
74a552a1