Re: [PATCH v7 1/3] PM: wakeup: Add routine to help fetch wakeup source object.

2019-10-21 Thread Rafael J. Wysocki
On Mon, Oct 21, 2019 at 5:49 AM Ran Wang wrote: > > Some user might want to go through all registered wakeup sources > and doing things accordingly. For example, SoC PM driver might need to > do HW programming to prevent powering down specific IP which wakeup > source depending on. So add this

Re: [PATCH v2 3/4] Documentation/ABI: mark /sys/kernel/fadump_* sysfs files deprecated

2019-10-21 Thread Hari Bathini
On 18/10/19 6:35 PM, Sourabh Jain wrote: > The /sys/kernel/fadump_* sysfs files are replicated under [...] > +Note: The following FADump sysfs files are deprecated. > + > +Deprecated Alternative > + >

[PATCH v1] usb: fsl: Check memory resource before releasing it

2019-10-21 Thread Yinbo Zhu
From: Nikhil Badola Check memory resource existence before releasing it to avoid NULL pointer dereference Signed-off-by: Nikhil Badola Reviewed-by: Ran Wang Reviewed-by: Peter Chen --- drivers/usb/gadget/udc/fsl_udc_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] lib/vdso: Make clock_getres() POSIX compliant again

2019-10-21 Thread Thomas Gleixner
A recent commit removed the NULL pointer check from the clock_getres() implementation causing a test case to fault. POSIX requires an explicit NULL pointer check for clock_getres() aside of the validity check of the clock_id argument for obscure reasons. Add it back for both 32bit and 64bit.

[PATCH v1] usb: gadget: Correct NULL pointer checking in fsl gadget

2019-10-21 Thread Yinbo Zhu
From: Nikhil Badola Correct NULL pointer checking for endpoint descriptor before it gets dereferenced Signed-off-by: Nikhil Badola Signed-off-by: Ran Wang Reviewed-by: Peter Chen --- drivers/usb/gadget/udc/fsl_udc_core.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git

Re: [PATCH 2/2] virtio_ring: Use DMA API if memory is encrypted

2019-10-21 Thread David Gibson
On Tue, Oct 15, 2019 at 09:35:01AM +0200, Christoph Hellwig wrote: > On Fri, Oct 11, 2019 at 06:25:19PM -0700, Ram Pai wrote: > > From: Thiago Jung Bauermann > > > > Normally, virtio enables DMA API with VIRTIO_F_IOMMU_PLATFORM, which must > > be set by both device and guest driver. However, as

Re: [PATCH 0/7] towards QE support on ARM

2019-10-21 Thread Rasmus Villemoes
On 18/10/2019 23.52, Li Yang wrote: > On Fri, Oct 18, 2019 at 3:54 PM Rasmus Villemoes > wrote: >> >> On 18/10/2019 22.16, Leo Li wrote: >>> There have been several attempts in the past few years to allow building the QUICC engine drivers for platforms other than PPC. This is

[PATCH v1] usb: fsl: Remove unused variable

2019-10-21 Thread Yinbo Zhu
From: Nikhil Badola Remove unused variable td_complete Signed-off-by: Nikhil Badola Reviewed-by: Ran Wang Reviewed-by: Peter Chen --- drivers/usb/gadget/udc/fsl_udc_core.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/usb/gadget/udc/fsl_udc_core.c

[RFC PATCH] powerpc/32: Switch VDSO to C implementation.

2019-10-21 Thread Christophe Leroy
This is a tentative to switch powerpc/32 vdso to generic C implementation. It will likely not work on 64 bits or even build properly at the moment. powerpc is a bit special for VDSO as well as system calls in the way that it requires setting CR SO bit which cannot be done in C. Therefore,

[PATCH 2/5] cxgb4/cxgb4vf: Remove superfluous void * cast in debugfs_create_file() call

2019-10-21 Thread Geert Uytterhoeven
There is no need to cast a typed pointer to a void pointer when calling a function that accepts the latter. Remove it, as the cast prevents further compiler checks. Signed-off-by: Geert Uytterhoeven --- drivers/net/ethernet/chelsio/cxgb4vf/cxgb4vf_main.c | 2 +- 1 file changed, 1 insertion(+),

Re: [PATCH] lib/vdso: Make clock_getres() POSIX compliant again

2019-10-21 Thread Christophe Leroy
Le 21/10/2019 à 12:07, Thomas Gleixner a écrit : A recent commit removed the NULL pointer check from the clock_getres() implementation causing a test case to fault. POSIX requires an explicit NULL pointer check for clock_getres() aside of the validity check of the clock_id argument for

[PATCH 1/5] crypto: nx - Improve debugfs_create_u{32, 64}() handling for atomics

2019-10-21 Thread Geert Uytterhoeven
Variables of type atomic{,64}_t can be used fine with debugfs_create_u{32,64}, when passing a pointer to the embedded counter. This allows to get rid of the casts, which prevented compiler checks. Signed-off-by: Geert Uytterhoeven --- drivers/crypto/nx/nx_debugfs.c | 18 +- 1

[PATCH] powerpc/security: Fix debugfs data leak on 32-bit

2019-10-21 Thread Geert Uytterhoeven
"powerpc_security_features" is "unsigned long", i.e. 32-bit or 64-bit, depending on the platform (PPC_FSL_BOOK3E or PPC_BOOK3S_64). Hence casting its address to "u64 *", and calling debugfs_create_x64() is wrong, and leaks 32-bit of nearby data to userspace on 32-bit platforms. While all

