Re: [PATCH v2 1/3] media: V3s: Add support for Allwinner CSI.

On Thu, Nov 23, 2017 at 09:14:44AM +0800, Yong wrote: > > On Wed, Nov 22, 2017 at 09:33:06AM +0800, Yong wrote: > > > > On Thu, Jul 27, 2017 at 01:01:35PM +0800, Yong Deng wrote: > > > > > Allwinner V3s SoC have two CSI module. CSI0 is used for MIPI interface > > > > > and CSI1 is used for

Re: [PATCH 20/43] x86/entry: Clean up SYSENTER_stack code

On Fri, Nov 24, 2017 at 06:23:48PM +0100, Ingo Molnar wrote: > From: Andy Lutomirski > > The existing code was a mess, mainly because C arrays are nasty. > Turn SYSENTER_stack into a struct, add a helper to find it, and do > all the obvious cleanups this enables. > >

Re: [PATCH 24/43] x86/mm/kaiser: Mark per-cpu data structures required for entry/exit

On Fri, 24 Nov 2017, Ingo Molnar wrote: > diff --git a/arch/x86/include/asm/desc.h b/arch/x86/include/asm/desc.h > index aab4fe9f49f8..300090d1c209 100644 > --- a/arch/x86/include/asm/desc.h > +++ b/arch/x86/include/asm/desc.h > @@ -46,7 +46,7 @@ struct gdt_page { > struct desc_struct

[PATCH] Staging: sm750fb: Fix coding style issue in ddk750_sii164.c

This is a patch to the ddk750_sii164.c file that fixes line length warnings found by the checkpatch.pl script Signed-off-by: Jeremy Lacomis --- drivers/staging/sm750fb/ddk750_sii164.c | 39 +++-- 1 file changed, 23 insertions(+), 16 deletions(-)

[PATCH 2/2] staging: sm750b: Fix coding style issues in sm750_accel.c

This is a patch to sm750_accel.c that fixes 80-character line length warnings found by checkpatch.pl. It also fixes some grammatical errors in comments and moves parameter-specific comments from inline to before the function. Signed-off-by: Jeremy Lacomis ---

[PATCH 1/2] staging: sm750fb: Fix coding style in ddk750_sii164.h

This patch to ddk750_sii164.h fixes line length warnings found by the checkpatch.pl script and reformats comments uniformly. Signed-off-by: Jeremy Lacomis --- drivers/staging/sm750fb/ddk750_sii164.h | 57 + 1 file changed, 22 insertions(+),

Re: [PATCH] x86/orc: Don't bail on stack overflow

On Sat, Nov 25, 2017 at 9:28 AM, Andy Lutomirski wrote: > If we overflow the stack into a guard page and then try to unwind > it with ORC, it should work perfectly: by construction, there can't > be any meaningful data in the guard page because no writes to the > guard page will

Re: [patch V4 01/11] Documentation: Add license-rules.rst to describe how to properly identify file licenses

On Sat 2017-11-25 09:11:58, Linus Torvalds wrote: > On Sat, Nov 25, 2017 at 9:04 AM, Pavel Machek wrote: > > > > That does not sound like he was deciding between /* */ and //. And > > actually this was in context of files with no existing license. You > > made the ugly patches. Stop

Re: [PATCH 42/43] x86/mm/kaiser: Allow KAISER to be enabled/disabled at runtime

On Fri, 24 Nov 2017, Ingo Molnar wrote: > From: Dave Hansen > > The KAISER CR3 switches are expensive for many reasons. Not all systems > benefit from the protection provided by KAISER. Some of them can not > pay the high performance cost. > > This patch adds a

[PATCH v6 03/11] x86: define IA32_FEATURE_CONTROL.SGX_ENABLE

From: Sean Christopherson When IA32_FEATURE_CONTROL.SGX_ENABLE and IA32_FEATURE_CONTROL.LOCK are set by the pre-boot firmware, SGX is usable by the OS. Signed-off-by: Sean Christopherson Signed-off-by: Jarkko Sakkinen

[PATCH v6 06/11] intel_sgx: driver for Intel Software Guard Extensions

Intel SGX is a set of CPU instructions that can be used by applications to set aside private regions of code and data. The code outside the enclave is disallowed to access the memory inside the enclave by the CPU access control. SGX driver provides a ioctl API for loading and initializing

[PATCH v6 10/11] intel_sgx: glue code for in-kernel LE

Glue code for hosting in-kernel Launch Enclave (LE) by using the user space helper framework. Tokens for launching enclaves are generated with by the following protocol: 1. The driver sends a SIGSTRUCT blob to the LE hosting process to the input pipe. 2. The LE hosting process reads the

[PATCH v6 11/11] intel_sgx: driver documentation

Signed-off-by: Jarkko Sakkinen --- Documentation/index.rst | 1 + Documentation/x86/intel_sgx.rst | 101 2 files changed, 102 insertions(+) create mode 100644 Documentation/x86/intel_sgx.rst diff --git

[PATCH v6 08/11] intel_sgx: in-kernel launch enclave

This commits implements the in-kernel launch enclave. It is wrapped into a user space program that reads SIGSTRUCT instances from stdin and outputs launch tokens to stdout. The commit also adds enclave signing tool that is used by kbuild to measure and sign the launch enclave.

[PATCH v6 04/11] x86: define IA32_FEATUE_CONTROL.SGX_LC

When IA32_FEATURE_CONTROL.SGX_LC identifies that the root key for enclave signatures can be configured either by the OS or pre-boot firmware. If this the case, IA32_SGXLEPUBKEYHASHn MSRs (0 < n < 4) can be used to set the SHA256 of the root key. IA32_FEATURE_CONTROL bit 17 controls whether the

[PATCH v6 09/11] fs/pipe.c: export create_pipe_files() and replace_fd()

Exported create_pipe_files() and replace_fd() because the SGX driver needs to be able to setup pipes in order to communicate with the helper process that hosts the Launch Enclave (LE). The pipe creation will be done in the init-callback supplied to call_usermodehelper_setup(). The driver will use

[PATCH 5/5] video: smscufx: Adjust three checks for null pointers

From: Markus Elfring Date: Sat, 25 Nov 2017 21:21:20 +0100 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script “checkpatch.pl” pointed information out like the following. Comparison to NULL could be written … Thus

Re: WTF? Re: [PATCH] License cleanup: add SPDX GPL-2.0 license identifier to files with no license

On Sat, 2017-11-25 at 20:12 +0100, Pavel Machek wrote: > [Linus Torvalds...] > > Having it be the first line of the file is good, it's obvious, and > > stands out, which is the point, you want it to, it's a license :) > > What is good about that? License is about the least interesting thing >

