Just use extern, saves a lot of sparse warnings.
fs/cifs/cifsglob.h:603:33: warning: symbol 'GlobalUidList' was not declared. 
Should it be static?
fs/cifs/cifsglob.h:606:32: warning: symbol 'GlobalSMBSessionList' was not 
declared. Should it be static?
fs/cifs/cifsglob.h:607:32: warning: symbol 'GlobalTreeConnectionList' was not 
declared. Should it be static?
fs/cifs/cifsglob.h:608:24: warning: symbol 'GlobalSMBSeslock' was not declared. 
Should it be static?
fs/cifs/cifsglob.h:610:32: warning: symbol 'GlobalOplock_Q' was not declared. 
Should it be static?
fs/cifs/cifsglob.h:613:32: warning: symbol 'GlobalDnotifyReqList' was not 
declared. Should it be static?
fs/cifs/cifsglob.h:615:32: warning: symbol 'GlobalDnotifyRsp_Q' was not 
declared. Should it be static?
fs/cifs/cifsglob.h:620:28: warning: symbol 'GlobalCurrentXid' was not declared. 
Should it be static?
fs/cifs/cifsglob.h:621:28: warning: symbol 'GlobalTotalActiveXid' was not 
declared. Should it be static?
fs/cifs/cifsglob.h:622:28: warning: symbol 'GlobalMaxActiveXid' was not 
declared. Should it be static?
fs/cifs/cifsglob.h:623:26: warning: symbol 'GlobalMid_Lock' was not declared. 
Should it be static?
fs/cifs/cifsglob.h:625:20: warning: symbol 'Local_System_Name' was not 
declared. Should it be static?
fs/cifs/cifsglob.h:630:24: warning: symbol 'sesInfoAllocCount' was not 
declared. Should it be static?
fs/cifs/cifsglob.h:631:24: warning: symbol 'tconInfoAllocCount' was not 
declared. Should it be static?
fs/cifs/cifsglob.h:632:24: warning: symbol 'tcpSesAllocCount' was not declared. 
Should it be static?
fs/cifs/cifsglob.h:633:24: warning: symbol 'tcpSesReconnectCount' was not 
declared. Should it be static?
fs/cifs/cifsglob.h:634:24: warning: symbol 'tconInfoReconnectCount' was not 
declared. Should it be static?
fs/cifs/cifsglob.h:637:24: warning: symbol 'bufAllocCount' was not declared. 
Should it be static?
fs/cifs/cifsglob.h:639:24: warning: symbol 'totBufAllocCount' was not declared. 
Should it be static?
fs/cifs/cifsglob.h:640:24: warning: symbol 'totSmBufAllocCount' was not 
declared. Should it be static?
fs/cifs/cifsglob.h:642:24: warning: symbol 'smBufAllocCount' was not declared. 
Should it be static?
fs/cifs/cifsglob.h:643:24: warning: symbol 'midCount' was not declared. Should 
it be static?
fs/cifs/cifsglob.h:646:28: warning: symbol 'multiuser_mount' was not declared. 
Should it be static?
fs/cifs/cifsglob.h:650:28: warning: symbol 'oplockEnabled' was not declared. 
Should it be static?
fs/cifs/cifsglob.h:651:28: warning: symbol 'experimEnabled' was not declared. 
Should it be static?
fs/cifs/cifsglob.h:652:28: warning: symbol 'lookupCacheEnabled' was not 
declared. Should it be static?
fs/cifs/cifsglob.h:653:28: warning: symbol 'extended_security' was not 
declared. Should it be static?
fs/cifs/cifsglob.h:655:28: warning: symbol 'sign_CIFS_PDUs' was not declared. 
Should it be static?
fs/cifs/cifsglob.h:656:28: warning: symbol 'linuxExtEnabled' was not declared. 
Should it be static?
fs/cifs/cifsglob.h:657:28: warning: symbol 'CIFSMaxBufSize' was not declared. 
Should it be static?
fs/cifs/cifsglob.h:658:28: warning: symbol 'cifs_min_rcv' was not declared. 
Should it be static?
fs/cifs/cifsglob.h:659:28: warning: symbol 'cifs_min_small' was not declared. 
Should it be static?
fs/cifs/cifsglob.h:660:28: warning: symbol 'cifs_max_pending' was not declared. 
Should it be static?

Signed-off-by: Harvey Harrison <[EMAIL PROTECTED]>
---
 fs/cifs/cifsglob.h |   76 ++++++++++++++++++++++++----------------------------
 1 files changed, 35 insertions(+), 41 deletions(-)

diff --git a/fs/cifs/cifsglob.h b/fs/cifs/cifsglob.h
index 5d32d8d..bb6ddf3 100644
--- a/fs/cifs/cifsglob.h
+++ b/fs/cifs/cifsglob.h
@@ -583,79 +583,73 @@ require use of the stronger protocol */
  *
  ****************************************************************************/
 
-#ifdef DECLARE_GLOBALS_HERE
-#define GLOBAL_EXTERN
-#else
-#define GLOBAL_EXTERN extern
-#endif
-
 /*
  * The list of servers that did not respond with NT LM 0.12.
  * This list helps improve performance and eliminate the messages indicating
  * that we had a communications error talking to the server in this list.
  */
 /* Feature not supported */
-/* GLOBAL_EXTERN struct servers_not_supported *NotSuppList; */
+/* extern struct servers_not_supported *NotSuppList; */
 
 /*
  * The following is a hash table of all the users we know about.
  */
-GLOBAL_EXTERN struct smbUidInfo *GlobalUidList[UID_HASH];
+extern struct smbUidInfo *GlobalUidList[UID_HASH];
 
