Re: [tyndur-devel] [PATCH] kernel: Parameterdreher bei memset gefixt

[Kevin Wolf <kevin@xxxxxxxxxx>]

On Fri, Nov 15, 2013 at 02:28:10PM +0100, Max Reitz wrote:
> On 15.11.2013 01:13, Stefan Schmidt wrote:
> >     ! kernel: memset erwartet (addr, value, len), nicht (addr, len, value)
> >
> > Signed-off-by: Stefan Schmidt <thrimbor.github@xxxxxxxxx>
> > ---
> >  src/kernel/src/mm/kmm.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/src/kernel/src/mm/kmm.c b/src/kernel/src/mm/kmm.c
> > index 0246abc..65cc99d 100644
> > --- a/src/kernel/src/mm/kmm.c
> > +++ b/src/kernel/src/mm/kmm.c
> > @@ -249,7 +249,7 @@ vaddr_t alloc_kernel_pages(int num)
> >              page_table = (page_table_t)(KERNEL_PAGE_TABLES_VADDR + (i << PAGE_SHIFT));
> >  
> >              // Page Table mit Nullen initialisieren
> > -            memset(page_table, PAGE_SIZE, 0);
> > +            memset(page_table, 0, PAGE_SIZE);
> >  
> >              kernel_pd_id++;
> >          }

Danke Stefan, der Patch ist jetzt drin.

> Na ja, nicht, dass der alte Kernel noch gepflegt wird, aber es macht ja
> auch Spaß, mal wieder ein Ack senden zu dürfen. ;-)

Um genau zu sein, habe ich ihn nicht einmal mehr kompilieren können, um
den Patch wenigstens einmal durchzubauen. Ich hab da gleich noch ein
paar Fixes hinterhergeschoben; nicht, dass kernel1 dadurch wieder
bootfähig wäre (mindestens die LIOv2-Syscalls fehlen), aber immerhin
baut er wieder.

Du hättest übrigens auch einfach committen dürfen, Max. ;-)

Kevin