18 Dec, 2009
8 commits
-
This reverts commit e4c570c4cb7a95dbfafa3d016d2739bf3fdfe319, as
requested by Alexey:"I think I gave a good enough arguments to not merge it.
To iterate:
* patch makes impossible to start using ext3 on EXT3_FS=n kernels
without reboot.
* this is done only for one pointer on task_struct"None of config options which define task_struct are tristate directly
or effectively."Requested-by: Alexey Dobriyan
Acked-by: Andrew Morton
Signed-off-by: Linus Torvalds -
This reverts commit e9496ff46a20a8592fdc7bdaaf41b45eb808d310. Quoth Al:
"it's dependent on a lot of other stuff not currently in mainline
and badly broken with current fs/namespace.c. Sorry, badly
out-of-order cherry-pick from old queue.PS: there's a large pending series reworking the refcounting and
lifetime rules for vfsmounts that will, among other things, allow to
rip a subtree away _without_ dissolving connections in it, to be
garbage-collected when all active references are gone. It's
considerably saner wrt "is the subtree busy" logics, but it's nowhere
near being ready for merge at the moment; this changeset is one of the
things becoming possible with that sucker, but it certainly shouldn't
have been picked during this cycle. My apologies..."Noticed-by: Eric Paris
Requested-by: Al Viro
Signed-off-by: Linus Torvalds -
* 'master' of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs-2.6:
kill I_LOCK
fold do_sync_file_range into sys_sync_file_range
fix up O_SYNC comments
VFS/fsstack: handle 32-bit smp + preempt + large files in fsstack_copy_inode_size
fsstack/ecryptfs: remove unused get_nlinks param to fsstack_copy_attr_all
vfs: remove extraneous NULL d_inode check from do_filp_open
fs: no games with DCACHE_UNHASHED
fs: anon_inodes implement dname
dio: fix use-after-free -
* 'for-linus' of git://gitserver.sunplusct.com/linux-2.6-score:
score: include asm-generic/param.h in asm/delay.h.
score: fixed pfn_valid define.
score: add flush_dcahce_page and PG_dcache_dirty define -
* 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/lrg/voltage-2.6: (27 commits)
regulator: wm831x_reg_read() failure unnoticed in wm831x_aldo_get_mode()
twl-regulator: Fix reg_disable functionality for 4030 and 6030
twl-regulator: Add turnon delay to reg_enable
twl-regulator: Restore REMAP configuration in regulator probe
twl-regulator: Add turnon-delay and REMAP config to twlreg_info struct
twl-regulator: Define critical regulators as always_on
twl-regulator: Add all twl4030 regulators to twlreg_info
regulator: mc13783-regulator: correct the probing time.
regulator: Fix unbalanced disables/enables in regulator_bulk_{enable,disable} error path
regulator: core.c: Small coding style cleanup (indentation fixup)
drivers/regulator: use PTR_ERR to get error code
regulator: consumer.h - fix build when consumer.h is #included first.
regulator/mc13783: various cleanups
regulator/mc13783: rename source file to match other drivers
Fix some AB3100 regulator issues
regulator: keep index within bounds in da9034_get_ldo12_voltage()
regulator: Ensure val is initialised in 88pm8607 choose_voltage()
regulator: Remove duplicate consts from ab3100
regulator: Handle regulators without suspend mode configuration
regulator: Factor out regulator name pretty printing
... -
After I_SYNC was split from I_LOCK the leftover is always used together with
I_NEW and thus superflous.Signed-off-by: Christoph Hellwig
Signed-off-by: Al Viro -
We recently go rid of all callers of do_sync_file_range as they're better
served with vfs_fsync or the filemap_write_and_wait. Now that
do_sync_file_range is down to a single caller fold it into it so that people
don't start using it again accidentally. While at it also switch it from
using __filemap_fdatawrite_range(..., WB_SYNC_ALL) to the more clear
filemap_fdatawrite_range().Signed-off-by: Christoph Hellwig
Signed-off-by: Al Viro -
Proper Posix O_SYNC handling only made it into 2.6.33, not 2.6.32.
Signed-off-by: Christoph Hellwig
Signed-off-by: Al Viro
17 Dec, 2009
32 commits
-
Copy the inode size and blocks from one inode to another correctly on 32-bit
systems with CONFIG_SMP, CONFIG_PREEMPT, or CONFIG_LBDAF. Use proper inode
spinlocks only when i_size/i_blocks cannot fit in one 32-bit word.Signed-off-by: Hugh Dickins
Signed-off-by: Erez Zadok
Signed-off-by: Al Viro -
This get_nlinks parameter was never used by the only mainline user,
ecryptfs; and it has never been used by unionfs or wrapfs either.Acked-by: Dustin Kirkland
Acked-by: Tyler Hicks
Signed-off-by: Erez Zadok
Signed-off-by: Al Viro -
We can't get to this point unless it's a valid pointer.
Signed-off-by: Jeff Layton
Cc: Al Viro
Cc: Christoph Hellwig
Signed-off-by: Andrew Morton
Signed-off-by: Al Viro -
Filesystems outside the regular namespace do not have to clear DCACHE_UNHASHED
in order to have a working /proc/$pid/fd/XXX. Nothing in proc prevents the
fd link from being used if its dentry is not in the hash.Also, it does not get put into the dcache hash if DCACHE_UNHASHED is clear;
that depends on the filesystem calling d_add or d_rehash.So delete the misleading comments and needless code.
Acked-by: Miklos Szeredi
Signed-off-by: Nick Piggin
Signed-off-by: Al Viro -
Add a d_dname method for anon_inodes filesystem, the same way pipefs and
sockfs pseudo filesystems. This allows us to remove the DCACHE_UNHASHED
hack from anon_inodes.c (see next patch).[AV: inumber is useless here, dropped from anon_inodefs_dname()]
Signed-off-by: Nick Piggin
Cc: Miklos Szeredi
Cc: Davide Libenzi
Cc: "David S. Miller"
Cc: Jens Axboe
Cc: Christoph Hellwig
Cc: Al Viro
Signed-off-by: Andrew Morton
Signed-off-by: Al Viro -
* 'for-33' of git://repo.or.cz/linux-kbuild: (29 commits)
net: fix for utsrelease.h moving to generated
gen_init_cpio: fixed fwrite warning
kbuild: fix make clean after mismerge
kbuild: generate modules.builtin
genksyms: properly consider EXPORT_UNUSED_SYMBOL{,_GPL}()
score: add asm/asm-offsets.h wrapper
unifdef: update to upstream revision 1.190
kbuild: specify absolute paths for cscope
kbuild: create include/generated in silentoldconfig
scripts/package: deb-pkg: use fakeroot if available
scripts/package: add KBUILD_PKG_ROOTCMD variable
scripts/package: tar-pkg: use tar --owner=root
Kbuild: clean up marker
net: add net_tstamp.h to headers_install
kbuild: move utsrelease.h to include/generated
kbuild: move autoconf.h to include/generated
drop explicit include of autoconf.h
kbuild: move compile.h to include/generated
kbuild: drop include/asm
kbuild: do not check for include/asm-$ARCH
...Fixed non-conflicting clean merge of modpost.c as per comments from
Stephen Rothwell (modpost.c had grown an include of linux/autoconf.h
that needed to be changed to generated/autoconf.h) -
This reverts commit 8546e3ce6e37c359979b69862442f94e0ef0e03b, as it's a
partial duplicate of commit 2f390380ca69e1617cdddb12d8da94f0a9f4319d
("fbdev: add palette register check to several drivers").The former went in first through the m68k tree, the latter through Andrew
Morton.Signed-off-by: Geert Uytterhoeven
Signed-off-by: Linus Torvalds -
The loop condition is fragile: we compare an unsigned value to zero, and
then decrement it by something larger than one in the loop. All the
callers should be passing in appropriately aligned buffer lengths, but
it's better to just not rely on it, and have some appropriate defensive
loop limits.Acked-by: Roland McGrath
Signed-off-by: Linus Torvalds -
Signed-off-by: Cui Bixiong
Signed-off-by: Chen Liqin -
Signed-off-by: Cui Bixiong
Signed-off-by: Chen Liqinmodified: arch/score/include/asm/page.h
modified: arch/score/kernel/setup.c
modified: arch/score/mm/init.c -
Signed-off-by: Cui Bixiong
Signed-off-by: Chen Liqinmodified: arch/score/include/asm/cacheflush.h
modified: arch/score/mm/cache.c -
ret should be signed to notice a failure in wm831x_reg_read().
Signed-off-by: Roel Kluin
Acked-by: Mark Brown
Signed-off-by: Liam Girdwood -
This change makes sure all regulator group assignments are cleared on
disable callSigned-off-by: Juha Keski-Saari
Acked-by: Mark Brown
Signed-off-by: Liam Girdwood -
This change implements a basic turnon delay in the regulator enable function
to make it less probable that reg_enable returns before the regulator
output is at target levelSigned-off-by: Juha Keski-Saari
Acked-by: Mark Brown
Signed-off-by: Liam Girdwood -
This change ensures the regulator REMAP register configuration is in a known
state so state transitions will function as intended regardless of
possible bootloader effects on itSigned-off-by: Juha Keski-Saari
Acked-by: Mark Brown
Signed-off-by: Liam Girdwood -
This change includes regulator turnon delay values and the REMAP reset
configuration to the twlreg_info struct, since they are basic attributes
of every TWL regulatorSigned-off-by: Juha Keski-Saari
Acked-by: Mark Brown
Signed-off-by: Liam Girdwood -
Defines VIO, VDD1, VDD2, VPLL1 and VINT* regulators as always_on by default
since they are critical to TWL and its master's functionality and should
be on in all cases where RegFW is usedSigned-off-by: Juha Keski-Saari
Acked-by: Mark Brown
Signed-off-by: Liam Girdwood -
Define all twl4030 regulators in the twlreg_info table, along with
appropriate VSEL tables for adjustable regulatorsSigned-off-by: Juha Keski-Saari
Acked-by: Mark Brown
Signed-off-by: Liam Girdwood -
When the mc13783-regulator driver is built in kernel, probing it during
the regulator subsystem initialisation result in a fault.That is because regulator subsystem is planned to be initialised very early
in the boot process, before the mfd subsystem initialisation.The mc12783-regulator probing process need to access to the mc13783-core
functionality to read/write mc13783 registers and so must be called after
the mc13783-core driver initialisation.The way to do this is to let the kernel probe the mc13783-regulator driver when
mc13783-core register his regulator subdevice.Signed-off-by: Alberto Panizzo
Acked-by: Mark Brown
Signed-off-by: Liam Girdwood -
Currently it is possible for regulator_bulk_{enable,disable} operations to
generate unbalanced regulator_{disable,enable} calls in its error path.
In case of an error only those regulators of the bulk operation which actually
had been enabled/disabled should get their original state restored.Signed-off-by: Lars-Peter Clausen
Acked-by: Mark Brown
Signed-off-by: Liam Girdwood -
Signed-off-by: Stefan Roese
Cc: Liam Girdwood
Cc: Mark Brown
Acked-by: Mark Brown
Signed-off-by: Liam Girdwood -
IS_ERR returns only 1 or 0. The callsite of setup_regulators expects a
negative integer in an error case. Thus, PTR_ERR has to be used to extract
it.The semantic match that finds this problem is as follows:
(http://coccinelle.lip6.fr/)//
@@
expression E,E1;
@@*E = IS_ERR(...)
... when != E = E1
*return E;
//Signed-off-by: Julia Lawall
Acked-by:
Signed-off-by: Liam Girdwood -
consumer.h requires device.h for stand alone build.
Signed-off-by: Liam Girdwood
-
- define needed registers and bits in the driver
- properly namespace functions and structs
- fix locking as required by patch
"mfd/mc13783: near complete rewrite"
- use platform_data as provided by "mfd/mc13783: near complete rewrite"
instead of accessing struct mc13783
- struct mc13783_regulator_priv.desc is (and was) unused and so can go
away
- use cpp magic to initialize mc13783_regulators
- bring MODULE_LICENSE in sync with actual copyright
- minor style fixesThis allows not including mc13783-private.h which I intend to remove
soon.Signed-off-by: Uwe Kleine-König
Cc: Sascha Hauer
Cc: Liam Girdwood
Cc: Mark Brown
Cc: Samuel Ortiz
Acked-by: Mark Brown
Signed-off-by: Liam Girdwood -
One annoying thing about the old name was that the module was just
called mc13783 which caused wrong expectations (at least for me).Signed-off-by: Uwe Kleine-König
Cc: Sascha Hauer
Cc: Liam Girdwood
Cc: Mark Brown
Cc: Samuel Ortiz
Acked-by: Mark Brown
Signed-off-by: Liam Girdwood -
This patch will remove surplus register writes on shut down of
LDO D (this magic was not needed), remove an unnecessary (!) error
check and really unregister the regulators when the module is
unloaded.Signed-off-by: Linus Walleij
Signed-off-by: Liam Girdwood -
If selector equals ARRAY_SIZE(da9034_ldo12_data), that is one too
large already.Signed-off-by: Roel Kluin
Acked-by: Mark Brown
Signed-off-by: Liam Girdwood -
If we fall through it means that we hit an unknown regulator/chip
combination so set -ENOENT as an explicit flag (the return code
is only used internally).Signed-off-by: Mark Brown
Signed-off-by: Liam Girdwood -
'static const int const' means the same thing as 'static const int'
and sparse complains about this.Signed-off-by: Mark Brown
Signed-off-by: Liam Girdwood -
Since some regulators in the system may not support suspend mode
configuration we need to allow some regulators to have a missing
suspend mode configuration. Do this by requiring that disabled
regulators are explicitly flagged and then skip over regulators
that have no state specified.Try to avoid surprises by warning the if we could set the state
but no configuration is provided. This also ensures that an all
zeros configuration generates a warning rather than silently
disabling the regulator.Reported-by: Joonyoung Shim
Signed-off-by: Mark Brown
Signed-off-by: Liam Girdwood -
Some of the regulator API functions have code to allow the machine
constraints to override the device supplied name for the regulator
in the constraints in order to help tie logging to supplies on the
board and disambiguate when there is more than one regulator chip
in the system. Factor this code out into a new rdev_get_name()
function and use it throughout the regulator API so that we always
use the same name.Signed-off-by: Mark Brown
Signed-off-by: Liam Girdwood -
When voltage or current constraints are either missing or specify
a range display the actual setting along with the constraints if
we can. This can aid debugging of configuration problems.Signed-off-by: Mark Brown
Signed-off-by: Liam Girdwood
