X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=vss.c;h=2e952eb4c6a1b0202a69ae79e0bab9a153fd0b04;hp=5c66c9813abad0b795fd1d3be12691c909c9afcd;hb=edeb499676e6d042ef1a913914a9fcb45a8cadde;hpb=b2d30c6f4ca248ca04d00edf40027e4cf3a0152d

diff --git a/vss.c b/vss.c
index 5c66c981..2e952eb4 100644
--- a/vss.c
+++ b/vss.c
@@ -169,6 +169,8 @@ struct fec_client {
 	int num_extra_slices;
 	/** Contains the FEC-encoded data. */
 	unsigned char *enc_buf;
+	/** Maximal packet size. */
+	int mps;
 };
 
 /**
@@ -207,26 +209,26 @@ static void write_fec_header(struct fec_client *fc, struct vss_task *vsst)
 	write_u32(buf + 14, g->bytes);
 
 	write_u8(buf + 18, fc->current_slice_num);
-	write_u16(buf + 20, p->max_slice_bytes - FEC_HEADER_SIZE);
+	write_u16(buf + 20, fc->mps - FEC_HEADER_SIZE);
 	write_u8(buf + 22, g->first_chunk? 0 : 1);
 	write_u8(buf + 23, vsst->header_len? 1 : 0);
 	memset(buf + 24, 0, 7);
 }
 
-static int need_audio_header(struct fec_client *fc, struct vss_task *vsst)
+static bool need_audio_header(struct fec_client *fc, struct vss_task *vsst)
 {
 	if (!mmd->current_chunk) {
 		tv_add(now, &vsst->header_interval, &fc->next_header_time);
-		return 0;
+		return false;
 	}
 	if (!vsst->header_buf)
-		return 0;
-	if (!vsst->header_len)
-		return 0;
+		return false;
+	if (vsst->header_len == 0)
+		return false;
 	if (fc->group.num && tv_diff(&fc->next_header_time, now, NULL) > 0)
-		return 0;
+		return false;
 	tv_add(now, &vsst->header_interval, &fc->next_header_time);
-	return 1;
+	return true;
 }
 
 static int num_slices(long unsigned bytes, int mps, int rs)
@@ -236,7 +238,7 @@ static int num_slices(long unsigned bytes, int mps, int rs)
 
 	assert(m > 0);
 	assert(rs > 0);
-	ret = (bytes + m - 1) / m;
+	ret = DIV_ROUND_UP(bytes, m);
 	if (ret + rs > 255)
 		return -E_BAD_CT;
 	return ret;
@@ -281,8 +283,7 @@ static int initialize_fec_client(struct fec_client *fc, struct vss_task *vsst)
 	if (ret < 0)
 		goto err;
 	hs = ret;
-	ret = num_slices(afh_get_largest_chunk_size(&mmd->afd.afhi),
-		mps, rs);
+	ret = num_slices(mmd->afd.max_chunk_size, mps, rs);
 	if (ret < 0)
 		goto err;
 	ds = ret;
@@ -304,7 +305,7 @@ static int initialize_fec_client(struct fec_client *fc, struct vss_task *vsst)
 	fc->extra_src_buf = para_realloc(fc->extra_src_buf, mps);
 	memset(fc->extra_src_buf, 0, mps);
 
-	fc->fcp->max_slice_bytes = mps;
+	fc->mps = mps;
 	fc->state = FEC_STATE_READY_TO_RUN;
 	fc->next_header_time.tv_sec = 0;
 	fc->stream_start = *now;
@@ -346,14 +347,13 @@ static int setup_next_fec_group(struct fec_client *fc, struct vss_task *vsst)
 		g->first_chunk += g->num_chunks;
 		g->num++;
 	}
-	slice_bytes = fc->fcp->max_slice_bytes - FEC_HEADER_SIZE;
+	slice_bytes = fc->mps - FEC_HEADER_SIZE;
 	PARA_CRIT_LOG("slice_bytes: %d\n", slice_bytes);
 	k = fc->fcp->data_slices_per_group + fc->num_extra_slices;
 	n = fc->fcp->slices_per_group + fc->num_extra_slices;
 	PARA_CRIT_LOG("k: %d, n: %d\n", k, n);
 	if (need_audio_header(fc, vsst)) {
-		ret = num_slices(vsst->header_len, fc->fcp->max_slice_bytes,
-			n - k);
+		ret = num_slices(vsst->header_len, slice_bytes, n - k);
 		if (ret < 0)
 			return ret;
 		g->num_header_slices = ret;
@@ -432,8 +432,7 @@ static int compute_next_fec_slice(struct fec_client *fc, struct vss_task *vsst)
 	}
 	write_fec_header(fc, vsst);
 	fec_encode(fc->parms, fc->src_data, fc->enc_buf + FEC_HEADER_SIZE,
-		fc->current_slice_num,
-		fc->fcp->max_slice_bytes - FEC_HEADER_SIZE);
+		fc->current_slice_num, fc->mps - FEC_HEADER_SIZE);
 	return 1;
 }
 
@@ -875,13 +874,8 @@ static void vss_send(struct vss_task *vsst)
 			&due, 1) < 0)
 		return;
 	list_for_each_entry_safe(fc, tmp_fc, &fec_client_list, node) {
-		switch (fc->state) {
-		case FEC_STATE_DISABLED:
+		if (fc->state == FEC_STATE_DISABLED)
 			continue;
-		case FEC_STATE_NONE:
-		case FEC_STATE_READY_TO_RUN:
-			break;
-		}
 		if (!next_slice_is_due(fc, NULL)) {
 			fec_active = 1;
 			continue;
@@ -889,9 +883,8 @@ static void vss_send(struct vss_task *vsst)
 		if (compute_next_fec_slice(fc, vsst) <= 0)
 			continue;
 		PARA_DEBUG_LOG("sending %d:%d (%u bytes)\n", fc->group.num,
-			fc->current_slice_num, fc->fcp->max_slice_bytes);
-		fc->fcp->send_fec(fc->sc, (char *)fc->enc_buf,
-				  fc->fcp->max_slice_bytes);
+			fc->current_slice_num, fc->mps);
+		fc->fcp->send_fec(fc->sc, (char *)fc->enc_buf, fc->mps);
 		fc->current_slice_num++;
 		fec_active = 1;
 	}