Openembedded Core Discussions
 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox