Currently bioset_free() will oops if passed a NULL argument. Other
kernel 'free' functions typically check for NULL arguments and
return. The rule of least surprise dictates that bioset_free() should
behave safely even if passed a NULL argument.
Add NULL argument check to bioset_free().
Signed-of
On Sat, Sep 30, 2017 at 2:26 PM, Julian Calaby wrote:
> Hi Chen-Yu,
>
> On Sat, Sep 30, 2017 at 3:58 PM, Chen-Yu Tsai wrote:
>> On Sat, Sep 30, 2017 at 1:35 PM, Julian Calaby
>> wrote:
>>> Hi Chen-Yu,
>>>
>>> On Fri, Sep 29, 2017 at 8:22 PM, Chen-Yu Tsai wrote:
On Fri, Sep 29, 2017 at 6:2
Hi Jason,
On Sat, Sep 16, 2017 at 3:00 PM, Jason A. Donenfeld wrote:
> This started out as just replacing the use of crypto/rng with
> get_random_bytes_wait, so that we wouldn't use bad randomness at boot time.
> But, upon looking further, it appears that there were even deeper
> underlying crypt
> -Original Message-
> From: Florian Fainelli [mailto:f.faine...@gmail.com]
> Sent: Friday, September 29, 2017 19:11
> To: Razvan Stefanescu ; Bogdan Purcareata
> ; gre...@linuxfoundation.org
> Cc: de...@driverdev.osuosl.org; linux-kernel@vger.kernel.org;
> net...@vger.kernel.org; ag...@sus
On Monday 02 October 2017 00:19:35 Pavel Machek wrote:
> Hi!
>
> > I'm trying to get qemu emulation of Nokia N900 to work, but
> > unfortunately i2c-omap.c breaks boot in the emulator (real hardware
> > works ok).
>
> I started bisection. v4.6 works, v4.7 is broken. (It still may be
> config diff
On Thu 28-09-17 09:02:20, Huang, Ying wrote:
> Hi, Michal,
>
> Michal Hocko writes:
>
> > On Wed 27-09-17 23:10:08, Minchan Kim wrote:
> >> On Wed, Sep 27, 2017 at 03:50:34PM +0200, Michal Hocko wrote:
> >> > On Wed 27-09-17 22:41:17, Minchan Kim wrote:
> >> > > On Wed, Sep 27, 2017 at 03:22:41P
On Thu 28-09-17 01:25:50, Yang Shi wrote:
>
>
> On 9/27/17 3:45 AM, Michal Hocko wrote:
> > On Wed 27-09-17 08:53:35, Yang Shi wrote:
> > > Kernel may panic when oom happens without killable process sometimes it
> > > is caused by huge unreclaimable slabs used by kernel.
> > >
> > > Although kdu
On Sat 30-09-17 20:02:41, Sasha Levin wrote:
> On Sat, Sep 30, 2017 at 03:57:27PM +0200, Vegard Nossum wrote:
> >On 30 September 2017 at 11:48, Steven Rostedt wrote:
> >> On Wed, 27 Sep 2017 17:02:07 +0200
> >> Michal Hocko wrote:
> >>
> >>> > Now that 2 years have passed, and all distros provide
On Sun, Oct 01, 2017 at 07:12:51AM +0800, kernel test robot wrote:
>
> FYI, we noticed the following commit (built with gcc-6):
>
> commit: a19b2e3d783964d48d2b494439648e929bcdc976 ("kprobes/x86: Remove IRQ
> disabling from ftrace-based/optimized kprobes")
> https://git.kernel.org/cgit/linux/ker
Code indent should use tabs wherever possible
Signed-off-by: Harsha Sharma
---
src/evaluate.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/src/evaluate.c b/src/evaluate.c
index e767542..5624ca2 100644
--- a/src/evaluate.c
+++ b/src/evaluate.c
@@ -122,7 +122,7 @@
Issues found using checkpatch.pl
As per linux-kernel coding style, code indent should use tabs wherever
possible and avoid unnecessary spaces.
Comparisons shoukd place the constant on the right side of the test.
static const char * array should be static const * char const array
Harsha Sharma (3
static const char * array should probably be static const char * const
array
as per linux-kernel coding style
Signed-off-by: Harsha Sharma
---
src/evaluate.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/evaluate.c b/src/evaluate.c
index b783054..f48801a 100644
---
Comparisons should place the constant on the right side of the test
as per linux-kernel coding style
Signed-off-by: Harsha Sharma
---
src/evaluate.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/evaluate.c b/src/evaluate.c
index 5624ca2..b783054 100644
--- a/src/eva
> On Oct 2, 2017, at 01:00 , Rob Herring wrote:
>
> On Thu, Sep 28, 2017 at 2:58 PM, Pantelis Antoniou
> wrote:
>> Hello again,
>>
>> Significant progress has been made on yamldt and is now capable of
>> not only generating yaml from DTS source but also compiling DTS sources
>> and being almos
On Sun, 01 Oct 2017, Jérémy Lefaure wrote:
> Using the ARRAY_SIZE macro improves the readability of the code. Also,
> it is not always useful to use a variable to store this constant
> calculated at compile time nor to re-invent the ARRAY_SIZE macro.
>
> Found with Coccinelle with the following se
On Fri, Sep 29, 2017 at 03:33:39PM +0530, Srishti Sharma wrote:
> Replace reference/unreference with get/put as it is consistent
> with the kernel coding style. Done using the following semantic
> patch by coccinelle.
>
> @r@
> expression e;
> @@
>
> -drm_gem_object_unreference_unlocked(e);
> +dr
This reverts commit fcaa4a07d2a4b541e91da7a55d8b3331f96d1865.
As noted by Masaki [1], 0x120A + trackpoint will not be used in mass
production machines, so remove the ID accordingly.
[1] http://www.spinics.net/lists/linux-input/msg53222.html
Signed-off-by: Kai-Heng Feng
---
drivers/hid/hid-ids.
On Fri, Sep 29, 2017 at 03:03:54PM +, Philippe CORNU wrote:
> Hi Benjamin,
> and many thanks for this cleanup patchset.
>
> Reviewed-by: Philippe Cornu
> Tested-by: Philippe Cornu
Can you pls either convert vc4 and dw-mipi too, or add an entry to our
todo list in Documentation/gpu/todo.rst?
On Thu, Sep 28, 2017 at 4:22 PM, Grygorii Strashko
wrote:
> Sry, but I do not agree with this series.
> - no prof that it can be re-used by other drivers than tegra
> (at least I do not see reasons to re-use it for any TI drivers)
This is not necessarily a blocker if it can be shown that others
On Wed 27-09-17 15:13:11, Tahsin Erdogan wrote:
> "mapping" parameter to balance_dirty_pages() is not used anymore.
>
> Fixes: dfb8ae567835 ("writeback: let balance_dirty_pages() work on the
> matching cgroup bdi_writeback")
balance_dirty_pages_ratelimited doesn't really need mapping as well. Al
The memory allocation used to be optional but became mandatory with:
commit 304419d8a7e9204c5d19b704467b814df8c8f5b1
Author: Linus Walleij
Date: Thu May 18 11:29:32 2017 +0200
mmc: core: Allocate per-request data using the block layer core
There is also a bug in mmc_init_request()
On Oct 02 2017 or thereabouts, Kai-Heng Feng wrote:
> This reverts commit fcaa4a07d2a4b541e91da7a55d8b3331f96d1865.
>
> As noted by Masaki [1], 0x120A + trackpoint will not be used in mass
> production machines, so remove the ID accordingly.
>
> [1] http://www.spinics.net/lists/linux-input/msg532
On machine with 5-level paging support a process can allocate
significant amount of memory and stay unnoticed by oom-killer and
memory cgroup. The trick is to allocate a lot of PUD page tables.
We don't account PUD page tables, only PMD and PTE.
We already addressed the same issue for PMD page tab
I thought commit 8e9b46679923 ("kbuild: use $(abspath ...) instead of
$(shell cd ... && /bin/pwd)") was a safe conversion, but it changed
the behavior.
$(abspath ...) / $(realpath ...) does not expand shell special
characters, such as '~'.
Here is a simple Makefile example:
>8-
On 29.09.2017 17:45, valdis.kletni...@vt.edu wrote:
> On Fri, 29 Sep 2017 19:56:41 +0530, Pintu Kumar said:
>
>> 1) If you have pointers on how to setup ssh/net connection on QEMU
>> with busybox, do let me know.
>
> Busybox doesn't do that as far as I know, as it's intended as a single-user
> /s
On Mon, Oct 2, 2017 at 12:44 AM, Bhumika Goyal wrote:
> Make these __ro_after_init as they are only stored as a reference in the
> ci_type field of a config_item structure during the init phase in the
> function config_group_init_type_name. None of the fields of these
> structures are modified aft
Sorry for the delay, but picked them up now.
On 10/01/2017 03:54 PM, Rakesh Pandit wrote:
On Sun, Oct 01, 2017 at 04:25:18PM +0300, Rakesh Pandit wrote:
Not all exported symbols are being used outside core and there were
some stable entries in lightnvm.h
If you can replace 'stable' with 'stale' both in subject or body while
picking up
On Sun, Oct 01, 2017 at 03:30:44PM -0400, Jérémy Lefaure wrote:
> Using the ARRAY_SIZE macro improves the readability of the code. Also,
> it is not always useful to use a variable to store this constant
> calculated at compile time nor to re-invent the ARRAY_SIZE macro.
>
> Found with Coccinelle
On Fri, Sep 08, 2017 at 11:39:44AM -0400, Sean Paul wrote:
> On Fri, Sep 8, 2017 at 11:35 AM, Daniel Vetter wrote:
> > fbdev is in maintenance only, except that it's still used by drm
> > through the drm fbdev emulation, to be able to use fbcon. And people
> > might want to sometimes extend fbcon
On 26 September 2017 at 16:55, Colin King wrote:
> From: Colin Ian King
>
> The functions sdhci_omap_set_ios, sdhci_omap_set_power and
> sdhci_omap_get_min_clock are local to the source and do not need
> to be in global scope, so make them static.
>
> Cleans up sparse warnings:
> symbol 'sdhci_om
On 3 September 2017 at 15:39, Colin King wrote:
> From: Colin Ian King
>
> Don't populate the const arrays mszs on the stack, instead make them
> static. Makes the object code smaller by over 310 bytes:
>
> Before:
>textdata bss dec hex filename
> 475278528 320 563
On 22 September 2017 at 14:36, Adrian Hunter wrote:
> Until mmc has blk-mq support fully implemented and tested, add a
> parameter use_blk_mq, default to false unless config option MMC_MQ_DEFAULT
> is selected.
>
> Signed-off-by: Adrian Hunter
Thanks, applied for next!
Kind regards
Uffe
> ---
On 22 September 2017 at 14:37, Adrian Hunter wrote:
> Export mmc_retune_hold_now() and mmc_retune_release() so they can be used
> by the block driver.
>
> Signed-off-by: Adrian Hunter
I decided to squash patch 10/11/12 as those are all related to the
same problem when porting to blkmq.
So, than
On 26 September 2017 at 02:03, Chanho Min wrote:
> The driver strength selection is missed and required when selecting
> hs400es. So, It is added here.
>
> v2: add stable tag.
>
> Fixes: 81ac2af65793ecf ("mmc: core: implement enhanced strobe support")
> Cc: sta...@vger.kernel.org
> Signed-off-by:
On 22 September 2017 at 14:37, Adrian Hunter wrote:
> Add CQE support to the block driver, including:
> - optionally using DCMD for flush requests
> - "manually" issuing discard requests
> - issuing read / write requests to the CQE
> - supporting block-layer timeouts
> - handli
On Thu, Sep 28, 2017 at 4:30 PM, Arnd Bergmann wrote:
> On Thu, Sep 28, 2017 at 6:09 AM, Andrey Ryabinin
> wrote:
>> On 09/27/2017 04:26 PM, Arnd Bergmann wrote:
>>> On Tue, Sep 26, 2017 at 9:49 AM, Andrey Ryabinin
>>> wrote:
>
>>> --- a/include/linux/string.h
>>> +++ b/include/linux/string.h
>>
Hi Luis,
On Thu, Sep 14, 2017 at 1:39 AM, Luis R. Rodriguez wrote:
[snipped]
> Would a fw_cache_hint(device, name_list) be reasonable then sometime *before*
> suspend? All this would do is ask the firmware API to extend the fw cache
> list with the entries. It would not load firmware immediately,
On Mon, 2017-10-02 at 14:14 +1100, Stewart Smith wrote:
> Shilpasri G Bhat writes:
> > This driver provides interface to mmap the OCC sensor area
> > to userspace to parse and read OCC inband sensors.
>
> Why?
>
> Is this for debug? If so, the existing exports interface should be used.
>
> If t
Add the command build/parse APIs for operating on DPSW objects through
the DPAA2 Management Complex.
Signed-off-by: Razvan Stefanescu
---
Changelog:
v2:
- use u8 for en parameter of dpsw_if_set_flooding/broadcast()
drivers/staging/fsl-dpaa2/Kconfig |8 +
drivers/staging/fsl-dp
Introduce the DPAA2 Ethernet Switch driver, which manages Datapath Switch
(DPSW) objects discovered on the MC bus.
Suggested-by: Alexandru Marginean
Signed-off-by: Razvan Stefanescu
---
Changelog:
v2:
- fix PVID cleanup in ethsw_port_add_vlan()
- rename err2 to ret in ethsw_port_add/del
On 29.09.2017 17:58, Jonathan Hunter wrote:
+static int bpmp_populate_dir(struct tegra_bpmp *bpmp, struct seqbuf
*seqbuf,
+ struct dentry *parent, uint32_t depth)
Do we need to use uint32_t here?
+{
+ int err;
+ uint32_t d, t;
And here?
It's part of the BPMP ABI th
This patchset introduces the Ethernet Switch Driver for Freescale/NXP SoCs
with DPAA2 (DataPath Acceleration Architecture v2). The driver manages
switch objects discovered on the fsl-mc bus. A description of the driver
can be found in the associated README file.
The patchset consists of:
* A set o
Add a README file describing the driver architecture, components and
interfaces.
Signed-off-by: Razvan Stefanescu
---
Changelog:
v2:
- no changes
drivers/staging/fsl-dpaa2/ethsw/README | 106 +
1 file changed, 106 insertions(+)
create mode 100644 drivers/st
Add a TODO file describing what needs to be added/changed before the driver
can be moved out of staging.
Signed-off-by: Razvan Stefanescu
---
Changelog:
v2:
- no changes
drivers/staging/fsl-dpaa2/ethsw/TODO | 14 ++
1 file changed, 14 insertions(+)
create mode 100644 drivers/s
Add driver information, link details and hardware statistics to be
reported via ethtool -S.
Signed-off-by: Razvan Stefanescu
---
Changelog:
v2:
- no changes
drivers/staging/fsl-dpaa2/ethsw/Makefile| 2 +-
drivers/staging/fsl-dpaa2/ethsw/dpsw-cmd.h | 13 ++
drivers/staging/f
Signed-off-by: Razvan Stefanescu
---
Changelog:
v2:
- no changes
MAINTAINERS | 6 ++
1 file changed, 6 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 2281af4..cfd4f74 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -4297,6 +4297,12 @@ L: linux-kernel@vger.kernel.org
S:
Hi!
> The memory allocation used to be optional but became mandatory with:
>
> commit 304419d8a7e9204c5d19b704467b814df8c8f5b1
> Author: Linus Walleij
> Date: Thu May 18 11:29:32 2017 +0200
>
> mmc: core: Allocate per-request data using the block layer core
>
> There is also a bu
The hardened strlen() function causes rather large stack usage
in at least one file in the kernel when CONFIG_KASAN is enabled:
drivers/media/usb/em28xx/em28xx-dvb.c: In function 'em28xx_dvb_init':
drivers/media/usb/em28xx/em28xx-dvb.c:2062:1: error: the frame size of 3256
bytes is larger than 20
On 29.09.2017 17:53, Jonathan Hunter wrote:
On 29/09/17 14:46, Timo Alho wrote:
Hi Jon,
On 21.09.2017 14:21, Jonathan Hunter wrote:
On 07/09/17 10:31, Timo Alho wrote:
Check return code in BPMP response message(s). The typical error case
is when clock operation is attempted with invalid
Hi Thomas,
Thanks a lot for looking at this.
On Wed, Sep 27, 2017 at 11:28 PM, Thomas Gleixner wrote:
> This code is gone in -next and replaced by a new vector allocator.
>
> git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86/apic
Nice, thanks for cleaning that up!
> But the real
On Mon, Oct 2, 2017 at 10:40 AM, Arnd Bergmann wrote:
>
> void fortify_panic(const char *name) __noreturn __cold;
> +
> +/* work around GCC PR82365 */
> +#if defined(CONFIG_KASAN) && !defined(__clang__) && GCC_VERSION <= 8
> +#define fortify_panic(x) \
> + do { \
> + asm v
The following patch set cleans up some code and builds upon this to replace
ccree custom logging macros with the generic device dev_* facilities,
handles the resulting fallout and further simplifies logging handling in
some OOM error handling code path exposed by checkpatch following the
change.
P
Simplify handling of memory allocation failures and remove
redundant log messages
Signed-off-by: Gilad Ben-Yossef
---
drivers/staging/ccree/ssi_cipher.c | 11 --
drivers/staging/ccree/ssi_driver.c | 1 -
drivers/staging/ccree/ssi_hash.c | 42 +---
The ccree cycle count mechanism was removed in
commit 7f821f0c6ffa ("staging: ccree: remove cycle count debug support")
but the sysfs interface lingered on. Remove it now.
Signed-off-by: Gilad Ben-Yossef
---
drivers/staging/ccree/ssi_sysfs.c | 266 --
1 file c
Move over from using macro wrappers around to printk to
dev_err, dev_dbg and friends and clean up resulting fallout.
Signed-off-by: Gilad Ben-Yossef
---
drivers/staging/ccree/ssi_aead.c| 226 ++
drivers/staging/ccree/ssi_buffer_mgr.c | 394 +++
Introduce a DEV macro to retrieve struct device from private
data structure in preparation to replacing custom logging
macros with proper dev_dbg and friends which require struct
device.
Signed-off-by: Gilad Ben-Yossef
---
drivers/staging/ccree/ssi_aead.c| 11 ---
drivers/staging/ccr
relocated and no_longmode are self-standing local functions, annotate
them as such.
Signed-off-by: Jiri Slaby
Cc: "H. Peter Anvin"
Cc: Thomas Gleixner
Cc: Ingo Molnar
Cc: x...@kernel.org
---
arch/x86/boot/compressed/head_32.S | 3 ++-
arch/x86/boot/compressed/head_64.S | 6 --
2 files cha
These are all functions which are invoked from elsewhere, so we annotate
them as global using the new SYM_FUNC_START. And their ENDPROC's by
SYM_FUNC_END.
And make sure ENTRY/ENDPROC is not defined on X86_64, given these were
the last users.
Signed-off-by: Jiri Slaby
Cc: "H. Peter Anvin"
Cc: Th
These are all functions which are invoked from elsewhere, so we annotate
them as global using the new SYM_FUNC_START (and their ENDPROC's by
SYM_FUNC_END.)
Now, we can finally force ENTRY/ENDPROC to be undefined on X86.
Signed-off-by: Jiri Slaby
Cc: "H. Peter Anvin"
Cc: Thomas Gleixner
Cc: Ing
Here, we change all code which is not marked as functions. In other
words, this code has been using END, not ENDPROC. So switch all of this
to appropriate new markings SYM_CODE_START and SYM_CODE_END.
And since we removed the last user of END on X86, make sure, that END is
not defined there.
Sign
Use the new SYM_CODE_INNER_LABEL_NOALIGN for WEAK entries in the middle
of x86 assembly functions.
And make sure WEAK is not defined for x86 anymore as these were the last
users.
Signed-off-by: Jiri Slaby
Cc: Thomas Gleixner
Cc: Ingo Molnar
Cc: "H. Peter Anvin"
Cc: x...@kernel.org
---
arch/x
Here, we change all code which is not marked as functions. In other
words, this code has been using END, not ENDPROC. So switch all of this
to appropriate new markings SYM_CODE_START and SYM_CODE_END.
Signed-off-by: Jiri Slaby
Cc: Thomas Gleixner
Cc: Ingo Molnar
Cc: "H. Peter Anvin"
Cc: x...@k
All these are functions which are invoked from elsewhere, but they are
not typical C functions. So we annotate them (as global) using the new
SYM_CODE_START. All these were not balanced with any END, so mark their
ends by SYM_CODE_END, appropriatelly.
Signed-off-by: Jiri Slaby
Cc: Boris Ostrovsky
All these are functions which are invoked from elsewhere, but they are
not typical C functions. So we annotate them (as global) using the new
SYM_CODE_START. All these were not balanced with any END, so mark their
ends by SYM_CODE_END, appropriatelly.
Signed-off-by: Jiri Slaby
Cc: Thomas Gleixner
purgatory used no annotations. So include linux/linkage.h and annotate
everything:
* code by SYM_CODE_*
* data by SYM_DATA_*
Signed-off-by: Jiri Slaby
Cc: Thomas Gleixner
Cc: Ingo Molnar
Cc: "H. Peter Anvin"
Cc: x...@kernel.org
---
arch/x86/purgatory/entry64.S | 21 -
GLOBAL is meant for global symbols, but not functions. Use the new
macros SYM_FUNC_START* and SYM_CODE_START* (depending on the type of the
function) which are dedicated for global functions. And since they both
require a closing by SYM_*_END, we do this here too.
startup_64, which does not use GL
Convert the remaining 32bit users and remove GLOBAL macro finally. In
particular, this means to use SYM_ENTRY for the singlestepping hack
region.
Signed-off-by: Jiri Slaby
Cc: Thomas Gleixner
Cc: Ingo Molnar
Cc: "H. Peter Anvin"
Cc: x...@kernel.org
---
arch/x86/entry/entry_32.S | 4 ++--
GLOBAL had several meanings and is going away. In this patch, convert
all the inner function labels marked with GLOBAL to use
SYM_CODE_INNER_LABEL_NOALIGN instead.
Note that retint_user needs not be global, perhaps since commit
2ec67971facc ("x86/entry/64/compat: Remove most of the fast system cal
There is a couple of assembly functions, which are invoked only locally
in the file they are defined. In C, we mark them "static". In assembly,
annotate them using SYM_{FUNC,CODE}_START_LOCAL (and switch their
ENDPROC to SYM_{FUNC,CODE}_END too). Whether FUNC or CODE depends on
ENDPROC/END for a pa
GLOBAL had several meanings and is going away. In this patch, convert
all the data marked using GLOBAL to use SYM_DATA_START or
SYM_DATA_SIMPLE instead.
Notes:
* SYM_DATA_END_LABEL is used to generate tr_gdt_end too.
* wakeup_idt is marked as LOCAL now as it is used only locally.
Signed-off-by: J
Relabel function_hook to be marked really as a function. It is called
from C and has the same expectations towards the stack etc.
Signed-off-by: Jiri Slaby
Cc: Thomas Gleixner
Cc: Ingo Molnar
Cc: "H. Peter Anvin"
Cc: x...@kernel.org
---
arch/x86/kernel/ftrace_32.S | 4 ++--
arch/x86/kernel/ft
Use the new SYM_DATA_START_LOCAL, and SYM_DATA_END* macros:
8 OBJECT LOCAL DEFAULT6 gdt
000832 OBJECT LOCAL DEFAULT6 gdt_start
0028 0 OBJECT LOCAL DEFAULT6 gdt_end
0028 256 OBJECT LOCAL DEFAULT6 early_stack
0128 0 OBJECT LOCAL DEFAULT6
_key_expansion_128 is an alias to _key_expansion_256a, __memcpy to
memcpy, xen_syscall32_target to xen_sysenter_target, and so on. Annotate
them all using the new SYM_FUNC_START_ALIAS, SYM_FUNC_START_LOCAL_ALIAS,
and SYM_FUNC_END_ALIAS. This will make the tools generating the
debuginfo happy.
Sign
Use the new SYM_DATA_SIMPLE and SYM_DATA_END in both 32 and 64 bit heads.
In the 64-bit version, define also SYM_DATA_START_PAGE_ALIGNED locally
using the new SYM_START. It is used in the code instead of NEXT_PAGE()
which was defined in this file and has been using the obsolete macro
GLOBAL().
Now
Use the new SYM_DATA_SIMPLE, SYM_DATA_START, and SYM_DATA_END* macros
for data.
Now, the data in the object file look sane:
Value Size TypeBind Vis Ndx Name
10 OBJECT GLOBAL DEFAULT3 efi32_boot_gdt
000a10 OBJECT LOCAL DEFAULT3 save_gdt
0014 8 OBJECT L
There are functions in relocate_kernel which are not annotated. This
will not allow for automatic DWARF annotations. So annotate all the
functions now.
Note that these are not C-like functions, so we do not use FUNC, but
CODE. Also they are not aligned, so we use NOALIGN versions:
- SYM_CODE_START
Use the new SYM_DATA_START and SYM_DATA_END_LABEL macros for vdso_start.
We get:
2376 OBJECT GLOBAL DEFAULT4 vdso_start
0948 0 OBJECT GLOBAL DEFAULT4 vdso_end
Signed-off-by: Jiri Slaby
Cc: Jeff Dike
Cc: Richard Weinberger
Cc: Thomas Gleixner
Cc: Ingo Molnar
Cc: "H. Pe
Use the newly added SYM_FUNC_START_LOCAL to annotate starts of all
functions which do not have ".globl" annotation, but their ends are
annotated by ENDPROC. This is needed to balance ENDPROC for tools that
are about to generate debuginfo.
To be symmetric, we also convert their ENDPROCs to the new
Place SYM_CODE_START_NOALIGN and SYM_CODE_END around the THUNK macro
body, given it generates:
1) non-C-like functions, and
2) was not marked as aligned.
The common tail .L_restore is put inside SYM_CODE_START_LOCAL_NOALIGN
and SYM_CODE_END too.
The result:
Value Size TypeBind Vis Nd
* annotate functions properly by SYM_CODE_START, SYM_CODE_START_LOCAL*
and SYM_CODE_END -- these are not C-like functions, so we have to
annotate them using CODE.
* use SYM_CODE_INNER_LABEL* for labels being in the middle of other
functions
[v4] alignments preserved
Signed-off-by: Jiri Slab
Use the newly added SYM_CODE_START_LOCAL* to annotate starts of all
pseudo-functions (those ending END) which do not have ".globl"
annotation. This is needed to balance END for tools that are about to
generate debuginfo. Note that we switch from END to SYM_CODE_END so that
everybody can see the pai
Introduce new C macros for annotations of functions and data in
assembly. There is a long-standing mess in macros like ENTRY, END,
ENDPROC and similar. They are used in different manners and sometimes
incorrectly.
So introduce macros with clear use to annotate assembly as follows:
a) Support macr
We make proper use of SYM_DATA_SIMPLE on global data instead of marking
them as ENTRY. ENTRY is intended for functions and shall be paired with
ENDPROC. ENTRY also aligns symbols which creates unnecessary holes
here between data.
ENTRY from saved_eip in wakeup_32 and many saved_* in wakeup_64 is
s
On Sun, Oct 01, 2017 at 05:57:20PM -0700, Darren Hart wrote:
> On Sat, Sep 30, 2017 at 10:12:05AM +0200, Greg Kroah-Hartman wrote:
> > On Fri, Sep 29, 2017 at 06:52:28PM -0700, Darren Hart wrote:
> > >
> > > On Wed, Sep 27, 2017 at 11:02:16PM -0500, Mario Limonciello wrote:
> > > > For WMI operati
On 01/10/17 15:26, Guenter Roeck wrote:
> On Thu, Sep 28, 2017 at 02:11:44PM +0100, Sudeep Holla wrote:
>> Create a driver to add support for SoC sensors exported by the System
>> Control Processor (SCP) via the System Control and Management Interface
>> (SCMI). The supported sensor types is one
From: Borislav Petkov
Now that lguest is gone, put it in the internal header which should be
used only by MCA/RAS code.
Add missing header guards while at it.
No functionality change.
Signed-off-by: Borislav Petkov
---
arch/x86/include/asm/mce.h| 1 -
arch/x86/kernel/cpu/mche
From: Borislav Petkov
Hi guys,
just minor stuff right now. Please queue.
Thx.
Borislav Petkov (1):
x86/mce: Hide mca_cfg
Nicolas Iooss (1):
RAS/CEC: Use the right length for "cec_disable"
arch/x86/include/asm/mce.h| 1 -
arch/x86/kernel/cpu/mcheck/mce-internal.h | 7
From: Nicolas Iooss
parse_cec_param() compares a string with "cec_disable" using only 7
characters of the 11-character-long string. Fix the length.
Signed-off-by: Nicolas Iooss
Fixes: 011d82611172 ("RAS: Add a Corrected Errors Collector")
Link:
http://lkml.kernel.org/r/20170903075440.30250-1-n
On Fri 2017-09-29 13:55:46, Sergey Senozhatsky wrote:
> Hello,
>
> (Cc-ing Andrew
> lkml.kernel.org/r/20170928120405.18273-1-sergey.senozhat...@gmail.com )
>
> On (09/28/17 21:13), Sergey Senozhatsky wrote:
> > (Cc-ing Sasha)
> >
> > On (09/28/17 21:04), Sergey Senozhatsky wrote:
> > [..]
> >
Hi Florian,
On 09/29/2017 07:11 PM, Florian Fainelli wrote:
> On September 29, 2017 6:59:18 AM PDT, Razvan Stefanescu
> wrote:
>>
>>
>>> -Original Message-
>>> From: Bogdan Purcareata
>>> Sent: Friday, September 29, 2017 16:36
>>> To: Razvan Stefanescu ;
>>> gre...@linuxfoundation.org
>>
On Sat, Sep 30, 2017 at 10:07:12PM +0530, Bhumika Goyal wrote:
> Make this const as it is only stored in the const field of a device
> structure. Make the declaration in header const too.
>
> Structure found using Coccinelle and changes done by hand.
>
> Signed-off-by: Bhumika Goyal
This looks
Make sure that bridge parameter is not NULL and can be safely
cast into a panel_bridge structure.
Signed-off-by: Benjamin Gaignard
Reviewed-by: Philippe Cornu
Tested-by: Philippe Cornu
---
drivers/gpu/drm/bridge/panel.c | 10 +-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --gi
The goal of this series is to simplify driver code when they need to clean up
a previously allocated panel bridge.
Few drivers have "is_panel_bridge" flag to be able to distinguish a
drm_panel_bridge from "simple" drm_bridge.
To remove this flag I propose to
- let drm_panel_bridge_remove() check if
This function is the pendant of drm_of_find_panel_or_bridge()
to remove a previously allocated panel_bridge.
Given a specific port and endpoint it remove the panel bridge.
Since drm_panel_bridge_remove() will check that bridge parameter
is not NULL and is a real drm_panel_bridge and no a simple bri
With a call to drm_of_panel_bridge_remove() we could remove
the bridge without store it in vc4_dpi internal driver structure.
Signed-off-by: Benjamin Gaignard
---
drivers/gpu/drm/vc4/vc4_dpi.c | 17 ++---
1 file changed, 6 insertions(+), 11 deletions(-)
diff --git a/drivers/gpu/drm/
When using drm_of_panel_bridge_remove() we can simplify the
code and remove is_panel_bridge from dw_mipi_dsi structure.
Signed-off-by: Benjamin Gaignard
Reviewed-by: Philippe Cornu
Tested-by: Philippe Cornu
---
drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 5 +
1 file changed, 1 insertio
With a call to drm_of_panel_bridge_remove() we could remove the bridge
without store it in ldtc internal driver structure.
Signed-off-by: Benjamin Gaignard
Reviewed-by: Philippe Cornu
Tested-by: Philippe Cornu
---
drivers/gpu/drm/stm/ltdc.c | 16 +---
drivers/gpu/drm/stm/ltdc.h |
On Tue, 19 Sep 2017, Dmitry Torokhov wrote:
> We should not try to bring HID device out of full power state before
> calling hid_hw_close(), so that transport driver operates on powered up
> device (making this inverse of the opening sequence).
>
> Signed-off-by: Dmitry Torokhov
Applied to for-
On Mon, 2 Oct 2017, Kai-Heng Feng wrote:
> This reverts commit fcaa4a07d2a4b541e91da7a55d8b3331f96d1865.
>
> As noted by Masaki [1], 0x120A + trackpoint will not be used in mass
> production machines, so remove the ID accordingly.
>
> [1] http://www.spinics.net/lists/linux-input/msg53222.html
>
1 - 100 of 713 matches
Mail list logo
