Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Imran Khan <imran.f.khan@oracle.com>
To:     vbabka@suse.cz, geert@linux-m68k.org, akpm@linux-foundation.org,
        ryabinin.a.a@gmail.com, glider@google.com, andreyknvl@gmail.com,
        dvyukov@google.com, maarten.lankhorst@linux.intel.com,
        mripard@kernel.org, tzimmermann@suse.de, airlied@linux.ie,
        daniel@ffwll.ch
Cc:     dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org,
        intel-gfx@lists.freedesktop.org, linux-mm@kvack.org
Subject: [PATCH RESEND v2 0/3] lib, stackdepot: check stackdepot handle before accessing slabs
Date:   Thu, 16 Sep 2021 03:53:18 +1000
Message-Id: <20210915175321.3472770-1-imran.f.khan@oracle.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?YpP0LyepXL/QqFfhuEDwNItqUOGPxs/9JBZdQcg6Ss0o8HdfxefqTxFwNXTy?=
 =?us-ascii?Q?LSCtOpBWgg6/DxXtVaUIegJu3CfG1YwkJcyo+YIvD+nxC8FLl5Ibuo6VRIGm?=
 =?us-ascii?Q?Q/Csfx0+OSRmvlGz9EJkMMUhAgdV9Ix0CeszsVI80Cpq2XvddxGNpfGFvqGP?=
 =?us-ascii?Q?P/Q92kqIU2fbC4Ro0Yp273op+QCEvICR9kuxH5FeuFE1O7XoQ0qtcK/VDjgI?=
 =?us-ascii?Q?pCZqIyQ/1P7rw7h331tcgnu54uDpFG/jg8vmd1orHQvElyxr2YmIWW1RzW39?=
 =?us-ascii?Q?ndwdZrhygrPTHCcgAaIueqbEU9oJz0q0IvuqzmCCguyzZuL8r2qhr0wtSOCj?=
 =?us-ascii?Q?YLTydZcg9fdb3tBDovvPuvZ8N06MoUpHyWx49x3jf+t4DZqFMiWwfbOTwYIj?=
 =?us-ascii?Q?fufzTdUwPKPK+mF/5N1xGqc6e9XzrrQkGXODzysraBT+VlVUXQK5Cj4otQlR?=
 =?us-ascii?Q?I0dmEti5uDL+Joq1fuASmQbG0j5OMjOnblvR3qNQbO+WZi2LPUM1Iuu/ddFQ?=
 =?us-ascii?Q?prWt3t+4M+N/Q9Bon0Gy1msmq4lZP8ZOqrNgIgcounrLJ+iQyP3NJG1PrDpA?=
 =?us-ascii?Q?af/5YZusTOt3amTm9j4XTRiYgpT4fa3sYiYUoA5SLLMKMl+YhqXnRaWOXP5p?=
 =?us-ascii?Q?MYYcE/+0QRrwN5qMRUhexKiG1BCMacomh3E8CpOLgetK32bVZoHqbxLIn55h?=
 =?us-ascii?Q?RovszUhID5suVkbLviLlIcU9/Zqmbko+IWSBKk3QeVDHUETpwiH99SGsL0CJ?=
 =?us-ascii?Q?ILaX8BdBhICl7AoLg+sGtkeYbckjxblDeG3y0p/JH5YAiIFWS2WhCy9Q1Nzf?=
 =?us-ascii?Q?Uapv+oB51MsFQgeCSMnUWpRXWyqgXYN4Ll6wrdrUtgZ/OqjwOFd8rwY8fBv8?=
 =?us-ascii?Q?DYu7tC9t8OU1KuI/KJa1f7Z31ffavL4lw12B7lfJn5BS4jCWefJr90W4K4FH?=
 =?us-ascii?Q?BVGs1f+bqDJc0AITDfqsbwtDsahC4MiLh6u5fVboIqiLouxeu3jLv0sVQsSu?=
 =?us-ascii?Q?MiCwO078dJ6nbdsW1mQ8MIpCnKvQroISR0ZRXKwFZLBElc80JxPsxYnwOind?=
 =?us-ascii?Q?6pJgD4bNlGZkXZODzEg/PAl8obmVIjNSGG1dhOsrHFnMqMEdRV+wHznizPY0?=
 =?us-ascii?Q?1SOVk/UNMQtfLOO8z/LngFyzGUFeWq7KCCdvGjXLiExvUTXM5GxcbMrtlPFM?=
 =?us-ascii?Q?K0mX3racEP2QqO8eRcUzQ3ObzpdazKhN2LFrqQmM9Vh4O2+r142TqZy7CfST?=
 =?us-ascii?Q?XQopUMcKHuLqVNfYGic1S9rUa73p0sHaic5kn0WVDR5GGi2VzPrvsEtyW1CN?=
 =?us-ascii?Q?zzPOAnLQ9X4kS5KOVtpKksJN?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f08f2a30-4fef-45a3-c2f9-08d97871b91f
