Re: [PATCH] neaten lguest boot code (again)

From: Andrew Morton
Date: Tue Jun 05 2007 - 22:48:50 EST


On Wed, 06 Jun 2007 12:37:39 +1000 Rusty Russell <rusty@xxxxxxxxxxxxxxx> wrote:

> (This cleanup seems to have gotten lost in rc3-mm1? It was in rc2-mm1
> called lguest-the-host-code-update-for-mm-simplify-boot_params.patch)
>
> Andrew patched up lguest after the boot parameters became a proper
> structure, but in fact it can be considerably neatened.
>
> Signed-off-by: Rusty Russell <rusty@xxxxxxxxxxxxxxx>
> ---
> drivers/lguest/lguest.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> ===================================================================
> --- a/drivers/lguest/lguest.c
> +++ b/drivers/lguest/lguest.c
> @@ -444,8 +444,7 @@ __init void lguest_init(void *boot)
> {
> /* Copy boot parameters first. */
> memcpy(&boot_params, boot, PARAM_SIZE);
> - memcpy(boot_command_line,
> - __va(*(unsigned long *)((void *)&boot_params + NEW_CL_POINTER)),
> + memcpy(boot_command_line, __va(boot_params.hdr.cmd_line_ptr),
> COMMAND_LINE_SIZE);
>
> paravirt_ops.name = "lguest";

I dropped it because it depended upon mm-simplify-boot_params.patch which
for some reason disappeared but now appears to have reappeared.

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