Merge pull request #80 from libfuse/master-proposed

Whitespace fix.

Fix valgrind warning

If we don't assign a value to padding, we get a warning about reading
uninitialized data when sending the iovec to the kernel.

Fix documentation of fuse_parse_cmdline().

For --help and --version, it returns -1.

Dropped example/null.c

This does not seem to be working. Maybe because it tries to treat the
mountpoint as a file rather than a directory?

Added timefs3 to test notify_inval_entry.

Fixes #32.

Add background and multithreading support to hello_ll and fuse_lo-plus

Update list of requests that can be answered with fuse_reply_none().

testfs2: Also test fuse_notify_retrieve().

Added timefs[12] examples.

These examplesdemonstrate the use of the `fuse_lowlevel_notify_store`
and `fuse_lowlevel_notify_inval_inode` functions.

Added safe_sleep()

Clarify that readdir() *may* report . and .. entries.

Removed ``-o big_writes`` option

This option is obsolete and should always be enabled. File systems that
want to limit the size of write requests should use the
``-o max_write=<N>`` option instead.

Test more combinations of low-level options.

Remove pointless aliasing of fuse_session *se to *f

This is an overlooked artifact of the fuse_ll-fuse_session merge.

Removed obsolete, unused *f member from struct fuse_session.

Merge struct fuse_ll into struct fuse_session.

This merge merges struct fuse_ll into struct fuse_session. Since
there is always a one-to-one correspondence between the two,
there is little reason to keep them separate. By merging them,
we save pointers and lines of code.

Merge fuse_ll into fuse_session (part 8)

Replaced "req->se" with "f" where the latter is already defined.

Merge fuse_ll into fuse_session (part 7)

Remove pointless aliasing of "struct fuse_session *se" to "struct
fuse_session *f".

Merge fuse_ll into fuse_session (part 6)

Fixup cuse_lowlevel_new().

Merge fuse_ll into fuse_session (part 5)

Fixup fuse_session_new().

Merge fuse_ll into fuse_session (part 4)

Merge fuse_ll_destroy() and fuse_session_destroy().

Merge fuse_ll into fuse_session (part 3)

Replace se->f with se.

Merge fuse_ll into fuse_session (part 2)

Replaced all references to req->f with req->se.

Merge fuse_ll into fuse_session (part 1)

Merged the structures, and replaced fuse_ll with fuse_session
in all type definitions.

fuse_mount_help(): clarify that listed options are FUSE specific

We also accept a number of mount options that are common to
all  file systems (nosuid, nodev, ro, etc).

Clarified purpose of helper.c, moved *version() to fuse.c

Merge branch 'fix-root-umount'

Commit e4015aca9b7 already fixed this problem in a different way, so
this is effectively a null-merge.

fuse_session_unmount(): fix unmounting as root

We have to store the channel before removing it from the session,
otherwise we cannot use it to do the unmount.

This problem was introduced in commit 5698ee09cf7.

Thanks to Michael Theall for reporting & debugging this.

Merge master fuse_chan into fuse_session.

This is a code simplification patch.

- It confines most of the implementation channel implementation into
  fuse_loop_mt (which is its only user).

- It makes it more obvious in the code that channels are only ever used
  when using -o clone_fd and multi-threaded main loop.

- It simplies the definition of both struct fuse_session and struct
  fuse_chan.

- Theoretically it should result in (minuscule) performance
  improvements when not using -o clone_fd.

- Overall, it removes a lot more lines of source code than it adds :-).

fuse_lowlevel_notify_poll(): use master channel

In theory, a poll handle could hang around much longer than the worker
thread that creates it. Furthermore, the thread that created the
pollhandle is no more likely to call fuse_lowlevel_notify_poll() than
any other thread.

In theory, this would have kept the channel alive for much longer than
necessary. In practice, there seems to have been a bug that prevented
this - and instead allowed the channel to be destroyed while there
was still a pollhandle referring to it.

Instead of fixing this by calling fuse_chan_get() and fuse_chan_put() in
do_poll() and fuse_pollhandle_destroy(), we simply transmit poll
notifications over the master channel now.

Store struct fuse_session* in struct fuse_req

fuse_session_process_buf(): improve documentation

Released 3.0.0pre0

Don't crash if valgrind or libtool are not installed.

