From nobody Wed Oct 8 14:20:57 2025 Received: from mail-wm1-f44.google.com (mail-wm1-f44.google.com [209.85.128.44]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9E50721C9EA; Fri, 27 Jun 2025 07:31:59 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.44 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751009521; cv=none; b=o6Xho52Yf5IVetSb/TnCOjH/dmF+wgxaUTGbLgKoySCPDXfNWSSGvJYTdmpjxBQkCsDv166BPymzjp5jNXahooX5KhTspxVu6yuxvKyTwNCantmsaEFTUOt2EwTLOC3shLqGohdIAL+NnRMcCRyFw/dym7zfklVpV2LrDNBqxNE= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751009521; c=relaxed/simple; bh=CsGo1GsbYINnY/8tnwOpQqo8uzAdi38TQdAiBPI/wtc=; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version; b=SSlZaRVxElUThdVhSbqusaJc6SWf/iUFLLWx9hg30N+viPDAwKytxau/Zs4LuEmi7HjyV/mnRNPAsrbC6whx58wOYtXywyhnd2b9udMlkyZtaoLzmBCnYs4RhJjYP1CFS/lXDxTCyOuzU1uPpD/DKxbY/ivRQYBemkIDp6e8c1E= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=kEiz9yF2; arc=none smtp.client-ip=209.85.128.44 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="kEiz9yF2" Received: by mail-wm1-f44.google.com with SMTP id 5b1f17b1804b1-453079c1e2eso2115365e9.3; Fri, 27 Jun 2025 00:31:59 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1751009518; x=1751614318; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=TScBpjlTpSBrukUzd8MltsW16ug3e51n2CNSK0/YESI=; b=kEiz9yF29EI/5oZICmy3GIxqFZ+gBXjxBqkPYYIyFm7iDqhvnnM5+irBPSEqsEqcL1 NyJXJxNsxew1nedR1F4wVTeTdN+NuffM9vZH8CGZdPS7sacvG3roG2V3DknrYrRTtfqZ ziNIonenKT9wE442exNqh445ewTOXkFZuCAXfOR1iqkD++hdVdzusIiBmhft4B0opkNr rWYZTbHy/BbkQrTd5nJIhj14kXUmB5mvNpvC5aac4KJTVkBuT2iNGPtSS4J0M9l1oFQR k6JXTt5kYkJ+YFKSrxfNLDno4P9VaPkUW7tEDrLi22yowpdi2PTO/+X9nHCUmzvwNJrv yEjA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1751009518; x=1751614318; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=TScBpjlTpSBrukUzd8MltsW16ug3e51n2CNSK0/YESI=; b=O9iODsIjnlUYRJF80g+J5wxj4WhcA8EmcpNWm3uvIbxikfNh1Ngyi7REm0XKRBmnMO NDBTUNcZW1RBkw0rCJDhsbGs3Ubh0gMqqein++5GKgBokdbQGnl6RGJPF6Rm7K/2VOPq E952egkrCQYx07tLz5JbuJAfP7MWLSqD55SBLyp3hLv5Ov116Tk/qsfnQu4C1mY3D0Oh E6Dv/20EDQXcrTnLJVUEUjGUwnNKSWm61D5YgEQfkp5w/aYEmGAcse0SUPwOwXMN0NU3 V1BCBIgpR99S+s7HeD3b0UkrVU3GJffH8hUEZIOQkUPS912NHjpwX6XFuAPGc6mpvChG ourQ== X-Forwarded-Encrypted: i=1; AJvYcCWJi06sNcFfAH7tMF1fOhMeGBwYcIK+nvKxm3MIAbXvs3Ba79sKjGj2PWg/MoRf5wKYP6M4riJdsxncLRs=@vger.kernel.org, AJvYcCXl0KbHvZ8x6se/CqMU5ZGozpLnaxqbqgubUV3PyOr8xXrOfNWKzir44RnEYH5ZNle0JVesYWIQZix9XX0=@vger.kernel.org X-Gm-Message-State: AOJu0Yyzegiu2L0J1zMQi7sh1giuHIweKrwC/07AI3T28blWKwDNm8UR uf1FAIXdoiZ/cUOml+xMMiJAqBUO9cWQUIQgZO7MYvFXBTNx7OVqSgY5 X-Gm-Gg: ASbGncu+GBA6HJ9Bhe3eplgp7eQkoHuS/xCLwInCaNuoaP5J6pux+ItpvfX+Q9bz1v+ eKIlpb3YCExNUMoxB52JeqIs2SmUCwn9feulbSZiF5URbm2raOAKOCUqwviqfaZDs5PPRVX1lSP zyorhVqzCi7mV+lTeFCMfGgxVsm2JwQRirj9zmTA9QUTRaNU2smV7XmNxECJxHnoE2EcwDZ5w6m F+fkHjQPtimvzHkVwPIJ+Tiz4L+6Kld6V+4psJzTUZuL9wecEW3FiU+ARQ4/+7ACQ1Z+ZHxWz/9 dTSxCgXS2o/Y3fr7NTwhmJTg2/nvuy2rFKFjZZvro3GmCgupgOiaDqG0uFaHGf9xncunWj2embc KrVS1IJefiQ8EAaAE9wE0UKAK X-Google-Smtp-Source: AGHT+IE76NO/bOabugCB8DyLAXSDRN+k6axz7/mxNvs22Q813ctMby/ju5/cUVSYpa3ZP5RqlSPsrg== X-Received: by 2002:a05:6000:2a06:b0:3a5:3399:51b1 with SMTP id ffacd0b85a97d-3a8fdeff8b9mr580958f8f.6.1751009517507; Fri, 27 Jun 2025 00:31:57 -0700 (PDT) Received: from localhost.localdomain ([156.208.147.213]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3a88c7e74fbsm1963481f8f.10.2025.06.27.00.31.55 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 27 Jun 2025 00:31:57 -0700 (PDT) From: Abdelrahman Fekry To: andy@kernel.org, hdegoede@redhat.com, mchehab@kernel.org, sakari.ailus@linux.intel.com, gregkh@linuxfoundation.org Cc: linux-kernel-mentees@lists.linux.dev, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, linux-staging@lists.linux.dev, skhan@linuxfoundation.com, dan.carpenter@linaro.org, Abdelrahman Fekry , Hans de Goede Subject: [PATCH v4] staging: media: atomisp: remove debug sysfs attributes active_bo and free_bo Date: Fri, 27 Jun 2025 10:29:39 +0300 Message-Id: <20250627072939.21447-1-abdelrahmanfekry375@gmail.com> X-Mailer: git-send-email 2.25.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" The sysfs attributes active_bo and free_bo expose internal buffer state used only for debugging purposes. These are not part of any standard kernel ABI, and need to be removed before this driver may be moved out of drivers/staging. - Remove active_bo and free_bo attributes - Remove group registration calls form hmm_init() and hmm_cleanup() Suggested-by: Hans de Goede Signed-off-by: Abdelrahman Fekry --- v4: - Fix typos v3: https://lore.kernel.org/all/20250624170746.47188-1-abdelrahmanfekry375@= gmail.com/ - remove blank line v2: https://lore.kernel.org/all/20250624144943.39297-1-abdelrahmanfekry375@= gmail.com/ - Add Suggested-by line - Remove unnecessary comments v1: https://lore.kernel.org/all/20250624130841.34693-1-abdelrahmanfekry375@= gmail.com/ drivers/staging/media/atomisp/pci/hmm/hmm.c | 92 --------------------- 1 file changed, 92 deletions(-) diff --git a/drivers/staging/media/atomisp/pci/hmm/hmm.c b/drivers/staging/= media/atomisp/pci/hmm/hmm.c index 84102c3aaf97..469de184b9b8 100644 --- a/drivers/staging/media/atomisp/pci/hmm/hmm.c +++ b/drivers/staging/media/atomisp/pci/hmm/hmm.c @@ -28,88 +28,6 @@ struct hmm_bo_device bo_device; static ia_css_ptr dummy_ptr =3D mmgr_EXCEPTION; static bool hmm_initialized; -/* - * p: private - * v: vmalloc - */ -static const char hmm_bo_type_string[] =3D "pv"; - -static ssize_t bo_show(struct device *dev, struct device_attribute *attr, - char *buf, struct list_head *bo_list, bool active) -{ - ssize_t ret =3D 0; - struct hmm_buffer_object *bo; - unsigned long flags; - int i; - long total[HMM_BO_LAST] =3D { 0 }; - long count[HMM_BO_LAST] =3D { 0 }; - int index1 =3D 0; - int index2 =3D 0; - - ret =3D scnprintf(buf, PAGE_SIZE, "type pgnr\n"); - if (ret <=3D 0) - return 0; - - index1 +=3D ret; - - spin_lock_irqsave(&bo_device.list_lock, flags); - list_for_each_entry(bo, bo_list, list) { - if ((active && (bo->status & HMM_BO_ALLOCED)) || - (!active && !(bo->status & HMM_BO_ALLOCED))) { - ret =3D scnprintf(buf + index1, PAGE_SIZE - index1, - "%c %d\n", - hmm_bo_type_string[bo->type], bo->pgnr); - - total[bo->type] +=3D bo->pgnr; - count[bo->type]++; - if (ret > 0) - index1 +=3D ret; - } - } - spin_unlock_irqrestore(&bo_device.list_lock, flags); - - for (i =3D 0; i < HMM_BO_LAST; i++) { - if (count[i]) { - ret =3D scnprintf(buf + index1 + index2, - PAGE_SIZE - index1 - index2, - "%ld %c buffer objects: %ld KB\n", - count[i], hmm_bo_type_string[i], - total[i] * 4); - if (ret > 0) - index2 +=3D ret; - } - } - - /* Add trailing zero, not included by scnprintf */ - return index1 + index2 + 1; -} - -static ssize_t active_bo_show(struct device *dev, struct device_attribute = *attr, - char *buf) -{ - return bo_show(dev, attr, buf, &bo_device.entire_bo_list, true); -} - -static ssize_t free_bo_show(struct device *dev, struct device_attribute *a= ttr, - char *buf) -{ - return bo_show(dev, attr, buf, &bo_device.entire_bo_list, false); -} - - -static DEVICE_ATTR_RO(active_bo); -static DEVICE_ATTR_RO(free_bo); - -static struct attribute *sysfs_attrs_ctrl[] =3D { - &dev_attr_active_bo.attr, - &dev_attr_free_bo.attr, - NULL -}; - -static struct attribute_group atomisp_attribute_group[] =3D { - {.attrs =3D sysfs_attrs_ctrl }, -}; - int hmm_init(void) { int ret; @@ -130,14 +48,6 @@ int hmm_init(void) */ dummy_ptr =3D hmm_alloc(1); - if (!ret) { - ret =3D sysfs_create_group(&atomisp_dev->kobj, - atomisp_attribute_group); - if (ret) - dev_err(atomisp_dev, - "%s Failed to create sysfs\n", __func__); - } - return ret; } @@ -145,12 +55,10 @@ void hmm_cleanup(void) { if (dummy_ptr =3D=3D mmgr_EXCEPTION) return; - sysfs_remove_group(&atomisp_dev->kobj, atomisp_attribute_group); /* free dummy memory first */ hmm_free(dummy_ptr); dummy_ptr =3D 0; - hmm_bo_device_exit(&bo_device); hmm_initialized =3D false; } -- 2.25.1