From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rafael J. Wysocki" Subject: Re: [RFC 03/15] PM / Hibernate: separate block_io Date: Thu, 25 Mar 2010 21:33:50 +0100 Message-ID: <201003252133.50206.rjw__9659.60198450393$1269549331$gmane$org@sisk.pl> References: <1269361063-3341-1-git-send-email-jslaby@suse.cz> <201003252112.25204.rjw@sisk.pl> <4BABC3D1.1050104@crca.org.au> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <4BABC3D1.1050104@crca.org.au> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-pm-bounces@lists.linux-foundation.org Errors-To: linux-pm-bounces@lists.linux-foundation.org To: Nigel Cunningham Cc: linux-pm@lists.linux-foundation.org, Jiri Slaby , linux-kernel@vger.kernel.org List-Id: linux-pm@vger.kernel.org On Thursday 25 March 2010, Nigel Cunningham wrote: > Hi. > > On 26/03/10 07:12, Rafael J. Wysocki wrote: > > On Thursday 25 March 2010, Nigel Cunningham wrote: > >> Hi again. > >> > >> On 25/03/10 09:58, Nigel Cunningham wrote: > >>> Hi. > >>> > >>> On 25/03/10 08:22, Jiri Slaby wrote: > >>>> On 03/24/2010 09:30 PM, Pavel Machek wrote: > >>>>> On Tue 2010-03-23 17:17:31, Jiri Slaby wrote: > >>>>>> +int sws_bio_read_page(pgoff_t page_off, void *addr, struct bio **bio_chain) > >>>>>> +{ > >>>>>> + return submit(READ, sws_resume_bdev, page_off * (PAGE_SIZE>> 9), > >>>>>> + virt_to_page(addr), bio_chain); > >>>>>> +} > >>>>> > >>>>> sws_ is kind of strange prefix. We were trying to get away from > >>>>> "swsuspend" name for quite some time... > >>>> > >>>> No problem to change the prefix to anything else. Do you (anybody) > >>>> suggest anything? > >>>> > >>> > >>> How about some abbreviation of hibernate? "hib"? > >> > >> On further reflection, how about "std" (suspend to disk)? I think that's > >> less ugly than the 'hib' suggestion :) > > > > But it also decodes as "standard" if someone is not in the right context. :-) > > Ah... > > > If the "bio" part of the name is not essential (ie. there's no conflicting name > > already), we could call it simply hibernate_read_page(). > > Yeah. So we're going with hibernate or hib_ if it needs abbreviating? I'd just use "hibernate" without abbreviating if reasonably possible. We can also use "image_" in some cases I guess. Rafael