Re: [PATCH] wlcore: mesh: handle failure case of pm_runtime_get_sync

2020-06-05 Thread Tony Lindgren
* Navid Emamdoost [200605 03:28]: > Calling pm_runtime_get_sync increments the counter even in case of > failure, causing incorrect ref count. Call pm_runtime_put if > pm_runtime_get_sync fails. Looks like we have a similar patch already in Linux next, care to check? Regards, Tony

Re: [PATCH v2] hwmon:(adm1275) Enable adm1278 ADM1278_TEMP1_EN

2020-06-05 Thread Manikandan
On Fri, May 29, 2020 at 10:30:16AM -0700, Guenter Roeck wrote: > On 5/29/20 5:46 AM, Manikandan Elumalai wrote: > > The adm1278 temperature sysfs attribute need it for one of the openbmc > > platform . > > This functionality is not enabled by default, so PMON_CONFIG needs to be > > modified in

Re: [PATCH] x86/split_lock: Sanitize userspace and guest error output

2020-06-05 Thread Prarit Bhargava
On 6/5/20 11:29 AM, Xiaoyao Li wrote: > On 6/5/2020 7:44 PM, Prarit Bhargava wrote: >> There are two problems with kernel messages in fatal mode that >> were found during testing of guests and userspace programs. >> >> The first is that no kernel message is output when the split lock detector

Re: [PATCH v2] ACPI: Drop rcu usage for MMIO mappings

2020-06-05 Thread Dan Williams
On Fri, Jun 5, 2020 at 9:22 AM Rafael J. Wysocki wrote: [..] > > The fix we are looking at now is to pre-map operation regions in a > > similar manner as the way APEI resources are pre-mapped. The > > pre-mapping would arrange for synchronize_rcu_expedited() to be elided > > on each dynamic

Lieber Freund,

2020-06-05 Thread AISHA GADDAFI
Assalamu Alaikum Lieber Freund, Darf ich dieses Medium nutzen, um eine gegenseitige Kommunikation mit Ihnen zu eröffnen und Ihre Akzeptanz für Investitionen in Ihr Land unter Ihrer Leitung als mein Partner zu suchen. Mein Name ist Aisha Gaddafi und ich lebe derzeit im Oman. Ich bin Witwe und

Re: slub freelist issue / BUG: unable to handle page fault for address: 000000003ffe0018

2020-06-05 Thread Vegard Nossum
On 2020-06-05 17:44, Kees Cook wrote: On Fri, Jun 05, 2020 at 04:44:51PM +0200, Vegard Nossum wrote: That's it :-) This fixes it for me: diff --git a/drivers/acpi/acpica/nsaccess.c b/drivers/acpi/acpica/nsaccess.c index 2566e2d4c7803..b76bbab917941 100644 --- a/drivers/acpi/acpica/nsaccess.c

[GIT PULL] Please pull powerpc/linux.git powerpc-5.8-1 tag

2020-06-05 Thread Michael Ellerman
Hi Linus, Please pull powerpc updates for 5.8. Unfortunately we've ended up with quite a few conflicts, which is primarily my fault for pushing things to next too late. Lesson learnt. I've pushed the result of my resolution of the conflicts to the powerpc/merge branch, if you want to look at

Re: [PATCH v4 04/11] watchdog: add support for sl28cpld watchdog

2020-06-05 Thread Guenter Roeck
On 6/5/20 9:04 AM, Andy Shevchenko wrote: > On Fri, Jun 5, 2020 at 6:05 PM Guenter Roeck wrote: >> On Fri, Jun 05, 2020 at 05:09:11PM +0300, Andy Shevchenko wrote: >>> On Fri, Jun 05, 2020 at 06:52:00AM -0700, Guenter Roeck wrote: On 6/5/20 3:50 AM, Andy Shevchenko wrote: > On Fri, Jun

Re: [PATCH net] net: phy: mscc: fix Serdes configuration in vsc8584_config_init

2020-06-05 Thread Florian Fainelli
On 6/5/2020 7:00 AM, Antoine Tenart wrote: > When converting the MSCC PHY driver to shared PHY packages, the Serdes > configuration in vsc8584_config_init was modified to use 'base_addr' > instead of 'base' as the port number. But 'base_addr' isn't equal to > 'addr' for all PHYs inside the

Re: [PATCH net 3/4] net: marvell: Fix OF_MDIO config check

2020-06-05 Thread Florian Fainelli
On 6/5/2020 7:01 AM, Dan Murphy wrote: > When CONFIG_OF_MDIO is set to be a module the code block is not > compiled. Use the IS_ENABLED macro that checks for both built in as > well as module. > > Fixes: cf41a51db8985 ("of/phylib: Use device tree properties to initialize > Marvell PHYs.") >

Re: [PATCH net 4/4] net: mscc: Fix OF_MDIO config check

2020-06-05 Thread Florian Fainelli
On 6/5/2020 7:01 AM, Dan Murphy wrote: > When CONFIG_OF_MDIO is set to be a module the code block is not > compiled. Use the IS_ENABLED macro that checks for both built in as > well as module. > > Fixes: 4f58e6dceb0e4 ("net: phy: Cleanup the Edge-Rate feature in Microsemi > PHYs.") >

Re: [PATCH net 1/4] net: dp83869: Fix OF_MDIO config check

2020-06-05 Thread Florian Fainelli
On 6/5/2020 7:01 AM, Dan Murphy wrote: > When CONFIG_OF_MDIO is set to be a module the code block is not > compiled. Use the IS_ENABLED macro that checks for both built in as > well as module. > > Fixes: 01db923e83779 ("net: phy: dp83869: Add TI dp83869 phy") > Signed-off-by: Dan Murphy

[PATCH stable 4.9 03/21] media: dvb/frontend.h: move out a private internal structure

2020-06-05 Thread Florian Fainelli
From: Mauro Carvalho Chehab commit f35afa4f60c868d7c7811ba747133acbf39410ac upstream struct dtv_cmds_h is just an ancillary struct used by the dvb_frontend.c to internally store frontend commands. It doesn't belong to the userspace header, nor it is used anywhere, except inside the DVB core.

Re: [PATCH net 2/4] net: dp83867: Fix OF_MDIO config check

2020-06-05 Thread Florian Fainelli
On 6/5/2020 7:01 AM, Dan Murphy wrote: > When CONFIG_OF_MDIO is set to be a module the code block is not > compiled. Use the IS_ENABLED macro that checks for both built in as > well as module. > > Fixes: 2a10154abcb75 ("net: phy: dp83867: Add TI dp83867 phy") > Signed-off-by: Dan Murphy

[PATCH stable 4.9 06/21] media: stv0288: get rid of set_property boilerplate

2020-06-05 Thread Florian Fainelli
From: Mauro Carvalho Chehab commit 473e4b4c1cf3046fc6b3437be9a9f3c89c2e61ef upstream This driver doesn't implement support for set_property(). Yet, it implements a boilerplate for it. Get rid of it. Signed-off-by: Mauro Carvalho Chehab Signed-off-by: Florian Fainelli ---

[PATCH stable 4.9 10/21] media: dvb_frontend: cleanup dvb_frontend_ioctl_properties()

2020-06-05 Thread Florian Fainelli
From: Mauro Carvalho Chehab commit 2b5df42b8dec69fb926a242007fd462343db4408 upstream Use a switch() on this function, just like on other ioctl handlers and handle parameters inside each part of the switch. That makes it easier to integrate with the already existing ioctl handler function.

[PATCH 04/16] dyndbg: rename __verbose section to __dyndbg

2020-06-05 Thread Jim Cromie
dyndbg populates its callsite info into __verbose section, change that to a more specific and descriptive name, __dyndbg. Also, per checkpatch: simplify __attribute(..) to __section(__dyndbg) declaration. and 1 spelling fix Signed-off-by: Jim Cromie --- include/asm-generic/vmlinux.lds.h |

[PATCH stable 4.9 07/21] media: stv6110: get rid of a srate dead code

2020-06-05 Thread Florian Fainelli
From: Mauro Carvalho Chehab commit 282996925b4d78f9795d176f7fb409281c98d56d upstream The stv6110 has a weird code that checks if get_property and set_property ioctls are defined. If they're, it initializes a "srate" var from properties cache. Otherwise, it sets to 15MBaud, with won't make any

[PATCH stable 4.9 11/21] media: dvb_frontend: cleanup ioctl handling logic

2020-06-05 Thread Florian Fainelli
From: Mauro Carvalho Chehab commit d73dcf0cdb95a47f7e4e991ab63dd30f6eb67b4e upstream Currently, there are two handlers for ioctls: - dvb_frontend_ioctl_properties() - dvb_frontend_ioctl_legacy() Despite their names, both handles non-legacy DVB ioctls. Besides that, there's no reason why to

[PATCH 08/16] dyndbg: refactor parse_linerange out of ddebug_parse_query

2020-06-05 Thread Jim Cromie
make the code-block reusable to later handle "file foo.c:101-200" etc. This should be a 90%+ code-move, with minimal adaptations; reindent, and scafolding. Signed-off-by: Jim Cromie --- lib/dynamic_debug.c | 61 + 1 file changed, 34 insertions(+), 27

[PATCH 14/16] dyndbg: prefer declarative init in caller, to memset in callee

2020-06-05 Thread Jim Cromie
drop memset in ddebug_parse_query, instead initialize the stack variable in the caller; let the compiler decide how to do it. Signed-off-by: Jim Cromie --- lib/dynamic_debug.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c index

[PATCH 12/16] dyndbg: add filter parameter to ddebug_parse_flags

2020-06-05 Thread Jim Cromie
Add a new *filter param to ddebug_parse_flags(), allowing it to communicate optional filter flags back to its caller: ddebug_change() Also, ddebug_change doesn't alter any of its arguments, including its 2 new ones; mods, filter. Say so by adding const modifier to them. Signed-off-by: Jim

[PATCH 15/16] dyndbg: add user-flag, negating-flags, and filtering on flags

2020-06-05 Thread Jim Cromie
1. Add a user-flag [u] which works like the [pfmlt] flags, but has no effect on callsite behavior; it allows incremental marking of arbitrary sets of callsites. 2. Add [PFMLTU] flags, which negate their counterparts; P===!p etc. And in ddebug_read_flags(): current code does: [pfmltu_] ->

[PATCH 13/16] dyndbg: extend ddebug_parse_flags to accept optional leading filter-flags

2020-06-05 Thread Jim Cromie
change ddebug_parse_flags to accept optional filterflags before OP. this now sets the parameter added in ~1 Signed-off-by: Jim Cromie --- .../admin-guide/dynamic-debug-howto.rst | 18 +++ lib/dynamic_debug.c | 30 ++- 2 files changed, 28

[PATCH 11/16] dyndbg: combine flags & mask into a struct, use that

2020-06-05 Thread Jim Cromie
combine flags & mask into a struct, and replace those 2 parameters in 3 functions: ddebug_change, ddebug_parse_flags, ddebug_read_flags, altering the derefs in them accordingly. This simplifies the 3 function sigs, preparing for more changes. We dont yet need mask from ddebug_read_flags, but will

[PATCH 16/16] dyndbg: allow negating flag-chars in modflags

