This patch adds the Documentation/x86/intel_bm.txt file with some
information about Intel Branch monitoring.
Signed-off-by: Megha Dey
Signed-off-by: Yu-Cheng Yu
---
Documentation/x86/intel_bm.txt | 216 +
On Thu, Nov 2, 2017 at 5:35 PM, Wanpeng Li wrote:
> From: Wanpeng Li
>
> According to the SDM, if the "load IA32_BNDCFGS" VM-entry controls is 1, the
> following checks are performed on the field for the IA32_BNDCFGS MSR:
> - Bits reserved in the
Use the new generic helper of_cpu_node_to_id() instead
of using our own version to map a device node to logical CPU
number.
Acked-by: Marc Zyngier
Signed-off-by: Suzuki K Poulose
---
Changes since V3:
- Reflect the change in the helper name and
Make use of the new generic helper to convert an of_node of a CPU
to the logical CPU id in parsing the topology.
Cc: Catalin Marinas
Cc: Leo Yan
Cc: Will Deacon
Acked-by: Mark Rutland
Signed-off-by: Suzuki
On Fri, Nov 03, 2017 at 01:46:02PM -0400, Roman Gushchin wrote:
> Delegatable cgroup v2 control files may require special handling
> (e.g. chowning), and the exact list of such files varies between
> kernel versions (and likely to be extended in the future).
>
> To guarantee correctness of this
* Andrew Morton [171103 17:34]:
> On Fri, 3 Nov 2017 09:00:12 -0700 Tony Lindgren wrote:
>
> > Looks like I'm still carrying patch "mm: distinguish CMA and MOVABLE
> > isolation in has_unmovable_pages" from Michal [1] for commit e1d753dff0fa
> >
On Wed, Nov 01, 2017 at 11:19:28AM -0500, Janakarajan Natarajan wrote:
> Add the EventSelect and Counter MSRs for AMD Core Perf Extension.
>
> Signed-off-by: Janakarajan Natarajan
> ---
> arch/x86/include/asm/msr-index.h | 12
> 1 file changed, 12
On Fri, 03 Nov 2017 01:44:44 +0800 "Yang Shi" wrote:
> I may not articulate it in the commit log
You should have done so ;)
Here's the changelog I ended up with:
: From: "Yang Shi"
: Subject: mm: use in_atomic() in print_vma_addr()
:
:
On Fri, Nov 03, 2017 at 10:11:52AM -0700, Sami Tolvanen wrote:
> From: Greg Hackmann
>
> LLVM bug 30792 causes clang's AArch64 backend to crash compiling
> arch/arm64/crypto/aes-ce-cipher.c. Replacing -mgeneral-regs-only with
> -mno-implicit-float is the suggested
On Fri, Nov 03, 2017 at 10:12:01AM -0700, Sami Tolvanen wrote:
> Clang's integrated assembler does not allow assembly macros defined
> in one inline asm block using the .macro directive to be used across
> separate asm blocks. LLVM developers consider this a feature and not a
> bug, recommending
On Fri, Nov 03, 2017 at 11:36:33AM -0700, Nick Desaulniers wrote:
> On Fri, Nov 3, 2017 at 11:29 AM, Mark Rutland wrote:
> > On Fri, Nov 03, 2017 at 11:07:04AM -0700, Nick Desaulniers wrote:
> >> On Fri, Nov 3, 2017 at 10:51 AM, Mark Rutland wrote:
>
The Lenovo Helix 2 and Dell XPS 12 (9Q33) have an extra button on the
front showing a 'Windows' logo, both reporting event codes 0xC2/0xC3
on press/release.
Tested on XPS 12, for info on the Helix 2 see:
https://www.spinics.net/lists/ibm-acpi-devel/msg03982.html
Signed-off-by: Stefan Brüns
Event code 0xcc is emitted by several convertibles (Dell XPS 12 9Q33 BIOS
A8, Dell XPS 13 2in1 9365, HP Spectre x360, Lenovo Thinkpad Helix) when
entering tablet mode, and 0xcd on return to laptop mode.
Signed-off-by: Stefan Brüns
---
The changes where tested on
On Fri, Nov 03, 2017 at 08:03:26PM +0100, Stefan Brüns wrote:
> Sync events are sent by sparse_keymap_report_entry for normal KEY_*
> events, and are generated by several drivers after generating
> SW_* events, so sparse_keymap_report_entry should do the same.
>
> Without the sync, events are
Hi
On Fri, 3 Nov 2017 15:44:53 +0100
Arnd Bergmann wrote:
> On Fri, Nov 3, 2017 at 1:44 PM, Build bot for Mark Brown
> wrote:
> >
> > Errors summary: 4
> > 2 ../include/linux/kernel.h:931:18: error: invalid type argument
> > of unary '*' (have
m_timer()
>> to pass the timer pointer explicitly.
>
> Reviewed and applied to drm-misc-next. Thanks!
Thanks!
I happened to notice that this was in next-20171102, but missing in
next-20171103. Did it get removed, or am I misunderstanding something?
-Kees
--
Kees Cook
Pixel Security
On Sat, 2017-11-04 at 03:08 +0800, Yang Shi wrote:
> checkpatch.pl still reports the below in_atomic warning:
>
> WARNING: use of in_atomic() is incorrect outside core kernel code
> + if (in_atomic())
>
> But, in_atomic() has been used outside kernel dir for a long time, and
> even
Sehr geehrte Damen und Herren,
Sie brauchen Geld? Sie sind auf der suche nach einem Darlehnen? Seriös und
unkompliziert?
Dann sind Sie hier bei uns genau richtig.
Durch unsere jahrelange Erfahrung und kompetente Beratung sind wir Europaweit
tätig.
Wir bieten jedem ein GÜNSTIGES Darlehnen zu
Just a follow up, since I said I would come back around this week.
I've come up with a couple patchsets that I'm going to attempt to test
internally before I sent out more widely. I'm thinking it's
appropriate to mail out two changes:
(1) Reject attempts to map physical memory addresses
struct timeval is deprecated because it cannot represent times
past 2038. In this driver, the only use of this structure is
to capture debug information. This is easily changed to ktime_t,
which we then format as needed when printing it later.
Signed-off-by: Arnd Bergmann
---
> [I missed this followup, other stuff]
>
> On Mon, Oct 23, 2017 at 03:41:49PM +0200, Peter Zijlstra wrote:
> > On Sat, Oct 21, 2017 at 10:21:11AM +1100, Dave Chinner wrote:
> > > On Fri, Oct 20, 2017 at 02:07:53PM +0300, Elena Reshetova wrote:
> > > IMO, that makes it way too hard to review
On Fri 03-11-17 00:46:18, Hugh Dickins wrote:
> On Thu, 2 Nov 2017, Michal Hocko wrote:
> > From: Michal Hocko
> >
> > syzkaller has reported the following lockdep splat
> > ==
> > WARNING: possible circular locking dependency
Add a new feature, VIRTIO_BALLOON_F_SG, which enables the transfer
of balloon (i.e. inflated/deflated) pages using scatter-gather lists
to the host.
The implementation of the previous virtio-balloon is not very
efficient, because the balloon pages are transferred to the
host one by one. Here is
Negotiation of the VIRTIO_BALLOON_F_FREE_PAGE_VQ feature indicates the
support of reporting hints of guest free pages to the host via
virtio-balloon. The host requests the guest to report the free pages by
sending commands via the virtio-balloon configuration registers.
When the guest starts to
From: Matthew Wilcox
The eXtensible Bitmap is a sparse bitmap representation which is
efficient for set bits which tend to cluster. It supports up to
'unsigned long' worth of bits, and this commit adds the bare bones --
xb_set_bit(), xb_clear_bit(), xb_clear_bit_range(),
On Fri 03-11-17 01:44:44, Yang Shi wrote:
>
>
> On 11/2/17 12:57 AM, Michal Hocko wrote:
> > On Thu 02-11-17 05:38:33, Yang Shi wrote:
> > > commit 3e51f3c4004c9b01f66da03214a3e206f5ed627b
> > > ("sched/preempt: Remove PREEMPT_ACTIVE unmasking off in_atomic()") makes
> > > in_atomic() just check
This patch series enhances the existing virtio-balloon with the following
new features:
1) fast ballooning: transfer ballooned pages between the guest and host in
chunks using sgs, instead of one array each time; and
2) free page block reporting: a new virtqueue to report guest free pages
to the
From: Matthew Wilcox
Add the following tests for xbitmap:
1) single bit test: single bit set/clear/find;
2) bit range test: set/clear a range of bits and find a 0 or 1 bit in
the range.
Signed-off-by: Wei Wang
Cc: Matthew Wilcox
From: "Michael S. Tsirkin"
fill_balloon doing memory allocations under balloon_lock
can cause a deadlock when leak_balloon is called from
virtballoon_oom_notify and tries to take same lock.
To fix, split page allocation and enqueue and do allocations outside
the lock.
Here's a detailed
This patch adds support to walk through the free page blocks in the
system and report them via a callback function. Some page blocks may
leave the free list after zone->lock is released, so it is the caller's
responsibility to either detect or prevent the use of such pages.
One use example of
On 10/22/2017 11:19 AM, Michael S. Tsirkin wrote:
On Fri, Oct 20, 2017 at 07:54:23PM +0800, Wei Wang wrote:
This patch series intends to summarize the recent contributions made by
Michael S. Tsirkin, Tetsuo Handa, Michal Hocko etc. via reporting and
discussing the related deadlock issues on the
From: Markus Elfring
Date: Fri, 3 Nov 2017 09:26:28 +0100
* Add a jump target so that a call of the function "mutex_unlock" is stored
only twice in this function implementation.
* Replace two calls by goto statements.
This issue was detected by using the
From: Colin Ian King
The check on bedata->ref is never true because ref is an unsigned
integer. Fix this by assigning signed int ret to the return of the
call to gnttab_claim_grant_reference so the -ve return can be checked.
Detected by CoverityScan, CID#1460358
Hi Frank,
On Thu, Nov 02, 2017 at 07:59:04PM +0100, Frank Kunz wrote:
> > > + {
> > > + pinctrl-names = "default";
> > > + pinctrl-0 = <_pins_a>;
> > > + vmmc-supply = <_vcc3v3>;
> >
> > What regulator is this connected to on the PMIC?
>
> Fixed, On the schematic it is finally connected to 3v0.
Hi Juergen,
On Wed, Nov 01, 2017 at 12:16:36PM +0100, Juergen Fitschen wrote:
> Hello Ludovic,
>
> Thank you very much for your feedback!
>
> On Tue, Oct 31, 2017 at 04:22:50PM +0100, Ludovic Desroches wrote:
> > On Fri, Oct 27, 2017 at 05:12:17PM +0200, Juergen Fitschen wrote:
> > > Some AT91
On 11/02/2017 07:49 PM, Tony Krowiak wrote:
> On 11/02/2017 11:53 AM, Christian Borntraeger wrote:
>>
>> On 11/02/2017 04:36 PM, Tony Krowiak wrote:
>>> On 11/02/2017 08:08 AM, Christian Borntraeger wrote:
On 10/16/2017 11:25 AM, Martin Schwidefsky wrote:
> On Fri, 13 Oct 2017 13:39:04
On 11/03/2017 09:14 AM, Harald Freudenberger wrote:
> I talked with the firmware guys about this.
> The answer is that the AP 3 instructions (NQAP, DQAP, PQAP) are always
> available.
That is true for basic mode and LPAR, but certainly not for z/VM or KVM guests.
> Tony is right with the
On Thu, Nov 2, 2017 at 12:51 PM, Jason Wang wrote:
>
>
> On 2017年11月02日 11:45, Michael S. Tsirkin wrote:
>>
>> On Thu, Nov 02, 2017 at 11:43:48AM +0800, Jason Wang wrote:
>>>
>>>
>>> On 2017年11月02日 09:11, Willem de Bruijn wrote:
On Tue, Oct 31, 2017 at 7:32 PM,
On 11/03/17 14:54 +0800, Xiao Guangrong wrote:
>
>
> On 11/03/2017 01:53 PM, Haozhong Zhang wrote:
> > Some reserved pages, such as those from NVDIMM DAX devices, are
> > not for MMIO, and can be mapped with cached memory type for better
> > performance. However, the above check misconceives
On 2017/11/3 15:31, Fan Li wrote:
> In current version, we preserve 8 pages of nat blocks as free nids,
> we build bitmaps for it and use them to allocate nids until its number
> drops below NAT_ENTRY_PER_BLOCK.
>
> After that, we have a problem, scan_free_nid_bits will scan the same
> 8 pages
This check was an attempt to protect against a race with put_pi_state()
by ensuring that the pi_state_list was consistent across the
unlock/lock of pi_lock.
However, as of commit 153fbd1226fb3 ("futex: Fix more put_pi_state() vs.
exit_pi_state_list() races"), this check is no longer necessary
On 11/3/17 12:41 PM, Joe Perches wrote:
On Sat, 2017-11-04 at 03:08 +0800, Yang Shi wrote:
checkpatch.pl still reports the below in_atomic warning:
WARNING: use of in_atomic() is incorrect outside core kernel code
+ if (in_atomic())
But, in_atomic() has been used outside kernel dir
On 11/03/2017 01:01 PM, a...@linux-foundation.org wrote:
>
> This mmotm tree contains the following patches against 4.14-rc7:
> (patches marked "*" will be included in linux-next)
>
> origin.patch
origin.patch has a problem. When CONFIG_SMP is not enabled (on x86_64 e.g.):
- if
Introduce environment variable KSFT_TAP_LEVEL to avoid printing
nested TAP headers for each test. lib.mk run_tests target prints
TAP header before invoking the test program or test script. Tests
need a way to suppress TAP headers if it is already printed out.
This new environment variable adds a
This first patch in this series adds environment variable KSFT_TAP_LEVEL
to avoid printing nested TAP headers for each test. lib.mk run_tests
target prints TAP header before invoking the test program or test script.
Tests need a way to suppress TAP headers if it is already printed out.
This new
Set KSFT_TAP_LEVEL before running tests to prevent nested TAP header
printing from tests.
Signed-off-by: Shuah Khan
---
tools/testing/selftests/lib.mk | 1 +
1 file changed, 1 insertion(+)
diff --git a/tools/testing/selftests/lib.mk b/tools/testing/selftests/lib.mk
On Fri, 2017-11-03 at 23:48 +0100, Arnd Bergmann wrote:
> When CDROM is enabled, but nothing else selects BLK_SCSI_REQUEST,
> we get this link error:
>
> cdrom.c:(.text+0x7a18): undefined reference to `scsi_cmd_blk_ioctl'
Hello Arnd,
Can you check whether this issue still occurs with Jens'
Hi Michael,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on char-misc/master]
url:
https://github.com/0day-ci/linux/commits/mikelley-exchange-microsoft-com/Drivers-hv-vmbus-Implement-Direct-Mode-for-stimer0/20171103-214519
config: x86_64-randconfig-h0
---
Documentation/kernel-hacking/hacking.rst | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/Documentation/kernel-hacking/hacking.rst
b/Documentation/kernel-hacking/hacking.rst
index daf3883b2694..91cadeb714e2 100644
--- a/Documentation/kernel-hacking/hacking.rst
On Fri, Nov 3, 2017 at 11:55 PM, Bart Van Assche wrote:
> On Fri, 2017-11-03 at 23:48 +0100, Arnd Bergmann wrote:
>> When CDROM is enabled, but nothing else selects BLK_SCSI_REQUEST,
>> we get this link error:
>>
>> cdrom.c:(.text+0x7a18): undefined reference to
DSA trees have a refcount used to automatically free the dsa_switch_tree
structure once there is no switch devices inside of it.
The refcount is incremented when a switch is added to the tree, and
decremented when it is removed from it.
But because of kref_init, the refcount is also incremented
Add dsa_port_parse_user, dsa_port_parse_dsa and dsa_port_parse_cpu
functions to factorize the code shared by both OF and pdata parsing.
They don't do much for the moment but will be extended later to support
tagging protocol resolution for example.
Signed-off-by: Vivien Didelot
Rename dsa_get_dst to dsa_tree_find since it doesn't increment the
reference counter, rename dsa_add_dst to dsa_tree_alloc for symmetry
with dsa_tree_free, and provide a convenient dsa_tree_touch function to
find or allocate a new tree.
Signed-off-by: Vivien Didelot
Similarly to a DSA switch and port, rename the tree index from "tree" to
"index" and make it an unsigned int because it isn't supposed to be less
than 0.
u32 is an OF specific data used to retrieve the value and has no need to
be propagated up to the tree index.
Signed-off-by: Vivien Didelot
Define the DSA switch index as an unsigned int, because it will never be
less than 0.
Signed-off-by: Vivien Didelot
---
include/net/dsa.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/net/dsa.h b/include/net/dsa.h
index
In preparation to enabling -Wimplicit-fallthrough, mark switch cases
where we are expecting to fall through.
Notice that in this particular case I replaced "Fall" with a proper
"fall through" comment, which is what GCC is expecting to find.
Signed-off-by: Gustavo A. R. Silva
Extend the dsa_port_parse_cpu() function to resolve the tagging protocol
at port parsing time, instead of waiting for the whole tree to be
complete.
Signed-off-by: Vivien Didelot
---
net/dsa/dsa2.c | 33 -
1 file changed, 16
On 11/03/2017 04:05 PM, Vivien Didelot wrote:
> Similarly to a DSA switch and port, rename the tree index from "tree" to
> "index" and make it an unsigned int because it isn't supposed to be less
> than 0.
>
> u32 is an OF specific data used to retrieve the value and has no need to
> be
On 11/03/2017 04:05 PM, Vivien Didelot wrote:
> DSA trees have a refcount used to automatically free the dsa_switch_tree
> structure once there is no switch devices inside of it.
>
> The refcount is incremented when a switch is added to the tree, and
> decremented when it is removed from it.
>
>
On 11/03/2017 04:05 PM, Vivien Didelot wrote:
> Provide convenient dsa_tree_get and dsa_tree_put functions scoping a DSA
> tree used to increment and decrement its reference counter, instead of
> poking directly its kref structure.
>
> Signed-off-by: Vivien Didelot
On Fri, 2017-11-03 at 20:55 -0200, Jean Carlo Machado wrote:
> ---
> Documentation/kernel-hacking/hacking.rst | 10 +-
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/Documentation/kernel-hacking/hacking.rst
> b/Documentation/kernel-hacking/hacking.rst
[]
> @@ -288,7
On Fri, 3 Nov 2017 15:23:56 +0800
lyang0 wrote:
> >> diff --git a/tools/testing/selftests/ftrace/config
> >> b/tools/testing/selftests/ftrace/config
> >> index 8a1c9f9..b01924c 100644
> >> --- a/tools/testing/selftests/ftrace/config
> >> +++
On Fri, Nov 03, 2017 at 07:18:28PM +0800, Eric Long wrote:
> This patch adds the watchdog driver for Spreadtrum SC9860 platform.
>
> Signed-off-by: Eric Long
> ---
> Changes since v4:
> - Remove sprd_wdt_remove().
> - Add devm_add_action() for sprd_wdt_disable().
>
>
Add a generic function for raising MSI irqs that can be used by all
DWC based controllers.
Note that certain controllers, like DRA7xx, have a special convenience
register for raising MSI irqs that doesn't require you to explicitly map
the MSI address. Therefore, it is likely that certain drivers
Assign pp->ops in *_add_pcie_port() to match how it is done in other
drivers like exynos, imx7, keystone, armada8k, artpec6, designware-plat,
hisi, kirin and spear13xx.
This is probably a remainder since when dev and ops were assigned as
members to pp. Since we now assign them as members to
From: Arnaldo Carvalho de Melo
For some unknown reason there is no entry in tracefs's syscalls for
kcmp, i.e. no tracefs/events/syscalls/sys_{enter,exit}_kcmp, so we need
to provide a data dictionary for the fields.
To beautify the 'type' argument we automatically generate a
From: Arnaldo Carvalho de Melo
We will use it to generate tables for beautifying kcmp's 'type' arg.
Cc: Adrian Hunter
Cc: Andrey Vagin
Cc: Cyrill Gorcunov
Cc: David Ahern
Cc: Jiri Olsa
From: Namhyung Kim
When libbfd is not used, addr2inlines() executes `addr2line -i` and
process output line by line. But it resets filename to NULL in the loop
so getline() allocates additional memory everytime instead of realloc.
Signed-off-by: Namhyung Kim
From: Namhyung Kim
When libbfd is not used, it doesn't show proper function name and reuse
the original symbol of the sample. That's because it passes the
original sym to inline_list__append(). As `addr2line -f` returns
function names as well, use that to create an
From: Arnaldo Carvalho de Melo
One that given a pid and a fd, will try to get the path for that fd.
Will be used in the upcoming kcmp's KCMP_FILE beautifier.
Cc: Adrian Hunter
Cc: Andrey Vagin
Cc: Cyrill Gorcunov
On 2017年11月03日 06:14, Shuah Khan wrote:
Hi Lei,
On 10/17/2017 08:10 PM, lei.y...@windriver.com wrote:
From: Lei Yang
Are you using git send-email to send patches. it is odd to see this
extra From: line.
I just find it's caused by my git configuration.
in
On Fri, 2017-11-03 at 11:55 +0100, Egil Hjelmeland wrote:
> Remove scripts/checkpatch.pl CHECKs by remove unnecessary parentheses
[]
> diff --git a/drivers/net/dsa/lan9303-core.c b/drivers/net/dsa/lan9303-core.c
[]
> @@ -483,7 +483,7 @@ static int lan9303_detect_phy_setup(struct lan9303 *chip)
>
Commit-ID: 1de3038d0082a1c2edd7a7b1b3381e38f42af0e7
Gitweb: https://git.kernel.org/tip/1de3038d0082a1c2edd7a7b1b3381e38f42af0e7
Author: Arnaldo Carvalho de Melo
AuthorDate: Tue, 31 Oct 2017 11:32:23 -0300
Committer: Arnaldo Carvalho de Melo
CommitDate:
Commit-ID: b7b75a60b291cc699ca9bb2a8517a1b3b08bbeb1
Gitweb: https://git.kernel.org/tip/b7b75a60b291cc699ca9bb2a8517a1b3b08bbeb1
Author: Namhyung Kim
AuthorDate: Tue, 31 Oct 2017 11:06:53 +0900
Committer: Arnaldo Carvalho de Melo
CommitDate: Wed, 1
Commit-ID: 021b462a51de48dd84f12f5046b5b57a362d6506
Gitweb: https://git.kernel.org/tip/021b462a51de48dd84f12f5046b5b57a362d6506
Author: Jiri Olsa
AuthorDate: Thu, 12 Oct 2017 13:21:01 +0200
Committer: Arnaldo Carvalho de Melo
CommitDate: Mon, 30 Oct
On Fri, Nov 03, 2017 at 03:07:32PM +0100, Juergen Fitschen wrote:
> Hello Ludovic,
>
> On Fri, Nov 03, 2017 at 09:46:02AM +0100, Ludovic Desroches wrote:
> > > > > diff --git a/drivers/i2c/busses/i2c-at91.h
> > > > > b/drivers/i2c/busses/i2c-at91.h
> > > > > index bb502c1..4a4fa67 100644
> > > >
On 03. nov. 2017 15:11, Joe Perches wrote:
On Fri, 2017-11-03 at 11:55 +0100, Egil Hjelmeland wrote:
Remove scripts/checkpatch.pl CHECKs by remove unnecessary parentheses
[]
diff --git a/drivers/net/dsa/lan9303-core.c b/drivers/net/dsa/lan9303-core.c
[]
@@ -483,7 +483,7 @@ static int
Hi Thomas, Peter,
I'm trying to work out what stops a thread being pre-empted and migrated between
calling get_online_cpus() and put_online_cpus().
According to __percpu_down_read(), its the pre-empt count:
> * Due to having preemption disabled the decrement happens on
> * the same CPU as the
On 11/01/2017 12:56 PM, Thomas Gleixner wrote:
> On Wed, 25 Oct 2017, Prarit Bhargava wrote:
>> diff --git a/arch/x86/include/asm/processor.h
>> b/arch/x86/include/asm/processor.h
>> index b390ff76e58f..f4ab1edf4e24 100644
>> --- a/arch/x86/include/asm/processor.h
>> +++
The SCMI is intended to allow OSPM to manage various functions that are
provided by the hardware platform it is running on, including power and
performance functions. SCMI provides two levels of abstraction, protocols
and transports. Protocols define individual groups of system control and
Hi Egil,
Egil Hjelmeland writes:
> If there are no disagreements, I can do a v2 with that.
>
> And thanks for teaching me about "git grep"!
If you send a v2, you may want to address the other parenthesis
alignment issues found when running ./scripts/checkpatch -f on
In preparation to enabling -Wimplicit-fallthrough, mark switch cases
where we are expecting to fall through.
Notice that in this particular case I replaced "Fall" with a proper
"fall through" comment, which is what GCC is expecting to find.
Signed-off-by: Gustavo A. R. Silva
On Fri, 2017-11-03 at 13:55 +0100, Christoph Hellwig wrote:
> On Fri, Nov 03, 2017 at 11:02:50AM +0100, Javier González wrote:
> > Signed-off-by: Javier González
[]
> > diff --git a/drivers/nvme/host/core.c b/drivers/nvme/host/core.c
[]
> > @@ -2108,7 +2108,7 @@ static
On Thu, Aug 31, 2017 at 06:04:30PM +0200, Greg KH wrote:
> On Mon, Aug 28, 2017 at 01:54:05PM +0300, laurentiu.tu...@nxp.com wrote:
> > From: Stuart Yoder
> >
> > Move the source files out of staging into their final locations:
> > -include files in
From: Tina Ruchandani
'struct frame' uses two variables to store the sent timestamp - 'struct
timeval' and jiffies. jiffies is used to avoid discrepancies caused by
updates to system time. 'struct timeval' is deprecated because it uses
32-bit representation for seconds
When CDROM is enabled, but nothing else selects BLK_SCSI_REQUEST,
we get this link error:
cdrom.c:(.text+0x7a18): undefined reference to `scsi_cmd_blk_ioctl'
The problem is that the CDROM helper code has become user-selectable
now when it used to only be compiled if anything else enabled it.
On Thu, Nov 2, 2017 at 6:49 PM, Thierry Reding wrote:
> here's the latest series of patches that implement the tighter IRQ chip
> integration. I've dropped the banked infrastructure for now as per the
> discussion with Grygorii.
And I really liked that part.
> The
In preparation to enabling -Wimplicit-fallthrough, mark switch cases
where we are expecting to fall through.
Notice that in this particular case I replaced "Fall" with a proper
"fall through" comment, which is what GCC is expecting to find.
Signed-off-by: Gustavo A. R. Silva
Currently __{get,put}_user_size() expand their ptr argument in several
places, and some callers pass in expressions with side effects.
For example, fs/binfmt_elf.c, passes sp++ as the ptr argument to a chain
of __put_user() calls.
So far this isn't a problem, as each of these uses is mutually
__{get,put}_user calls are designed to be fast and have no checks,
relying on the caller to have made the appropriate calls previously.
It's very easy to forget a check though, leaving the kernel vulnerable
to exploits. Add an option to do the checks and kill the kernel if it
catches something
When parsing a port, simply use of_property_read_bool which checks the
presence of a given property, instead of parsing the link phandle.
Signed-off-by: Vivien Didelot
---
net/dsa/dsa2.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
This patch removes the unnecessary index argument from the
dsa_dst_add_ds and dsa_dst_del_ds functions and renames them to
dsa_tree_add_switch and dsa_tree_remove_switch respectively.
In addition to a more explicit scope, we now check the presence of an
existing switch with the same index
We will need a reference to the dsa_switch_tree when parsing a CPU port,
so fetch it right after parsing the member and before parsing ports.
Signed-off-by: Vivien Didelot
---
net/dsa/dsa2.c | 18 ++
1 file changed, 10 insertions(+), 8
When registering a DSA switch, there is basically two stages.
The first stage is the parsing of the switch device, from either device
tree or platform data. It fetches the DSA tree to which it belongs, and
validates its ports. The switch device is then added to the tree, and
the second stage is
When parsing a switch, we have to identify to which tree it belongs and
parse its ports. Provide two functions to separate the OF and platform
data specific paths.
Also use the of_property_read_variable_u32_array function to parse the
OF member array instead of calling of_property_read_u32_index
Provide convenient dsa_tree_get and dsa_tree_put functions scoping a DSA
tree used to increment and decrement its reference counter, instead of
poking directly its kref structure.
Signed-off-by: Vivien Didelot
---
net/dsa/dsa2.c | 40
On 11/03/2017 04:05 PM, Vivien Didelot wrote:
> Define the DSA switch index as an unsigned int, because it will never be
> less than 0.
>
> Signed-off-by: Vivien Didelot
Reviewed-by: Florian Fainelli
--
Florian
Function move_msr_up is used to _manually_ swap MSR entries in MSR array.
This function can be removed and replaced using the swap macro instead.
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva
---
The new lines are over 80
On 11/03/2017 04:05 PM, Vivien Didelot wrote:
> Rename dsa_get_dst to dsa_tree_find since it doesn't increment the
> reference counter, rename dsa_add_dst to dsa_tree_alloc for symmetry
> with dsa_tree_free, and provide a convenient dsa_tree_touch function to
> find or allocate a new tree.
>
>
On 11/03/2017 10:56 AM, Mike Kravetz wrote:
> On 11/03/2017 10:41 AM, David Herrmann wrote:
>> Hi
>>
>> On Fri, Nov 3, 2017 at 6:12 PM, Mike Kravetz wrote:
>>> On 11/03/2017 10:03 AM, David Herrmann wrote:
Hi
On Tue, Oct 31, 2017 at 7:40 PM, Marc-André
601 - 700 of 1758 matches
Mail list logo