SEM Key Clash with Open-iSCSI initiator and LFS correction for ipc socket

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


doing some regression test on a kvm cluster I had to run 
the Open-iSCSI initiator (iscsid) and tgtd on the same machine.
This brings up these to issues...

1. Logging of iscsd and tgtd broken

But this breaks logging of both iscsd and tgtd because they both
run the logging semaphore with the same unix ipc key 0x000000a7
(I think the reason is that the logging code from one project is
 reused in the other project).
The error messages found in syslog are:
 tgtd: semop up failed
 iscsid: semop down failed 22

2. IPC Namespace Socket in /tmp

I suggest strongly to move the ipc communication socket to /var/run
to avoid the socket file is accidentally deleted by admins or 
cron system maintenance scripts. 
I think it is also more FHS conform this way.

A test case for the breakage of logging facility can found in this
redhat bugzilla:

And this is my suggested patch to fix these issues:
--- tgt-1.0.14.ori/usr/log.c    2011-03-01 12:53:45.000000000 +0100
+++ tgt-1.0.14/usr/log.c        2011-03-03 16:13:53.758746516 +0100
@@ -33,7 +33,7 @@
 #include "log.h"
-#define SEMKEY 0xA7L
+#define SEMKEY 0x54475444L /* TGTD */
 #define LOGDBG 0
diff -Naur tgt-1.0.14.ori/usr/tgtadm.h tgt-1.0.14/usr/tgtadm.h
--- tgt-1.0.14.ori/usr/tgtadm.h 2011-03-01 12:53:45.000000000 +0100
+++ tgt-1.0.14/usr/tgtadm.h     2011-03-03 16:14:01.612496609 +0100
@@ -1,7 +1,7 @@
 #ifndef TGTADM_H
 #define TGTADM_H
+#define TGT_IPC_NAMESPACE      "/var/run/tgtd.ipc_abstract_namespace"
 #define TGT_LLD_NAME_LEN       64
 #define GLOBAL_TID (~0U)

Kind Regards,

Roland.Friedwagner@xxxxxxxx            Phone: +43 1 31336 5377
IT Services - WU (Vienna University of Economics and Business) 
To unsubscribe from this list: send the line "unsubscribe stgt" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at

[Index of Archives]     [Linux SCSI]     [Linux RAID]     [Linux Clusters]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Free Online Dating]     [Linux Kernel]     [XFree86]
  Powered by Linux