-/* GLOBAL_EXTERN struct list_head GlobalServerList; BB not implemented yet */
-GLOBAL_EXTERN struct list_head GlobalSMBSessionList;
-GLOBAL_EXTERN struct list_head GlobalTreeConnectionList;
-GLOBAL_EXTERN rwlock_t GlobalSMBSeslock;  /* protects list inserts on 3 above 
*/
+/* extern struct list_head GlobalServerList; BB not implemented yet */
+extern struct list_head GlobalSMBSessionList;
+extern struct list_head GlobalTreeConnectionList;
+extern rwlock_t GlobalSMBSeslock;  /* protects list inserts on 3 above */
 
-GLOBAL_EXTERN struct list_head GlobalOplock_Q;
+extern struct list_head GlobalOplock_Q;
 
 /* Outstanding dir notify requests */
-GLOBAL_EXTERN struct list_head GlobalDnotifyReqList;
+extern struct list_head GlobalDnotifyReqList;
 /* DirNotify response queue */
-GLOBAL_EXTERN struct list_head GlobalDnotifyRsp_Q;
+extern struct list_head GlobalDnotifyRsp_Q;
 
 /*
  * Global transaction id (XID) information
  */
-GLOBAL_EXTERN unsigned int GlobalCurrentXid;   /* protected by GlobalMid_Sem */
-GLOBAL_EXTERN unsigned int GlobalTotalActiveXid; /* prot by GlobalMid_Sem */
-GLOBAL_EXTERN unsigned int GlobalMaxActiveXid; /* prot by GlobalMid_Sem */
-GLOBAL_EXTERN spinlock_t GlobalMid_Lock;  /* protects above & list operations 
*/
+extern unsigned int GlobalCurrentXid;  /* protected by GlobalMid_Sem */
+extern unsigned int GlobalTotalActiveXid; /* prot by GlobalMid_Sem */
+extern unsigned int GlobalMaxActiveXid;        /* prot by GlobalMid_Sem */
+extern spinlock_t GlobalMid_Lock;  /* protects above & list operations */
                                          /* on midQ entries */
-GLOBAL_EXTERN char Local_System_Name[15];
+extern char Local_System_Name[15];
 
 /*
  *  Global counters, updated atomically
  */
-GLOBAL_EXTERN atomic_t sesInfoAllocCount;
-GLOBAL_EXTERN atomic_t tconInfoAllocCount;
-GLOBAL_EXTERN atomic_t tcpSesAllocCount;
-GLOBAL_EXTERN atomic_t tcpSesReconnectCount;
-GLOBAL_EXTERN atomic_t tconInfoReconnectCount;
+extern atomic_t sesInfoAllocCount;
+extern atomic_t tconInfoAllocCount;
+extern atomic_t tcpSesAllocCount;
+extern atomic_t tcpSesReconnectCount;
+extern atomic_t tconInfoReconnectCount;
 
 /* Various Debug counters to remove someday (BB) */
-GLOBAL_EXTERN atomic_t bufAllocCount;    /* current number allocated  */
+extern atomic_t bufAllocCount;    /* current number allocated  */
 #ifdef CONFIG_CIFS_STATS2
-GLOBAL_EXTERN atomic_t totBufAllocCount; /* total allocated over all time */
-GLOBAL_EXTERN atomic_t totSmBufAllocCount;
+extern atomic_t totBufAllocCount; /* total allocated over all time */
+extern atomic_t totSmBufAllocCount;
 #endif
-GLOBAL_EXTERN atomic_t smBufAllocCount;
-GLOBAL_EXTERN atomic_t midCount;
+extern atomic_t smBufAllocCount;
+extern atomic_t midCount;
 
 /* Misc globals */
-GLOBAL_EXTERN unsigned int multiuser_mount; /* if enabled allows new sessions
+extern unsigned int multiuser_mount; /* if enabled allows new sessions
                                to be established on existing mount if we
                                have the uid/password or Kerberos credential
                                or equivalent for current user */
-GLOBAL_EXTERN unsigned int oplockEnabled;
-GLOBAL_EXTERN unsigned int experimEnabled;
-GLOBAL_EXTERN unsigned int lookupCacheEnabled;
-GLOBAL_EXTERN unsigned int extended_security;  /* if on, session setup sent
+extern unsigned int oplockEnabled;
+extern unsigned int experimEnabled;
+extern unsigned int lookupCacheEnabled;
+extern unsigned int extended_security; /* if on, session setup sent
                                with more secure ntlmssp2 challenge/resp */
-GLOBAL_EXTERN unsigned int sign_CIFS_PDUs;  /* enable smb packet signing */
-GLOBAL_EXTERN unsigned int linuxExtEnabled;/*enable Linux/Unix CIFS 
extensions*/
-GLOBAL_EXTERN unsigned int CIFSMaxBufSize;  /* max size not including hdr */
-GLOBAL_EXTERN unsigned int cifs_min_rcv;    /* min size of big ntwrk buf pool 
*/
-GLOBAL_EXTERN unsigned int cifs_min_small;  /* min size of small buf pool */
-GLOBAL_EXTERN unsigned int cifs_max_pending; /* MAX requests at once to 
server*/
+extern unsigned int sign_CIFS_PDUs;  /* enable smb packet signing */
+extern unsigned int linuxExtEnabled;/*enable Linux/Unix CIFS extensions*/
+extern unsigned int CIFSMaxBufSize;  /* max size not including hdr */
+extern unsigned int cifs_min_rcv;    /* min size of big ntwrk buf pool */
+extern unsigned int cifs_min_small;  /* min size of small buf pool */
+extern unsigned int cifs_max_pending; /* MAX requests at once to server*/
 
-- 
1.5.4.1.1278.gc75be



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to