lkml.org 
[lkml]   [2008]   [Jun]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [patch 2/3] Add flags parameter to reserve_bootmem_generic()
    * Johannes Weiner [2008-06-09 00:01]:
    >
    > > /*
    > > @@ -811,11 +812,11 @@ void __init reserve_bootmem_generic(unsi
    > > * firmware tables:
    > > */
    > > if (pfn < max_pfn_mapped)
    > > - return;
    > > + return -EFAULT;
    >
    > This seemed to be `just do nothing' behaviour. Wouldn't 0 be more
    > correct here? Or something else so there is a difference between the
    > path that does not print a warning (the one below) and the path that
    > does?

    Well, I don't think that we should return success when memory
    allocation fails. For kdump, I think if the memory has not been
    reserved, then the function should failed, for whatever reason it
    failed. Because we cannot load the crashkernel.

    So IMO the code should look like

    [...]
    int ret;

    if (pfn >= end_pfn)
    return -EFAULT;

    /* Should check here against the e820 map to avoid double free */
    #ifdef CONFIG_NUMA
    nid = phys_to_nid(phys);
    [...]


    Bernhard
    --
    Bernhard Walle, SUSE LINUX Products GmbH, Architecture Maintenance


    \
     
     \ /
      Last update: 2008-06-09 18:27    [W:0.022 / U:0.076 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site