Re: [PATCH v2 2/3] iio: adc: ina2xx: Use LSB specifier instead of divider in config

On Sunday, November 19, 2017 5:15:03 PM CET Jonathan Cameron wrote: > On Sat, 28 Oct 2017 23:12:47 +0200 > > Stefan Brüns wrote: > > While the config uses the physical value corresponding to the LSB > > for both the power and the bus voltage register, the shunt

Re: [PATCH 42/43] x86/mm/kaiser: Allow KAISER to be enabled/disabled at runtime

> On Nov 25, 2017, at 1:05 PM, Thomas Gleixner wrote: > > On Sat, 25 Nov 2017, Andy Lutomirski wrote: On Nov 25, 2017, at 12:18 PM, Thomas Gleixner wrote: On Fri, 24 Nov 2017, Ingo Molnar wrote: From: Dave Hansen

[PATCH] NFC: use timer_setup() helper

Use timer_setup function instead of initializing timer with the function and data fields. net/nfc/core.c:1097:2-12: Use setup_timer function for function on line 1098 net/nfc/hci/core.c:1007:1-11: Use setup_timer function for function on line 1008 net/nfc/hci/llc_shdlc.c:766:1-11: Use setup_timer

Re: [PATCH] ASoC: fsl_ssi: Override bit clock rate based on slot number

Hi Nicolin, > On Wed, Sep 13, 2017 at 10:02:20AM +0200, Arnaud Mouiche wrote: > > > >Could you please give me a few set of examples of how you set > > >set_sysclk(), set_tdm_slot() with the current driver? The idea > > >here is to figure out a way to calculate the bclk in hw_params > > >without

Re: [GIT PULL] afs: Fixes

Linus Torvalds wrote: > However, even when you do that, the page can be writable in other > mappings. At least fork(), for example, only clears the dirty bit, > doesn't mark it write-protected. I assumed the rmap walk done by page_mkclean() would take care of that

Re: [PATCH] Input: elantech - add new icbody type 15

On Sun, Nov 19, 2017 at 10:56:47PM +0800, Aaron Ma wrote: > The touchpad of Lenovo Thinkpad L480 reports it's version as 15. > > Cc: sta...@vger.kernel.org > Signed-off-by: Aaron Ma Applied, thank you. > --- > drivers/input/mouse/elantech.c | 2 +- > 1 file changed, 1

Re: [PATCH v2] xfs: handle register_shrinker error

Dave Chinner wrote: > IOWs, we don't actually need to touch this code, but if you really > must, just remove the KM_NOFS tag. OK. Then, please remove KM_NOFS. GFP_KERNEL is safer than GFP_NOFS in the sense that it won't cause OOM lockup due to unable to invoke the OOM killer.

[lib/rbtree,drm/mm] 653c9f9a4d: BUG:kernel_hang_in_boot_stage

FYI, we noticed the following commit (built with gcc-5): commit: 653c9f9a4dd8037ffc5afbb1040d15566aa8f533 ("lib/rbtree,drm/mm: Add rbtree_replace_node_cached()") git://anongit.freedesktop.org/drm-intel topic/core-for-CI in testcase: boot on test machine: qemu-system-i386 -enable-kvm -smp 2 -m

Re: [PATCH v2] checkpatch: Add a warning for log messages that don't end in a new line

On Sat, 25 Nov 2017, Logan Gunthorpe wrote: > Check for lines with a log function using a relatively strict regular > expression catching only printk, dev_* and pr_* functions. Once > one is found, accumulate further lines for any functions that > are split over multiple lines. I don't

[PATCH] auxdisplay: img-ascii-lcd: add MODULE_LICENSE("GPL")

This matches the header at the top of the file and squashes: WARNING: modpost: missing MODULE_LICENSE() in drivers/auxdisplay/img-ascii-lcd.o see include/linux/module.h for more information Signed-off-by: Daniel Axtens --- drivers/auxdisplay/img-ascii-lcd.c | 2 ++ 1 file

d7be102f29 ("cfg80211: initialize regulatory keys/database later"): kernel BUG at crypto/asymmetric_keys/public_key.c:80!

FYI, we noticed the following commit (built with gcc-4.8): commit: d7be102f2945a626f55e0501e52bb31ba3e77b81 ("cfg80211: initialize regulatory keys/database later") https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git master in testcase: boot on test machine: qemu-system-x86_64

Re: [PATCHv2 2/2] userns: control capabilities of some user namespaces

Quoting Mahesh Bandewar (mah...@bandewar.net): > From: Mahesh Bandewar > > With this new notion of "controlled" user-namespaces, the controlled > user-namespaces are marked at the time of their creation while the > capabilities of processes that belong to them are controlled

[PATCH] Staging: comedi: das16: Fixed a const struct coding style issue

Fixed a coding style issue. Signed-off-by: Alex Frappier Lachapelle --- drivers/staging/comedi/drivers/das16.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/comedi/drivers/das16.c

Re: [PATCH 1/6] perf: Add new type PERF_TYPE_PROBE

On 11/24/17 12:28 AM, Peter Zijlstra wrote: On Thu, Nov 23, 2017 at 10:31:29PM -0800, Alexei Starovoitov wrote: unfortunately 32-bit is more screwed than it seems: $ cat align.c #include struct S { unsigned long long a; } s; struct U { unsigned long long a; } u; int main() {

Re: [PATCH] x86/orc: Don't bail on stack overflow

On Sat, Nov 25, 2017 at 6:40 PM, Josh Poimboeuf wrote: > On Sat, Nov 25, 2017 at 04:16:23PM -0800, Andy Lutomirski wrote: >> Can you send me whatever config and exact commit hash generated this? >> I can try to figure out why it failed. > > Sorry, I've been traveling. I just

Re: [PATCH] x86/orc: Don't bail on stack overflow

On Sat, Nov 25, 2017 at 08:25:12PM -0800, Andy Lutomirski wrote: > On Sat, Nov 25, 2017 at 6:40 PM, Josh Poimboeuf wrote: > > On Sat, Nov 25, 2017 at 04:16:23PM -0800, Andy Lutomirski wrote: > >> Can you send me whatever config and exact commit hash generated this? > >> I can

[PATCH v2] checkpatch: Add a warning for log messages that don't end in a new line

Check for lines with a log function using a relatively strict regular expression catching only printk, dev_* and pr_* functions. Once one is found, accumulate further lines for any functions that are split over multiple lines. Stop accumulating the log function line when we find a ';' or match the

Re: [PATCHv2 1/2] capability: introduce sysctl for controlled user-ns capability whitelist

Quoting Mahesh Bandewar (mah...@bandewar.net): > From: Mahesh Bandewar > > Add a sysctl variable kernel.controlled_userns_caps_whitelist. This > takes input as capability mask expressed as two comma separated hex > u32 words. The mask, however, is stored in kernel as

Re: [PATCH] input: pegasus_notetaker: add license information

Hi Martin, On Sat, Nov 18, 2017 at 09:45:18AM +0100, Martin Kepplinger wrote: > This adds an SPDX license identifier to this driver I wrote some time back. > > Signed-off-by: Martin Kepplinger > --- > drivers/input/tablet/pegasus_notetaker.c | 1 + > 1 file changed, 1

Sparse warnings from sched.h

Hi! Did these: ./include/linux/sched.h:476:62: error: dubious one-bit signed bitfield ./include/linux/sched.h:477:62: error: dubious one-bit signed bitfield ./include/linux/sched.h:478:62: error: dubious one-bit signed bitfield ./include/linux/sched.h:479:62: error: dubious one-bit signed

