• KOSAKI Motohiro's avatar
    mm, fadvise: don't return -EINVAL when filesystem cannot implement fadvise() · 3d3727cd
    KOSAKI Motohiro authored
    Eric Wong reported his test suite failex when /tmp is tmpfs.
    
    https://lkml.org/lkml/2012/2/24/479
    
    
    
    Currentlt the input check of POSIX_FADV_WILLNEED has two problems.
    
    - requires a_ops->readpage.  But in fact, force_page_cache_readahead()
      requires that the target filesystem has either ->readpage or ->readpages.
    
    - returns -EINVAL when the filesystem doesn't have ->readpage.  But
      posix says that fadvise is merely a hint.  Thus fadvise() should return
      0 if filesystem has no means of implementing fadvise().  The userland
      application should not know nor care whcih type of filesystem backs the
      TMPDIR directory, as Eric pointed out.  There is nothing which userspace
      can do to solve this error.
    
    So change the return value to 0 when filesytem doesn't support readahead.
    
    [akpm@linux-foundation.org: checkpatch fixes]
    Signed-off-by: default avatarKOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
    Cc: Hugh Dickins <hughd@google.com>
    Cc: Hillf Dan...
    3d3727cd
fadvise.c 3.61 KB