minates the following warning in debugfs.c:
> > drivers/acpi/debugfs.c:16:13: warning: no previous prototype for
> > ‘acpi_debugfs_init’ [-Wmissing-prototypes]
> >
> > Signed-off-by: Rashika Kheria
> > Reviewed-by: Josh Triplett
> > ---
> > drivers/acpi
On Mon, Jan 06, 2014 at 04:48:39PM +0900, Alexandre Courbot wrote:
> On Sat, Jan 4, 2014 at 5:22 AM, Josh Triplett wrote:
> > On Fri, Jan 03, 2014 at 08:07:22PM +0900, Alexandre Courbot wrote:
> >> On Thu, Jan 2, 2014 at 10:07 PM, Linus Walleij
> >> wrote:
> >
[CCing build-system folks and others likely to know about potential
issues.]
Does anyone have any objection to the use of "#pragma once" instead of
the usual #ifndef-#define-...-#endif include guard? GCC, LLVM/clang,
and the latest Sparse all support either method just fine. (I added
support to
On Mon, Jan 06, 2014 at 01:00:03PM -0800, Andrew Morton wrote:
> On Mon, 6 Jan 2014 12:47:07 -0800 Josh Triplett wrote:
>
> > Does anyone have any objection to the use of "#pragma once" instead of
> > the usual #ifndef-#define-...-#endif include guard?
>
> S
On Fri, Dec 20, 2013 at 01:23:52AM -0800, Joe Perches wrote:
> On Fri, 2013-12-20 at 01:16 -0800, Josh Triplett wrote:
> > Duplicating the regex seems awful. Could you put it in a variable?
>
> awful is a strong word and I don't have any
> issue with the duplication. Usi
s prototype for
> ‘smtcfb_setmode’ [-Wmissing-prototypes]
>
> Signed-off-by: Rashika Kheria
Reviewed-by: Josh Triplett
> drivers/staging/sm7xxfb/sm7xxfb.c |2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/sm7xxfb/sm7xxfb.c
&
totype for
> ‘t3e3_port_del_stats’ [-Wmissing-prototypes]
>
> Signed-off-by: Rashika Kheria
Reviewed-by: Josh Triplett
> drivers/staging/sbe-2t3e3/ctrl.c | 14 +++---
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/staging/sbe-2t3e3/ctrl.c
vious prototype for
> ‘dwc2_set_param_uframe_sched’ [-Wmissing-prototypes]
>
> Signed-off-by: Rashika Kheria
Reviewed-by: Josh Triplett
> drivers/staging/dwc2/core.c |2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/dwc2/core.c b
ce.c:
> drivers/staging/dgnc/dgnc_trace.c:180:6: warning: no previous prototype for
> ‘dgnc_tracer_free’ [-Wmissing-prototypes]
>
> Signed-off-by: Rashika Kheria
Reviewed-by: Josh Triplett
> drivers/staging/dgnc/dgnc_trace.c |1 +
> 1 file changed, 1 insertion(+)
>
ce.c:
> drivers/staging/dgap/dgap_trace.c:181:6: warning: no previous prototype for
> ‘dgap_tracer_free’ [-Wmissing-prototypes]
>
> Signed-off-by: Rashika Kheria
Reviewed-by: Josh Triplett
> drivers/staging/dgap/dgap_trace.c |1 +
> 1 file changed, 1 insertion(+)
>
p/dgap_parse.c:1252:7: warning: no previous prototype for
> ‘dgap_create_config_string’ [-Wmissing-prototypes]
> drivers/staging/dgap/dgap_parse.c:1311:7: warning: no previous prototype for
> ‘dgap_get_config_letters’ [-Wmissing-prototypes]
>
> Signed-off-by: Rashika Kheria
Reviewed-by:
s’
> [-Wmissing-prototypes]
>
> Signed-off-by: Rashika Kheria
Reviewed-by: Josh Triplett
> drivers/staging/bcm/Qos.c |6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/staging/bcm/Qos.c b/drivers/staging/bcm/Qos.c
> index 1609a2
rototype for
> ‘PropagateCalParamsFromEEPROMToMemory’ [-Wmissing-prototypes]
>
> Signed-off-by: Rashika Kheria
Reviewed-by: Josh Triplett
> drivers/staging/bcm/nvm.c | 61
> -
> 1 file changed, 61 deletions(-)
>
> diff --git a/drivers/staging/bcm/nvm.c
The kernel already has this information, and individual drivers
shouldn't duplicate that. This also eliminates the use of __DATE__ and
__TIME__, which make the build non-deterministic.
Signed-off-by: Josh Triplett
---
drivers/mtd/nand/denali_pci.c | 1 -
1 file changed, 1 deletion(-)
The kernel already has this information, and individual drivers
shouldn't duplicate that. This also eliminates the use of __DATE__ and
__TIME__, which make the build non-deterministic.
Signed-off-by: Josh Triplett
---
drivers/staging/wlags49_h2/wl_version.h | 4
1 file chang
The kernel already has this information, and individual drivers
shouldn't duplicate that. This also eliminates the use of __TIME__,
which makes the build non-deterministic. (And, without __DATE__,
__TIME__ provided little useful information to begin with.)
Signed-off-by: Josh Tri
ned-off-by: Josh Triplett
---
arch/x86/math-emu/errors.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/arch/x86/math-emu/errors.c b/arch/x86/math-emu/errors.c
index 59d353d..a544908 100644
--- a/arch/x86/math-emu/errors.c
+++ b/arch/x86/math-emu/errors.c
@@ -330,11 +330,6 @@ asmli
The kernel already has this information, and individual drivers
shouldn't duplicate that. This also eliminates the use of __DATE__ and
__TIME__, which make the build non-deterministic.
Signed-off-by: Josh Triplett
---
drivers/staging/rtl8188eu/os_dep/usb_intf.c | 1 -
1 file chang
(optionally) records this information at build time
in a single place; other kernel code should not duplicate that.
Signed-off-by: Josh Triplett
---
Makefile | 3 +++
1 file changed, 3 insertions(+)
diff --git a/Makefile b/Makefile
index 14d592c..188eea7 100644
--- a/Makefile
+++ b/Makefile
The kernel already has this information, and individual drivers
shouldn't duplicate that. This also eliminates the use of __DATE__ and
__TIME__, which make the build non-deterministic.
Signed-off-by: Josh Triplett
---
drivers/net/wireless/brcm80211/brcmfmac/dhd_common.c | 7 ---
1
ry space after function pointer
> name\n" . $herecurr);
> + }
> +
> +# unnecessary space "type (*funcptr) (args...)"
> + if (defined $pre_args_space &&
> + $pre_args_space =~ /^\s/) {
> +
On Thu, Dec 26, 2013 at 11:20:29AM -0800, Joe Perches wrote:
> Add the ability to fix and overwrite existing files/patches instead
> of creating a new file ".EXPERIMENTAL-checkpatch-fixes".
>
> Suggested-by: Manfred Spraul
> Signed-off-by: Joe Perches
Reviewed-by: J
if so.
>
> Signed-off-by: Joe Perches
Reviewed-by: Josh Triplett
> scripts/checkpatch.pl | 14 ++
> 1 file changed, 14 insertions(+)
>
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index db72b88..c12234e 100755
> --- a/scripts/checkpatch.
On Thu, Dec 26, 2013 at 12:07:23PM -0800, Josh Triplett wrote:
> On Thu, Dec 26, 2013 at 11:20:28AM -0800, Joe Perches wrote:
> > Kernel style uses function pointers in this form:
> > "type (*funcptr)(args...)"
> >
> > Emit warnings when
//
>
> Signed-off-by: Julia Lawall
For this patch, I think it would make sense to drop
initializations of preferred/quirks/modes to 0, since a designated
initializer may skip fields to leave them initialized to 0.
With that change:
Reviewed-by: Josh Triplett
&
> //
>
> Signed-off-by: Julia Lawall
One comment below about formatting. With that addressed:
Reviewed-by: Josh Triplett
> --- a/drivers/video/fbdev/aty/aty128fb.c
> +++ b/drivers/video/fbdev/aty/aty128fb.c
> @@ -324,14 +324,57 @@ struct aty128_meminfo {
> };
>
> /* va
0 or NULL,
> where some of the fields already use labels, and where there are nested
> structures.
I responded to patches 6 and 8 with comments; for the rest (1-5, 7, 9):
Reviewed-by: Josh Triplett
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the bo
On Sat, Aug 23, 2014 at 05:32:37PM +0200, Julia Lawall wrote:
> On Sat, 23 Aug 2014, Josh Triplett wrote:
> > On Sat, Aug 23, 2014 at 01:20:30PM +0200, Julia Lawall wrote:
> > > From: Julia Lawall
> > >
> > > Use c99 initializers for structures.
> > >
t; identifier i1,fld;
> type T;
> field list[n] fs;
> @@
>
> struct i1 {
> fs
> T fld;
> ...};
>
> @bad@
> identifier decl.i1,i2;
> expression e;
> initializer list[decl.n] is;
> @@
>
> struct i1 i2 = { is,
> + .fld = e
> - e
> ,...};
> /
e code is clear enough such as:
>
> /* clear interrupts */
> clear_interrupts(denali);
>
> Signed-off-by: Masahiro Yamada
All of the cases seem to match the preferred style; seems reasonable.
And thanks for applying the additional cleanup of removing entirely
unnecessary comments
On Mon, Sep 08, 2014 at 05:10:11PM +0900, Masahiro Yamada wrote:
> The variable "retry" in wait_for_irq() is set, but not referenced.
>
> Signed-off-by: Masahiro Yamada
Sure enough.
Reviewed-by: Josh Triplett
>
> drivers/mtd/nand/denali.c | 2 --
> 1 file chang
dence of the relevant operators is always clear.
Reviewed-by: Josh Triplett
>
> drivers/mtd/nand/denali.c | 16
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/mtd/nand/denali.c b/drivers/mtd/nand/denali.c
> index d37c2e1..ed37b76 1
pr_err("Your NAND chip OOB is not large enough to "
> +"contain 8bit ECC correction codes");
Likewise.
- Josh Triplett
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
ff-by: Masahiro Yamada
Looks sensible, since the header above the first diff hunk is the same
modulo filename.
(Side note: could we get checkpatch to complain about comment headers
containing the name of the file they appear in? They're always either
redundant or wrong.)
Review
e itself doesn't need to be visible if there are no items
> selectable (i.e. on arch/frv) or RCU_NOCB_CPU is not defined.
>
> Signed-off-by: Stefan Hengelein
> Signed-off-by: Andreas Ruprecht
Makes sense; nice cleanup.
Reviewed-by: Josh Triplett
> init/Kconfig | 6 +++---
On Sat, Aug 30, 2014 at 07:09:26AM -0700, Joe Perches wrote:
> On Fri, 2014-08-29 at 11:30 -0700, Josh Triplett wrote:
> > (Side note: could we get checkpatch to complain about comment headers
> > containing the name of the file they appear in? They're always either
&g
On Mon, Aug 04, 2014 at 01:19:59PM +0100, Matt Fleming wrote:
> On Fri, 01 Aug, at 09:11:54AM, Josh Triplett wrote:
> >
> > The original bug report was about an allocation failure for a fairly
> > reasonable BGRT size. We can certainly prohibit absurdly huge ones (for
>
his provides a
better baseline for embedded systems or efforts to reduce kernel size.
Signed-off-by: Josh Triplett
---
arch/x86/Makefile| 6 ++
arch/x86/configs/tiny.config | 5 +
2 files changed, 11 insertions(+)
create mode 100644 arch/x86/configs/tiny.config
diff --git
On Wed, Aug 06, 2014 at 03:06:11PM -0700, David Rientjes wrote:
> On Wed, 6 Aug 2014, Josh Triplett wrote:
> > Since commit 5d2acfc7b974bbd3858b4dd3f2cdc6362dd8843a ("kconfig: make
> > allnoconfig disable options behind EMBEDDED and EXPERT") in 3.15-rc1,
> > &q
path.
Signed-off-by: Josh Triplett
---
Hopefully the above changelog message makes the origin of this patch
clear; it didn't seem appropriate to keep a Signed-off-by when modifying
the patch, but I wanted to preserve the credit.
arch/x86/Makefile | 10 +++---
1 file changed, 7 inser
his provides a
better baseline for embedded systems or efforts to reduce kernel size.
Signed-off-by: Josh Triplett
---
v2: Based this on Luis's helper, added in patch 1/2, which eliminates
the previously duplicated implementation from kvmconfig.
arch/x86/Makefile| 5 +
arch/x
On Wed, Aug 06, 2014 at 03:33:45PM -0700, David Rientjes wrote:
> On Wed, 6 Aug 2014, Josh Triplett wrote:
> > The new mergeconfig helper makes it easier to add other partial
> > configurations similar to kvmconfig.
> >
> > Based on a patch by Luis R. Rodriguez .
&
On Wed, Aug 06, 2014 at 03:38:56PM -0700, David Rientjes wrote:
> On Wed, 6 Aug 2014, Josh Triplett wrote:
> > diff --git a/arch/x86/Makefile b/arch/x86/Makefile
> > index 1568678..2b5ca16 100644
> > --- a/arch/x86/Makefile
> > +++ b/arch/x86/Makefile
> > @@ -25
On Wed, Aug 06, 2014 at 04:37:54PM -0700, David Rientjes wrote:
> On Wed, 6 Aug 2014, Josh Triplett wrote:
> > On Wed, Aug 06, 2014 at 03:38:56PM -0700, David Rientjes wrote:
> > > On Wed, 6 Aug 2014, Josh Triplett wrote:
> > > > diff --git a/arch/x86/Makefile b/a
On Thu, Aug 07, 2014 at 03:13:41AM +0200, Nick Krause wrote:
> Architectures which use generic BUG() have warnings like
>
> kernel/sched/core.c:2692:1: warning: control reaches end of non-void function
> [-Wreturn-type]
> net/core/ethtool.c:236:1: warning: control reaches end of non-void functio
(), this one doesn't call panic() or anything else flagged as
__noreturn, so it seems reasonable to add a call to unreachable() here
to tell the compiler that the assembly will trap and not return.
Reviewed-by: Josh Triplett
> arch/sh/include/asm/bug.h | 1 +
> 1 file changed, 1 ins
On Thu, Aug 07, 2014 at 06:03:52PM +0200, Sam Ravnborg wrote:
> On Wed, Aug 06, 2014 at 03:31:54PM -0700, Josh Triplett wrote:
> > Since commit 5d2acfc7b974bbd3858b4dd3f2cdc6362dd8843a ("kconfig: make
> > allnoconfig disable options behind EMBEDDED and EXPERT") in 3.15-r
, and factor out more of the common file path.
Signed-off-by: Josh Triplett
---
Note that this leaves kvm_guest.config in arch/x86/configs/ . A KVM
expert is welcome to follow up with a patch splitting the options in
kvm_guest.config between arch/x86/configs/ and kernel/configs/ .
arch/x86
his provides a
better baseline for embedded systems or efforts to reduce kernel size.
Signed-off-by: Josh Triplett
---
Any architecture with architecture-specific tinification options just
needs to create arch/${arch}/configs/tiny.config, and "make tinyconfig"
will automatically
I noticed with the series: the new read functions, when given
an integer type, don't appear to do any range-checking to make sure the
returned value falls within the range of that type. They should do so,
and return -EOVERFLOW if so. And the documentation of the new functions
should note tha
On Fri, Aug 01, 2014 at 10:19:49AM +0100, Matt Fleming wrote:
> (Including akpm, the __GFP_NOWARN police)
Andrew suggested __GFP_NOWARN here in the first place.
> On Thu, 31 Jul, at 09:11:33AM, Josh Triplett wrote:
> >
> > I started to add an explicit limit, but any reaso
I'll take the series through the tiny tree when it's ready
> > to merge.
>
> I think it's rather pointless - there is very little sendfile code,
> so you'd rather want to disable splice.
That's the plan, but since sendfile depends on some of the splice bits,
s
On Tue, Oct 21, 2014 at 02:13:56AM -0700, Christoph Hellwig wrote:
> On Tue, Oct 21, 2014 at 02:04:22AM -0700, Josh Triplett wrote:
> > That's the plan, but since sendfile depends on some of the splice bits,
> > sendfile needs to be optional as well; SENDFILE_SYSC
ad"
> > - default y
> > - help
>
> The above is not in mainline, nor in -next?
> Seems it only exists in a patch series you sent yourself?
>
> Confused...
See the previous thread. The previous patch adds this config option to
make the current default behavior o
On Wed, Oct 29, 2014 at 08:12:49PM +0100, Geert Uytterhoeven wrote:
> On Wed, Oct 29, 2014 at 7:15 PM, Josh Triplett wrote:
> > Document several common practices and conventions regarding conditional
> > compilation, most notably the preference for ifdefs in headers rather
&g
On Wed, Oct 29, 2014 at 05:35:02PM -0700, Randy Dunlap wrote:
> On 10/29/14 12:12, Geert Uytterhoeven wrote:
> > On Wed, Oct 29, 2014 at 7:15 PM, Josh Triplett
> > wrote:
> >> Document several common practices and conventions regarding conditional
> >> compila
e it to ALLOW_DEV_COREDUMP to better capture its semantics.
>
> Signed-off-by: Johannes Berg
Seems reasonable. The original "DISABLE" was really "FORCE_DISABLE"
(even if used), and the inverse of "FORCE_DISABLE" is not "ENABLE", it's
"ALLOW
ater builds. This commit therefore runs
> the kernel out of the results directory so that less hand-editing is
> required to re-run a previous test.
>
> Signed-off-by: Paul E. McKenney
Reviewed-by: Josh Triplett
> tools/testing/selftests/rcutorture/bin/kvm-test-1-run.sh | 4 ++
; I had attached one in local tree for a while. or even sent it before
> several years ago.
I don't think we need the full generality of printf in the decompression
stub. I prefer Kees' patch, though I'd still like to see __puthex made
conditional.
- Josh Triplett
--
To unsubs
er, for size-constrained systems that have nowhere
for the output to go at all, it still makes sense to compile all of the
printing infrastructure out of the decompression stub, especially since
that code itself does not get compressed and thus has a higher impact
than code in vmlinux.
- Josh Triple
On Wed, Oct 29, 2014 at 10:00:54PM +0100, Thomas Gleixner wrote:
> On Wed, 29 Oct 2014, Josh Triplett wrote:
> > v3: Eliminated several #ifdefs, and in particular almost all #ifdefs in
> > .c files, by adding a macro INIT_SET_IOPL_MASK to use in place of
> >
, as requested by Thomas Gleixner.
I've verified that this compiles after each patch.
Josh Triplett (10):
x86: process: Unify 32-bit and 64-bit copy_thread I/O bitmap handling
x86: tss: Eliminate fragile calculation of TSS segment limit
x86: processor.h: Introduce macros to initiali
; move the resulting identical code to a
static inline in a common header.
Signed-off-by: Josh Triplett
Acked-by: Kees Cook
---
arch/x86/kernel/process-io.h | 22 ++
arch/x86/kernel/process_32.c | 28
arch/x86/kernel/process_64.c | 25
No functional change, just splitting out parts of the existing
INIT_THREAD and INIT_TSS macros. This will simplify making those fields
optional later.
Signed-off-by: Josh Triplett
---
arch/x86/include/asm/processor.h | 18 +++---
1 file changed, 11 insertions(+), 7 deletions
This will allow making set_iopl_mask optional later.
Signed-off-by: Josh Triplett
---
arch/x86/include/asm/paravirt_types.h | 1 +
arch/x86/kernel/paravirt.c| 2 +-
arch/x86/xen/enlighten.c | 4 ++--
3 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/arch
That then removes the last use of IO_BITMAP_OFFSET, so delete it.
Signed-off-by: Josh Triplett
Acked-by: Alexander van Heukelum
---
arch/x86/include/asm/desc.h | 11 +--
arch/x86/include/asm/processor.h | 4 +++-
2 files changed, 4 insertions(+), 11 deletions(-)
diff --git a/arch/x
This will make it easier to make the io_bitmap_ptr field optional later.
Signed-off-by: Josh Triplett
---
arch/x86/kernel/process-io.h | 5 +
arch/x86/kernel/process_32.c | 4 ++--
arch/x86/kernel/process_64.c | 2 +-
3 files changed, 8 insertions(+), 3 deletions(-)
diff --git a/arch/x86
This simplifies __switch_to a bit, and will make it easier to make iopl
optional later.
Signed-off-by: Josh Triplett
---
arch/x86/kernel/process-io.h | 13 +
arch/x86/kernel/process_32.c | 9 +
2 files changed, 14 insertions(+), 8 deletions(-)
diff --git a/arch/x86/kernel
-related fields optional
later.
Signed-off-by: Josh Triplett
---
arch/x86/include/asm/processor.h | 14 ++
arch/x86/kernel/cpu/common.c | 12 ++--
2 files changed, 16 insertions(+), 10 deletions(-)
diff --git a/arch/x86/include/asm/processor.h b/arch/x86/include/asm/processor.h
8576 384 -8192
Signed-off-by: Josh Triplett
---
arch/x86/Kconfig | 10 ++
arch/x86/include/asm/paravirt.h | 2 ++
arch/x86/include/asm/paravirt_types.h | 4
arch/x86/include/asm/processor.h | 19 ++-
arch/x86/include/asm
This simplifies __switch_to_xtra, and will make it easier to make IO
optional later.
Signed-off-by: Josh Triplett
---
arch/x86/kernel/process-io.h | 23 +++
arch/x86/kernel/process.c| 14 +-
2 files changed, 24 insertions(+), 13 deletions(-)
diff --git a
This makes the two major functions of exit_thread (IO and FPU) more
obvious, and will make it easier to make IO optional later.
Signed-off-by: Josh Triplett
---
arch/x86/kernel/process-io.h | 20
arch/x86/kernel/process.c| 20 +++-
2 files changed, 23
ny amount of core mm code that supports a gate area,
> and possibly 4k for a wasted pagetable. The latter is because the
> vsyscall addresses are misaligned and fit poorly in the fixmap.
>
> Signed-off-by: Andy Lutomirski
One minor nit below, but with or without that change,
Re
On Tue, Sep 23, 2014 at 10:36:03AM +0200, Ingo Molnar wrote:
> * Josh Triplett wrote:
> > I've dropped tiny/no-io from tiny/next; I'll poke at it further
> > and resubmit for the x86 tree later, likely not for the next
> > merge window at this point.
> >
&
That then removes the last use of IO_BITMAP_OFFSET, so delete it.
Signed-off-by: Josh Triplett
Acked-by: Alexander van Heukelum
---
v3: No changes in this patch, only in patch 3/3.
arch/x86/include/asm/desc.h | 11 +--
arch/x86/include/asm/processor.h | 4 +++-
2 files changed, 4
257-107
__switch_to_xtra 234 123-111
sys_ioperm 240 --240
init_tss8576 384 -8192
Signed-off-by: Josh Triplett
---
v3: Eliminated several #ifdefs, and in particular almost all
; move the resulting identical code to a
static inline in a common header.
Signed-off-by: Josh Triplett
Acked-by: Kees Cook
---
v3: No changes in this patch, only in patch 3/3.
arch/x86/kernel/process-io.h | 22 ++
arch/x86/kernel/process_32.c | 28
Document several common practices and conventions regarding conditional
compilation, most notably the preference for ifdefs in headers rather
than .c files.
Signed-off-by: Josh Triplett
---
I found myself explaining a few of these unwritten rules in patch
feedback, so I figured I'd doc
fine IS_ERR_VALUE(x) unlikely((x) >= (unsigned long)-MAX_ERRNO)
...
static inline bool __must_check IS_ERR(__force const void *ptr)
{
return IS_ERR_VALUE((unsigned long)ptr);
}
So unless GCC fails to propagate expected-value handling through static
inline functions, you should never need t
a way to
> > switch off DEV_COREDUMP regardless if any drivers select
> > WANT_DEV_COREDUMP.
> >
> > This patch renames the option to ENABLE_DEV_COREDUMP and setting it to
> > 'n' (as in allnconfig or tinyconfig) will effectively disable device
> >
The buffers sized by CONFIG_LOG_BUF_SHIFT and
CONFIG_LOG_CPU_MAX_BUF_SHIFT do not exist if CONFIG_PRINTK=n, so don't
ask about their size at all.
Signed-off-by: Josh Triplett
---
Posting for review. I can upstream this through the tiny tree.
init/Kconfig | 2 ++
1 file changed, 2 inser
on FUTEX. With this change, the
subsequent items display as part of the EXPERT menu again; the EMBEDDED
menu now appears as the next top-level item in the General Setup menu,
which makes General Setup much shorter and more usable.
Signed-off-by: Josh Triplett
---
Posting for review. I can upstr
On Fri, Oct 03, 2014 at 04:36:36PM -0700, Randy Dunlap wrote:
> On 10/03/14 16:31, Josh Triplett wrote:
> > commit 03b8c7b623c80af264c4c8d6111e5c6289933666 ("futex: Allow
> > architectures to skip futex_atomic_cmpxchg_inatomic() test") added the
> > HAVE_FUTEX_C
On Fri, Oct 03, 2014 at 05:00:18PM -0700, Randy Dunlap wrote:
> On 10/03/14 16:47, Josh Triplett wrote:
> > On Fri, Oct 03, 2014 at 04:36:36PM -0700, Randy Dunlap wrote:
> >> On 10/03/14 16:31, Josh Triplett wrote:
> >>> commit 03b8c7b623c80af264c4c8d61
han waiting for 3.18.
Josh Triplett (2):
init/Kconfig: Hide printk log config if CONFIG_PRINTK=n
init/Kconfig: Fix HAVE_FUTEX_CMPXCHG to not break up the EXPERT menu
init/Kconfig | 3 +++
1 file changed, 3 insertions(+)
--
To unsubscribe from this list: send the line "unsubscribe lin
--
Tinification for 3.18
----
Josh Triplett (10):
x86, platform, kconfig: move kvmconfig functionality to a helper
x86: Add "make tinyconfig" to configure the tiniest possible kernel
x86
On Sat, Oct 04, 2014 at 01:04:15PM -0400, Pranith Kumar wrote:
> Now that we have removed configs based on kernel version, we can also remove
> the
> kversion parameter in kvm.sh.
>
> Signed-off-by: Pranith Kumar
Reviewed-by: Josh Triplett
> .../selftests/rcutorture/bin
n' into
tiny/next (2014-09-22 23:14:40 -0700)
Tinification for 3.18
----
Josh Triplett (10):
x86, platform, kconfig: move kvmconfig functionality to a helper
x86: Add "make tinyconfig" to co
On Thu, Oct 23, 2014 at 06:41:08PM -0700, Alexei Starovoitov wrote:
> introduce two configs:
> - hidden CONFIG_BPF to select eBPF interpreter that classic socket filters
> depend on
> - visible CONFIG_BPF_SYSCALL (default off) that tracing and sockets can use
>
> that solves several problems:
>
On Thu, Oct 23, 2014 at 10:32:50PM -0700, Alexei Starovoitov wrote:
> On Thu, Oct 23, 2014 at 8:23 PM, Josh Triplett wrote:
> > On Thu, Oct 23, 2014 at 06:41:08PM -0700, Alexei Starovoitov wrote:
> >> introduce two configs:
> >> - hidden CONFIG_BPF to select eBPF inte
e this to the user via Kconfig?
> > >
> > > No sane person would even turn it on if they don't need it.
> >
> > IOW, I think you should just make it:
> >
> > config FLEX_ARRAY
> > def_bool n
>
> Joe Pershes complained on a similar patc
es to the MAINTAINERS
file (perhaps in the same entry as get_maintainer.pl), so that
get_maintainer.pl does not fall back to git history for them?
- Josh Triplett
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
Mor
;
> Signed-off-by: Darren Hart
> Cc: Josh Triplett
One comment below; with that addressed:
Reviewed-by: Josh Triplett
> +def main(argv):
> +try:
> +opts, args = getopt.getopt(argv[1:], "dh", ["help"])
> +except getopt.GetoptError
On Sat, Nov 08, 2014 at 12:49:29PM -0800, Joe Perches wrote:
> On Sat, 2014-11-08 at 11:01 -0800, Josh Triplett wrote:
> > On Sat, Nov 08, 2014 at 08:37:27PM +0200, Oded Gabbay wrote:
> > > CREDITS | 7 +++
> > > MAINTAINERS | 10 ++
> >
d catch, thanks. Not clear why that build failure didn't show up on
x86.
Looks like the two compat syscalls need cond_syscall lines as well.
I'll add those to the patch.
- Josh Triplett
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body
On Mon, Nov 03, 2014 at 12:10:49PM +, One Thousand Gnomes wrote:
> On Sun, 2 Nov 2014 09:33:01 -0800
> Josh Triplett wrote:
>
> > On the vast majority of modern systems, no processes will use the
> > userspsace IO syscalls, iopl and ioperm. Add a new config option,
>
gt;
> Signed-off-by: Mark Rustad
> Tested-by: Aaron Brown
> Signed-off-by: Jeff Kirsher
Good catch.
Reviewed-by: Josh Triplett
> include/linux/compiler.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/linux/compiler.h b/include/li
On Mon, Nov 03, 2014 at 09:47:40AM -0800, Joe Perches wrote:
> On Mon, 2014-11-03 at 11:46 -0500, Jonathan Corbet wrote:
> > On Wed, 29 Oct 2014 11:15:17 -0700
> > Josh Triplett wrote:
> >
> > > Document several common practices and conventions regarding condi
8780f96b54a2 (""Tree RCU": scalable
> classic RCU implementation") since it is not needed here either.
>
> We leave some tags like MODULE_AUTHOR for documentation purposes.
>
> Cc: "Paul E. McKenney"
> Cc: Josh Triplett
> Cc: Steven Rostedt
> Cc:
from early_ioremap. We need to never follow the pointer in the
> > first place after a kexec, unless we have some way to know that it's
> > actually valid.
>
> So you assume that the information from ACPI is always correct then?
> The pointer is correct, the information it po
901 - 1000 of 1376 matches
Mail list logo