[PATCH 4/5] power: avs: smartreflex: Remove superfluous cast in debugfs_create_file() call

2019-10-21 Thread Geert Uytterhoeven
There is no need to cast a typed pointer to a void pointer when calling a function that accepts the latter. Remove it, as the cast prevents further compiler checks. Signed-off-by: Geert Uytterhoeven --- drivers/power/avs/smartreflex.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH 5/5] ionic: Use debugfs_create_bool() to export bool

2019-10-21 Thread Geert Uytterhoeven
Currently bool ionic_cq.done_color is exported using debugfs_create_u8(), which requires a cast, preventing further compiler checks. Fix this by switching to debugfs_create_bool(), and dropping the cast. Signed-off-by: Geert Uytterhoeven --- drivers/net/ethernet/pensando/ionic/ionic_debugfs.c

[PATCH 3/5] drm/amdgpu: Remove superfluous void * cast in debugfs_create_file() call

2019-10-21 Thread Geert Uytterhoeven
There is no need to cast a typed pointer to a void pointer when calling a function that accepts the latter. Remove it, as the cast prevents further compiler checks. Signed-off-by: Geert Uytterhoeven --- drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH 0/5] debugfs: Remove casts in debugfs_create_*() callers

2019-10-21 Thread Geert Uytterhoeven
Hi all, Casting parameters in debugfs_create_*() calls prevents the compiler from performing some checks. Hence this patch series removes superfluous casts, or reworks code to no longer need the casts. All patches can be applied independently, there are no dependencies. Thanks for your

RE: passing NULL to clock_getres (VDSO): terminated by unexpected signal 11

2019-10-21 Thread David Laight
From: Thomas Gleixner > Sent: 20 October 2019 20:53 > On Sun, 20 Oct 2019, Andreas Schwab wrote: > > On Okt 20 2019, Thomas Gleixner wrote: > > > > > POSIX does not mention anything about the validity of the pointer handed > > > to > > > clock_getres(). > > > > Sure it does: "If the argument res

[Bug 199561] sungem: RX MAC fifo overflow smac[03910440]

2019-10-21 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=199561 Erhard F. (erhar...@mailbox.org) changed: What|Removed |Added CC|

[Bug 205283] New: BUG: KASAN: global-out-of-bounds in _copy_to_iter+0x3d4/0x5a8

2019-10-21 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205283 Bug ID: 205283 Summary: BUG: KASAN: global-out-of-bounds in _copy_to_iter+0x3d4/0x5a8 Product: File System Version: 2.5 Kernel Version: 5.4-rc4 Hardware: PPC-32

[Bug 205283] BUG: KASAN: global-out-of-bounds in _copy_to_iter+0x3d4/0x5a8

2019-10-21 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205283 --- Comment #1 from Erhard F. (erhar...@mailbox.org) --- Created attachment 285607 --> https://bugzilla.kernel.org/attachment.cgi?id=285607=edit 5.4.0-rc4 kernel .config (PowerMac G4 DP) -- You are receiving this mail because: You are

Re: [RFC PATCH] powerpc/32: Switch VDSO to C implementation.

2019-10-21 Thread Thomas Gleixner
On Mon, 21 Oct 2019, Christophe Leroy wrote: > This is a tentative to switch powerpc/32 vdso to generic C implementation. > It will likely not work on 64 bits or even build properly at the moment. > > powerpc is a bit special for VDSO as well as system calls in the > way that it requires setting

Re: [PATCH 0/7] towards QE support on ARM

2019-10-21 Thread Li Yang
On Mon, Oct 21, 2019 at 3:46 AM Rasmus Villemoes wrote: > > On 18/10/2019 23.52, Li Yang wrote: > > On Fri, Oct 18, 2019 at 3:54 PM Rasmus Villemoes > > wrote: > >> > >> On 18/10/2019 22.16, Leo Li wrote: > >>> > > There have been several attempts in the past few years to allow

Re: [PATCH v4 3/3] powerpc/prom_init: Use -ffreestanding to avoid a reference to bcmp

2019-10-21 Thread Nathan Chancellor
On Fri, Oct 18, 2019 at 03:02:10PM -0500, Segher Boessenkool wrote: > On Fri, Oct 18, 2019 at 12:00:22PM -0700, Nathan Chancellor wrote: > > Just as an FYI, there was some more discussion around the availablity > > and use of bcmp in this LLVM bug which spawned > > commit 5f074f3e192f

Re: [PATCH 5/5] ionic: Use debugfs_create_bool() to export bool

2019-10-21 Thread Shannon Nelson
On 10/21/19 7:51 AM, Geert Uytterhoeven wrote: Currently bool ionic_cq.done_color is exported using debugfs_create_u8(), which requires a cast, preventing further compiler checks. Fix this by switching to debugfs_create_bool(), and dropping the cast. Signed-off-by: Geert Uytterhoeven

RE: [PATCH 0/7] towards QE support on ARM

2019-10-21 Thread Qiang Zhao
On Mon, Oct 22, 2019 at 6:11 AM Leo Li wrote > -Original Message- > From: Li Yang > Sent: 2019年10月22日 6:11 > To: Rasmus Villemoes > Cc: Timur Tabi ; Greg Kroah-Hartman > ; linux-ker...@vger.kernel.org; > linux-ser...@vger.kernel.org; Jiri Slaby ; > linuxppc-dev@lists.ozlabs.org;