On Mon, 24 Sep 2012 12:52:52 +0200 Or Gerlitz <ogerlitz@xxxxxxxxxxxx> wrote: > On 18/09/2012 03:37, FUJITA Tomonori wrote: >> + //xiaoxi added here >> + pthread_t myThreadId=pthread_self(); >> + int myTID=0; >> +// eprintf("in bs_sheepdog_request,systid=%u\n",myThreadId); >> + int i; >> + for(i = 0; i < info->nr_worker_threads; i++) >> + { >> + dprintf("in >> for:TID=%d:sysid=%u,myid=%u\n",i,info->worker_thread[i],myThreadId); >> + if((uint32_t)info->worker_thread[i] == (uint32_t)myThreadId) >> + { >> + dprintf("equal\n"); >> + myTID=i; >> + break; >> + } >> + } >> +// eprintf("here,myTID=%d\n",myTID); >> + struct sheepdog_access_info *ai = (struct sheepdog_access_info >> *)(info + 1); >> +// >> eprintf("ai_base:%p,TID:%d,ai_addr:%p,fd=%d\n",ai,myTID,ai+myTID,(ai+myTID)->fd); >> + > > Hi Tomo, > > Is this RFC or code you are going to merge? seems that this way or > another, some cleanups are needed here... people names in the code... > and commented out debug prints left Thanks, looks like some cleanup is necessary, as you said. I post it just because I was asked. I wanted to see if anyone is interested in it. -- To unsubscribe from this list: send the line "unsubscribe stgt" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html