Re: [PATCH v4] remoteproc: Add device awake calls in rproc boot and shutdown path

2025-04-03 Thread Mukesh Ojha
(rproc->dev.parent); > ret = mutex_lock_interruptible(&rproc->lock); > if (ret) { > dev_err(dev, "can't lock rproc %s: %d\n", rproc->name, ret); > @@ -2027,6 +2030,7 @@ int rproc_shutdown(struct rproc *rproc) > rproc-&g

Re: Query: DMA device assigned to remoteproc usage by Linux

2025-01-19 Thread Mukesh Ojha
On Mon, Dec 23, 2024 at 11:27 PM Mukesh Ojha wrote: > > Hi All, > > Wanted to check if we have encountered remoteproc use case where a device > with dma is assigned to a remoteproc with its own streamid and iommu > translation context. This DMA device can have a selective DMA

Query: DMA device assigned to remoteproc usage by Linux

2024-12-23 Thread Mukesh Ojha
Hi All, Wanted to check if we have encountered remoteproc use case where a device with dma is assigned to a remoteproc with its own streamid and iommu translation context. This DMA device can have a selective DMA range within the remoteproc carveout memory that needs to be iommu mapped before the

Re: [PATCH v3] remoteproc: Add a new remoteproc state RPROC_DEFUNCT

2024-10-25 Thread Mukesh Ojha
On Fri, Oct 25, 2024 at 09:08:03AM -0600, Mathieu Poirier wrote: > On Fri, Oct 25, 2024 at 01:40:45PM +0530, Mukesh Ojha wrote: > > On Mon, Oct 21, 2024 at 09:12:47AM -0600, Mathieu Poirier wrote: > > > Hi Mukesh, > > > > > > On Wed, Oct 16, 2024 at

Re: [PATCH v3] remoteproc: Add a new remoteproc state RPROC_DEFUNCT

2024-10-25 Thread Mukesh Ojha
On Mon, Oct 21, 2024 at 09:12:47AM -0600, Mathieu Poirier wrote: > Hi Mukesh, > > On Wed, Oct 16, 2024 at 10:25:46AM +0530, Mukesh Ojha wrote: > > Multiple call to glink_subdev_stop() for the same remoteproc can happen > > if rproc_stop() fails from Process-A that leav

[PATCH v3] remoteproc: Add a new remoteproc state RPROC_DEFUNCT

2024-10-15 Thread Mukesh Ojha
V Unable to handle kernel NULL pointer dereference at virtual address 0358 Signed-off-by: Mukesh Ojha --- Changes in v3: - Fix kernel test report

Re: [PATCH v2] remoteproc: Add a new remoteproc state RPROC_DEFUNCT

2024-10-14 Thread Mukesh Ojha
On Tue, Oct 15, 2024 at 02:01:18AM +0530, Mukesh Ojha wrote: > Multiple call to glink_subdev_stop() for the same remoteproc can happen > if rproc_stop() fails from Process-A that leaves the rproc state to > RPROC_CRASHED state later a call to recovery_store from user space in > Proces

[PATCH v2] remoteproc: Add a new remoteproc state RPROC_DEFUNCT

2024-10-14 Thread Mukesh Ojha
V Unable to handle kernel NULL pointer dereference at virtual address 0358 Signed-off-by: Mukesh Ojha --- Changes in v2: - Removed NULL pointer

Re: [PATCH 6/6] remoteproc: qcom: Enable map/unmap and SHM bridge support

2024-10-11 Thread Mukesh Ojha
On Fri, Oct 11, 2024 at 09:23:05AM +0300, Dmitry Baryshkov wrote: > On Fri, Oct 11, 2024 at 10:35:18AM GMT, Shiraz Hashim wrote: > > On Thu, Oct 10, 2024 at 08:57:56AM +0200, neil.armstr...@linaro.org wrote: > > > On 08/10/2024 08:21, Mukesh Ojha wrote: > > > > On M

Re: [PATCH 6/6] remoteproc: qcom: Enable map/unmap and SHM bridge support

2024-10-07 Thread Mukesh Ojha
On Mon, Oct 07, 2024 at 08:22:39PM +0530, Mukesh Ojha wrote: > On Mon, Oct 07, 2024 at 10:05:08AM +0200, neil.armstr...@linaro.org wrote: > > On 04/10/2024 23:23, Mukesh Ojha wrote: > > > For Qualcomm SoCs runnning with Qualcomm EL2 hypervisor(QHEE), IOMMU > > > transl

Re: [PATCH] remoteproc: qcom: Fix NULL pointer in glink_subdev_stop()

2024-10-07 Thread Mukesh Ojha
On Tue, Oct 01, 2024 at 02:15:52PM -0700, Bjorn Andersson wrote: > On Tue, Oct 01, 2024 at 12:06:17PM +0530, Mukesh Ojha wrote: > > On Fri, Sep 27, 2024 at 02:59:09PM -0700, Bjorn Andersson wrote: > > > On Sat, Sep 28, 2024 at 01:07:43AM +0530, Mukesh Ojha wrote: > > > &

Re: [PATCH 0/6] Peripheral Image Loader support for Qualcomm SoCs

2024-10-07 Thread Mukesh Ojha
On Sun, Oct 06, 2024 at 10:34:19PM +0300, Dmitry Baryshkov wrote: > On Sat, Oct 05, 2024 at 02:53:53AM GMT, Mukesh Ojha wrote: > > Qualcomm is looking to enable remote processors on the SA8775p SoC > > running KVM Linux host and is currently trying to figure out an > > upstrea

Re: [PATCH 1/6] dt-bindings: remoteproc: qcom,pas-common: Introduce iommus and qcom,devmem property

2024-10-07 Thread Mukesh Ojha
On Sun, Oct 06, 2024 at 10:38:01PM +0300, Dmitry Baryshkov wrote: > On Sat, Oct 05, 2024 at 02:53:54AM GMT, Mukesh Ojha wrote: > > From: Shiraz Hashim > > > > Qualcomm’s PAS implementation for remote processors only supports a > > single stage of IOMMU translation and

Re: [PATCH 6/6] remoteproc: qcom: Enable map/unmap and SHM bridge support

2024-10-07 Thread Mukesh Ojha
On Mon, Oct 07, 2024 at 10:05:08AM +0200, neil.armstr...@linaro.org wrote: > On 04/10/2024 23:23, Mukesh Ojha wrote: > > For Qualcomm SoCs runnning with Qualcomm EL2 hypervisor(QHEE), IOMMU > > translation for remote processors is managed by QHEE and if the same SoC > > run

Re: [PATCH 3/6] remoteproc: qcom: Add helper function to support IOMMU devmem translation

2024-10-07 Thread Mukesh Ojha
On Mon, Oct 07, 2024 at 10:08:16AM +0200, neil.armstr...@linaro.org wrote: > On 04/10/2024 23:23, Mukesh Ojha wrote: > > From: Shiraz Hashim > > > > Qualcomm SoCs runnning with Qualcomm EL2 hypervisor(QHEE), IOMMU > > translation set up for remote processors is managed

Re: [PATCH 0/6] Peripheral Image Loader support for Qualcomm SoCs

2024-10-04 Thread Mukesh Ojha
Subject got wrapped and later part got ignored, Quoting it again here for reference. "Peripheral Image Loader support for Qualcomm SoCs on KVM host system" -Mukesh

[PATCH 6/6] remoteproc: qcom: Enable map/unmap and SHM bridge support

2024-10-04 Thread Mukesh Ojha
tear down and apart from this, SHM bridge also need to set up to enable memory protection on both remoteproc meta data memory as well as for the carveout region. Enable the support required to run Qualcomm remoteprocs on non-QHEE hypervisors. Signed-off-by: Mukesh Ojha --- drivers/remoteproc

[PATCH 5/6] remoteproc: qcom: Add support of SHM bridge to enable memory protection

2024-10-04 Thread Mukesh Ojha
. Make qcom_tzmem_init_area() and qcom_tzmem_cleanup_area() exported symbol so that it could be used to create SHM bridge for remoteproc region. Signed-off-by: Mukesh Ojha --- drivers/firmware/qcom/qcom_scm.c | 29 +++- drivers/firmware/qcom/qcom_tzmem.c | 14

[PATCH 4/6] remoteproc: qcom: Add support to parse qcom,devmem property

2024-10-04 Thread Mukesh Ojha
qcom,devmem information. Signed-off-by: Shiraz Hashim Co-developed-by: Mukesh Ojha Signed-off-by: Mukesh Ojha --- drivers/remoteproc/qcom_q6v5_pas.c | 55 ++ 1 file changed, 55 insertions(+) diff --git a/drivers/remoteproc/qcom_q6v5_pas.c b/drivers/remoteproc

[PATCH 3/6] remoteproc: qcom: Add helper function to support IOMMU devmem translation

2024-10-04 Thread Mukesh Ojha
: Mukesh Ojha Signed-off-by: Mukesh Ojha --- drivers/remoteproc/qcom_common.c | 96 drivers/remoteproc/qcom_common.h | 35 2 files changed, 131 insertions(+) diff --git a/drivers/remoteproc/qcom_common.c b/drivers/remoteproc/qcom_common.c index

[PATCH 2/6] remoteproc: qcom: Add iommu map_unmap helper function

2024-10-04 Thread Mukesh Ojha
le in qcom_q6v5_adsp.c. So, in motivation to reuse code, introduce common exported functions like qcom_map_unmap_carveout() such that it can be used by both qcom_q6v5_adsp and qcom_q6v5_pas. Signed-off-by: Komal Bajaj Co-Developed-by: Mukesh Ojha Signed-off-by: Mukesh Ojha --- drivers/remot

[PATCH 1/6] dt-bindings: remoteproc: qcom,pas-common: Introduce iommus and qcom,devmem property

2024-10-04 Thread Mukesh Ojha
either QHEE or a KVM hypervisor (CPUs booted in EL2). The qcom,devmem property provides IOMMU devmem translation information intended for non-QHEE based systems. Signed-off-by: Shiraz Hashim Co-Developed-by: Mukesh Ojha Signed-off-by: Mukesh Ojha --- .../bindings/remoteproc/qcom,pas

[PATCH 0/6] Peripheral Image Loader support for Qualcomm SoCs

2024-10-04 Thread Mukesh Ojha
the iommus properties and mapping/unmapping carveout and device memory based on it. Komal Bajaj (1): remoteproc: qcom: Add iommu map_unmap helper function Mukesh Ojha (2): remoteproc: qcom: Add support of SHM bridge to enable memory protection remoteproc: qcom: Enable map/unmap and

Re: [PATCH] remoteproc: qcom: Fix NULL pointer in glink_subdev_stop()

2024-09-30 Thread Mukesh Ojha
On Fri, Sep 27, 2024 at 02:59:09PM -0700, Bjorn Andersson wrote: > On Sat, Sep 28, 2024 at 01:07:43AM +0530, Mukesh Ojha wrote: > > On Wed, Sep 25, 2024 at 08:41:55PM -0700, Bjorn Andersson wrote: > > > On Wed, Sep 25, 2024 at 04:03:51PM +0530, Mukesh Ojha wrote: > &

Re: [PATCH] remoteproc: qcom: Fix NULL pointer in glink_subdev_stop()

2024-09-27 Thread Mukesh Ojha
On Wed, Sep 25, 2024 at 08:41:55PM -0700, Bjorn Andersson wrote: > On Wed, Sep 25, 2024 at 04:03:51PM +0530, Mukesh Ojha wrote: > > Multiple call to glink_subdev_stop() for the same remoteproc can happen > > if rproc_stop() fails from Process-A that leaves the rproc state to &g

[PATCH] remoteproc: qcom: Fix NULL pointer in glink_subdev_stop()

2024-09-25 Thread Mukesh Ojha
ble to handle kernel NULL pointer dereference at virtual address 0358 Signed-off-by: Mukesh Ojha --- - We can do this NULL check in qcom_glink_smem_unregister() as it is exported function however, there is only one user of this. So, doing it with c

Re: [PATCH] rpmsg: char: fix rpmsg_eptdev structure documentation

2024-05-20 Thread Mukesh Ojha
eptdev' drivers/rpmsg/rpmsg_char.c:75: warning: Function parameter or struct member 'remote_flow_updated' not described in 'rpmsg_eptdev' Fixes: 5550201c0fe2 ("rpmsg: char: Add RPMSG GET/SET FLOWCONTROL IOCTL support") Signed-off-by: Arnaud Pouliquen Reviewed-by: Mukesh Ojha -Mukesh

Re: [PATCH] remoteproc: mediatek: Zero out only remaining bytes of IPI buffer

2024-05-20 Thread Mukesh Ojha
ize - len); Although, it does not make any difference apart from a write of len bytes, still a good improvement to do .. Acked-by: Mukesh Ojha -Mukesh handler(scp->share_buf, len, ipi_desc[id].priv); scp_ipi_unlock(scp, id);

Re: [PATCH v9 1/3] soc: qcom: Add qcom_rproc_minidump module

2024-05-03 Thread Mukesh Ojha
On 4/30/2024 7:08 PM, Bjorn Andersson wrote: On Tue, Mar 26, 2024 at 07:43:12PM +0530, Mukesh Ojha wrote: Add qcom_rproc_minidump module in a preparation to remove minidump specific code from driver/remoteproc/qcom_common.c and provide needed exported API, this as well helps to abstract

[PATCH v10] remoteproc: qcom: Move minidump related layout and API to soc/qcom directory

2024-05-03 Thread Mukesh Ojha
. Guards the qcom_minidump() with CONFIG_QCOM_RPROC_MINIDUMP. 3. Selects this QCOM_RPROC_MINIDUMP config when QCOM_RPROC_COMMON enabled. 4. Added new header qcom_minidump.h . Signed-off-by: Mukesh Ojha --- Changes in v10: - Converted all 3 changes sent in v9 to a single to properly reflect

Re: [PATCH v9 1/3] soc: qcom: Add qcom_rproc_minidump module

2024-04-29 Thread Mukesh Ojha
Gentle ping.. -Mukesh On 3/26/2024 7:43 PM, Mukesh Ojha wrote: Add qcom_rproc_minidump module in a preparation to remove minidump specific code from driver/remoteproc/qcom_common.c and provide needed exported API, this as well helps to abstract minidump specific data layout from qualcomm&#

[PATCH v9 3/3] remoteproc: qcom: Remove minidump related data from qcom_common.c

2024-03-26 Thread Mukesh Ojha
As minidump specific data structure and functions move under config QCOM_RPROC_MINIDUMP, so remove minidump specific data from driver/remoteproc/qcom_common.c . Signed-off-by: Mukesh Ojha --- Changes in v9: - Change in patch order. - rebased it. v8: https://lore.kernel.org/lkml

[PATCH v9 2/3] remoteproc: qcom_q6v5_pas: Use qcom_rproc_minidump()

2024-03-26 Thread Mukesh Ojha
s use qcom_rproc_minidump() and we will be removing qcom_minidump() and minidump related stuff from driver/remoteproc/qcom_common.c . Signed-off-by: Mukesh Ojha --- Changes in v9: - Change in patch order from its last version. - Rebased it. v8: https://lore.kernel.org/lkml/20240131105734.13090-1-qu

[PATCH v9 1/3] soc: qcom: Add qcom_rproc_minidump module

2024-03-26 Thread Mukesh Ojha
idump() functionality from driver/remoteproc/qcom_common.c into a separate file under qcom_rproc_minidump(). Signed-off-by: Mukesh Ojha --- Changes in v9: - Added source file driver/remoteproc/qcom_common.c copyright to qcom_rproc_minidump.c - Dissociated it from minidump series as this can go separ

Re: [PATCH v7 4/4] arm64: dts: qcom: sm8650: add missing qlink_logging reserved memory for mpss

2024-01-23 Thread Mukesh Ojha
On 1/23/2024 2:21 PM, Neil Armstrong wrote: The qlink_logging memory region is also used by the modem firmware, add it to the reserved memories and add it to the MPSS memory regions. Signed-off-by: Neil Armstrong LGTM, Reviewed-by: Mukesh Ojha -Mukesh --- arch/arm64/boot/dts/qcom

Re: [PATCH v7 3/4] remoteproc: qcom: pas: Add SM8650 remoteproc support

2024-01-23 Thread Mukesh Ojha
= &sm8550_cdsp_resource}, { .compatible = "qcom,sm8550-mpss-pas", .data = &sm8550_mpss_resource}, + { .compatible = "qcom,sm8650-adsp-pas", .data = &sm8550_adsp_resource}, Same as sm8550; + { .compatible = "qcom,sm8650-cdsp-pas", .data = &sm8650_cdsp_resource}, + { .compatible = "qcom,sm8650-mpss-pas", .data = &sm8650_mpss_resource}, LGTM, Acked-by: Mukesh Ojha -Mukesh { }, }; MODULE_DEVICE_TABLE(of, adsp_of_match);

Re: [PATCH V3] remoteproc: qcom: q6v5: Get crash reason from specific SMEM partition

2023-12-18 Thread Mukesh Ojha
On 12/18/2023 11:40 AM, Vignesh Viswanathan wrote: q6v5 fatal and watchdog IRQ handlers always retrieves the crash reason information from SMEM global partition (QCOM_SMEM_HOST_ANY). For some targets like IPQ9574 and IPQ5332, crash reason information is present in target specific partition du

Re: [PATCH v3 2/3] remoteproc: qcom: pas: make region assign more generic

2023-11-06 Thread Mukesh Ojha
ms[offset], + &perm, 1); + if (ret < 0) + dev_err(adsp->dev, "unassign memory failed\n"); In case you are going to send another version, you can print offset similar to the assign call failure., Othe

Re: [PATCH v2 2/3] remoteproc: qcom: pas: make region assign more generic

2023-11-02 Thread Mukesh Ojha
On 11/2/2023 3:56 PM, neil.armstr...@linaro.org wrote: On 01/11/2023 15:42, Mukesh Ojha wrote: On 10/31/2023 10:36 PM, Neil Armstrong wrote: Hi, On 30/10/2023 14:10, Mukesh Ojha wrote: On 10/30/2023 3:33 PM, Neil Armstrong wrote: The current memory region assign only supports a

Re: [PATCH] eventfs: Fix kerneldoc of eventfs_remove_rec()

2023-11-01 Thread Mukesh Ojha
On 11/2/2023 1:30 AM, Steven Rostedt wrote: On Mon, 30 Oct 2023 21:57:13 +0530 Mukesh Ojha wrote: On 10/30/2023 9:45 PM, Steven Rostedt wrote: From: "Steven Rostedt (Google)" The eventfs_remove_rec() had some missing parameters in the kerneldoc comment above it. Also, re

Re: [PATCH v2 2/3] remoteproc: qcom: pas: make region assign more generic

2023-11-01 Thread Mukesh Ojha
On 10/31/2023 10:36 PM, Neil Armstrong wrote: Hi, On 30/10/2023 14:10, Mukesh Ojha wrote: On 10/30/2023 3:33 PM, Neil Armstrong wrote: The current memory region assign only supports a single memory region. But new platforms introduces more regions to make the memory requirements more

Re: [PATCH] tracing/kprobes: Fix the order of argument descriptions

2023-10-31 Thread Mukesh Ojha
const char *name, const char *loc, ...) Fixes: 2a588dd1d5d6 ("tracing: Add kprobe event command generation functions") Suggested-by: Mukesh Ojha Signed-off-by: Yujie Liu Thanks. Reviewed-by: Mukesh Ojha -Mukesh --- kernel/trace/trace_kprobe.c | 2 +- 1 file changed, 1 insertion(+),

Re: [PATCH 7/9] arm64: dts: qcom: sc7280: Add CDSP node

2023-10-30 Thread Mukesh Ojha
On 10/30/2023 8:33 PM, Doug Anderson wrote: Hi, On Mon, Oct 30, 2023 at 7:43 AM Luca Weiss wrote: On Mon Oct 30, 2023 at 3:11 PM CET, Doug Anderson wrote: Hi, On Mon, Oct 30, 2023 at 2:12 AM Luca Weiss wrote: On Mon Oct 30, 2023 at 10:04 AM CET, Mukesh Ojha wrote: On 10/27/2023 7

Re: [PATCH] eventfs: Fix kerneldoc of eventfs_remove_rec()

2023-10-30 Thread Mukesh Ojha
eventfs: Remove eventfs_file and just use eventfs_inode"); Reported-by: kernel test robot Closes: https://lore.kernel.org/oe-kbuild-all/202310052216.4sgqaswo-...@intel.com/ Signed-off-by: Steven Rostedt (Google) Reviewed-by: Mukesh Ojha -Mukesh --- fs/tracefs/event_inode.c | 6 --

Re: [PATCH v2 2/3] remoteproc: qcom: pas: make region assign more generic

2023-10-30 Thread Mukesh Ojha
On 10/30/2023 3:33 PM, Neil Armstrong wrote: The current memory region assign only supports a single memory region. But new platforms introduces more regions to make the memory requirements more flexible for various use cases. Those new platforms also shares the memory region between the DSP

Re: [PATCH 7/9] arm64: dts: qcom: sc7280: Add CDSP node

2023-10-30 Thread Mukesh Ojha
On 10/27/2023 7:50 PM, Luca Weiss wrote: Add the node for the ADSP found on the SC7280 SoC, using standard Qualcomm firmware. The memory region for sc7280-chrome-common.dtsi is taken from msm-5.4 yupik.dtsi since the other areas also seem to match that file there, though I cannot be sure ther

Re: [PATCH] tracing/kprobes: Fix the description of variable length arguments

2023-10-27 Thread Mukesh Ojha
ctions") Reported-by: kernel test robot Closes: https://lore.kernel.org/oe-kbuild-all/202310190437.pai6lyjf-...@intel.com/ Signed-off-by: Yujie Liu Not related to this patch, but I see order of the argument as well is not proper in the document of the __kprobe_event_gen_cmd_start(), if you c