Re: [PATCH] x86/orc: Don't bail on stack overflow

On Sat, Nov 25, 2017 at 04:16:23PM -0800, Andy Lutomirski wrote: > Can you send me whatever config and exact commit hash generated this? > I can try to figure out why it failed. Sorry, I've been traveling. I just got some time to take a look at this. I think there are at least two unwinder

[PATCH] fs: befs: btree: Fixed some coding standard issues

Fixed some coding standard issues. Signed-off-by: Alex Frappier Lachapelle --- fs/befs/btree.c | 97 ++--- 1 file changed, 51 insertions(+), 46 deletions(-) diff --git a/fs/befs/btree.c b/fs/befs/btree.c

Re: [PATCH] x86/orc: Don't bail on stack overflow

On Sat, Nov 25, 2017 at 10:41:15PM -0600, Josh Poimboeuf wrote: > On Sat, Nov 25, 2017 at 08:25:12PM -0800, Andy Lutomirski wrote: > > On Sat, Nov 25, 2017 at 6:40 PM, Josh Poimboeuf wrote: > > > On Sat, Nov 25, 2017 at 04:16:23PM -0800, Andy Lutomirski wrote: > > >> Can you

Re: [PATCH v2] checkpatch: Add a warning for log messages that don't end in a new line

On Sun, 2017-11-26 at 06:51 +0100, Julia Lawall wrote: > > On Sat, 25 Nov 2017, Logan Gunthorpe wrote: > > > Check for lines with a log function using a relatively strict regular > > expression catching only printk, dev_* and pr_* functions. Once > > one is found, accumulate further lines for

Re: perf test LLVM & clang 6 failing

On 11/24/17 11:09 AM, Arnaldo Carvalho de Melo wrote: Em Fri, Nov 24, 2017 at 04:16:52PM +0100, Daniel Borkmann escreveu: [ +Yonghong ] On 11/24/2017 03:47 PM, Arnaldo Carvalho de Melo wrote: FYI, just noticed, recently updated clang to version 6, from its upstream git repo. Do you recall

Re: [PATCH v2 1/5] mm: memory_hotplug: Memory hotplug (add) support for arm64

On Fri, Nov 24, 2017 at 4:23 PM, Maciej Bielski wrote: > On Fri, Nov 24, 2017 at 09:42:33AM +, Andrea Reale wrote: >> Hi Arun, >> >> >> On Fri 24 Nov 2017, 11:25, Arun KS wrote: >> > On Thu, Nov 23, 2017 at 4:43 PM, Maciej Bielski >> >

[PATCH v2 2/5] video: smscufx: Less checks in ufx_usb_probe() after error detection

From: Markus Elfring Date: Sun, 26 Nov 2017 08:18:20 +0100 Up to four checks could be repeated by the ufx_usb_probe() function during error handling even if the relevant properties can be determined for the involved variables before by source code analysis. *

linux-next: Signed-off-by missing for commit in the hdlcd tree

Hi Liviu, Commit 7f2189b12359 ("drm: Fix checkpatch issue: "WARNING: braces {} are not necessary for single statement blocks."") is missing a Signed-off-by from its committer. -- Cheers, Stephen Rothwell

[PATCH v2] Staging: sm750fb: Fix coding style issue in ddk750_sii164.c

This is a patch to the ddk750_sii164.c file that fixes line length warnings found by the checkpatch.pl script Signed-off-by: Jeremy Lacomis --- Changes in v2: - Change definition of i2cReadReg/i2cWriteReg - Remove unnecessary casts drivers/staging/sm750fb/ddk750_sii164.c |

Re: [PATCH 2/2] staging: rtl8188eu: Fix private WEXT IOCTL calls

