tree:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core.git 
driver-core-testing
head:   3bd418f13312393093d17a312ec0033395d6892e
commit: 055cd38ba6715e5affe753600b907acccfc85b7e [21/31] debugfs: prevent 
access to possibly dead file_operations at file open
reproduce: make htmldocs

All warnings (new ones prefixed by >>):

>> fs/debugfs/file.c:71: warning: No description found for parameter 'dentry'
>> fs/debugfs/file.c:71: warning: Excess function parameter 'file' description 
>> in 'debugfs_use_file_start'
   include/linux/jbd2.h:439: warning: No description found for parameter 
'i_transaction'
   include/linux/jbd2.h:439: warning: No description found for parameter 
'i_next_transaction'
   include/linux/jbd2.h:439: warning: No description found for parameter 
'i_list'
   include/linux/jbd2.h:439: warning: No description found for parameter 
'i_vfs_inode'
   include/linux/jbd2.h:439: warning: No description found for parameter 
'i_flags'
   include/linux/jbd2.h:495: warning: No description found for parameter 
'h_rsv_handle'
   include/linux/jbd2.h:495: warning: No description found for parameter 
'h_reserved'
   include/linux/jbd2.h:495: warning: No description found for parameter 
'h_type'
   include/linux/jbd2.h:495: warning: No description found for parameter 
'h_line_no'
   include/linux/jbd2.h:495: warning: No description found for parameter 
'h_start_jiffies'
   include/linux/jbd2.h:495: warning: No description found for parameter 
'h_requested_credits'
   include/linux/jbd2.h:495: warning: No description found for parameter 
'h_lockdep_map'
   include/linux/jbd2.h:1038: warning: No description found for parameter 
'j_chkpt_bhs[JBD2_NR_BATCH]'
   include/linux/jbd2.h:1038: warning: No description found for parameter 
'j_devname[BDEVNAME_SIZE+24]'
   include/linux/jbd2.h:1038: warning: No description found for parameter 
'j_average_commit_time'
   include/linux/jbd2.h:1038: warning: No description found for parameter 
'j_min_batch_time'
   include/linux/jbd2.h:1038: warning: No description found for parameter 
'j_max_batch_time'
   include/linux/jbd2.h:1038: warning: No description found for parameter 
'j_commit_callback'
   include/linux/jbd2.h:1038: warning: No description found for parameter 
'j_failed_commit'
   include/linux/jbd2.h:1038: warning: No description found for parameter 
'j_chksum_driver'
   include/linux/jbd2.h:1038: warning: No description found for parameter 
'j_csum_seed'
   include/linux/jbd2.h:1038: warning: Excess struct/union/enum/typedef member 
'j_history' description in 'journal_s'
   include/linux/jbd2.h:1038: warning: Excess struct/union/enum/typedef member 
'j_history_max' description in 'journal_s'
   include/linux/jbd2.h:1038: warning: Excess struct/union/enum/typedef member 
'j_history_cur' description in 'journal_s'
   fs/jbd2/transaction.c:429: warning: No description found for parameter 
'rsv_blocks'
   fs/jbd2/transaction.c:429: warning: No description found for parameter 
'gfp_mask'
   fs/jbd2/transaction.c:429: warning: No description found for parameter 'type'
   fs/jbd2/transaction.c:429: warning: No description found for parameter 
'line_no'
   fs/jbd2/transaction.c:505: warning: No description found for parameter 'type'
   fs/jbd2/transaction.c:505: warning: No description found for parameter 
'line_no'
   fs/jbd2/transaction.c:635: warning: No description found for parameter 
'gfp_mask'

vim +/dentry +71 fs/debugfs/file.c

    55   * and debugfs_remove_recursive() will block. Since associated private
    56   * file data may only get freed after a successful return of any of
    57   * the removal functions, you may safely access it after a successful
    58   * call to debugfs_use_file_start() without worrying about
    59   * lifetime issues.
    60   *
    61   * If -%EIO is returned, the file has already been removed and thus,
    62   * it is not safe to access any of its data. If, on the other hand,
    63   * it is allowed to access the file data, zero is returned.
    64   *
    65   * Regardless of the return code, any call to
    66   * debugfs_use_file_start() must be followed by a matching call
    67   * to debugfs_use_file_finish().
    68   */
    69  static int debugfs_use_file_start(const struct dentry *dentry, int 
*srcu_idx)
    70          __acquires(&debugfs_srcu)
  > 71  {
    72          *srcu_idx = srcu_read_lock(&debugfs_srcu);
    73          barrier();
    74          if (d_unlinked(dentry))
    75                  return -EIO;
    76          return 0;
    77  }
    78  
    79  /**

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: Binary data

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to