lkml.org 
[lkml]   [2006]   [Jan]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [RFC/RFT][PATCH -mm 0/5] swsusp: userland interface (rev. 2)
Date
Hi,

On Friday, 6 January 2006 00:30, Pavel Machek wrote:
> > This is the second "preview release" of the swsusp userland interface patches.
> > They have changed quite a bit since the previous post, as I tried to make the
> > interface more robust against some potential user space bugs (or outright
> > attempts to abuse it).
>
> Works for me, thanks.
>
> Perhaps it is time to get 1/4 and 3/4 into -mm? You get my signed-off
> on them...

OK, I'll prepare them in a while.

> 2/4 needs to allocate official major/minor. 1/13 would be nice :-).

Well, you said you liked the patch with a misc device (ie. major = 10).

Actually the code is somewhat simpler in that case so I'd prefer it.

Now, if we used a misc device, which minor would be suitable? 231?

> 4/4... I'm not sure. It would be nice to make swsusp.c disappear. It
> is really wrong name. That means we need to only delete from it for a
> while...

Anyway I think it would be nice to move the code that does not really belong
to the snapshot and is used by both the user interface and disk.c/swap.c to
a separate file. I have no preference as far as the name of the file is
concerned, though.

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

\
 
 \ /
  Last update: 2006-01-06 22:18    [W:0.173 / U:0.288 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site