Hi, What was broken was private/device specific IOCTL calls implemented by this driver. The standard IOCTL calls worked and the driver worked as it was in client mode. But in AP mode with hostapd (https://w1.fi/hostapd/) the rtl871xdrv driver of hostapd (which is required for using devices

Re: [PATCH 20/43] x86/entry: Clean up SYSENTER_stack code

> On Nov 25, 2017, at 9:50 AM, Thomas Gleixner wrote: > > On Sat, 25 Nov 2017, Borislav Petkov wrote: >>> - >>> +wrmsr(MSR_IA32_SYSENTER_ESP, (unsigned long)(cpu_SYSENTER_stack(cpu) + >>> 1), 0); >>>wrmsr(MSR_IA32_SYSENTER_EIP, (unsigned long)entry_SYSENTER_32, 0);

Re: [PATCH v2 2/2] x86: disable IRQs before changing CR4

On Sat, 25 Nov 2017, Nadav Amit wrote: > Thomas Gleixner wrote: > > > On Fri, 24 Nov 2017, Nadav Amit wrote: > >> /* Set in this cpu's CR4. */ > >> -static inline void cr4_set_bits(unsigned long mask) > >> +static inline void cr4_set_bits_irqs_off(unsigned long mask) > > > >

Re: [PATCH 20/43] x86/entry: Clean up SYSENTER_stack code

On Sat, Nov 25, 2017 at 9:10 AM, Borislav Petkov wrote: > On Sat, Nov 25, 2017 at 06:03:36PM +0100, Thomas Gleixner wrote: >> > Maybe I should rename cpu_tss to cpu_tss_rw in that patch. >> >> For clarity that would be nice. > > + a comment stating the alias mapping. It took tglx

hi3521a syscon-reboot issue, reboot fails oddly.

Greetings, Having a slight issue with getting reboot to work on the board I'm tinkering with; according to the documentation writing any value to 0x12050004 should reset the system, as such I have the following snippet in my dts to make it work: sysctrl: system-controller@1205 {

Re: [PATCH v2] VSOCK: Don't call vsock_stream_has_data in atomic context

From: Jorgen Hansen Date: Fri, 24 Nov 2017 06:25:28 -0800 > When using the host personality, VMCI will grab a mutex for any > queue pair access. In the detach callback for the vmci vsock > transport, we call vsock_stream_has_data while holding a spinlock, > and

Re: [PATCH] hwmon: (pmbus) Use 64bit math for DIRECT format values

Hi Robert, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on hwmon/hwmon-next] [also build test WARNING on v4.14 next-20171124] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [patch V4 01/11] Documentation: Add license-rules.rst to describe how to properly identify file licenses

On Sat, Nov 25, 2017 at 9:53 AM, Mauro Carvalho Chehab wrote: > > Let me see if I got it straight. At drivers/media/common/siano/smsir.c, > we have, currently: .. snip snip .. > With is completely out of standard. I'd like to add an SPDX tag there > and, while doing

[PATCH] ALSA: ice1724: Fix resume issues with Prodigy 7.1 HiFi

There are two issues after resuming from suspend on the Audiotrak Prodigy 7.1 HiFi: - the output volume is set to 100% - microphone input isn't working anymore This patch fixes these issues by reinitializing both codecs of the device and restoring the previous volumes during resume.

Re: [PATCH] Support TrackStick of Thinkpad L570

On Mon, Nov 20, 2017 at 04:55:30PM +0900, Masaki Ota wrote: > From: Masaki Ota > - The issue is that Thinkpad L570 TrackStick does not work. Because the main > interface of Thinkpad L570 device is SMBus, so ALPS overlooked PS2 interface > Firmware setting of TrackStick.

Re: [PATCH] x86/mm/kaiser: Fix IRQ entries text section mapping

On Sat, 25 Nov 2017, Ingo Molnar wrote: > kaiser_add_user_map_ptrs_early(__entry_text_start, __entry_text_end, > __PAGE_KERNEL_RX | _PAGE_GLOBAL); > + kaiser_add_user_map_ptrs_early(__irqentry_text_start, > __irqentry_text_end, > +

[PATCH 2/2] video: ssd1307fb: Improve a size determination in ssd1307fb_probe()

From: Markus Elfring Date: Sat, 25 Nov 2017 16:50:26 +0100 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style

Re: [PATCH 20/43] x86/entry: Clean up SYSENTER_stack code

On Sat, Nov 25, 2017 at 06:03:36PM +0100, Thomas Gleixner wrote: > > Maybe I should rename cpu_tss to cpu_tss_rw in that patch. > > For clarity that would be nice. + a comment stating the alias mapping. It took tglx and me a while on IRC to figure it out. :-) Thx. -- Regards/Gruss, Boris.

Re: [PATCH v1 03/10] v4l: platform: Add Renesas CEU driver

Hi Sakari! On Sat, Nov 25, 2017 at 05:56:14PM +0200, Sakari Ailus wrote: > On Fri, Nov 17, 2017 at 10:33:55AM +0100, jacopo mondi wrote: > > Hi Sakari! > > [snip] > > I would like to make sure we're all on the same page with this. My > > preference would be: > > > > 1) Have renesas-ceu.c driver

Re: [PATCH v1 08/10] media: i2c: ov772x: Remove soc_camera dependencies

On Fri, Nov 17, 2017 at 10:14:51AM +0100, jacopo mondi wrote: > Hi Sakari! > > On Fri, Nov 17, 2017 at 02:43:15AM +0200, Sakari Ailus wrote: > > Hi Jacopo, > > > > On Wed, Nov 15, 2017 at 11:56:01AM +0100, Jacopo Mondi wrote: > > > > > [snip] > > > > +#include > > > #include > > > #include

