[stgt] tgtd startup problem

Hannes Reinecke hare at suse.de
Wed Mar 11 10:12:38 CET 2009


Hi all,

Albert Pauw wrote:
> Yes, that's it!
> 
> Works fine again.
> 
> Thanks!
>>
>> This works (against the latest git, again)?
>>
>>
>> diff --git a/usr/tgtadm.h b/usr/tgtadm.h
>> index 5d63485..60b984d 100644
>> --- a/usr/tgtadm.h
>> +++ b/usr/tgtadm.h
>> @@ -1,7 +1,7 @@
>>  #ifndef TGTADM_H
>>  #define TGTADM_H
>>  
>> -#define TGT_IPC_NAMESPACE    "TGT_IPC_ABSTRACT_NAMESPACE"
>> +#define TGT_IPC_NAMESPACE    "/tmp/.TGT_IPC_ABSTRACT_NAMESPACE"
>>  #define TGT_LLD_NAME_LEN    64
>>  
>>  #include "tgtadm_error.h"
>>   
But doesn't this counteract the idea of an abstract namespace?
IE you have to have /tmp mounted and accessible here, as the
socket is created in the /tmp directory, not the abstract
namespace as previously.

Not that is matters much here, as the daemon most certainly
will only run when the root fs is set up properly, but using
/tmp as a socket storage is a bad move. And it's certainly
_not_ an abstract namespace anymore, so we can as well
rename it to something sensible and move it to /var/run
or somesuch.

Cheers,

Hannes
-- 
Dr. Hannes Reinecke		      zSeries & Storage
hare at suse.de			      +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: Markus Rex, HRB 16746 (AG Nürnberg)
--
To unsubscribe from this list: send the line "unsubscribe stgt" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html



More information about the stgt mailing list