Re: [PATCH] eventfs: Fix typo in eventfs_inode union comment

2023-10-25 Thread Mukesh Ojha
ostedt (Google) Reviewed-by: Mukesh Ojha -Mukesh --- fs/tracefs/internal.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/tracefs/internal.h b/fs/tracefs/internal.h index 298d3ecaf621..64fde9490f52 100644 --- a/fs/tracefs/internal.h +++ b/fs/tracefs/internal.h @@ -37,7

Re: [PATCH] tracing/histograms: Simplify last_cmd_set()

2023-10-23 Thread Mukesh Ojha
n., Thanks. Reviewed-by: Mukesh ojha -Mukesh - if (file) { call = file->event_call; system = call->class->system;

Re: [PATCH] tracefs/eventfs: Modify mismatched function name

2023-10-23 Thread Mukesh Ojha
=6939 Signed-off-by: Jiapeng Chong Reviewed-by: Mukesh Ojha -Mukesh --- fs/tracefs/event_inode.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/tracefs/event_inode.c b/fs/tracefs/event_inode.c index 1ccd100bc565..ba9d1cb0d24c 100644 --- a/fs/tracefs/event_inode.c

Re: [PATCH] eventfs: Fix failure path in eventfs_create_events_dir()

2023-10-23 Thread Mukesh Ojha
c-95f-1117-706c2c22...@inria.fr/ Fixes: 5790b1fb3d67 ("eventfs: Remove eventfs_file and just use eventfs_inode") Reported-by: Julia Lawall Signed-off-by: Steven Rostedt (Google) Reviewed-by: Mukesh Ojha -Mukesh --- fs/tracefs/event_inode.c | 3 ++- 1 file changed, 2 inser

Re: [PATCH v3] pstore: Add mem_type property DT parsing support

2021-03-30 Thread Mukesh Ojha
Hi Kees/All, Could you please review again?  I have addressed your comment made on last patch version. Thanks, Mukesh On 3/23/2021 12:12 AM, Mukesh Ojha wrote: There could be a scenario where we define some region in normal memory and use them store to logs which is later retrieved by

[PATCH v3] pstore: Add mem_type property DT parsing support

2021-03-22 Thread Mukesh Ojha
cacheable could improve performance. This commit gives control to change mem_type from Device tree, and also documents the value for normal memory. Signed-off-by: Mukesh Ojha --- Changes in v3: - Minor code and documentation update done as per comment given by Kees. Changes in v2: - if-else

Re: [RESEND PATCH v2 1/2] pstore: Add mem_type property DT parsing support

2021-03-17 Thread Mukesh Ojha
Hi All, can you please review this ? Thanks, Mukesh On 3/2/2021 1:59 PM, Mukesh Ojha wrote: Hi Kees, i have updated the patch based on your last comments. please review. Thanks, Mukesh On 2/25/2021 9:30 PM, Mukesh Ojha wrote: There could be a sceanario where we define some region in

Re: [RESEND PATCH v2 1/2] pstore: Add mem_type property DT parsing support

2021-03-02 Thread Mukesh Ojha
Hi Kees, i have updated the patch based on your last comments. please review. Thanks, Mukesh On 2/25/2021 9:30 PM, Mukesh Ojha wrote: There could be a sceanario where we define some region in normal memory and use them store to logs which is later retrieved by bootloader during warm reset

[RESEND PATCH v2 2/2] pstore: Add buffer start check during init

2021-02-25 Thread Mukesh Ojha
So add the start check to avoid. Signed-off-by: Huang Yiwei Signed-off-by: Mukesh Ojha --- change in v2: - this is on top of first patchset. fs/pstore/ram_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/pstore/ram_core.c b/fs/pstore/ram_core.c index 0da012f..a157

[RESEND PATCH v2 1/2] pstore: Add mem_type property DT parsing support

2021-02-25 Thread Mukesh Ojha
cacheable could improve performance. This commit gives control to change mem_type from Device tree, and also documents the value for normal memory. Signed-off-by: Mukesh Ojha --- Changes in v2: - if-else converted to switch case - updated MODULE_PARM_DESC with new memory type. - default setting

[PATCH v2 1/2] pstore: Add mem_type property DT parsing support

2021-02-23 Thread Mukesh Ojha
From: Mukesh Ojha There could be a sceanario where we define some region in normal memory and use them store to logs which is later retrieved by bootloader during warm reset. In this scenario, we wanted to treat this memory as normal cacheable memory instead of default behaviour which is an

[PATCH 2/2] pstore: Add buffer start check during init

2021-02-23 Thread Mukesh Ojha
So add the start check to avoid. Signed-off-by: Huang Yiwei Signed-off-by: Mukesh Ojha --- change in v2: - this is on top of first patchset. fs/pstore/ram_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/pstore/ram_core.c b/fs/pstore/ram_core.c index 0da012f..a157

Re: [PATCH] pstore/ram : Add support for cached pages

2021-02-11 Thread Mukesh Ojha
On 2/11/2021 1:47 AM, Kees Cook wrote: On Wed, Feb 10, 2021 at 08:22:21PM +0530, Mukesh Ojha wrote: There could be a sceanario where we define some region in normal memory and use them store to logs which is later retrieved by bootloader during warm reset. In this scenario, we wanted to

Re: Pstore : Query on using ramoops driver for DDR

2021-02-10 Thread Mukesh Ojha
On 2/10/2021 2:32 AM, Luck, Tony wrote: Can we use existing backend pstore ram driver (fs/pstore/ram.c) for DDR instead of SRAM ? The expectation for pstore is that the system will go through a reset when it crashes. Most systems do not preserve DDR contents across reset. to support DDR, If w

[PATCH] pstore/ram : Add support for cached pages

2021-02-10 Thread Mukesh Ojha
cacheable could improve performance. This commit gives control to change mem_type from Device tree, and also documents the value for normal memory. Signed-off-by: Huang Yiwei Signed-off-by: Mukesh Ojha --- Documentation/admin-guide/ramoops.rst | 4 +++- fs/pstore/ram.c | 1

Pstore : Query on using ramoops driver for DDR

2021-02-09 Thread Mukesh Ojha
Hi All, Can we use existing backend pstore ram driver (fs/pstore/ram.c) for DDR instead of SRAM ? Was the current driver written only to support persistant RAM like SRAM or it can accept further change to support DDR, If we have a mechanism to copy stored data from DDR to external device after

[PATCH] thermal: Fix NULL pointer dereference issue

2020-11-16 Thread Mukesh Ojha
Cooling stats variable inside thermal_cooling_device_stats_update() can get NULL. We should add a NULL check on stat inside for sanity. Signed-off-by: Mukesh Ojha --- drivers/thermal/thermal_sysfs.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/thermal/thermal_sysfs.c b/drivers

[PATCH 3/3] cpuidle: Trivial fixes

2019-10-15 Thread Mukesh Ojha
iterrupts => interrupts stratight => straight Minor comment correction. Signed-off-by: Mukesh Ojha --- kernel/sched/idle.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel/sched/idle.c b/kernel/sched/idle.c index 8dad5aa..2df8ae1 100644 --- a/kernel/sched/

[PATCH 2/3] time: Fix spelling mistake in comment

2019-10-15 Thread Mukesh Ojha
witin => within Signed-off-by: Mukesh Ojha --- kernel/time/time.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/time/time.c b/kernel/time/time.c index 5c54ca6..d31661c4 100644 --- a/kernel/time/time.c +++ b/kernel/time/time.c @@ -179,7 +179,7 @@

[PATCH 1/3] time/jiffies: Fixes some typo

2019-10-15 Thread Mukesh Ojha
accuratly => accurately while at it change `clock source` to clocksource to make it align with its usage at other places. Signed-off-by: Mukesh Ojha --- kernel/time/jiffies.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel/time/jiffies.c b/kernel/time/jiffie

[PATCH] seqlock: Minor comment correction

2019-10-04 Thread Mukesh Ojha
write_seqcountbeqin => write_seqcount_begin Signed-off-by: Mukesh Ojha --- include/linux/seqlock.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/seqlock.h b/include/linux/seqlock.h index bcf4cf2..370ef8f 100644 --- a/include/linux/seqlock.h +++ b/incl

Re: [PATCH V5 1/1] perf: event preserve and create across cpu hotplug

2019-09-24 Thread Mukesh Ojha
On 8/12/2019 4:12 PM, Jiri Olsa wrote: On Fri, Aug 02, 2019 at 12:16:53AM +0530, Mukesh Ojha wrote: Perf framework doesn't allow preserving CPU events across CPU hotplugs. The events are scheduled out as and when the CPU walks offline. Moreover, the framework also doesn't allow the

Re: [PATCH] perf test: fix spelling mistake "allos" -> "allocate"

2019-09-17 Thread Mukesh Ojha
On 9/11/2019 8:51 PM, Colin King wrote: From: Colin Ian King There is a spelling mistake in a TEST_ASSERT_VAL message. Fix it. Signed-off-by: Colin Ian King Reviewed-by: Mukesh Ojha Thanks, Mukesh --- tools/perf/tests/event_update.c | 2 +- 1 file changed, 1 insertion(+), 1

Re: [PATCH V5 1/1] perf: event preserve and create across cpu hotplug

2019-09-05 Thread Mukesh Ojha
On 8/12/2019 4:12 PM, Jiri Olsa wrote: On Fri, Aug 02, 2019 at 12:16:53AM +0530, Mukesh Ojha wrote: Perf framework doesn't allow preserving CPU events across CPU hotplugs. The events are scheduled out as and when the CPU walks offline. Moreover, the framework also doesn't allow the

Re: [PATCH V1]Perf: Return error code for perf_session__new function on failure

2019-08-20 Thread Mukesh Ojha
: Mukesh Ojha Thanks, Mukesh --- tools/perf/builtin-annotate.c |5 +++-- tools/perf/builtin-buildid-cache.c |5 +++-- tools/perf/builtin-buildid-list.c |5 +++-- tools/perf/builtin-c2c.c |6 -- tools/perf/builtin-diff.c |9 + tools

[tip:locking/core] locking/mutex: Use mutex flags macro instead of hard code

2019-08-06 Thread tip-bot for Mukesh Ojha
Commit-ID: a037d269221c0ae15f47046757afcbd1a7177bbf Gitweb: https://git.kernel.org/tip/a037d269221c0ae15f47046757afcbd1a7177bbf Author: Mukesh Ojha AuthorDate: Wed, 31 Jul 2019 20:35:04 +0530 Committer: Peter Zijlstra CommitDate: Tue, 6 Aug 2019 12:49:16 +0200 locking/mutex: Use mutex

[tip:locking/core] locking/mutex: Make __mutex_owner static to mutex.c

2019-08-06 Thread tip-bot for Mukesh Ojha
Commit-ID: 5f35d5a66b3ec62cb5ec4ec2ad9aebe2ac325673 Gitweb: https://git.kernel.org/tip/5f35d5a66b3ec62cb5ec4ec2ad9aebe2ac325673 Author: Mukesh Ojha AuthorDate: Wed, 31 Jul 2019 20:35:03 +0530 Committer: Peter Zijlstra CommitDate: Tue, 6 Aug 2019 12:49:16 +0200 locking/mutex: Make

Re: [PATCH] perf tools: Fix a typo in Makefile

2019-08-01 Thread Mukesh Ojha
On 8/1/2019 8:58 AM, Masanari Iida wrote: This patch fix a spelling typo in Makefile. Signed-off-by: Masanari Iida Reviewed-by: Mukesh Ojha -Mukesh --- tools/perf/Documentation/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/perf/Documentation

[PATCH V5 0/1] perf: Add CPU hotplug support for events

2019-08-01 Thread Mukesh Ojha
ander Shishkin Cc: Jiri Olsa Cc: Alexei Starovoitov Mukesh Ojha (1): perf: event preserve and create across cpu hotplug include/linux/perf_event.h | 1 + kernel/events/core.c | 122 + 2 files changed, 79 insertions(+), 44 deletions(-)

[PATCH V5 1/1] perf: event preserve and create across cpu hotplug

2019-08-01 Thread Mukesh Ojha
o Ananta Signed-off-by: Mukesh Ojha Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Arnaldo Carvalho de Melo Cc: Alexander Shishkin Cc: Jiri Olsa Cc: Alexei Starovoitov --- Change in V5: = - Rebased it. Change in V4: = - Released, __get_cpu_context would not be correct

[PATCH 2/2 v3] locking/mutex: Use mutex flags macro instead of hard code

2019-07-31 Thread Mukesh Ojha
Use the mutex flag macro instead of hard code value inside __mutex_owner(). Signed-off-by: Mukesh Ojha --- Changes in v3: - no change. Changes in v2: - Framed the commit according the changes done in 1/2 of the patchset. kernel/locking/mutex.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH 1/2 v3] locking/mutex: Make __mutex_owner static to mutex.c

2019-07-31 Thread Mukesh Ojha
gacy thing intact move them as well and export them. Move mutex_waiter structure also to keep it private to the file. Signed-off-by: Mukesh Ojha --- Changes in v3: - Moved mutex_waiter and removed inline from mutex_trylock_recursive() mutex_is_owner(). Changes in v2: - On Peterz suggest

Re: [PATCH 2/2] locking/mutex: Use mutex flags macro instead of hard code value

2019-07-30 Thread Mukesh Ojha
On 7/30/2019 9:32 PM, Peter Zijlstra wrote: On Tue, Jul 30, 2019 at 06:10:49PM +0530, Mukesh Ojha wrote: To make it static , i have to export mutex_is_locked() after moving it inside mutex.c, so that other module can use it. Yep, see below -- completely untested. Also are we thinking of

[PATCH 2/2 v2] locking/mutex: Use mutex flags macro instead of hard code value

2019-07-30 Thread Mukesh Ojha
Use the mutex flag macro instead of hard code value inside __mutex_owner(). Signed-off-by: Mukesh Ojha --- Changes in v2: - Framed the commit according the changes done in 1/2 of the patchset. kernel/locking/mutex.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel

[PATCH 1/2 v2] locking/mutex: Make __mutex_owner static to mutex.c

2019-07-30 Thread Mukesh Ojha
ing intact move them as well and export them. Signed-off-by: Mukesh Ojha --- Changes in v2: - On Peterz suggestion, moved __mutex_owner() to mutex.c to make it static to mutex.c. - Exported mutex_is_owner() and mutex_trylock_recursive() to keep the existing thing intact as there are loadable