Re: [GIT PULL] afs: Fixes

On Fri, Nov 24, 2017 at 4:22 AM, David Howells wrote: > > (2) Don't write to a page that's being written out, but wait for it to > complete. So I see in the commit message why afs needs to do this, but it's worth pointing out that it's (a) impossible to avoid the

Re: [PATCH 42/43] x86/mm/kaiser: Allow KAISER to be enabled/disabled at runtime

On Sat, 25 Nov 2017, Andy Lutomirski wrote: > > On Nov 25, 2017, at 12:18 PM, Thomas Gleixner wrote: > >> On Fri, 24 Nov 2017, Ingo Molnar wrote: > >> > >> From: Dave Hansen > >> > >> The KAISER CR3 switches are expensive for many reasons. Not

Re: [PATCH] x86/mm/kaiser: Fix IRQ entries text section mapping

On Sat, 25 Nov 2017 17:08:08 +0100 (CET) Thomas Gleixner wrote: > Tracing uses it to stop the printout of the function graph. Should be safe > to print the C-Code functions. It uses it to filter out interrupts, as well as to print the: "<==" "==>"

[PATCH 0/5] video-SMSC UFX: Adjustments for five function implementations

From: Markus Elfring Date: Sat, 25 Nov 2017 21:38:42 +0100 A few update suggestions were taken into account from static source code analysis. Markus Elfring (5): Delete an error message for a failed memory allocation in two functions Less checks in

Re: [GIT PULL] afs: Fixes

Linus Torvalds wrote: > So I see in the commit message why afs needs to do this, but it's > worth pointing out that it's > > (a) impossible to avoid the "inconsistent data" case for writable mmap'ed > pages Doesn't clear_page_dirty_for_io() write-protect the

Re: regression: 4.13 cannot follow symlinks on some ext3 fs

Am 25.11.2017 um 23:32 schrieb Dave Chinner: On Fri, Nov 24, 2017 at 03:03:37PM -0700, Andreas Dilger wrote: Any worse an idea than running a new kernel on an old system? Newer e2fsck fixes a lot of bugs that are present in older e2fsck as well... I'm running with everything up to date

Re: [patch V4 01/11] Documentation: Add license-rules.rst to describe how to properly identify file licenses

Pavel: On Sat, Nov 25, 2017 at 7:51 PM, Pavel Machek wrote: > On Fri 2017-11-17 15:06:39, Mauro Carvalho Chehab wrote: >> Hi Thomas, >> >> Em Fri, 17 Nov 2017 11:00:33 +0100 (CET) >> Thomas Gleixner escreveu: >> >> > Subject: Documentation: Add

Re: [PATCH v2] powerpc: fix boot on BOOK3S_32 with CONFIG_STRICT_KERNEL_RWX

On Thu, Nov 23, 2017 at 11:04 PM, Michael Ellerman wrote: > Christophe LEROY writes: >> Le 22/11/2017 à 12:48, Michael Ellerman a écrit : >>> Christophe LEROY writes: Le 22/11/2017 à 00:07, Balbir Singh a écrit : >

Re: [PATCH] ASoC: fsl_ssi: Override bit clock rate based on slot number

On Sat, Nov 25, 2017 at 11:29:48PM +0100, Lukasz Majewski wrote: > Nicolin, do you know what happened with this patch? I couldn't find it > in current linux/master. > > Has it been applied to any asoc tree for being upstreamed? A similar patch with an updated subject got applied:

[PATCH 1/2] video: ssd1307fb: Delete an error message for a failed memory allocation in ssd1307fb_probe()

From: Markus Elfring Date: Sat, 25 Nov 2017 16:45:56 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH 0/2] video: ssd1307fb: Adjustments for ssd1307fb_probe()

From: Markus Elfring Date: Sat, 25 Nov 2017 16:56:46 +0100 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Delete an error message for a failed memory allocation Improve a size determination

Re: [PATCH v5 1/2] iio : Add cm3218 smbus ara and acpi support

On Wed, 22 Nov 2017 23:52:33 +0100 Marc CAPDEVILLE wrote: > On asus T100, Capella cm3218 chip is implemented as ambiant light > sensor. This chip expose an smbus ARA protocol device on standard > address 0x0c. The chip is not functional before all alerts are >

Re: [PATCH 1/2] iio: adc: ina2xx: Make calibration register value fixed

On Wed, 22 Nov 2017 16:32:14 +0100 Maciej Purski wrote: > Calibration register is used for calculating current register in > hardware according to datasheet: > current = shunt_volt * calib_register / 2048 (ina 226) > current = shunt_volt * calib_register / 4096 (ina 219) >