X-MS-Exchange-CrossTenant-AuthSource: CO1PR10MB4468.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Sep 2021 17:53:28.3493
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: A53XU9TPD0hHVP3L9IJGlbQ3G4S83zapq6kVzRm/0U+wR9NEkb6tDzWqUPIOCcBPPNs+0CqMQUpB4o4hJWHqqw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR10MB1568
X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10108 signatures=668682
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0 phishscore=0 malwarescore=0
 adultscore=0 mlxscore=0 mlxlogscore=999 suspectscore=0 bulkscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2109030001
 definitions=main-2109150105
X-Proofpoint-GUID: VIxzGFAGkzzfl0H1FEwK_6vsWlQ_vQqV
X-Proofpoint-ORIG-GUID: VIxzGFAGkzzfl0H1FEwK_6vsWlQ_vQqV
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Changes in v2:
 - Fixed compilation error [1] due to typo in patch-3 (stack_depot_print
   used in place of stack_depot_snprint)
   This compilation error appears with CONFIG_DRM_I915_DEBUG_RUNTIME_PM=y
   and this was missed by my test config (x86_64_defconfig)

[1] https://patchwork.freedesktop.org/series/94696/

Original cover letter
------------------------------------------
This patch series consolidates the changes submitted and reviewed at [1]
and [2].
The patches at [1] and [2] were submitted separarely, but they have some
inter dependency (later patches were created on top of earlier ones).
As both sets are still under review, I have put them in a single
change set here, so that it can be reviewed/included together and also
to avoid automation build failures where git am fails because of absent
parent.

I have included Acked-by (from Vlastimil) and Reviewed-by (from Alexander)
tags obtained so far for these changes and have also addressed last review
comment from Vlastimil [3].

To summarize, the changes in this set are as follows:

PATCH-1: Checks validity of a stackdepot handle before proceeding
to access stackdepot slab/objects.

PATCH-2: Adds a helper in stackdepot, to allow users to print
stack entries just by specifying the stackdepot handle. It also
changes such users to use this new interface.   

PATCH-3: Adds a helper in stackdepot, to allow users to print
stack entries into buffers just by specifying the stackdepot handle and
destination buffer. It also changes such users to use this new interface.

[1] https://lore.kernel.org/lkml/20210902000154.1096484-1-imran.f.khan@oracle.com/
[2] https://lore.kernel.org/lkml/20210910141001.1622130-1-imran.f.khan@oracle.com/
[3] https://lore.kernel.org/lkml/ef0aa660-0cb6-dc21-f2ce-368b34f8af3d@suse.cz/

Imran Khan (3):
  lib, stackdepot: check stackdepot handle before accessing slabs.
  lib, stackdepot: Add helper to print stack entries.
  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              |  5 +++
 lib/stackdepot.c                        | 45 +++++++++++++++++++++++++
 mm/kasan/report.c                       | 15 ++-------
 mm/page_owner.c                         | 18 +++-------
 8 files changed, 66 insertions(+), 52 deletions(-)

-- 
2.30.2