Re: [PATCH] sheepdog backing store support

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

 



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


[Index of Archives]     [Linux SCSI]     [Linux RAID]     [Linux Clusters]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]

  Powered by Linux