2020-06-05 Thread Jim Cromie
Extend flags modifications to allow [PFMLTU] negating flags. This allows control-queries like: #> Q () { echo file inode.c $* > control } # to type less #> Q -P # same as +p #> Q +U # same as -u #> Q u-P # same as u+p This allows flags in a callsite to be simultaneously

[PATCH 10/16] dyndbg: refactor ddebug_read_flags out of ddebug_parse_flags

2020-06-05 Thread Jim Cromie
Currently, ddebug_parse_flags accepts [+-=][pflmt_]+ as flag-spec strings. If we allow [pflmt_]*[+-=][pflmt_]+ instead, the (new) 1st flagset can be used as a filter to select callsites, before applying changes in the 2nd flagset. 1st step is to split out the flags-reader so we can use it again.

[PATCH 06/16] dyndbg: fix a BUG_ON in ddebug_describe_flags

2020-06-05 Thread Jim Cromie
ddebug_describe_flags currently fills a caller provided string buffer, after testing its size (also passed) in a BUG_ON. Fix this by replacing them with a known-big-enough string buffer wrapped in a struct, and passing that instead. Also simplify the flags parameter, and instead de-ref the flags

[PATCH stable 4.9 19/21] media: dvb_frontend: Add commands implementation for compat ioct

2020-06-05 Thread Florian Fainelli
From: Jaedon Shin commit 18192a77f0810933ab71a46c1b260d230d7352ee upstream The dtv_properties structure and the dtv_property structure are different sizes in 32-bit and 64-bit system. This patch provides FE_SET_PROPERTY and FE_GET_PROPERTY ioctl commands implementation for 32-bit user space

[PATCH stable 4.9 15/21] media: dvb_frontend: dtv_property_process_set() cleanups

2020-06-05 Thread Florian Fainelli
From: Satendra Singh Thakur commit media: dvb_frontend: dtv_property_process_set() cleanups upstream Since all properties in the func dtv_property_process_set() use at most 4 bytes arguments, change the code to pass u32 cmd and u32 data as function arguments, instead of passing a pointer to the

[PATCH 09/16] dyndbg: accept 'file foo.c:func1' and 'file foo.c:10-100'

2020-06-05 Thread Jim Cromie
Accept these additional query forms: echo "file $filestr +_" > control path/to/file.c:100 # as from control, column 1 path/to/file.c:1-100 # or any legal line-range path/to/file.c:func_A# as from an editor/browser path/to/file.c:drm_\*# wildcards

[PATCH 07/16] dyndbg: make ddebug_tables list LIFO for add/remove_module

2020-06-05 Thread Jim Cromie
loadable modules are the last in on this list, and are the only modules that could be removed. ddebug_remove_module() searches from head, but ddebug_add_module() uses list_add_tail(). Change it to list_add() for a micro-optimization. Signed-off-by: Jim Cromie --- lib/dynamic_debug.c | 2 +- 1

[PATCH 02/16] dyndbg: drop obsolete comment on ddebug_proc_open

2020-06-05 Thread Jim Cromie
commit 4bad78c55002 ("lib/dynamic_debug.c: use seq_open_private() instead of seq_open()")' The commit was one of a tree-wide set which replaced open-coded boilerplate with a single tail-call. It therefore obsoleted the comment about that boilerplate, clean that up now. Signed-off-by: Jim

[PATCH stable 4.9 13/21] media: dvb_frontend: better document the -EPERM condition

2020-06-05 Thread Florian Fainelli
From: Mauro Carvalho Chehab commit da5516b5e81d45a96291823620f6c820178dc055 upstream Two readonly ioctls can't be allowed if the frontend device is opened in read only mode. Explain why. Reviewed by: Shuah Khan Signed-off-by: Mauro Carvalho Chehab Signed-off-by: Florian Fainelli ---

[PATCH 01/16] dyndbg-docs: eschew file /full/path query in docs

2020-06-05 Thread Jim Cromie
Regarding: commit 2b6783191da7 ("dynamic_debug: add trim_prefix() to provide source-root relative paths") commit a73619a845d5 ("kbuild: use -fmacro-prefix-map to make __FILE__ a relative path") 2nd commit broke dynamic-debug's "file $fullpath" query form, but nobody noticed because 1st commit

[PATCH 05/16] dyndbg: fix overcounting of ram used by dyndbg

2020-06-05 Thread Jim Cromie
during dyndbg init, verbose logging prints its ram overhead. It counted strlens of struct _ddebug's 4 string members, in all callsite entries, which would be approximately correct if each had been mallocd. But they are pointers into shared .rodata; for example, all 10 kobject callsites have

[PATCH 03/16] dyndbg: refine debug verbosity; 1 is basic, 2 more chatty

2020-06-05 Thread Jim Cromie
The verbose/debug logging done for `cat $MNT/dynamic_debug/control` is voluminous (2 per control file entry + 2 per PAGE). Moreover, it just prints pointer and sequence, which is not useful to a dyndbg user. So just drop them. Also require verbose>=2 for several other debug printks that are a

[PATCH stable 4.9 21/21] media: dvb_frontend: fix return error code

2020-06-05 Thread Florian Fainelli
From: Mauro Carvalho Chehab commit 330dada5957e3ca0c8811b14c45e3ac42c694651 upstream The correct error code when a function is not defined is -ENOTSUPP. It was typoed wrong as -EOPNOTSUPP, with, unfortunately, exists, but it is not used by the DVB core. Thanks-to: Geert Uytterhoeven

[PATCH 00/16] dynamic_debug: cleanups, 2 features

