Again

Author: chorkin

src/nvme/mi.c

@@ -2176,33 +2176,33 @@ static int validate_occ_list_update_ctx(
 							struct nvme_mi_aem_ctx *ctx,
 							bool check_generation_num)
 {
-	printf("File: %s, Line: %d\n", __FILE__, __LINE__);
+	fprintf(stderr,"File: %s, Line: %d\n", __FILE__, __LINE__);
 	//Make sure header fields have valid data
 	if (len < sizeof(*occ_header)) {
 		errno = EPROTO;
-		printf("File: %s, Line: %d\n", __FILE__, __LINE__);
+		fprintf(stderr,"File: %s, Line: %d\n", __FILE__, __LINE__);
 		goto err_cleanup;
 	} else if (occ_header->aelver != 0 ||
 			   occ_header->aeolhl != sizeof(*occ_header)) {
 		//Make sure header is the right version and length
 		errno = EPROTO;
-		printf("File: %s, Line: %d\n", __FILE__, __LINE__);
+		fprintf(stderr,"File: %s, Line: %d\n", __FILE__, __LINE__);
 		goto err_cleanup;
 	} else if (nvme_mi_aem_aeolli_get_aeoltl(occ_header->aeolli) > len) {
 		//Full length is bigger than the data that was received
 		errno = EPROTO;
-		printf("File: %s, Line: %d\n", __FILE__, __LINE__);
+		fprintf(stderr,"File: %s, Line: %d\n", __FILE__, __LINE__);
 		goto err_cleanup;
 	} else if (check_generation_num &&
 		ctx->last_generation_num ==
 			(int) nvme_mi_aem_aemti_get_aemgn(occ_header->aemti)) {
 		//This is a duplicate and shouldn't be parsed.
 		//Let's just act like there's no updates
-		printf("File: %s, Line: %d\n", __FILE__, __LINE__);
+		fprintf(stderr,"File: %s, Line: %d\n", __FILE__, __LINE__);
 
 		occ_header->numaeo = 0;
 	} else if (check_generation_num) {
-		printf("File: %s, Line: %d\n", __FILE__, __LINE__);
+		fprintf(stderr,"File: %s, Line: %d\n", __FILE__, __LINE__);
 		ctx->last_generation_num = 
 			nvme_mi_aem_aemti_get_aemgn(occ_header->aemti);
 	}
@@ -2222,11 +2222,11 @@ static int validate_occ_list_update_ctx(
 	for (int i = 0; i < occ_header->numaeo; i++) {
 		//Validate this item
 		if (current->aelhlen != sizeof(*current)) {
-			printf("File: %s, Line: %d\n", __FILE__, __LINE__);
+			fprintf(stderr,"File: %s, Line: %d\n", __FILE__, __LINE__);
 			errno = EPROTO;
 			goto err_cleanup;
 		} else if (!ctx->callbacks.enabled_map.enabled[current->aeoui.aeoi]) {
-			printf("File: %s, Line: %d\n", __FILE__, __LINE__);
+			fprintf(stderr,"File: %s, Line: %d\n", __FILE__, __LINE__);
 			//This is unexpected as this AE shouldn't be enabled
 			errno = EPROTO;
 			goto err_cleanup;
@@ -2237,8 +2237,8 @@ static int validate_occ_list_update_ctx(
 
 		bytes_so_far += offset;
 		if (bytes_so_far > nvme_mi_aem_aeolli_get_aeoltl(occ_header->aeolli)) {
-			printf("Get() = %d\n", nvme_mi_aem_aeolli_get_aeoltl(occ_header->aeolli));
-			printf("File: %s, Line: %d\n", __FILE__, __LINE__);
+			fprintf(stderr,"Get() = %d and bsf=%zd\n", nvme_mi_aem_aeolli_get_aeoltl(occ_header->aeolli), bytes_so_far);
+			fprintf(stderr,"File: %s, Line: %d\n", __FILE__, __LINE__);
 			errno = EPROTO;
 			goto err_cleanup;
 		}