> -----Original Message----- > From: Rafael J. Wysocki [mailto:rjw@xxxxxxxxxxxxx] > Sent: Monday, March 19, 2018 5:11 AM > To: Limonciello, Mario <Mario_Limonciello@xxxxxxxx> > Cc: linux-acpi@xxxxxxxxxxxxxxx; LKML <linux-kernel@xxxxxxxxxxxxxxx> > Subject: Re: [PATCH] power/hibernate: Make passing hibernate offsets more friendly > > On Tuesday, March 6, 2018 11:53:40 AM CET Mario Limonciello wrote: > > Currently the only way to specify a hibernate offset for a > > swap file is on the kernel command line. > > > > Add a new /sys/power/disk_offset that lets userspace > > specify the offset and disk to use when initiating a hibernate > > cycle. > > > > Also split up the parsing routine to re-use the same code > > for the /sys/power/resume and /sys/power/disk_offset parsing. > > > > Signed-off-by: Mario Limonciello <mario.limonciello@xxxxxxxx> > > Well, IMO /sys/power/resume and /sys/power/disk_offset would be confusingly > similar after this patch. Yeah they do function pretty similarly. > > I wonder if adding a resume_offset sysfs attr to simply allow the value of > resume offset alone to be passed to the kernel at run time (in addition to > the command line) would work? > > Thanks! I guess what's your expected flow for how the disk gets selected though? Should the kernel be "guessing" which disk to use? Or should userspace be setting offset via /sys/power/resume_offset followed by disk in /sys/power/resume (which will show an error in syslog about no partition found) If the latter, then I'd like to also modify the behavior to downgrade that missing hibernate partition to debug instead too. -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