2020-06-05 Thread Jim Cromie
Patchset starts with 7 "cleanups"; - it changes section name from vague "__verbose" to "__dyndbg" - cleaner docs, drop obsolete comment & useless debug prints, refine verbosity, fix a BUG_ON, ram reporting miscounts. It adds a few query parsing conveniences; accept combined file:line &

[PATCH stable 4.9 08/21] media: friio-fe: get rid of set_property()

2020-06-05 Thread Florian Fainelli
From: Mauro Carvalho Chehab commit b2c41ca9632e686e79f6c9db9c5f75666d37926e upstream This callback is not actually doing anything but making it to return an error depending on the DTV frontend command. Well, that could break userspace for no good reason, and, if needed, should be implemented,

[PATCH stable 4.9 18/21] media: dvb_frontend: Add compat_ioctl callback

2020-06-05 Thread Florian Fainelli
From: Jaedon Shin commit c2dfd2276cec63a0c6f6ce18ed83800d96fde542 upstream Adds compat_ioctl for 32-bit user space applications on a 64-bit system. [m.che...@osg.samsung.com: add missing include compat.h] Signed-off-by: Jaedon Shin Signed-off-by: Mauro Carvalho Chehab Signed-off-by: Florian

[PATCH stable 4.9 14/21] media: dvb_frontend: fix return values for FE_SET_PROPERTY

2020-06-05 Thread Florian Fainelli
From: Mauro Carvalho Chehab commit 259a41d9ae8f3689742267f340ad2b159d00b302 upstream There are several problems with regards to the return of FE_SET_PROPERTY. The original idea were to return per-property return codes via tvp->result field, and to return an updated set of values. However, that

[PATCH stable 4.9 09/21] media: dvb_frontend: get rid of set_property() callback

2020-06-05 Thread Florian Fainelli
From: Mauro Carvalho Chehab commit 6680e73b5226114992acfc11f9cf5730f706fb01 upstream Now that all clients of set_property() were removed, get rid of this callback. Signed-off-by: Mauro Carvalho Chehab Signed-off-by: Florian Fainelli --- drivers/media/dvb-core/dvb_frontend.c | 7 ---

[PATCH stable 4.9 16/21] media: dvb_frontend: be sure to init dvb_frontend_handle_ioctl() return code

2020-06-05 Thread Florian Fainelli
From: Mauro Carvalho Chehab commit a9cb97c3e628902e37583d8a40bb28cf76522cf1 upstream As smatch warned: drivers/media/dvb-core/dvb_frontend.c:2468 dvb_frontend_handle_ioctl() error: uninitialized symbol 'err'. The ioctl handler actually got a regression here: before changeset

[PATCH stable 4.9 12/21] media: dvb_frontend: get rid of property cache's state

2020-06-05 Thread Florian Fainelli
From: Mauro Carvalho Chehab commit ef2cc27cf860b79874e9fde1419dd67c3372e41c upstream In the past, I guess the idea was to use state in order to allow an autofush logic. However, in the current code, it is used only for debug messages, on a poor man's solution, as there's already a debug message

[PATCH stable 4.9 05/21] media: dvb_frontend: get rid of get_property() callback

2020-06-05 Thread Florian Fainelli
From: Mauro Carvalho Chehab commit 8f8a19fcc1a89b83d0ab6d7cf2bcdd272dbd4334 upstream Only lg2160 implement gets_property, but there's no need for that, as no other driver calls this callback, as get_frontend() does the same, and set_frontend() also calls lg2160 get_frontend(). So, get rid of

[PATCH stable 4.9 20/21] media: dvb_frontend: fix wrong cast in compat_ioctl

2020-06-05 Thread Florian Fainelli
From: Katsuhiro Suzuki commit 5c6c9c4830b76d851d38829611b3c3e4be0f5cdf upstream FE_GET_PROPERTY has always failed as following situations: - Use compatible ioctl - The array of 'struct dtv_property' has 2 or more items This patch fixes wrong cast to a pointer 'struct dtv_property' from a

[PATCH stable 4.9 04/21] media: dvb/frontend.h: document the uAPI file

2020-06-05 Thread Florian Fainelli
From: Mauro Carvalho Chehab commit 8220ead805b6bab4ade2839857a198e9708b07de upstream Most of the stuff at the Digital TV frontend header file are documented only at the Documentation. However, a few kernel-doc markups are there, several of them with parsing issues. Add the missing

[PATCH stable 4.9 17/21] media: dvb_frontend: Add unlocked_ioctl in dvb_frontend.c

2020-06-05 Thread Florian Fainelli
From: Jaedon Shin commit a2282fd1fe2ebcda480426dbfaaa7c4e87e27399 upstream Adds unlocked ioctl function directly in dvb_frontend.c instead of using dvb_generic_ioctl(). Signed-off-by: Jaedon Shin Signed-off-by: Mauro Carvalho Chehab Signed-off-by: Florian Fainelli ---

[PATCH stable 4.9 00/21] Unbreak 32-bit DVB applications on 64-bit kernels

2020-06-05 Thread Florian Fainelli
Hi all, This long patch series was motivated by backporting Jaedon's changes which add a proper ioctl compatibility layer for 32-bit applications running on 64-bit kernels. We have a number of Android TV-based products currently running on the 4.9 kernel and this was broken for them. Thanks to

[PATCH stable 4.9 01/21] media: dvb_frontend: ensure that inital front end status initialized

2020-06-05 Thread Florian Fainelli
From: Colin Ian King commit a9e4998073d49a762a154a6b48a332ec6cb8e6b1 upstream The fe_status variable s is not initialized meaning it can have any random garbage status. This could be problematic if fe->ops.tune is false as s is not updated by the call to fe->ops.tune() and a subsequent check

