lkml.org 
[lkml]   [2015]   [Dec]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH v2 1/2] scripts: Add a recorduidiv program
    From
    Date
    On 2015-11-30 16:40, Michal Marek wrote:
    > On 2015-11-30 16:32, Russell King - ARM Linux wrote:
    >> On Mon, Nov 30, 2015 at 04:11:16PM +0100, Michal Marek wrote:
    >>> On 2015-11-26 00:47, Russell King - ARM Linux wrote:
    >>>> Do we have any resolution on these programs which modify the object
    >>>> files in-place, rather than breaking any hard-links which may be
    >>>> present (eg, as a result of using ccache in hard-link mode) ?
    >>>
    >>> Good point, but I do not think anybody is using CCACHE_HARDLINK with the
    >>> kernel.
    >>
    >> That's wrong then, because I've been using it for a very long time with
    >> my nightly builds. :) Therefore, there is somebody!
    >
    > OK.

    So, both recordmcount and the new recordudiv program are idempotent.
    They check if the to-be-added section is already present and do nothing.
    So the result is correct even with CCACHE_HARDLINK, just the
    intermediate file might be incorrect. If this still is considered an
    issue, I suggest clearing CCACHE_HARDLINK when using any of these
    postprocessors, so as not to penalize other use cases.

    The perl recordmcount does not modify the file in place.

    Michal


    \
     
     \ /
      Last update: 2015-12-01 17:21    [W:5.664 / U:0.188 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site