mbox series

[0/1] lib, stackdepot: Add helper to print stack entries into buffer.

Message ID 20210910141001.1622130-1-imran.f.khan@oracle.com (mailing list archive)
Headers show
Series lib, stackdepot: Add helper to print stack entries into buffer. | expand

Message

Imran Khan Sept. 10, 2021, 2:10 p.m. UTC
This change is in response to discussion at [1].
The patch has been created on top of my earlier changes [2] and [3].
If needed I can resend all of these patches together, though my
earlier patches have been Acked.

[1] https://lore.kernel.org/lkml/e6f6fb85-1d83-425b-9e36-b5784cc9e69a@suse.cz/
[2] https://lore.kernel.org/lkml/fe94ffd8-d235-87d8-9c3d-80f7f73e0c4e@suse.cz/
[3] https://lore.kernel.org/lkml/85f4f073-0b5a-9052-0ba9-74d450608656@suse.cz/

Imran Khan (1):
  lib, stackdepot: Add helper to print stack entries into buffer.

 drivers/gpu/drm/drm_dp_mst_topology.c   |  5 +----
 drivers/gpu/drm/drm_mm.c                |  5 +----
 drivers/gpu/drm/i915/i915_vma.c         |  5 +----
 drivers/gpu/drm/i915/intel_runtime_pm.c | 20 +++++---------------
 include/linux/stackdepot.h              |  3 +++
 lib/stackdepot.c                        | 23 +++++++++++++++++++++++
 mm/page_owner.c                         |  5 +----
 7 files changed, 35 insertions(+), 31 deletions(-)

Comments

Vlastimil Babka Sept. 14, 2021, 9:03 a.m. UTC | #1
On 9/10/21 16:10, Imran Khan wrote:
> This change is in response to discussion at [1].
> The patch has been created on top of my earlier changes [2] and [3].
> If needed I can resend all of these patches together, though my
> earlier patches have been Acked.

I think you sent those at the beginning of merge window, so it would be best
to gather everything in a self-contained series now and resend. I suggested
another change for one of those anyway.

You can of course resend including the Acks you already got, as you did
already with "[PATCH v2 1/1] lib, stackdepot: Add helper to print stack
entries into buffer."

> [1] https://lore.kernel.org/lkml/e6f6fb85-1d83-425b-9e36-b5784cc9e69a@suse.cz/
> [2] https://lore.kernel.org/lkml/fe94ffd8-d235-87d8-9c3d-80f7f73e0c4e@suse.cz/
> [3] https://lore.kernel.org/lkml/85f4f073-0b5a-9052-0ba9-74d450608656@suse.cz/
> 
> Imran Khan (1):
>   lib, stackdepot: Add helper to print stack entries into buffer.
> 
>  drivers/gpu/drm/drm_dp_mst_topology.c   |  5 +----
>  drivers/gpu/drm/drm_mm.c                |  5 +----
>  drivers/gpu/drm/i915/i915_vma.c         |  5 +----
>  drivers/gpu/drm/i915/intel_runtime_pm.c | 20 +++++---------------
>  include/linux/stackdepot.h              |  3 +++
>  lib/stackdepot.c                        | 23 +++++++++++++++++++++++
>  mm/page_owner.c                         |  5 +----
>  7 files changed, 35 insertions(+), 31 deletions(-)
>
Imran Khan Sept. 15, 2021, 3:33 a.m. UTC | #2
Hi Vlastimil,

On 14/9/21 7:03 pm, Vlastimil Babka wrote:
> On 9/10/21 16:10, Imran Khan wrote:
>> This change is in response to discussion at [1].
>> The patch has been created on top of my earlier changes [2] and [3].
>> If needed I can resend all of these patches together, though my
>> earlier patches have been Acked.
> 
> I think you sent those at the beginning of merge window, so it would be best
> to gather everything in a self-contained series now and resend. I suggested
> another change for one of those anyway.
> 
> You can of course resend including the Acks you already got, as you did
> already with "[PATCH v2 1/1] lib, stackdepot: Add helper to print stack
> entries into buffer."
> 

Thanks for clarifying this. I have resend consolidated patch as per your 
suggestion. This self-contained change set can be seen at [1].

[1] 
https://lore.kernel.org/lkml/20210915014806.3206938-1-imran.f.khan@oracle.com/

>> [1] https://urldefense.com/v3/__https://lore.kernel.org/lkml/e6f6fb85-1d83-425b-9e36-b5784cc9e69a@suse.cz/__;!!ACWV5N9M2RV99hQ!aF3HSKJ9eit2OvdDk5j_DTKT8rL2SkG6cYL1lLVvegHz5_YTJhN7dSkHgBKeJjupkw$
>> [2] https://urldefense.com/v3/__https://lore.kernel.org/lkml/fe94ffd8-d235-87d8-9c3d-80f7f73e0c4e@suse.cz/__;!!ACWV5N9M2RV99hQ!aF3HSKJ9eit2OvdDk5j_DTKT8rL2SkG6cYL1lLVvegHz5_YTJhN7dSkHgBITooNU1w$
>> [3] https://urldefense.com/v3/__https://lore.kernel.org/lkml/85f4f073-0b5a-9052-0ba9-74d450608656@suse.cz/__;!!ACWV5N9M2RV99hQ!aF3HSKJ9eit2OvdDk5j_DTKT8rL2SkG6cYL1lLVvegHz5_YTJhN7dSkHgBJcGhmlHA$
>>
>> Imran Khan (1):
>>    lib, stackdepot: Add helper to print stack entries into buffer.
>>
>>   drivers/gpu/drm/drm_dp_mst_topology.c   |  5 +----
>>   drivers/gpu/drm/drm_mm.c                |  5 +----
>>   drivers/gpu/drm/i915/i915_vma.c         |  5 +----
>>   drivers/gpu/drm/i915/intel_runtime_pm.c | 20 +++++---------------
>>   include/linux/stackdepot.h              |  3 +++
>>   lib/stackdepot.c                        | 23 +++++++++++++++++++++++
>>   mm/page_owner.c                         |  5 +----
>>   7 files changed, 35 insertions(+), 31 deletions(-)
>>
>