Re: [PATCH 2/2] locking/mutex: Use mutex flags macro instead of hard code value

2019-07-30 Thread Mukesh Ojha
On 7/30/2019 1:33 PM, Peter Zijlstra wrote: On Tue, Jul 30, 2019 at 01:23:13PM +0530, Mukesh Ojha wrote: On 7/29/2019 4:37 PM, Peter Zijlstra wrote: On Mon, Jul 29, 2019 at 04:22:58PM +0530, Mukesh Ojha wrote: Let's use the mutex flag macro(which got moved from mutex.c to linux/mutex

Re: [PATCH 2/2] locking/mutex: Use mutex flags macro instead of hard code value

2019-07-30 Thread Mukesh Ojha
On 7/29/2019 4:37 PM, Peter Zijlstra wrote: On Mon, Jul 29, 2019 at 04:22:58PM +0530, Mukesh Ojha wrote: Let's use the mutex flag macro(which got moved from mutex.c to linux/mutex.h in the last patch) instead of hard code value which was used in __mutex_owner(). Signed-off-by: Mukesh

[PATCH 2/2] locking/mutex: Use mutex flags macro instead of hard code value

2019-07-29 Thread Mukesh Ojha
Let's use the mutex flag macro(which got moved from mutex.c to linux/mutex.h in the last patch) instead of hard code value which was used in __mutex_owner(). Signed-off-by: Mukesh Ojha --- include/linux/mutex.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/

[PATCH 1/2] locking/mutex: Move mutex flag macros to linux/mutex.h

2019-07-29 Thread Mukesh Ojha
There exist a place where we use hard code value for mutex flag(e.g in mutex.h __mutex_owner()). Let's move the mutex flag macros to header linux/mutex.h, so that it could be reused at other places as well. Signed-off-by: Mukesh Ojha --- include/linux/mutex.h | 15 +++ k

[PATCH] rcu: Fix spelling mistake "greate"->"great"

2019-07-29 Thread Mukesh Ojha
There is a spelling mistake in file tree_exp.h, fix this. Signed-off-by: Mukesh Ojha --- kernel/rcu/tree_exp.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/rcu/tree_exp.h b/kernel/rcu/tree_exp.h index af7e7b9..609fc87 100644 --- a/kernel/rcu/tree_exp.h +++ b/kernel