[PATCH stable 4.9 02/21] media: dvb_frontend: initialize variable s with FE_NONE instead of 0

2020-06-05 Thread Florian Fainelli
From: Colin Ian King commit 450694c3b9f47b826a002089c463b9454b4bbe42 upstream GIT_AUTHOR_NAME=Colin King GIT_AUTHOR_EMAIL=colin.k...@canonical.com In a previous commit, we added FE_NONE as an unknown fe_status. Initialize variable s to FE_NONE instead of the more opaque value 0.

Re: [PATCH v5 17/19] mm: memcg/slab: use a single set of kmem_caches for all allocations

2020-06-05 Thread Vlastimil Babka
On 5/28/20 12:34 AM, Roman Gushchin wrote: > diff --git a/mm/slab.h b/mm/slab.h > index c49a863adb63..57b425d623e5 100644 > --- a/mm/slab.h > +++ b/mm/slab.h ... > @@ -526,8 +430,7 @@ static inline struct kmem_cache *cache_from_obj(struct > kmem_cache *s, void *x) >* When kmemcg is not

Re: [PATCH v2] crypto: DRBG - always try to free Jitter RNG instance

2020-06-05 Thread Eric Biggers
On Fri, Jun 05, 2020 at 08:52:57AM +0200, Stephan Mueller wrote: > Am Freitag, 5. Juni 2020, 08:16:46 CEST schrieb Eric Biggers: > > Hi Eric, > > > On Fri, Jun 05, 2020 at 07:58:15AM +0200, Stephan Mueller wrote: > > > Am Freitag, 5. Juni 2020, 02:43:36 CEST schrieb Eric Biggers: > > > > > > Hi

Re: [PATCH v2] ACPI: Drop rcu usage for MMIO mappings

2020-06-05 Thread Rafael J. Wysocki
On Fri, Jun 5, 2020 at 6:18 PM Dan Williams wrote: > > On Fri, Jun 5, 2020 at 6:32 AM Rafael J. Wysocki wrote: > > > > On Fri, May 8, 2020 at 1:55 AM Dan Williams > > wrote: > > > > > > Recently a performance problem was reported for a process invoking a > > > non-trival ASL program. The

Re: [PATCH v3] binfmt_misc: pass binfmt_misc flags to the interpreter

2020-06-05 Thread Laurent Vivier
Ping? Thanks, Laurent Le 28/01/2020 à 14:25, Laurent Vivier a écrit : > It can be useful to the interpreter to know which flags are in use. > > For instance, knowing if the preserve-argv[0] is in use would > allow to skip the pathname argument. > > This patch uses an unused auxiliary vector,

Re: [PATCH] software node: recursively unregister child swnodes

2020-06-05 Thread Jordan Hand
On 6/5/20 12:54 AM, Greg Kroah-Hartman wrote: Right now, the way the driver model and sysfs/kobjects work is that all objects must be removed in child-first order. The problem of your change where you want to try to remove the devices in parent-first order is that you do not really know if

Re: [PATCH] iomap: Handle I/O errors gracefully in page_mkwrite

2020-06-05 Thread Darrick J. Wong
On Fri, Jun 05, 2020 at 05:48:26AM -0700, Matthew Wilcox wrote: > On Fri, Jun 05, 2020 at 01:07:58PM +1000, Dave Chinner wrote: > > On Thu, Jun 04, 2020 at 07:24:51PM -0700, Matthew Wilcox wrote: > > > On Fri, Jun 05, 2020 at 10:31:59AM +1000, Dave Chinner wrote: > > > > On Thu, Jun 04, 2020 at

Re: [PATCH v2] ACPI: Drop rcu usage for MMIO mappings

2020-06-05 Thread Dan Williams
On Fri, Jun 5, 2020 at 6:32 AM Rafael J. Wysocki wrote: > > On Fri, May 8, 2020 at 1:55 AM Dan Williams wrote: > > > > Recently a performance problem was reported for a process invoking a > > non-trival ASL program. The method call in this case ends up > > repetitively triggering a call path

Re: [RFC][PATCH 5/7] irq_work, smp: Allow irq_work on call_single_queue