Run tests under valgrind when available.

Fixes #50.

Import os.path.join as pjoin

Makes the code much shorter :-).

Added fuse_session_fd()

Fixes #59.

Document that client pid/gid/uid may be zero.

Fixes #67.

Merged fuse_session.c into fuse_lowlevel.c

This distinction no longer makes sens. fuse_lowlevel.c already contains
several session related functions, and fuse_session.c contains various
stuff that is more related to the channel interface.

Merged fuse_mt.c into fuse.c

There is no point in having a separate file for a 10 line function.

Extended per-file comments.

This should make more clear what file contains code for what
purpose.

Don't handle --help and --version in fuse_session_new().

Help and version messages can be generated using the new
fuse_lowlevel_help(), fuse_lowlevel_version(), fuse_mount_help(), and
fuse_mount_version() functions.

The fuse_parse_cmdline() function has been made more powerful
to do this automatically, and is now explicitly intended only
for low-level API users.

This is a code simplication patch. We don't have to parse for --help and
--version in quite as many places, and we no longer have a low-level
initialization function be responsible for the (super-high level) task
of printing a program usage message.

In the high-level API, we can now handle the command line parsing
earlier and avoid running other initialization code if we're just going
to abort later on.

fuse_mount_opts: don't keep max_read= option.

There is no consumer of it down the line.

Add section headings for --help output

Also, do not include "General options" in usage message.

Edited comments for clarity.

Added test for fuse_lo-plus.

Run hello_ll single-threaded explicitly.

Fixed permissions of source files.

Turn struct fuse_chan into an implementation detail

The only struct fuse_chan that's accessible to the user application is
the "master" channel that is returned by fuse_mount and stored in struct
fuse_session.

When using the multi-threaded main loop with the "clone_fd" option, each
worker thread gets its own struct fuse_chan. However, none of these are
available to the user application, nor do they hold references to struct
fuse_session (the pointer is always null).

Therefore, any presence of struct fuse_chan can be removed
without loss of functionality by relying on struct fuse_session instead.

This reduces the number of API functions and removes a potential source
of confusion (since the new API no longer looks as if it might be
possible to add multiple channels to one session, or to share one
channel between multiple sessions).

Fixes issue #17.

Renamed fuse_lowlevel_new() to fuse_session_new().

Re-order declarations to reflect typical order of use

Introduce separate mount/umount functions for low-level API.

Tell emacs to use tabs for linkerscript.

Turned fuse_session_{process,receive}_buf into wrapper functions.

Inlined fuse_session_new()

This function is only used in one place.

Improve documentation of argument parsing.

Added notes for libfuse hackers.

Documentation improvements.

Added TAGS to .gitignore

Fixed up permissions.

fuse_lowlevel_notify_*: take struct fuse_session instead of struct fuse_chan

The only struct fuse_chan that's available to the user application is
the one that is returned by fuse_mount. However, this is also
permanently available from struct fuse_session.

A later patch will therefore remove struct fuse_chan from the
public API completely. This patch prepares for this by changing the
fuse_lowlevel_notify_* functions to take a struct fuse_session
parameter instead of a struct fuse_chan parameter.

Dropped fuse_unmount_compat22

This function was for backwards compatibility in FUSE 2.x, and
is no longer exposed by FUSE 3.

Fixed typo.

Don't hardcode /sbin, /lib & /etc

Default values for MOUNT_FUSE_PATH, UDEV_RULES_PATH and INIT_D_PATH should be based on directory variables from GNU Coding Standarts.
Directory variables left unexpanded because installation directory options may be changed via "make install prefix=/foo".

Disable thread sanitizer, use newer gcc.

There's no point in having tests failing all the time,
we will re-enable thread sanitizer once we have figured
out if the errors are bugs or false positives (and, if they
are bugs, fixed them).

Updated ChangeLog

Started from most recent FUSE 2.9.7 ChangeLog, and added FUSE 3.0
changes based on inspection of total diff to master.

Include test suite in tarball