Re: [PATCH v2 2/2] x86: disable IRQs before changing CR4

Thomas Gleixner wrote: > On Sat, 25 Nov 2017, Nadav Amit wrote: >> Thomas Gleixner wrote: >> >>> On Fri, 24 Nov 2017, Nadav Amit wrote: /* Set in this cpu's CR4. */ -static inline void cr4_set_bits(unsigned long mask) +static inline void

Re: [PATCH 2/2] iio: chemical: sgpxx: triggered buffer support

On Tue, 21 Nov 2017 17:11:29 +0100 Andreas Brauchli wrote: > Support triggered buffer for use with e.g. hrtimer for automated > polling to ensure that the sensor's internal baseline is correctly > updated independently of the use-case. Given the really strict timing

Re: [PATCH v5 10/11] intel_sgx: glue code for in-kernel LE

On Fri, Nov 17, 2017 at 03:07:05PM -0800, Darren Hart wrote: > No incremental cleanup here - appears to all be handled through > sgx_le_stop - do I have that right? Yes. This is correct. /Jarkko

Re: [patch V4 01/11] Documentation: Add license-rules.rst to describe how to properly identify file licenses

On Wed 2017-11-22 14:48:04, Greg Kroah-Hartman wrote: > On Wed, Nov 22, 2017 at 09:51:17AM -0200, Mauro Carvalho Chehab wrote: > > Em Wed, 22 Nov 2017 12:12:04 +0100 (CET) > > Thomas Gleixner escreveu: > > > > > On Fri, 17 Nov 2017, Christoph Hellwig wrote: > > > > On Fri,

[PATCH v4] drm: bridge: synopsys/dw-hdmi: Enable cec clock

Support the "cec" optional clock. The documentation already mentions "cec" optional clock and it is used by several boards, but currently the driver doesn't enable it, thus preventing cec from working on those boards. And even worse: a /dev/cecX device will appear for those boards, but it won't

Re: [PATCH] lib: memmove: Use optimised memcpy if possible

Paul's original patch should have been separated into two patches to begin with. The patch does two different things and one part goes through the MIPS tree and one part goes through Andrew, probably. On Sat, Nov 25, 2017 at 10:52:04PM +0530, PrasannaKumar Muralidharan wrote: > Hi, > > On 4

Re: [PATCH] net: openvswitch: datapath: fix data type in queue_gso_packets

On Sat, Nov 25, 2017 at 2:14 PM, Gustavo A. R. Silva wrote: > gso_type is being used in binary AND operations together with SKB_GSO_UDP. > The issue is that variable gso_type is of type unsigned short and > SKB_GSO_UDP expands to more than 16 bits: > > SKB_GSO_UDP = 1 <<

[PATCH] nvmem: core: let stride and word_size default to 1

If the caller doesn't set stride and/or word_size in struct nvmem_config then nvmem_register accepts this but we may face strange effects later due to both values being 0. Therefore use 1 as default for both values. Signed-off-by: Heiner Kallweit --- drivers/nvmem/core.c

Re: [PATCH] Support TrackStick of Thinkpad L570

Hi, On Mon, Nov 20, 2017 at 04:55:30PM +0900, Masaki Ota wrote: > From: Masaki Ota > - The issue is that Thinkpad L570 TrackStick does not work. Because the main > interface of Thinkpad L570 device is SMBus, so ALPS overlooked PS2 interface > Firmware setting of

Re: [PATCH] x86/orc: Don't bail on stack overflow

Can you send me whatever config and exact commit hash generated this? I can try to figure out why it failed. On Sat, Nov 25, 2017 at 3:13 PM, Thomas Gleixner wrote: > On Sat, 25 Nov 2017, Andy Lutomirski wrote: > >> On Sat, Nov 25, 2017 at 9:28 AM, Andy Lutomirski

Re: [PATCH v1 03/10] v4l: platform: Add Renesas CEU driver

On Fri, Nov 17, 2017 at 10:33:55AM +0100, jacopo mondi wrote: > Hi Sakari! > > On Fri, Nov 17, 2017 at 02:36:51AM +0200, Sakari Ailus wrote: > > Hi Jacopo, > > > > On Wed, Nov 15, 2017 at 03:25:11PM +0100, jacopo mondi wrote: > > > Hi Sakari, > > >thanks for review! > > > > You're welcome! >

Re: [PATCH] arm64: allwinner: a64: Enable AXP803 for Orangepi Win

On Thu, Nov 23, 2017 at 11:05:36PM +0530, Jagan Teki wrote: > Enable AXP803 PMIC and regulators for Orangepi Win. > > Signed-off-by: Jagan Teki Applied, thanks! Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com

Re: [PATCH 20/43] x86/entry: Clean up SYSENTER_stack code