2020-06-05 Thread Peter Zijlstra
On Fri, Jun 05, 2020 at 05:02:08PM +0200, Frederic Weisbecker wrote: > On Fri, Jun 05, 2020 at 11:37:04AM +0200, Peter Zijlstra wrote: > > struct irq_work { > > - struct llist_node llnode; > > - atomic_t flags; > > + union { > > + struct __call_single_node node; > > +

[PATCH v2] media: budget-core: Improve exception handling in budget_register()

2020-06-05 Thread Chuhong Yuan
budget_register() has no error handling after its failure. Add the missed undo functions for error handling to fix it. Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2") Signed-off-by: Chuhong Yuan --- Changes in v2: - Modify the label of error handler. drivers/media/pci/ttpci/budget-core.c | 11

[PATCH v2] docs: deprecated.rst: Add zero-length and one-element arrays

2020-06-05 Thread Gustavo A. R. Silva
Add zero-length and one-element arrays to the list. While I continue replacing zero-length and one-element arrays with flexible-array members, I need a reference to point people to, so they don't introduce more instances of such arrays. And while here, add a note to the "open-coded arithmetic in

Re: [PATCH v7 01/13] tools/libperf: introduce notion of static polled file descriptors

2020-06-05 Thread Alexey Budankov
On 05.06.2020 14:38, Jiri Olsa wrote: > On Fri, Jun 05, 2020 at 12:50:54PM +0200, Jiri Olsa wrote: >> On Wed, Jun 03, 2020 at 06:52:59PM +0300, Alexey Budankov wrote: >>> >>> Implement adding of file descriptors by fdarray__add_stat() to >>> fix-sized (currently 1) stat_entries array located at

[PATCH v2] fbdev: geode: Add the missed pci_disable_device() in gx1fb_map_video_memory()

2020-06-05 Thread Chuhong Yuan
Although gx1fb_probe() has handled the failure of gx1fb_map_video_memory() partly, it does not call pci_disable_device() as gx1fb_map_video_memory() calls pci_enable_device(). Add the missed function call to fix the bug. Fixes: 53eed4ec8bcd ("[PATCH] fbdev: geode updates]") Signed-off-by: Chuhong

Re: [RFC][PATCH 7/7] sched: Replace rq::wake_list

2020-06-05 Thread Eric Biggers
On Fri, Jun 05, 2020 at 09:41:54AM +0200, Peter Zijlstra wrote: > On Thu, Jun 04, 2020 at 05:24:33PM -0700, Eric Biggers wrote: > > On Thu, Jun 04, 2020 at 07:18:37AM -0700, Guenter Roeck wrote: > > > On Tue, May 26, 2020 at 06:11:04PM +0200, Peter Zijlstra wrote: > > > > The recent commit:

[PATCH v2] clk: clk-st: Add missed return value checks in st_clk_probe()

2020-06-05 Thread Chuhong Yuan
Return values were not checked after calls of the following functions. - clk_hw_register_mux - clk_hw_register_gate - devm_clk_hw_register_clkdev Thus add error detection and the corresponding exception handling. Return the value from the function call "devm_clk_hw_register_clkdev" at the

Re: [RFC PATCH v4 07/10] vfio/pci: introduce a new irq type VFIO_IRQ_TYPE_REMAP_BAR_REGION

2020-06-05 Thread Alex Williamson
On Thu, 4 Jun 2020 22:02:31 -0400 Yan Zhao wrote: > On Wed, Jun 03, 2020 at 10:10:58PM -0600, Alex Williamson wrote: > > On Wed, 3 Jun 2020 22:42:28 -0400 > > Yan Zhao wrote: > > > > > On Wed, Jun 03, 2020 at 05:04:52PM -0600, Alex Williamson wrote: > > > > On Tue, 2 Jun 2020 21:40:58

Re: BUG: using smp_processor_id() in preemptible code in radix_tree_node_alloc

2020-06-05 Thread Eric Biggers
On Fri, Jun 05, 2020 at 04:29:22AM -0700, Matthew Wilcox wrote: > On Thu, Jun 04, 2020 at 08:55:55PM -0700, Eric Biggers wrote: > > Possibly a bug in lib/radix-tree.c? this_cpu_ptr() in > > radix_tree_node_alloc() > > can be reached without a prior preempt_disable(). Or is the caller of > >

Re: [PATCH v4 04/11] watchdog: add support for sl28cpld watchdog

2020-06-05 Thread Andy Shevchenko
On Fri, Jun 5, 2020 at 6:05 PM Guenter Roeck wrote: > On Fri, Jun 05, 2020 at 05:09:11PM +0300, Andy Shevchenko wrote: > > On Fri, Jun 05, 2020 at 06:52:00AM -0700, Guenter Roeck wrote: > > > On 6/5/20 3:50 AM, Andy Shevchenko wrote: > > > > On Fri, Jun 5, 2020 at 1:24 PM Michael Walle wrote: >

[PATCH v2 3/3] exec: Move path_noexec() check earlier

2020-06-05 Thread Kees Cook
The path_noexec() check, like the regular file check, was happening too late, letting LSMs see impossible execve()s. Check it earlier as well in may_open() and collect the redundant fs/exec.c path_noexec() test under the same robustness comment as the S_ISREG() check. My notes on the call path,

[PATCH v2 2/3] exec: Move S_ISREG() check earlier

2020-06-05 Thread Kees Cook
The execve(2)/uselib(2) syscalls have always rejected non-regular files. Recently, it was noticed that a deadlock was introduced when trying to execute pipes, as the S_ISREG() test was happening too late. This was fixed in commit 73601ea5b7b1 ("fs/open.c: allow opening only regular files during

[PATCH v2 1/3] exec: Change uselib(2) IS_SREG() failure to EACCES

2020-06-05 Thread Kees Cook
Change uselib(2)' S_ISREG() error return to EACCES instead of EINVAL so the behavior matches execve(2), and the seemingly documented value. The "not a regular file" failure mode of execve(2) is explicitly documented[1], but it is not mentioned in uselib(2)[2] which does, however, say that open(2)

[PATCH v2 0/3] Relocate execve() sanity checks

2020-06-05 Thread Kees Cook
Hi, While looking at the code paths for the proposed O_MAYEXEC flag, I saw some things that looked like they should be fixed up. exec: Change uselib(2) IS_SREG() failure to EACCES This just regularizes the return code on uselib(2). exec: Move S_ISREG() check earlier This

Re: [GIT PULL] sh: remove sh5 support

2020-06-05 Thread Rich Felker
On Fri, Jun 05, 2020 at 05:47:34PM +0200, John Paul Adrian Glaubitz wrote: > On 6/5/20 5:43 PM, Rich Felker wrote: > >> Can you include the patch as well? > > > > This one is outside arch/sh and I'm not sure it's permissible to go up > > through my tree. I was also under the impression that only

Re: [PATCH] regulator: do not balance 'boot-on' coupled regulators without constraints

2020-06-05 Thread Mark Brown
On Fri, Jun 05, 2020 at 03:37:32PM +0200, Marek Szyprowski wrote: > On 05.06.2020 12:20, Mark Brown wrote: > > No, this is not what boot-on means at all. It is there for cases where > > we can't read the enable status from the hardware. Trying to infer > > *anything* about the runtime behaviour

Re: [PATCH] crypto: hisilicon - fix strncpy warning with strlcpy

2020-06-05 Thread Eric Biggers
On Fri, Jun 05, 2020 at 11:26:20PM +0800, Zhangfei Gao wrote: > > > On 2020/6/5 下午8:17, Herbert Xu wrote: > > On Fri, Jun 05, 2020 at 05:34:32PM +0800, Zhangfei Gao wrote: > > > Will add a check after the copy. > > > > > >     strlcpy(interface.name, pdev->driver->name, > > >

Re: [GIT PULL] sh: remove sh5 support

2020-06-05 Thread John Paul Adrian Glaubitz
On 6/5/20 5:43 PM, Rich Felker wrote: >> Can you include the patch as well? > > This one is outside arch/sh and I'm not sure it's permissible to go up > through my tree. I was also under the impression that only part 1 was > needed to fix the immediate problem on sh and tha part 2 was for >

Re: [PATCH 5.6 00/43] 5.6.17-rc1 review

2020-06-05 Thread Greg Kroah-Hartman
On Fri, Jun 05, 2020 at 09:30:36AM -0600, Shuah Khan wrote: > On 6/5/20 8:14 AM, Greg Kroah-Hartman wrote: > > This is the start of the stable review cycle for the 5.6.17 release. > > There are 43 patches in this series, all will be posted as a response > > to this one. If anyone has any issues

[PATCH] Add missing bound checks for software 842 decompressor

2020-06-05 Thread Joan Bruguera
The software 842 decompressor receives, through the initial value of the 'olen' parameter, the capacity of the buffer pointed to by 'out'. If this capacity is insufficient to decode the compressed bitstream, -ENOSPC should be returned. However, the bounds checks are missing for index references

Re: slub freelist issue / BUG: unable to handle page fault for address: 000000003ffe0018

2020-06-05 Thread Kees Cook
On Fri, Jun 05, 2020 at 04:44:51PM +0200, Vegard Nossum wrote: > On 2020-06-05 16:08, Vlastimil Babka wrote: > > On 6/5/20 3:12 PM, Rafael J. Wysocki wrote: > > > On Fri, Jun 5, 2020 at 2:48 PM Vegard Nossum > > > wrote: > > > > > > > > On 2020-06-05 11:36, Vegard Nossum wrote: > > > > > > > >

Re: [GIT PULL] sh: remove sh5 support

2020-06-05 Thread Rich Felker
On Fri, Jun 05, 2020 at 05:38:18PM +0200, John Paul Adrian Glaubitz wrote: > Hi Rich! > > On 6/2/20 3:33 AM, Rich Felker wrote: > [PATCH 1/2] arch/sh: vmlinux.scr > https://marc.info/?l=linux-sh=158429470120959=2 > >> > >> OK. > > > > Included in -mm. > > I just had a look at your

Re: [RFC] dt-bindings: mailbox: add doorbell support to ARM MHU

2020-06-05 Thread Jassi Brar
On Fri, Jun 5, 2020 at 3:58 AM Sudeep Holla wrote: > > > > > >> bash-1526 [000] 1149.472553: scmi_xfer_begin: > > > > >> transfer_id=1538 msg_id=6 protocol_id=21 seq=0 poll=0 > > > > >> -0 [001] 1149.472733: scmi_xfer_begin: > > > > >> transfer_id=1539 msg_id=7

[PATCH 1/9] doc: networking: wireless: fix wiki website url

2020-06-05 Thread Flavio Suligoi
In the files: - regulatory.rst - mac80211-injection.rst the wiki url is still the old "wireless.kernel.org" instead of the new "wireless.wiki.kernel.org" Signed-off-by: Flavio Suligoi --- Documentation/networking/mac80211-injection.rst | 2 +- Documentation/networking/regulatory.rst |

[PATCH 4/9] net: wireless: atmel: fix wiki website url

2020-06-05 Thread Flavio Suligoi
In at76c50x-usb.c the wiki url is still the old "wireless.kernel.org" instead of the new "wireless.wiki.kernel.org" Signed-off-by: Flavio Suligoi --- drivers/net/wireless/atmel/at76c50x-usb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 2/9] net: wireless: fix wiki website url in main Kconfig

2020-06-05 Thread Flavio Suligoi
The wiki url is still the old "wireless.kernel.org" instead of the new "wireless.wiki.kernel.org" Signed-off-by: Flavio Suligoi --- drivers/net/wireless/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/wireless/Kconfig b/drivers/net/wireless/Kconfig index

[PATCH 7/9] net: wireless: intersil: fix wiki website url

2020-06-05 Thread Flavio Suligoi
In some Intesil files, the wiki url is still the old "wireless.kernel.org" instead of the new "wireless.wiki.kernel.org" Signed-off-by: Flavio Suligoi --- drivers/net/wireless/intersil/Kconfig | 2 +- drivers/net/wireless/intersil/p54/Kconfig | 6 +++---

[PATCH 8/9] include: fix wiki website url in netlink interface header

2020-06-05 Thread Flavio Suligoi
The wiki url is still the old "wireless.kernel.org" instead of the new "wireless.wiki.kernel.org" Signed-off-by: Flavio Suligoi --- include/uapi/linux/nl80211.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/uapi/linux/nl80211.h b/include/uapi/linux/nl80211.h index

[PATCH 6/9] net: wireless: intel: fix wiki website url

2020-06-05 Thread Flavio Suligoi
In some Intel files, the wiki url is still the old "wireless.kernel.org" instead of the new "wireless.wiki.kernel.org" Signed-off-by: Flavio Suligoi --- drivers/net/wireless/intel/iwlegacy/4965-mac.c | 2 +- drivers/net/wireless/intel/iwlwifi/Kconfig | 2 +- 2 files changed, 2

[PATCH 9/9] net: fix wiki website url mac80211 and wireless files

2020-06-05 Thread Flavio Suligoi
In the files: - net/mac80211/rx.c - net/wireless/Kconfig the wiki url is still the old "wireless.kernel.org" instead of the new "wireless.wiki.kernel.org" Signed-off-by: Flavio Suligoi --- net/mac80211/rx.c| 2 +- net/wireless/Kconfig | 2 +- 2 files changed, 2 insertions(+), 2

[PATCH 0/9] net: wireless: fix wireless wiki website url

2020-06-05 Thread Flavio Suligoi
In some files, related to the net wireless sub-system, the wireless wiki URL is still the old "wireless.kernel.org" instead of the new "wireless.wiki.kernel.org" Flavio Suligoi (9): doc: networking: wireless: fix wiki website url net: wireless: fix wiki website url in main Kconfig net:

[PATCH 5/9] net: wireless: broadcom: fix wiki website url

2020-06-05 Thread Flavio Suligoi
In some b43 files, the wiki url is still the old "wireless.kernel.org" instead of the new "wireless.wiki.kernel.org" Signed-off-by: Flavio Suligoi --- drivers/net/wireless/broadcom/b43/main.c | 2 +- drivers/net/wireless/broadcom/b43legacy/main.c | 4 ++-- 2 files changed, 3

[PATCH 3/9] net: wireless: ath: fix wiki website url

2020-06-05 Thread Flavio Suligoi
In some ath files, the wiki url is still the old "wireless.kernel.org" instead of the new "wireless.wiki.kernel.org" Signed-off-by: Flavio Suligoi --- drivers/net/wireless/ath/Kconfig | 4 ++-- drivers/net/wireless/ath/ath9k/Kconfig| 5 +++-- drivers/net/wireless/ath/ath9k/hw.c

Re: [PATCH v3 01/18] nitro_enclaves: Add ioctl interface definition

2020-06-05 Thread Paraschiv, Andra-Irina
On 05/06/2020 11:15, Stefan Hajnoczi wrote: On Mon, Jun 01, 2020 at 10:20:18AM +0300, Paraschiv, Andra-Irina wrote: On 01/06/2020 06:02, Benjamin Herrenschmidt wrote: On Wed, 2020-05-27 at 09:49 +0100, Stefan Hajnoczi wrote: What about feature bits or a API version number field? If you

Re: [PATCH v4 2/3] media: stm32-dcmi: Set minimum cpufreq requirement

2020-06-05 Thread Valentin Schneider
On 05/06/20 14:05, Benjamin Gaignard wrote: > Before start streaming set cpufreq minimum frequency requirement. > The cpufreq governor will adapt the frequencies and we will have > no latency for handling interrupts. > The frequency requirement is retrieved from the device-tree node. > >

Re: [GIT PULL] sh: remove sh5 support

2020-06-05 Thread John Paul Adrian Glaubitz
Hi Rich! On 6/2/20 3:33 AM, Rich Felker wrote: [PATCH 1/2] arch/sh: vmlinux.scr https://marc.info/?l=linux-sh=158429470120959=2 >> >> OK. > > Included in -mm. I just had a look at your tree and it looks you forgot to merge the second patch of the series, see: >

Re: [PATCH 1/5] media: videodev2: add Compressed Framebuffer pixel formats

2020-06-05 Thread Nicolas Dufresne
Le vendredi 05 juin 2020 à 11:35 -0400, Nicolas Dufresne a écrit : > Le jeudi 04 juin 2020 à 15:53 +0200, Neil Armstrong a écrit : > > From: Maxime Jourdan > > > > Add two generic Compressed Framebuffer pixel formats to be used > > with a modifier when imported back in another subsystem like

Re: [PATCH 1/5] media: videodev2: add Compressed Framebuffer pixel formats

2020-06-05 Thread Nicolas Dufresne
Le jeudi 04 juin 2020 à 15:53 +0200, Neil Armstrong a écrit : > From: Maxime Jourdan > > Add two generic Compressed Framebuffer pixel formats to be used > with a modifier when imported back in another subsystem like DRM/KMS. > > These pixel formats represents generic 8bits and 10bits compressed

Re: [PATCH] tpm: ibmvtpm: Wait for ready buffer before probing for TPM2 attributes

2020-06-05 Thread Stefan Berger
On 6/5/20 2:37 AM, David Gibson wrote: The tpm2_get_cc_attrs_tbl() call will result in TPM commands being issued, which will need the use of the internal command/response buffer. But, we're issuing this *before* we've waited to make sure that buffer is allocated. This can result in

Re: [PATCH 5.6 00/43] 5.6.17-rc1 review

2020-06-05 Thread Shuah Khan
On 6/5/20 8:14 AM, Greg Kroah-Hartman wrote: This is the start of the stable review cycle for the 5.6.17 release. There are 43 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know. Responses should be made

RE: [PATCH] scsi: megaraid_sas: fix kdump kernel boot hung caused by JBOD

2020-06-05 Thread Chandrakanth Patil
>Subject: Re: [PATCH] scsi: megaraid_sas: fix kdump kernel boot hung caused >by JBOD > >On 2020/06/05 Fri 01:05, Chandrakanth Patil wrote: >> >>Hi Kai Liu, >> >>Gen3 (Invader) and Gen3.5 (Ventura/Aero) generations of controllers are >>affected. > >Hi Chandrakanth, > >My card is not one of these

<    1   2   3   4   5   6   7   8   9   10   >