All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Robert Yang <liezhi.yang@windriver.com>
Cc: bitbake-devel@lists.openembedded.org
Subject: Re: [PATCH 0/3] V5 Disk space monitoring
Date: Sun, 26 Feb 2012 11:08:31 +0000	[thread overview]
Message-ID: <1330254511.13788.61.camel@ted> (raw)
In-Reply-To: <cover.1330245845.git.liezhi.yang@windriver.com>

On Sun, 2012-02-26 at 16:48 +0800, Robert Yang wrote:
> Hi Richard,
> 
> Thank you very much for your detailed review, here is V5,
> please take your review.
> 
> Changes:
> 
> * Add the patch "Fix finish_now when no running task" which is from you.
> 
> * Change the code as you suggested.
> 
> * Change the default disk interval value from "10M,50" to "50M,5K",
>   this seems a reasonable value during the testing(not too many or
>   few warnings).
> 
> * Add more explanations in local.conf.sample
> 
> // Robert
> 
> The following changes since commit 2ff96d664e8e003f64de42187fce9ad0f9646070:
> 
>   crumbs: add back progress implementation for depexp (2012-02-25 11:55:45 +0000)
> 
> are available in the git repository at:
>   git://git.pokylinux.org/poky-contrib robert/disk-space-monitor
>   http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=robert/disk-space-monitor
> 
> Robert Yang (3):
>   Fix finish_now when no running task
>   Disk space monitoring
>   Add config sample for disk space monitoring

Merged to master, thanks. I applied the config sample to
local.conf.sample.extended though.

Cheers,

Richard




      parent reply	other threads:[~2012-02-26 11:17 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-26  8:48 [PATCH 0/3] V5 Disk space monitoring Robert Yang
2012-02-26  8:48 ` [PATCH 1/3] V5 Fix finish_now when no running task Robert Yang
2012-02-26  8:48 ` [PATCH 2/3] V5 Disk space monitoring Robert Yang
2012-02-26  8:48 ` [PATCH 3/3] V5 Add config sample for disk " Robert Yang
2012-02-26 11:08 ` Richard Purdie [this message]

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=1330254511.13788.61.camel@ted \
    --to=richard.purdie@linuxfoundation.org \
    --cc=bitbake-devel@lists.openembedded.org \
    --cc=liezhi.yang@windriver.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.