All of lore.kernel.org
 help / color / mirror / Atom feed
From: Fengguang Wu <fengguang.wu@intel.com>
To: kernel-janitors@vger.kernel.org
Subject: [cifs:for-next 61/66] fs/cifs/file.c:708:1: sparse: function 'cifs_find_fid_lock_conflict' with exte
Date: Fri, 14 Sep 2012 23:47:11 +0000	[thread overview]
Message-ID: <20120914234711.GB31637@localhost> (raw)

Hi Pavel,

FYI, there are new sparse warnings show up in

tree:   git://git.samba.org/sfrench/cifs-2.6.git for-next
head:   c246704dfe86f2c1875b59a8d60ca69c1f48acb4
commit: 2485496555e7b3f0280e2958934e10fb2672fcf5 [61/66] CIFS: Check for mandatory brlocks on read/write

All sparse warnings:

+ fs/cifs/file.c:708:1: sparse: function 'cifs_find_fid_lock_conflict' with external linkage has definition
+ fs/cifs/file.c:708:1: sparse: symbol 'cifs_find_fid_lock_conflict' was not declared. Should it be static?

vim +708 fs/cifs/file.c
   698	cifs_del_lock_waiters(struct cifsLockInfo *lock)
   699	{
   700		struct cifsLockInfo *li, *tmp;
   701		list_for_each_entry_safe(li, tmp, &lock->blist, blist) {
   702			list_del_init(&li->blist);
   703			wake_up(&li->block_q);
   704		}
   705	}
   706	
   707	extern bool
 > 708	cifs_find_fid_lock_conflict(struct cifs_fid_locks *fdlocks, __u64 offset,
   709				    __u64 length, __u8 type, struct cifsFileInfo *cfile,
   710				    struct cifsLockInfo **conf_lock, bool rw_check)
   711	{
   712		struct cifsLockInfo *li;
   713		struct cifsFileInfo *cur_cfile = fdlocks->cfile;
   714		struct TCP_Server_Info *server = tlink_tcon(cfile->tlink)->ses->server;
   715	
   716		list_for_each_entry(li, &fdlocks->locks, llist) {
   717			if (offset + length <= li->offset ||
   718			    offset >= li->offset + li->length)

---
0-DAY kernel build testing backend         Open Source Technology Centre
Fengguang Wu, Yuanhan Liu                              Intel Corporation

                 reply	other threads:[~2012-09-14 23:47 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120914234711.GB31637@localhost \
    --to=fengguang.wu@intel.com \
    --cc=kernel-janitors@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.