【发布时间】:2014-10-09 17:49:58
【问题描述】:
我正在模拟一个设备,它会在时钟下降沿时接收数据字节并写入寄存器。我创建了一个单独的 pthread 来解释时钟在高低之间的转换。
现在我的问题是,在主线程中,我让控制台要求用户输入十六进制值以写入“设备”,然后它调用 SendCommand(uint32_t addr, uint16_t data) 函数来执行此操作。由于它应该在每个时钟周期一次发送一位,因此我将它放在一个 for 循环中。这里是发送命令函数:
void SendCmd(uint32_t loadAddr, uint16_t data)
{
dest = &loadAddr;
int i;
printf("%d\n\n",status);
for(i=15; i>=0; --i) {
pthread_cond_wait(&rising_edge, &data_lock);
*dest = (*dest << 1)|(data & 0x8000) >> 15;
// ^^^^^^^^ get MSB
// ^^^^^ move the MSB down to the LSB, assuming the
// peripheral wants the bit in the LSB.
pthread_cond_wait(&falling_edge, &data_lock);
data <<= 1;
printf("%d\ti=%d\n",*dest,i);
}
pthread_mutex_unlock(&data_lock);
status = ENABLED_IDLE;
}
当 i=0 时,循环应该退出并返回到主函数,它会要求用户输入另一个十六进制值。但它没有,我似乎无法找到为什么它会在退出循环之前被卡住。
编辑:冻结实际上是在函数执行之前发生的:
int main()
{
State status = DISABLED;
Clock clk = LOW;
int exit_flag = 0;
char Command[20];
pthread_t clk_t;
pthread_cond_init(&en_sig, NULL);
pthread_cond_init(&rising_edge, NULL);
pthread_cond_init(&falling_edge, NULL);
pthread_mutex_init(&clk_lock, NULL);
pthread_mutex_init(&data_lock, NULL);
pthread_create(&clk_t, NULL, simClk, NULL);
while(!exit_flag)
{
if(status != ENABLED_ACTIVE)
{
fputs("(enable/disable/write [addr] [word_0] [word_1*]/quit): ", stdout);
fflush(stdout);
if ( fgets(Command, sizeof Command, stdin) != NULL )
{
char *newline = strchr(Command, '\n');
if ( newline != NULL )
{
*newline = '\0';
}
}
if(strcasecmp(Command,"quit")==0)
{
printf("\nShutting Down!\n\n");
pthread_cancel(&clk_t);
pthread_mutex_destroy(&clk_lock);
exit_flag = ~exit_flag;
continue;
}
if(strcasecmp(Command,"enable")==0)
{
if(status != DISABLED)
{
printf("\nSynthesizer is already enabled!\n\n");
continue;
}
status = ENABLED_IDLE;
nanosleep(&ecsWait, NULL);
printf("\nEnabled!\n\n");
pthread_cond_signal(&en_sig);
continue;
}
if(strcasecmp(Command,"disable")==0)
{
status = DISABLED;
// pthread_mutex_lock(&clk_lock);
printf("\nDisabled!\n\n");
continue;
}
if(strcasecmp(Command,"w")==0)
{
if(status != ENABLED_IDLE)
{
printf("\nSynthesizer is not enabled!\n\n");
continue;
}
printf("\nWriting!\n\n");
status = ENABLED_ACTIVE; //FREEZE OCCURS HERE!
SendCmd(NREGISTER_ADDR,0xF13F);
continue;
}
printf("Invalid command!\n");
}
}
return 0;
}
【问题讨论】:
-
dest = &loadAddr看起来不正确,这意味着您正在写入局部变量loadAddr的地址。如果loadAddr是一个实际地址,也许你打算写类似volatile uint8_t *dest = (uint8_t *) loadAddr的东西。 -
您是否看到每次调用有 16 行输出?
-
pthread_mutex_lock(&data_lock)在哪里?在同一个函数中匹配成对的锁和解锁通常是个好主意 -
pthread_cond_wait()的文档部分说“应在调用线程或未定义行为结果锁定的情况下调用 [互斥锁]”。看起来您遇到的 UB 包含pthread_cond_wait()永远不会返回。 -
当你调用
pthread_cond_wait()时你应该持有互斥锁,但如果你已经持有它就不需要再次获取它。事实上,尝试这样做可能会导致程序死锁,具体取决于互斥体的属性。看起来您应该在循环开始之前锁定互斥锁一次,而不是在函数的其他地方。正如@dohashi 观察到的那样,一个互斥锁的获取与一个互斥锁的释放是配对的。