add error checking to setuid calls (#66)

libfuse/fuse_daemonize:  wait until daemon child process is ready (#55)

Mounting a FUSE file system remotely using SSH in combination with
pseudo-terminal allocation (-t), results in "Transport endpoint is
not connected" errors when trying to access the file system contents.

For example:

  # ssh -t root@localhost  "cmsfs-fuse /dev/disk/by-path/ccw-0.0.0190 /CMSFS"
  Connection to localhost closed.
  # ls /CMSFS
  ls: cannot access '/CMSFS': Transport endpoint is not connected

The cmsfs-fuse main program (which can also be any other FUSE file
system) calls into the fuse_main() libfuse library function.
The fuse_main() function later calls fuse_daemonize() to fork the
daemon process to handle the FUSE file system I/O.

The fuse_daemonize() function calls fork() as usual.  The child
proceeds with setsid() and then redirecting its file descriptors
to /dev/null etc.  The parent process, simply exits.

The child's functions and the parent's exit creates a subtle race.
This is seen with an SSH connection.  The SSH command above calls
cmsfs-fuse on an allocated pseudo-terminal device (-t option).

If the parent exits, SSH receives the command completion and closes
the connection, that means, it closes the master side of the
pseudo-terminal.  This causes a HUP signal being sent to the process
group on the pseudo-terminal.  At this point in time, the child might
not have completed the setsid() call and, hence, becomes terminated.
Note that fuse daemon sets up its signal handlers after fuse_daemonize()
has completed.

Even if the child has the chance to disassociate from its parent process
group to become it's own process group with setsid(), the child still
has the pseudo-terminal opened as stdin, stdout, and stderr.  So the
pseudo-terminal still behave as controlling terminal and might cause a
SIGHUP at closing the the master side.

To solve the problem, the parent has to wait until the child (the fuse
daemon process) has completed its processing, that means, has become
its own process group with setsid() and closed any file descriptors
pointing to the pseudo-terminal.

Closes: #27
Reported-by: Ofer Baruch <oferba@il.ibm.com>
Reviewed-by: Gerald Schaefer <gerald.schaefer@de.ibm.com>
Signed-off-by: Hendrik Brueckner <brueckner@linux.vnet.ibm.com>

Automatically run CI tests with clang

Fixes: #51

Inlined calculate_dirent*_size functions

Don't use IFTODT macro.

IFTODT is provided by libc to convert from mode_t (as included in e.g.
struct fstat) to d_type (unsigned char element of struct dirent, as
returned by e.g. readdir). However, fuse actually uses a different
struct fuse_dirent with a "type" field of type uint32.
In other words, the semantics of (struct fuse_dirent).type are
not necessarily the same as those of (struct dirent).d_type.

Inlined fuse_add_dirent

Merge branch 'add-dirent-strlen3' of git://github.com/trapexit/libfuse into trapexit-add-dirent-strlen3

Add 'traceback' as suspicious test output pattern.

Added Travis integration

Merge remote-tracking branch 'origin/master'

Merge branch 'ioctl-test'

Add ioctl tests.

Fix ioctl example

Fixes issue #39.

remove double calculation of filename length and dirent size

lib/fuse_lowlevel.c: fix small typo in fuse_ll_help (#44)

s/disable remove file locking/disable remote file locking/

Fixup commit 482a49c.

Merge remote-tracking branch 'origin/master'

Remove leading _ on header guards to comply with reserved identifier requirements (#29)

Remove leading _ on header guards to comply with reserved identifier requirements

Merge pull request #35 from 1ace/master

Fix spelling mistake in comment

Fix spelling mistake

Added test_fsel

Added unit tests for fusexmp and hello

Added basic unit tests.

Fixes issue #33.

Integrate tests with autotools

Whitespace cleanup.

Applied (whitespace-cleanup) to each file. Having whitespace changes
in the VCS is ugly, but it ensures that in the future committers
can run this function to *avoid* commiting any whitespace.

Added .dir-locals.el to customize emacs settings

Inlined fuse_chan_fd

Inlined fuse_chan_{send, recv}

Merge pull request #24 from jblumsch/doc-statfs-frsize

doc fix: The f_frsize field is not ignored by the statfs operation

cuse_lowlevel.setup(): fix double free of local variable 'args'

Signed-off-by: Winfried Koehler <w_scan@gmx-topmail.de>

fuse.h doc fix: The f_frsize field is not ignored by the statfs operation.

Apparently f_frsize has been passed on transparently since
2b4781100812d42e704c39c51303cd28ad3f9aa6 (Nov 28, 2005).

Improve description of issue #15.