Re: [PATCH RESEND V4 0/1] perf: Add CPU hotplug support for events

2019-07-26 Thread Mukesh Ojha
Hi All, Can you please review this ? Thanks, Mukesh On 6/24/2019 2:31 PM, Mukesh Ojha wrote: Friendly ping. On 6/18/2019 7:16 PM, Mukesh Ojha wrote: The embedded world, specifically Android mobile SoCs, rely on CPU hotplugs to manage power and thermal constraints. These hotplugs can happen

Re: [PATCH v5] driver core: Fix use-after-free and double free on glue directory

2019-07-24 Thread Mukesh Ojha
On 7/24/2019 7:11 PM, Mukesh Ojha wrote: On 7/18/2019 4:49 PM, Muchun Song wrote: There is a race condition between removing glue directory and adding a new device under the glue directory. It can be reproduced in following test: path 1: Add the child device under glue dir device_add

Re: [PATCH v5] driver core: Fix use-after-free and double free on glue directory

2019-07-24 Thread Mukesh Ojha
ttps://lkml.org/lkml/2019/5/1/3 +ref = kref_read(&glue_dir->kref); +if  (!kobject_has_children(glue_dir) && --ref) We have seen many pattern of this issue which is coming from different driver e.g uinput as well in 4.14 kernel.(https://lkml.org/lkml/2019/4/10/146)

Re: [PATCH RESEND V4 1/1] perf: event preserve and create across cpu hotplug

2019-07-02 Thread Mukesh Ojha
Friendly Ping. More explanation of the usecase added in the coverletter [PATCH RESEND V4 0/1] perf: Add CPU hotplug support for events. -Mukesh On 6/18/2019 7:16 PM, Mukesh Ojha wrote: Perf framework doesn't allow preserving CPU events across CPU hotplugs. The events are scheduled out a

Re: Perf framework : Cluster based counter support

2019-07-01 Thread Mukesh Ojha
On 6/28/2019 10:29 PM, Mark Rutland wrote: On Fri, Jun 28, 2019 at 10:23:10PM +0530, Mukesh Ojha wrote: Hi All, Hi Mukesh, Is it looks considerable to add cluster based event support to add in current perf event framework and later in userspace perf to support such events ? Could you

Perf framework : Cluster based counter support

2019-06-28 Thread Mukesh Ojha
Hi All, Is it looks considerable to add cluster based event support to add in current perf event framework and later in userspace perf to support such events ? Thanks, Mukesh

Re: [PATCH RESEND V4 0/1] perf: Add CPU hotplug support for events

2019-06-24 Thread Mukesh Ojha
Friendly ping. On 6/18/2019 7:16 PM, Mukesh Ojha wrote: The embedded world, specifically Android mobile SoCs, rely on CPU hotplugs to manage power and thermal constraints. These hotplugs can happen at a very rapid pace. Adjacently, they also relies on many perf event counters for its management

[PATCH RESEND V4 1/1] perf: event preserve and create across cpu hotplug

2019-06-18 Thread Mukesh Ojha
o Ananta Signed-off-by: Mukesh Ojha Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Arnaldo Carvalho de Melo Cc: Alexander Shishkin Cc: Jiri Olsa Cc: Alexei Starovoitov --- Change in V4: = - Released, __get_cpu_context would not be correct way to get the cpu context of the cpu which is

[PATCH RESEND V4 0/1] perf: Add CPU hotplug support for events

2019-06-18 Thread Mukesh Ojha
243819 cycles 9.6300522877102438 cycles 9.839848225 337454 cycles 10.049645048 644072 cycles 10.2594762461855410 cycles Mukesh Ojha (1): perf: event preserve and create across cpu hotpl

Re: [PATCH V4] perf: event preserve and create across cpu hotplug

2019-06-18 Thread Mukesh Ojha
On 6/18/2019 5:53 PM, Peter Zijlstra wrote: On Tue, Jun 18, 2019 at 02:24:51PM +0530, Mukesh Ojha wrote: Perf framework doesn't allow preserving CPU events across CPU hotplugs. The events are scheduled out as and when the CPU walks offline. Moreover, the framework also doesn't

[PATCH V4] perf: event preserve and create across cpu hotplug

2019-06-18 Thread Mukesh Ojha
o Ananta Signed-off-by: Mukesh Ojha Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Arnaldo Carvalho de Melo Cc: Alexander Shishkin Cc: Jiri Olsa Cc: Alexei Starovoitov --- Change in V4: = - Released, __get_cpu_context would not be correct way to get the cpu context of the cpu which is

[PATCH V3] perf: event preserve and create across cpu hotplug

2019-06-12 Thread Mukesh Ojha
o Ananta Signed-off-by: Mukesh Ojha Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Arnaldo Carvalho de Melo Cc: Alexander Shishkin Cc: Jiri Olsa Cc: Alexei Starovoitov --- Change in V3: - Jiri has tried perf stat -a and removed one of the cpu from the other terminal. This resulted in a crash.

Re: [PATCH V2 1/1] perf: event preserve and create across cpu hotplug

2019-06-04 Thread Mukesh Ojha
On 6/3/2019 11:23 PM, Jiri Olsa wrote: On Fri, May 31, 2019 at 07:09:16PM +0530, Mukesh Ojha wrote: Perf framework doesn't allow preserving CPU events across CPU hotplugs. The events are scheduled out as and when the CPU walks offline. Moreover, the framework also doesn't allow the

  1   2   3   4   >