Rooba
/
boinc
mirror of https://github.com/BOINC/boinc.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1635 from BOINC/suspicious_results 

Validator: implement "suspicious" results
This commit is contained in:
David Anderson 2017-01-31 01:18:30 -08:00 committed by GitHub
parent a484b7dd6b 998a927956
commit 04e1396360
2 changed files with 19 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
sched/validate_util2.cpp
View File

@ -80,6 +80,7 @@ int check_set(
had_error[i] = false;
}
int good_results = 0;
int suspicious_results = 0;
for (i=0; i<n; i++) {
retval = init_result(results[i], data[i]);
if (retval == ERR_OPENDIR) {
@ -89,6 +90,12 @@ int check_set(
);
retry = true;
had_error[i] = true;
} else if (retval == VAL_RESULT_SUSPICIOUS) {
log_messages.printf(MSG_NORMAL,
"[RESULT#%lu %s] considered to be suspicious\n",
results[i].id, results[i].name
);
suspicious_results++;
} else if (retval) {
log_messages.printf(MSG_CRITICAL,
"check_set: init_result([RESULT#%lu %s]) failed: %s\n",
@ -101,6 +108,14 @@ int check_set(
good_results++;
}
}
// don't count a single "suspicious" result as "good",
// but do if there are more results to compare it with
//
if (suspicious_results > 1 || good_results > 0) {
good_results += suspicious_results;
}
if (good_results < wu.min_quorum) goto cleanup;
// Compare results

4
sched/validate_util2.h
View File

@ -18,6 +18,9 @@
#ifndef _VALIDATE_UTIL2_
#define _VALIDATE_UTIL2_
// return value of init_result if an "adaptive replication" result looks suspicious
#define VAL_RESULT_SUSPICIOUS 1
#include <vector>
#include "boinc_db_types.h"
@ -31,4 +34,5 @@ extern int check_set(
DB_ID_TYPE& canonicalid, double& credit_deprecated, bool& retry
);
extern void check_pair(RESULT& r1, RESULT& r2, bool& retry);
#endif