[sheepdog] [PATCH 6/7] sheep: split check_request

Liu Yuan namei.unix at gmail.com
Thu May 31 16:22:05 CEST 2012


On 05/31/2012 05:48 PM, Christoph Hellwig wrote:

> +	/*
> +	 * If we go for a cached object, we don't care if it is busy
> +	 * or being recovered.
> +	 */
> +	if (req->local_oid &&
> +	    (!sys->enable_write_cache ||
> +	     !(req->rq.flags & SD_FLAG_CMD_CACHE) ||
> +	     !object_is_cached(req->rq.obj.oid))) {
> +		if (check_request_busy(req) < 0)
> +			return;
> +	}


You mentioned you were writing object lock for farm, if so I think we'd
better put it in this series, then is_access_to_busy_objects() and
resume_pending_requests() can be wiped out.

Thanks,
Yuan



More information about the sheepdog mailing list