From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f202.google.com (mail-pf1-f202.google.com [209.85.210.202]) (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 0A659284678 for ; Thu, 9 Oct 2025 21:27:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.202 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1760045226; cv=none; b=r8wSl+mJB+L5MpweeMRBU2gtKMIX/IZaCjLYnqfzCHJ0MQ5tdIJ6VUTbglJoZcrWlg5x785waWSpDlgevqyn0trf9NleUDqbtMtwd1F0J9zaUCnatFgbOh0bBUuXvtyOi+iGZM+CvGIVqtKg2YJHITiKmkAb6vHaNy07QPnV+AY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1760045226; c=relaxed/simple; bh=QUyhg2tPz5e6+QEJVc5tbnozxAJnMMfwwidrggXUHNo=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=d8YKHXBLI5wazsqaDGCJY/VWm0zgO6ZE/PEd1s1gNAC6liAt0zedGgDa4xWfrKBM4NjakpYW5QjcTpAIKRyxHJBTJJoOdkzKIwNYGNtWNvVyp7a7OqiG7lWznd1esWJIc3yCFanxyMeW+KMfW0s5aYzjbbWXKIIkz+D45UD9i14= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--ackerleytng.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=euQH7L7x; arc=none smtp.client-ip=209.85.210.202 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=flex--ackerleytng.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="euQH7L7x" Received: by mail-pf1-f202.google.com with SMTP id d2e1a72fcca58-78102ba5966so2093524b3a.2 for ; Thu, 09 Oct 2025 14:27:04 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1760045224; x=1760650024; darn=vger.kernel.org; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:from:to:cc:subject:date:message-id:reply-to; bh=VyG7Vshnn1y3ko5tY/OsDeeGP+hnaQE19ZgqRlKar30=; b=euQH7L7xwI49CNkcocNjnYTGXwNznmWhXcpNLI5tT3lTh4ELiJsBNxA5aL/Jon84CH lmL89GYvbGnIKSXW3o7owp4BW2wrFm0UMmLlftPOP5c9FtY85L6p3bQlCPOrIkS5Rj5T p+4ndpn6Ta1CeNbORZSWCOJ5p8q+wENbIZDbfIA+KWCfiNrqgEkpvoqVgSJEn2E0edSA d9ifZ/JbcUSvm8Hio8UtUKHNSE153jhJ+b40mpoe57hf2CKDucFFArrQs6SCy7mlNe6S ST0RfH0/NnPiG5LP3wRzI6pmHJ4Q43SWA6Q5o6pekg59S0blyd1H9wude1FqS9NUKy+q 2t0Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1760045224; x=1760650024; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=VyG7Vshnn1y3ko5tY/OsDeeGP+hnaQE19ZgqRlKar30=; b=FBRjp0KEPeH1kKvtptqjd5fLXjwhsb0JLk5cmcz3bW/arnss30baX8q4aIj17k6dAv 51913+T6Za1wI7QRwOP2Lf9OICubWamnQvzOrwBJV5cpqTMHTMqpwcdF4EbOJRKS7Org Lfj3MQxa7nvMr+KmqzuahuRR4l5UIz19ALJsE1EksBenXcPhjExOrwrlarURnLBjPEn+ TPxlNnzOBFP99OSWi/3zz8EVy7JLI5cS1sAIxgt+8WDbneoBgVLC9/Zuybjlaw5Hxyv+ KD/Ijy3Xv0bPb+MAojgVeg5wH2ZYUOU1zVrBp++VUcwz0W+wFO9E5FUAah2kaX2fiCSg bzxA== X-Forwarded-Encrypted: i=1; AJvYcCVSS3/PMxaFP1ns30MLZBUJdgbUFYG3/EjGFjD8Idh6zdLH/TfKGcVs9NXK1qi586/VrzZWC+hz0ysoPXw=@vger.kernel.org X-Gm-Message-State: AOJu0YwdDxEOIkt8Qvp6mPXNnjJvC6HnYDEjePFbklwbuD1FR4JBZNM4 5LTprViCalrjU8F7cO/yWuCaLFZEaGo5b4z9FHBIjiXYKX9Uh3IkyqqcxNEdMNiERtkraikI0qD V7Lyx6IEMHkIGl68RjMpNy54eRg== X-Google-Smtp-Source: AGHT+IEXSCT7Ge93/eS1ZLL+wIsHzqCvUWVK2zdip6A/DIomUySbaHDcPWrDH/GuqrKumWfw/V6m+o1hEtdfCxTDkQ== X-Received: from pjbms8.prod.google.com ([2002:a17:90b:2348:b0:33b:51fe:1a80]) (user=ackerleytng job=prod-delivery.src-stubby-dispatcher) by 2002:a05:6a21:33a6:b0:32a:6d26:fb5a with SMTP id adf61e73a8af0-32da80b8305mr11558495637.4.1760045224327; Thu, 09 Oct 2025 14:27:04 -0700 (PDT) Date: Thu, 09 Oct 2025 14:27:02 -0700 In-Reply-To: <20251007221420.344669-3-seanjc@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20251007221420.344669-1-seanjc@google.com> <20251007221420.344669-3-seanjc@google.com> Message-ID: Subject: Re: [PATCH v12 02/12] KVM: guest_memfd: Add macro to iterate over gmem_files for a mapping/inode From: Ackerley Tng To: Sean Christopherson , Marc Zyngier , Oliver Upton , Paolo Bonzini Cc: linux-arm-kernel@lists.infradead.org, kvmarm@lists.linux.dev, kvm@vger.kernel.org, linux-kernel@vger.kernel.org, David Hildenbrand , Fuad Tabba , Shivank Garg , Ashish Kalra , Vlastimil Babka Content-Type: text/plain; charset="UTF-8" Sean Christopherson writes: > Add a kvm_gmem_for_each_file() to make it more obvious that KVM is > iterating over guest_memfd _files_, not guest_memfd instances, as could > be assumed given the name "gmem_list". > Can we also add to .clang-format: diff --git i/.clang-format w/.clang-format index 48405c54ef271..e4df86f2d3cf7 100644 --- i/.clang-format +++ w/.clang-format @@ -541,6 +541,7 @@ ForEachMacros: - 'kvm_for_each_memslot' - 'kvm_for_each_memslot_in_gfn_range' - 'kvm_for_each_vcpu' + - 'kvm_gmem_for_each_file' - 'libbpf_nla_for_each_attr' - 'list_for_each' - 'list_for_each_codec' Reviewed-by: Ackerley Tng Tested-by: Ackerley Tng > No functional change intended. > > Signed-off-by: Sean Christopherson > --- > virt/kvm/guest_memfd.c | 9 +++++---- > 1 file changed, 5 insertions(+), 4 deletions(-) > > diff --git a/virt/kvm/guest_memfd.c b/virt/kvm/guest_memfd.c > index 3c57fb42f12c..9b9e239b3073 100644 > --- a/virt/kvm/guest_memfd.c > +++ b/virt/kvm/guest_memfd.c > @@ -22,6 +22,9 @@ struct gmem_file { > struct list_head entry; > }; > > +#define kvm_gmem_for_each_file(f, mapping) \ > + list_for_each_entry(f, &(mapping)->i_private_list, entry) > + > /** > * folio_file_pfn - like folio_file_page, but return a pfn. > * @folio: The folio which contains this index. > @@ -159,13 +162,12 @@ static void __kvm_gmem_invalidate_begin(struct gmem_file *f, pgoff_t start, > static void kvm_gmem_invalidate_begin(struct inode *inode, pgoff_t start, > pgoff_t end) > { > - struct list_head *gmem_list = &inode->i_mapping->i_private_list; > enum kvm_gfn_range_filter attr_filter; > struct gmem_file *f; > > attr_filter = kvm_gmem_get_invalidate_filter(inode); > > - list_for_each_entry(f, gmem_list, entry) > + kvm_gmem_for_each_file(f, inode->i_mapping) > __kvm_gmem_invalidate_begin(f, start, end, attr_filter); > } > > @@ -184,10 +186,9 @@ static void __kvm_gmem_invalidate_end(struct gmem_file *f, pgoff_t start, > static void kvm_gmem_invalidate_end(struct inode *inode, pgoff_t start, > pgoff_t end) > { > - struct list_head *gmem_list = &inode->i_mapping->i_private_list; > struct gmem_file *f; > > - list_for_each_entry(f, gmem_list, entry) > + kvm_gmem_for_each_file(f, inode->i_mapping) > __kvm_gmem_invalidate_end(f, start, end); > } > > -- > 2.51.0.710.ga91ca5db03-goog