On Sat, 25 Nov 2017, Andy Lutomirski wrote: > > On Nov 25, 2017, at 9:50 AM, Thomas Gleixner wrote: > > > > On Sat, 25 Nov 2017, Borislav Petkov wrote: > >>> - > >>> +wrmsr(MSR_IA32_SYSENTER_ESP, (unsigned long)(cpu_SYSENTER_stack(cpu) > >>> + 1), 0); > >>>

Re: [PATCH] lib: memmove: Use optimised memcpy if possible

Hi, On 4 October 2017 at 22:26, PrasannaKumar Muralidharan wrote: > When there is no overlap between src and dst use optimised memcpy if it > is available. > > Signed-off-by: Paul Burton > Signed-off-by: PrasannaKumar Muralidharan

Re: [PATCH 1/2] iio: chemical: sgpxx: Support Sensirion SGPxx sensors

On Tue, 21 Nov 2017 22:46:07 +0100 (CET) Peter Meerwald-Stadler wrote: > Hello, > > some quick comments on this driver below A few additional bits from me but I think Peter got most of the key stuff. > > I think documentation is missing and the ABI is a bit problematic and

Re: [patch V4 01/11] Documentation: Add license-rules.rst to describe how to properly identify file licenses

On Fri 2017-11-17 15:06:39, Mauro Carvalho Chehab wrote: > Hi Thomas, > > Em Fri, 17 Nov 2017 11:00:33 +0100 (CET) > Thomas Gleixner escreveu: > > > Subject: Documentation: Add license-rules.rst to describe how to properly > > identify file licenses > > From: Thomas

Re: [PATCH net] net: dsa: fix 'increment on 0' warning

From: Vivien Didelot Date: Fri, 24 Nov 2017 11:36:06 -0500 > Setting the refcount to 0 when allocating a tree to match the number of > switch devices it holds may cause an 'increment on 0; use-after-free', > if CONFIG_REFCOUNT_FULL is enabled. > > To fix

Re: [patch V4 01/11] Documentation: Add license-rules.rst to describe how to properly identify file licenses

On Sat, Nov 25, 2017 at 9:17 AM, Pavel Machek wrote: > > There's logical place in the comment, and it should look like this: > > /* > * Driver for SMSC USB3503 USB 2.0 hub controller driver > * > * SPDX-License-Identifier: GPL-2.0+ > * Copyright (c) 2012-2013 Dongjin Kim

[PATCH 3/5] video: smscufx: Return an error code only as a constant in ufx_realloc_framebuffer()

From: Markus Elfring Date: Sat, 25 Nov 2017 19:56:44 +0100 * Return an error code without storing it in an intermediate variable. * Delete the label "error" and local variable "retval" which became unnecessary with this refactoring. Signed-off-by: Markus

[PATCH 2/5] video: smscufx: Less checks in ufx_usb_probe() after error detection

From: Markus Elfring Date: Sat, 25 Nov 2017 19:47:51 +0100 Up to four checks could be repeated by the ufx_usb_probe() function during error handling even if the relevant properties can be determined for the involved variables before by source code analysis. *

Re: [PATCH 28/43] x86/mm/kaiser: Map cpu entry area

On Fri, 24 Nov 2017, Ingo Molnar wrote: > +void kaiser_add_mapping_cpu_entry(int cpu) > +{ > + kaiser_add_user_map_early(get_cpu_gdt_ro(cpu), PAGE_SIZE, > + __PAGE_KERNEL_RO); > + > + /* includes the entry stack */ > +

[PATCH v2] staging: sm750b: Fix coding style issues in sm750_accel.c

This is a patch to sm750_accel.c that fixes 80-character line length warnings found by checkpatch.pl. It also fixes some grammatical errors in comments and moves parameter-specific comments from inline to before the function. Signed-off-by: Jeremy Lacomis --- Changes in v2:

Re: [PATCH 42/43] x86/mm/kaiser: Allow KAISER to be enabled/disabled at runtime

On Sat, 25 Nov 2017, Andy Lutomirski wrote: > > On Nov 25, 2017, at 1:05 PM, Thomas Gleixner wrote: > > On Sat, 25 Nov 2017, Andy Lutomirski wrote: > >> Keep in mind that, for a static_branch, actually setting the thing needs > >> to be deferred, but that's straightforward. >

Re: [GIT PULL] afs: Fixes

On Sat, Nov 25, 2017 at 10:35:43PM +, David Howells wrote: > Linus Torvalds wrote: > > > So I see in the commit message why afs needs to do this, but it's > > worth pointing out that it's > > > > (a) impossible to avoid the "inconsistent data" case for

Re: [PATCH 0/5] Consolidate init_task handling

I've updated my git branch with changed descriptions, but the content is the same. David

  1   2   3   4   >