Re: [PATCH 1/2] mm/swap: fix stale comment on swap_info_struct::cluster_info
Youngjun Park <[email protected]> Wed, 5 Aug 2026 01:36:18 +0900
| Newsgroups | org.kvack.linux-mm,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <anIVAs1a0N37oba1@yjaykim-PowerEdge-T330> |
On Tue, Aug 04, 2026 at 05:48:51PM +0800, Kairui Song wrote: > On Wed, Jul 29, 2026 at 12:43 AM Youngjun Park <[email protected]> wrote: > > > > From: Youngjun Park <[email protected]> > > > > From: Youngjun Park <[email protected]> > > Hi Youngjun, Thanks for the patch. > > Double From:? I think you want to keep the later one right? my git config was wrong :( fixed it locally > > > > setup_swap_clusters_info() allocates cluster_info for every swap area, > > not only for SSDs. > > > > Signed-off-by: Youngjun Park <[email protected]> > > --- > > include/linux/swap.h | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/include/linux/swap.h b/include/linux/swap.h > > index 2cb1d29307c5..e12c73891b48 100644 > > --- a/include/linux/swap.h > > +++ b/include/linux/swap.h > > @@ -246,7 +246,7 @@ struct swap_info_struct { > > struct plist_node list; /* entry in swap_active_head */ > > signed char type; /* strange name for an index */ > > unsigned int max; /* size of this swap device */ > > - struct swap_cluster_info *cluster_info; /* cluster info. Only for SSD */ > > + struct swap_cluster_info *cluster_info; /* one per cluster, on every device */ > > Nice catch. > > Acked-by: Kairui Song <[email protected]>