Re: [OE-core] [PATCH 08/20] udev-cache: Create cache asynchronously

2014-08-04 Thread Richard Tollerton
Otavio Salvador writes: > On Mon, Aug 4, 2014 at 6:37 PM, Richard Tollerton > wrote: >> Otavio Salvador writes: >> >>> On Mon, Aug 4, 2014 at 6:18 PM, Richard Tollerton >>> wrote: Otavio Salvador writes: > I am not sure about this one. I see the value you are adding here but >

Re: [OE-core] [PATCH 08/20] udev-cache: Create cache asynchronously

2014-08-04 Thread Otavio Salvador
On Mon, Aug 4, 2014 at 6:37 PM, Richard Tollerton wrote: > Otavio Salvador writes: > >> On Mon, Aug 4, 2014 at 6:18 PM, Richard Tollerton >> wrote: >>> Otavio Salvador writes: >>> I am not sure about this one. I see the value you are adding here but I worry how often something can be

Re: [OE-core] [PATCH 08/20] udev-cache: Create cache asynchronously

2014-08-04 Thread Richard Tollerton
Otavio Salvador writes: > On Mon, Aug 4, 2014 at 6:18 PM, Richard Tollerton > wrote: >> Otavio Salvador writes: >> >>> I am not sure about this one. I see the value you are adding here but >>> I worry how often something can be connected during this process and >>> change the contents along th

Re: [OE-core] [PATCH 08/20] udev-cache: Create cache asynchronously

2014-08-04 Thread Otavio Salvador
On Mon, Aug 4, 2014 at 6:18 PM, Richard Tollerton wrote: > Otavio Salvador writes: > >> I am not sure about this one. I see the value you are adding here but >> I worry how often something can be connected during this process and >> change the contents along the way. Did you see something as that

Re: [OE-core] [PATCH 08/20] udev-cache: Create cache asynchronously

2014-08-04 Thread Richard Tollerton
Otavio Salvador writes: > I am not sure about this one. I see the value you are adding here but > I worry how often something can be connected during this process and > change the contents along the way. Did you see something as that > during your tests? No, but point taken. I didn't really test

Re: [OE-core] [PATCH 08/20] udev-cache: Create cache asynchronously

2014-08-04 Thread Otavio Salvador
On Mon, Aug 4, 2014 at 3:41 PM, Ben Shelton wrote: > From: Richard Tollerton > > Don't hold up the boot while the cache tarball is created. > > Natinst-Rally-ID: TA44427 > Acked-by: Gratian Crisan > Natinst-ReviewBoard-ID: 58620 > Signed-off-by: Richard Tollerton I am not sure about this one.

[OE-core] [PATCH 08/20] udev-cache: Create cache asynchronously

2014-08-04 Thread Ben Shelton
From: Richard Tollerton Don't hold up the boot while the cache tarball is created. Natinst-Rally-ID: TA44427 Acked-by: Gratian Crisan Natinst-ReviewBoard-ID: 58620 Signed-off-by: Richard Tollerton --- meta/recipes-core/udev/udev/udev-cache | 8 +--- 1 file changed, 5 insertions(+), 3 dele