Re: [PATCH -mm 0/6] swsusp: Add support for swap files

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi.

On Sun, 2006-09-24 at 00:59 +0200, Pavel Machek wrote:
> Hi!
> 
> > > > > We could use a regular (non-swap) file like this but that would require us to
> > > > > use some dangerous code (ie. one that writes directly to blocks belonging to
> > > > > certain file bypassing the filesystem).  IMHO this isn't worth it, provided
> > > > > the kernel's swap-handling code can do this for us and is known to work. ;-)
> > > > 
> > > > It's not that dangerous once you debug it.
> > > 
> > > Certainly.  Still, let me repeat: if there is some code that does pretty much
> > > the same and has _already_ been debugged, I prefer using it to writing some
> > > new code, debugging it etc.
> > 
> > Look at Suspend2 then. I know you won't want it in exactly that form,
> > but it's there and have been tested and debugged.
> 
> Well, but any testing/debugging would have to be invalidated for a
> merge, sorry. suspend2 has diverged too much.
> 
> "normal files" are not big priority for us, and you could probably do
> them in userland just now.... Anyway, diff -u for kernel or preferably
> for uswsusp parts would be welcome.
> 
> Telling us how much suspend2 rocks with each and every mail... is not
> that welcome. 

That's not what I was doing. I was saying that Rafael could, if he
wanted, at least look at how Suspend2 has done it and use that (if he
thinks its helpful) as a basis for improving or whatever. If he does
essentially the same thing for swsusp, he could again look at suspend2
loops etc to help in double checking his algorithm and so on.

Regards,

Nigel

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux