Start using our new "busy" criteria for objcores

bin/varnishd/cache/cache_hash.c

@@ -398,8 +398,9 @@ HSH_Lookup(struct req *req, struct objcore **ocp, struct objcore **bocp,
 		if (oc->flags & OC_F_FAILED)
 			continue;
 
-		if (oc->flags & OC_F_BUSY) {
+		if (oc->boc != NULL && oc->boc->state < BOS_STREAM) {
 			CHECK_OBJ_ORNULL(oc->boc, BOC_MAGIC);
+
 			if (req->hash_ignore_busy)
 				continue;
 

bin/varnishd/cache/cache_req_fsm.c

@@ -203,7 +203,9 @@ cnt_deliver(struct worker *wrk, struct req *req)
 			HSH_DerefBusy(wrk, req->objcore);
 			boc = NULL;
 		} else if (!boc->busyobj->do_stream) {
-			ObjWaitState(boc, BOS_FINISHED);
+			/* XXX These should go away */
+			xxxassert(boc->state >= BOS_STREAM);
+			xxxassert(boc->state >= BOS_FINISHED);
 			HSH_DerefBusy(wrk, req->objcore);
 			boc = NULL;
 		}

bin/varnishtest/tests/c00013.vtc

@@ -43,5 +43,5 @@ client c2 {
 
 client c1 -wait
 
-varnish v1 -expect busy_sleep == 1
-varnish v1 -expect busy_wakeup == 1
+varnish v1 -expect busy_sleep >= 1
+varnish v1 -expect busy_wakeup >= 1