git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH v3 00/17] Fix some mkdir/rmdir races

From
Michael Haggerty <mhagger@alum.mit.edu>
Date
Jan 18, 2014, 22:48 UTC
Message-ID
<1390085341-2553-1-git-send-email-mhagger@alum.mit.edu>
Sorry, I forgot to send this re-roll to the mailing list.
This version differs only very slightly from v2:
* Rebased to the current master (there were no conflicts).
* Rename a couple of local variables from "attempts" to
  "attempts_remaining".
* Patches 13, 16, and 17 have slightly improved commit messages.
Michael Haggerty (17):
  safe_create_leading_directories(): fix format of "if" chaining
  safe_create_leading_directories(): reduce scope of local variable
  safe_create_leading_directories(): add explicit "slash" pointer
  safe_create_leading_directories(): rename local variable
  safe_create_leading_directories(): split on first of multiple slashes
  safe_create_leading_directories(): always restore slash at end of loop
  safe_create_leading_directories(): introduce enum for return values
  cmd_init_db(): when creating directories, handle errors conservatively
  safe_create_leading_directories(): add new error value SCLD_VANISHED
  lock_ref_sha1_basic(): on SCLD_VANISHED, retry
  lock_ref_sha1_basic(): if locking fails with ENOENT, retry
  remove_dir_recurse(): tighten condition for removing unreadable dir
  remove_dir_recurse(): handle disappearing files and directories
  rename_ref(): extract function rename_tmp_log()
  rename_tmp_log(): handle a possible mkdir/rmdir race
  rename_tmp_log(): limit the number of remote_empty_directories()
    attempts
  rename_tmp_log(): on SCLD_VANISHED, retry
 builtin/init-db.c |  9 +++---
 cache.h           | 25 +++++++++++++--
 dir.c             | 27 +++++++++++-----
 merge-recursive.c |  2 +-
 refs.c            | 92 ++++++++++++++++++++++++++++++++++++++++---------------
 sha1_file.c       | 67 ++++++++++++++++++++++------------------
 6 files changed, 155 insertions(+), 67 deletions(-)
-- 
1.8.5.2
Next: Michael Haggerty
Message 1 of 18 in “Fix some mkdir/rmdir races”
  1. 00/17 Fix some mkdir/rmdir racesMichael Haggerty, Jan 18, 2014
  2. 01/17 safe_create_leading_directories(): fix format of "if" chainingMichael Haggerty, Jan 18, 2014
  3. 02/17 safe_create_leading_directories(): reduce scope of local variableMichael Haggerty, Jan 18, 2014
  4. 03/17 safe_create_leading_directories(): add explicit "slash" pointerMichael Haggerty, Jan 18, 2014
  5. 04/17 safe_create_leading_directories(): rename local variableMichael Haggerty, Jan 18, 2014
  6. 05/17 safe_create_leading_directories(): split on first of multiple slashesMichael Haggerty, Jan 18, 2014
  7. 06/17 safe_create_leading_directories(): always restore slash at end of loopMichael Haggerty, Jan 18, 2014
  8. 07/17 safe_create_leading_directories(): introduce enum for return valuesMichael Haggerty, Jan 18, 2014
  9. 08/17 cmd_init_db(): when creating directories, handle errors conservativelyMichael Haggerty, Jan 18, 2014
  10. 09/17 safe_create_leading_directories(): add new error value SCLD_VANISHEDMichael Haggerty, Jan 18, 2014
  11. 10/17 lock_ref_sha1_basic(): on SCLD_VANISHED, retryMichael Haggerty, Jan 18, 2014
  12. 11/17 lock_ref_sha1_basic(): if locking fails with ENOENT, retryMichael Haggerty, Jan 18, 2014
  13. 12/17 remove_dir_recurse(): tighten condition for removing unreadable dirMichael Haggerty, Jan 18, 2014
  14. 13/17 remove_dir_recurse(): handle disappearing files and directoriesMichael Haggerty, Jan 18, 2014
  15. 14/17 rename_ref(): extract function rename_tmp_log()Michael Haggerty, Jan 18, 2014
  16. 15/17 rename_tmp_log(): handle a possible mkdir/rmdir raceMichael Haggerty, Jan 18, 2014
  17. 16/17 rename_tmp_log(): limit the number of remote_empty_directories() attemptsMichael Haggerty, Jan 18, 2014
  18. 17/17 rename_tmp_log(): on SCLD_VANISHED, retryMichael Haggerty, Jan 18, 2014

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.