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

[Max Reitz <max@xxxxxxxxxx>]

On 16.11.2013 01:32, Kevin Wolf wrote:
> 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

Aber dann hätte ich ja kernel1 fixen müssen. *g*

Max