From mboxrd@z Thu Jan 1 00:00:00 1970 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 24B5831A05B for ; Tue, 16 Sep 2025 14:35:06 +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=1758033307; cv=none; b=UNLHZRt55h4gWMceETybyyVhvxGm6fc91lyXauwewmCf5q0Bs9243+2OgaobpifnG5kvac74lbYEAk6/4TdvtGUPivGg+CeRWuzrijnlYGMU2FNa0wglU57FlQlzx42Aq/k5Sa3GQxR2ZcnOgJvo2XrxsN1rCF5yOpWoSE0noOM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758033307; c=relaxed/simple; bh=Nu11bmOVRte2aokBqHmrTHWTKZU8JcQy9nmoUexKGS0=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=IznFGAACAIiSuITbuXzwVaXsufLnOcaVp/u8kewgZHSYe+9Ed6w3TmX1KgRKUM4GtBtIroWaepy87D6DI1AEz8/4E9wjqAIoovYiY1S6X5tMNFsqdq1ZqAuT9I+Hxk6mcYA665QhjBKUelTnQ9UQp59RW8xN5rz72B9d4Vv4pbs= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=CVyMXgYf; arc=none smtp.client-ip=209.85.128.44 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="CVyMXgYf" Received: by mail-wm1-f44.google.com with SMTP id 5b1f17b1804b1-45f30011eceso58925e9.1 for ; Tue, 16 Sep 2025 07:35:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1758033304; x=1758638104; darn=lists.linux.dev; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to; bh=CT4KnFMC9K253+uFB3sGHZvOxPtog/sC9euCmcEFSfI=; b=CVyMXgYf/bzr7RDFGXWhuKguWJiYbEUQ20GHBFSiDo9ykcVa2o/ve4U43DFzpp6ubH K2GBpNhmLqvrcx7wIlJj7Uh6iR8q3x6OzxXUvfS+/nwn/QDOJMM7sTzXvoQLqA4Qun2y z6melJpE9W8z9AamVpWXX7IR24isiqB9gFnuwKpLNFgAkxHULwr65gZtKE9S5Fv6hDKu 7v3vradggw11fFPnj0/i+xyLDMNKBdABRNvv4L1rn9C/BNHlzbgqhvTyvZ31PjxQJ7YK M+3eSDb/lU6F83JBwqeWjvEfiXxjxQN76/dE6Y061I5/jgtrLpSIOGq54MScFPxgELdp 1lPQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1758033304; x=1758638104; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=CT4KnFMC9K253+uFB3sGHZvOxPtog/sC9euCmcEFSfI=; b=ZnVyFgmcd6yNserXRC8fDAE+GnqcQthrIn+40boLYiAeg34+DA+8PKWH/4KgzHNxY4 2B36a+uO6S5+V8w59yPRUxfO+Tz+HBLuHkN9Cm58j59d0nAdCSKkKGvRfx7qz01XpXMy BqBaybFKPhfR8jnFzIXvrlWWRRo29CZkOjQHklRFL+KP4AHURpMiS0U4QsVGhlBJPw4f LML27rfqtvkpIVsoOInToO9vpuDLc7mnb65rxIKFIEh3EDjP32N1BZYplVkHesymxWdA t20LwuTrSFhr9tVgBbjGMyKjdS6ZxOzZOve1nH2hit4mhLW+SsLTf7JOJPx0NCdM6CDR DvuA== X-Forwarded-Encrypted: i=1; AJvYcCUoJAY7LT09KekpotKEvriLO5BtoeVW64VbzzwW6TgPDDLwArUl2xg+2HS9UwRDaZTUECi0lZI=@lists.linux.dev X-Gm-Message-State: AOJu0Yxatp6G9tWpqZ0SpzrL2Eo2bNzb2viZ7hml6GQvX7XCixotS7/w 1Ga1eS7EvxAYUichHJ2jNyuAelVAJMMc+84JRiubPNmHsUhFms6DuLjws35GtypE2g== X-Gm-Gg: ASbGncv7YZcfFnYw5hDxJk4abgiOfniZSIYt0LoGWPWQGGRqa28Md9S/FySKQQxBYcP kARw4SGs4VrIf6uDXUgSdlDGhipoR6fuUeycgH8kjvFnzEoJ4ynZDPSUSQvIYPwo/Gbr1zX/qLv 6JfFxwW0OuHKdQsa1EiY3uJjzxveVVjvdGsTffldmaCgznitsVPh8xrOIUYuFrn1b2fp6KMuNRI wfR8XoLy0+jnEKlSbRxo8/1hUKEju5+Z3nLAv4sWtPkO6RjEbs7LkXtkdC5t4aaqTx6PG1ljBnP 91SnI+ZBP6me8Ajfx26Ea7q6PhSBB3/Jvh5RjVhimY4VH/dlavBBZk7RxTaF7MJBcDW07eDNS9F d6fkukENTFnhTvJYH2tYLJPzuoxEnef5nOc/oXu4rLedsexoxRabCaQFauFxESp6emT4vUw== X-Google-Smtp-Source: AGHT+IECpwDExsg0Dr7B4qD1+CeVxxyJjTc2PPzhq2qhSmxiB4I92vX7At74X9XKcTLJsKMwvH5o1A== X-Received: by 2002:a05:600c:8b35:b0:45a:2861:3625 with SMTP id 5b1f17b1804b1-45f320ea2dfmr1951735e9.4.1758033304305; Tue, 16 Sep 2025 07:35:04 -0700 (PDT) Received: from google.com (157.24.148.146.bc.googleusercontent.com. [146.148.24.157]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3e9cf04db65sm10884027f8f.3.2025.09.16.07.35.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 16 Sep 2025 07:35:03 -0700 (PDT) Date: Tue, 16 Sep 2025 14:35:00 +0000 From: Mostafa Saleh To: Daniel Mentz Cc: linux-kernel@vger.kernel.org, kvmarm@lists.linux.dev, linux-arm-kernel@lists.infradead.org, iommu@lists.linux.dev, maz@kernel.org, oliver.upton@linux.dev, joey.gouly@arm.com, suzuki.poulose@arm.com, yuzenghui@huawei.com, catalin.marinas@arm.com, will@kernel.org, robin.murphy@arm.com, jean-philippe@linaro.org, qperret@google.com, tabba@google.com, jgg@ziepe.ca, mark.rutland@arm.com, praan@google.com Subject: Re: [PATCH v4 16/28] iommu/arm-smmu-v3-kvm: Create array for hyp SMMUv3 Message-ID: References: <20250819215156.2494305-1-smostafa@google.com> <20250819215156.2494305-17-smostafa@google.com> Precedence: bulk X-Mailing-List: kvmarm@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: On Tue, Sep 09, 2025 at 11:30:48AM -0700, Daniel Mentz wrote: > On Tue, Aug 19, 2025 at 2:55 PM Mostafa Saleh wrote: > > > > + if (kvm_arm_smmu_array[i].mmio_size < SZ_128K) { > > + pr_err("SMMUv3(%s) has unsupported size(0x%lx)\n", np->name, > > + kvm_arm_smmu_array[i].mmio_size); > > Use format specifier %pOF to print device tree node. > If mmio_size is a size_t type, use format specifier %zx. > Align language of error message with kernel driver which prints "MMIO > region too small (%pr)\n". Thanks for catching that, I will fix it in v5. > I'm wondering if we should use kvm_err instead of pr_err. I am not sure, kvm_err seems to be used from core arch code only, but I don't see why not. Thanks, Mostafa