All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Tollerton <rich.tollerton@ni.com>
To: openembedded-core@lists.openembedded.org
Cc: otavio@ossystems.com.br
Subject: [PATCH v2 00/12] udev-cache related changes
Date: Fri, 22 Aug 2014 16:30:46 -0500	[thread overview]
Message-ID: <cover.1408742574.git.rich.tollerton@ni.com> (raw)

This is v2 of the changes submitted by Ben Shelton on August 4. I've reworked the
patches quite a bit, larger changes include:

- All non-udev-cache-related patches have been omitted. I'll submit those
  separately.
- Use `find` to build `tar` filelist. This eliminates the need for GNU- and
  busybox-specific tar options.
- No new settings have been added to udev-cache.default in order to preserve
  compatibility with older overrides of udev-cache.default. Note, this means a
  lot of new settings are duplicated between initscripts.
- System config is now generated immediately before the tarball is generated,
  and is manipulated at the file level instead of the shell variable level.
- Internal/corporate commit message fields stripped.
- Spurious whitespace changes should all be fixed.

There is one requested fix I'm pushing back on: Copying both $CMP_FILE_LIST and
/dev to tmpfs in a single operation to minimize race conditions. For a variety
of reasons, I abandoned trying to make this work in a single command. Instead
it's done in two: grabbing the system configuration (i.e reading /proc files and
udev rules) is done first, then /dev is archived as an uncompressed tarball.
Notably, using tar instead of cp means that I can use `find -xdev` and `find
-type` to eliminate a lot of other complexity. I think this approach will wind
up being faster than the straight-up copy approach, although I have not made any
attempt to confirm that.

The following changes since commit 47d1fc9f5c38f3d092937c47bd4c2f45adaa7fe6:

  qemu: fix Darwin cross-compilation (2014-08-18 20:43:24 +0100)

are available in the git repository at:

  git://github.com/rtollert/openembedded-core dev/rtollert/udev-cache-7
  https://github.com/rtollert/openembedded-core/tree/dev/rtollert/udev-cache-7

Richard Tollerton (12):
  udev-cache: Update cache tarball atomically
  udev-cache: Compress the cache
  udev-cache: choose a more descriptive cache filename
  busybox: enable `tar -m`
  udev-cache: strip timestamps on extract
  udev-cache: omit sockets and filesystems mounted under /dev
  udev-cache: Don't ignore error messages from cache extract
  udev-cache: parametrize sysconf file paths
  udev-cache: get system config immediately before cache
  udev-cache: invalidate on rules.d changes
  udev-cache: Update cache asynchronously
  udev-cache: refactor; improve verbosity and error handling

 meta/recipes-core/busybox/busybox/defconfig    |  2 +-
 meta/recipes-core/udev/udev/init               | 53 ++++++++++++--------------
 meta/recipes-core/udev/udev/udev-cache         | 42 ++++++++++++++++++--
 meta/recipes-core/udev/udev/udev-cache.default |  2 +-
 4 files changed, 64 insertions(+), 35 deletions(-)

-- 
2.0.4



             reply	other threads:[~2014-08-22 21:31 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-22 21:30 Richard Tollerton [this message]
2014-08-22 21:30 ` [PATCH v2 01/12] udev-cache: Update cache tarball atomically Richard Tollerton
2014-08-22 21:30 ` [PATCH v2 02/12] udev-cache: Compress the cache Richard Tollerton
2014-08-22 21:30 ` [PATCH v2 03/12] udev-cache: choose a more descriptive cache filename Richard Tollerton
2014-08-22 21:30 ` [PATCH v2 04/12] busybox: enable `tar -m` Richard Tollerton
2014-08-22 21:30 ` [PATCH v2 05/12] udev-cache: strip timestamps on extract Richard Tollerton
2014-08-22 21:30 ` [PATCH v2 06/12] udev-cache: omit sockets and filesystems mounted under /dev Richard Tollerton
2014-08-22 21:30 ` [PATCH v2 07/12] udev-cache: Don't ignore error messages from cache extract Richard Tollerton
2014-08-22 21:30 ` [PATCH v2 08/12] udev-cache: parametrize sysconf file paths Richard Tollerton
2014-08-22 21:30 ` [PATCH v2 09/12] udev-cache: get system config immediately before cache Richard Tollerton
2014-08-23 11:11   ` Richard Purdie
2014-08-25 18:20     ` Richard Tollerton
2014-08-22 21:30 ` [PATCH v2 10/12] udev-cache: invalidate on rules.d changes Richard Tollerton
2014-08-22 21:30 ` [PATCH v2 11/12] udev-cache: Update cache asynchronously Richard Tollerton
2014-08-22 21:30 ` [PATCH v2 12/12] udev-cache: refactor; improve verbosity and error handling Richard Tollerton

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1408742574.git.rich.tollerton@ni.com \
    --to=rich.tollerton@ni.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=otavio@ossystems.com.br \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.