From 9df43aacab0f311c15837b230761a11750f8b9cb Mon Sep 17 00:00:00 2001 From: Michael Meffie Date: Wed, 1 Aug 2012 17:26:33 -0400 Subject: [PATCH] comerr: compile_et -emit option for parallel make Add the -emit option to the compile_et command to support parallel make. The -emit option allows make to generate the header and the source files independently, instead of building two files at the some time. This avoids the issue where one command creates two separate files, which is difficult to handle correctly for parallel makes. Change-Id: Ib44a8e358643cf19b4834b3bd4d5b88db6cd0ccf Reviewed-on: http://gerrit.openafs.org/7921 Tested-by: BuildBot Reviewed-by: Marc Dionne Reviewed-by: Benjamin Kaduk Reviewed-by: Derrick Brashear --- doc/man-pages/pod1/afs_compile_et.pod | 27 ++++- src/comerr/compile_et.c | 152 ++++++++++++++++---------- src/comerr/error_table.y | 12 +- src/comerr/error_table_nt.c | 12 +- 4 files changed, 137 insertions(+), 66 deletions(-) diff --git a/doc/man-pages/pod1/afs_compile_et.pod b/doc/man-pages/pod1/afs_compile_et.pod index eaa1b4907..e60593d0b 100644 --- a/doc/man-pages/pod1/afs_compile_et.pod +++ b/doc/man-pages/pod1/afs_compile_et.pod @@ -9,7 +9,8 @@ afs_compile_et - Produce error text tables for compilation B [B<-debug>] S<<< [B<-language> >] >>> S<<< [B<-prefix> >] >>> S<<< [B<-v> >] >>> - S<<< [B<-h> >] >>> > + S<<< [B<-h> >] >>> >] >>> + I> =for html @@ -26,6 +27,13 @@ The error table specification should exist in the current working directory or in the directory specified with B<-prefix> and should be named F. +By default, B generates two files in one invocation. This is +problematic for parallel build systems. The B<-emit> option may be used to +generate the output files independently with two separate invocations of +B for a given error table. This allows parallel build systems +to generate the source and header files, and the targets which depend on the +generated source and headers files, in parallel. + =head1 CAUTIONS This command is used internally within the build process for OpenAFS. @@ -79,6 +87,13 @@ The B<-h> option does not affect the source file generated by B. Specified the type of output file: valid values are 1 (the default, for C files) or 2, for B<.msf> file generation. +=item B<-emit> > + +Specifies which program file to generate; the header file or the source file. +Specify B<-emit header> (or B<-emit h>) to generate the F<.h> header file. +Specify B<-emit source> (or B<-emit c>) to generate the F<.c> (or F<.msf>) +source file. + =back =head1 EXAMPLES @@ -92,6 +107,16 @@ The following command generates K&R style files instead: % afs_compile_et -p path/to/src/ptserver -lang 'k&r-c' pterror +The following command generates the F file, but not the F +file. + + % afs_compile_et -p path/to/src/ptserver -emit header pterror + +The following command generates the F file, but not the F +file. + + % afs_compile_et -p path/to/src/ptserver -emit source pterror + =head1 SEE ALSO L diff --git a/src/comerr/compile_et.c b/src/comerr/compile_et.c index be75156e7..3f3f02da9 100644 --- a/src/comerr/compile_et.c +++ b/src/comerr/compile_et.c @@ -27,9 +27,11 @@ extern char *current_token; extern int table_number, current; char buffer[BUFSIZ]; char *table_name = NULL; -FILE *hfile, *cfile, *msfile; +FILE *hfile = NULL, *cfile = NULL, *msfile = NULL; int version = 1; int use_msf = 0; +int emit_source = 0; +int emit_header = 0; /* lex stuff */ extern FILE *yyin; @@ -78,6 +80,12 @@ static const char *const prefix_args[] = { 0, }; +static const char *const emit_args[] = { + "e", + "emit", + 0, +}; + static const char *const language_names[] = { "C", "K&R C", @@ -122,7 +130,7 @@ static void usage(void) { fprintf(stderr, - "%s: usage: %s ERROR_TABLE [-debug] [-language LANG] [-h INCLUDE] [-p prefix] [-v version]\n", + "%s: usage: %s ERROR_TABLE [-debug] [-language LANG] [-h INCLUDE] [-p PREFIX] [-v VERSION] [-emit source|header]\n", whoami, whoami); exit(1); } @@ -145,6 +153,7 @@ main(int argc, char **argv) int got_language = 0; char *got_include = 0; char *got_prefix = "."; + int got_emit = 0; char lcname[6]; #ifdef AFS_AIX32_ENV @@ -237,6 +246,24 @@ main(int argc, char **argv) } if (version == 2) use_msf = 1; + } else if (check_arg(emit_args, arg)) { + arg = *++argv; + argc--; + got_emit = 1; + if (!strcasecmp(arg, "source")) { + emit_source = 1; + } else if (!strcasecmp(arg, "c")) { + emit_source = 1; + } else if (!strcasecmp(arg, "header")) { + emit_header = 1; + } else if (!strcasecmp(arg, "h")) { + emit_header = 1; + } else { + fprintf(stderr, "%s: unknown emit argument - `%s'\n", + whoami, arg); + usage(); + exit(1); + } } else { fprintf(stderr, "%s: unknown control argument -`%s'\n", whoami, arg); @@ -254,6 +281,9 @@ main(int argc, char **argv) exit(1); } + if (!got_emit) { + emit_source = emit_header = 1; /* generate both by default */ + } p = strrchr(filename, '/'); if (p == NULL) @@ -270,15 +300,19 @@ main(int argc, char **argv) *p = 0; } - if (use_msf) { - sprintf(msf_file, "%s.msf", ename); - } else { - sprintf(c_file, "%s.c", ename); + if (emit_source) { + if (use_msf) { + sprintf(msf_file, "%s.msf", ename); + } else { + sprintf(c_file, "%s.c", ename); + } } - if (got_include) { - sprintf(h_file, "%s.h", got_include); - } else { - sprintf(h_file, "%s.h", ename); + if (emit_header) { + if (got_include) { + sprintf(h_file, "%s.h", got_include); + } else { + sprintf(h_file, "%s.h", ename); + } } p = strrchr(filename, '.'); if (p == NULL) { @@ -300,13 +334,15 @@ main(int argc, char **argv) yyout = stdout; } - hfile = fopen(h_file, "w"); - if (hfile == NULL) { - perror(h_file); - exit(1); + if (emit_header) { + hfile = fopen(h_file, "w"); + if (hfile == NULL) { + perror(h_file); + exit(1); + } + fprintf(hfile, warning, h_file); } - fprintf(hfile, warning, h_file); - if (got_include) { + if (emit_header && got_include) { char buffer[BUFSIZ]; char prolog_h_file[MAXPATHLEN]; FILE *prolog_hfile; @@ -340,14 +376,14 @@ main(int argc, char **argv) } } - if (use_msf) { + if (emit_source && use_msf) { msfile = fopen(msf_file, "w"); if (msfile == NULL) { perror(msf_file); exit(1); } fprintf(msfile, msf_warning, msf_file); - } else { + } else if (emit_source) { cfile = fopen(c_file, "w"); if (cfile == NULL) { perror(c_file); @@ -371,46 +407,52 @@ main(int argc, char **argv) fclose(yyin); /* bye bye input file */ if (!use_msf) { - fputs(" 0\n};\n\n", cfile); - fprintf(cfile, - "static const struct error_table et = { text, %ldL, %d };\n\n", - (long int)table_number, current); - fputs("static struct et_list etlink = { 0, &et};\n\n", cfile); - fprintf(cfile, "void initialize_%s_error_table(void) {\n", - table_name); - fputs(" afs_add_to_error_table(&etlink);\n", cfile); - fputs("}\n", cfile); - fclose(cfile); - - - fprintf(hfile, "extern void initialize_%s_error_table(void);\n", - table_name); + if (cfile) { + fputs(" 0\n};\n\n", cfile); + fprintf(cfile, + "static const struct error_table et = { text, %ldL, %d };\n\n", + (long int)table_number, current); + fputs("static struct et_list etlink = { 0, &et};\n\n", cfile); + fprintf(cfile, "void initialize_%s_error_table(void) {\n", + table_name); + fputs(" afs_add_to_error_table(&etlink);\n", cfile); + fputs("}\n", cfile); + fclose(cfile); + } + if (hfile) { + fprintf(hfile, "extern void initialize_%s_error_table(void);\n", + table_name); + } } else { - fprintf(hfile, "#define initialize_%s_error_table(void)\n", - table_name); + if (hfile) { + fprintf(hfile, "#define initialize_%s_error_table(void)\n", + table_name); + } } - fprintf(hfile, "#define ERROR_TABLE_BASE_%s (%ldL)\n", table_name, - (long int)table_number); - /* compatibility... */ - fprintf(hfile, "\n/* for compatibility with older versions... */\n"); - fprintf(hfile, "#define init_%s_err_tbl initialize_%s_error_table\n", - table_name, table_name); - fprintf(hfile, "#define %s_err_base ERROR_TABLE_BASE_%s\n", table_name, - table_name); - fprintf(hfile, "\n/* for compatibility with other users... */\n"); - lcstring(lcname, table_name, sizeof(lcname)); - fprintf(hfile, "#define ERROR_TABLE_BASE_%s (%ldL)\n", lcname, - (long int)table_number); - fprintf(hfile, "#define init_%s_err_tbl initialize_%s_error_table\n", - lcname, table_name); - fprintf(hfile, - "#define initialize_%s_error_table initialize_%s_error_table\n", - lcname, table_name); - fprintf(hfile, "#define %s_err_base ERROR_TABLE_BASE_%s\n", lcname, - lcname); - fclose(hfile); /* bye bye include file */ - if (use_msf) + if (hfile) { + fprintf(hfile, "#define ERROR_TABLE_BASE_%s (%ldL)\n", table_name, + (long int)table_number); + /* compatibility... */ + fprintf(hfile, "\n/* for compatibility with older versions... */\n"); + fprintf(hfile, "#define init_%s_err_tbl initialize_%s_error_table\n", + table_name, table_name); + fprintf(hfile, "#define %s_err_base ERROR_TABLE_BASE_%s\n", + table_name, table_name); + fprintf(hfile, "\n/* for compatibility with other users... */\n"); + lcstring(lcname, table_name, sizeof(lcname)); + fprintf(hfile, "#define ERROR_TABLE_BASE_%s (%ldL)\n", lcname, + (long int)table_number); + fprintf(hfile, "#define init_%s_err_tbl initialize_%s_error_table\n", + lcname, table_name); + fprintf(hfile, + "#define initialize_%s_error_table initialize_%s_error_table\n", + lcname, table_name); + fprintf(hfile, "#define %s_err_base ERROR_TABLE_BASE_%s\n", lcname, + lcname); + fclose(hfile); /* bye bye include file */ + } + if (msfile) fclose(msfile); return 0; } diff --git a/src/comerr/error_table.y b/src/comerr/error_table.y index afb2bdf0f..58113f44a 100644 --- a/src/comerr/error_table.y +++ b/src/comerr/error_table.y @@ -183,7 +183,7 @@ void add_ec(const char *name, const char *description) #else fprintf(msfile, "%d %s\n", current, description); #endif /* !sun */ - } else { + } else if (cfile){ fprintf(cfile, "\t\"%s\",\n", description); } if (error_codes == NULL) { @@ -206,7 +206,7 @@ void add_ec_val(const char *name, const char *val, const char *description) } while (ncurrent > current) { - if (!msfile) + if (cfile) fputs("\tNULL,\n", cfile); current++; } @@ -217,7 +217,7 @@ void add_ec_val(const char *name, const char *val, const char *description) #else fprintf(msfile, "%d %s\n", current, description); #endif /* ! sun */ - } else { + } else if (cfile) { fprintf(cfile, "\t\"%s\",\n", description); } if (error_codes == NULL) { @@ -233,6 +233,8 @@ void add_ec_val(const char *name, const char *val, const char *description) void put_ecs(void) { int i; + if (!hfile) + return; for (i = 0; i < current; i++) { if (error_codes[i] != NULL) fprintf(hfile, "#define %-40s (%ldL)\n", @@ -275,7 +277,7 @@ int char_to_num(char c) void set_table_num(char *string) { - if (msfile) { + if (use_msf) { set_table_1num(string); return; } @@ -312,7 +314,7 @@ void set_table_fun(char *astring) exit(1); } } - if (msfile) + if (use_msf) table_number += (atoi(astring)) << 28; } diff --git a/src/comerr/error_table_nt.c b/src/comerr/error_table_nt.c index f5ea2b2be..6bcef5b6c 100755 --- a/src/comerr/error_table_nt.c +++ b/src/comerr/error_table_nt.c @@ -999,7 +999,7 @@ add_ec(const char *name, const char *description) #else fprintf(msfile, "%d %s\n", current, description); #endif /* !sun */ - } else { + } else if (cfile) { fprintf(cfile, "\t\"%s\",\n", description); } if (error_codes == NULL) { @@ -1021,7 +1021,7 @@ add_ec_val(const char *name, const char *val, const char *description) } while (ncurrent > current) { - if (!msfile) + if (cfile) fputs("\tNULL,\n", cfile); current++; } @@ -1032,7 +1032,7 @@ add_ec_val(const char *name, const char *val, const char *description) #else fprintf(msfile, "%d %s\n", current, description); #endif /* ! sun */ - } else { + } else if (cfile) { fprintf(cfile, "\t\"%s\",\n", description); } if (error_codes == NULL) { @@ -1048,6 +1048,8 @@ void put_ecs(void) { int i; + if (!hfile) + return; for (i = 0; i < current; i++) { if (error_codes[i] != NULL) fprintf(hfile, "#define %-40s (%ldL)\n", error_codes[i], @@ -1087,7 +1089,7 @@ char_to_num(char c) void set_table_num(char *string) { - if (msfile) { + if (use_msf) { set_table_1num(string); return; } @@ -1124,7 +1126,7 @@ set_table_fun(char *astring) exit(1); } } - if (msfile) + if (use_msf) table_number += (atoi(astring)) << 28; } -- 2.39.5