(2013/04/20 0:36), Michael Holzheu wrote:
On s390 the makedumpfile tool sometimes is used directly by
users on the command line. Currently the check if the dump
file already exists is done after the filtering function has
been called. Therefore, for large dumps the user has to wait
for filtering and after some time he gets the error message
"open_dump_file: Can't open the dump file(out). File exists".
This patch improves this by adding an early check for the
existence of the dump file. In case the -f (force) option has
been specified it is checked that an existing file is writable.
Signed-off-by: Michael Holzheu <[email protected]>
---
makedumpfile.c | 37 ++++++++++++++++++++++++++++++++++++-
1 file changed, 36 insertions(+), 1 deletion(-)
--- a/makedumpfile.c
+++ b/makedumpfile.c
@@ -730,6 +730,24 @@ open_dump_file(void)
}
int
+check_dump_file(const char *path)
+{
+ char *err_str;
+
+ if (access(path, F_OK) != 0)
+ return TRUE; /* File does not exist */
+ if (info->flag_force) {
+ if (access(path, W_OK) == 0)
+ return TRUE; /* We have write permission */
+ err_str = strerror(errno);
+ } else {
+ err_str = "File exists";
How about strerror(EEXIST)? It's better to avoid hard code to use the
same string as what libc returns.
--
Thanks.
HATAYAMA, Daisuke
_______________________________________________
kexec mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/kexec
