Linux PARISC architecture development
 help / color / mirror / Atom feed
From: Grant Grundler <grundler@dsl2.external.hp.com>
To: Bruno Vidal <bruno_vidal@hpfrcu03.france.hp.com>
Cc: "parisc-linux@lists.parisc-linux.org"
	<parisc-linux@lists.parisc-linux.org>
Subject: Re: [parisc-linux] dump driver for parisc-linux: patch vs CVS
Date: Mon, 04 Feb 2002 10:40:39 -0700	[thread overview]
Message-ID: <20020204174039.DAF3B482A@dsl2.external.hp.com> (raw)
In-Reply-To: Message from Bruno Vidal <bruno_vidal@hpfrcu03.france.hp.com> of "Mon, 04 Feb 2002 12:54:50 +0100." <3C5E768A.573CC5A2@admin.france.hp.com>

Bruno Vidal wrote:
> 	Hi.
> I've a dump driver available for 2.4.17-pa3 and I'll
> upgrade my source to latest CVS bits and redo the 
> modifications. But do you think it is possible to
> put in the CVS these modifications ? 


As Randolph noted, we need to see the diff and what was changed
in the generic kernel. Is the kernel and user space code GPL?

> I have modified:
> ./Makefile 
> ./arch/parisc/config.in
> ./arch/parisc/kernel/traps.c
> ./arch/parisc/mm/init.c
> ./drivers/Makefile 
> ./drivers/char/sysrq.c
> ./init/main.c
> ./kernel/ksyms.c
> ./kernel/panic.c
> ./kernel/sched.c

The last four might cause trouble....

> 
> And I've added:
> ./include/linux/dump.h 
> ./drivers/dump/Makefile
> ./drivers/dump/dump_base.c
> ./drivers/dump/dump_rle.c
> ./drivers/dump/dump_parisc.c

and we need to ask 2.4 maintainer if this can be added upstream too...

> So my question: 
> -Do you think it is a good idea to have a dump driver available 
> in the standart kernel ?

I do. But it's really up to linus/et al to decide that.
I can choose to commit this code to *our* tree (parisc-linux) only.
(and that's really not just my decision - need some consensus)

> -If yes, who is able to do the code review, and put this dump 
> module in the CVS ?

Ideally, I'd like to see outstanding non-arch code changes merged
upstream first..  Otherwise, I don't mind maintaining that code in our tree.

thanks!
grant

  parent reply	other threads:[~2002-02-04 17:40 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-02-04 11:54 [parisc-linux] dump driver for parisc-linux: patch vs CVS Bruno Vidal
2002-02-04 15:38 ` Randolph Chung
2002-02-04 17:40 ` Grant Grundler [this message]
2002-02-05  7:53   ` Bruno Vidal
2002-02-06  2:09     ` Randolph Chung
2002-02-06  7:32     ` Grant Grundler

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20020204174039.DAF3B482A@dsl2.external.hp.com \
    --to=grundler@dsl2.external.hp.com \
    --cc=bruno_vidal@hpfrcu03.france.hp.com \
    --cc=parisc-linux@lists.parisc-linux.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox