From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933910AbZHEA35 (ORCPT ); Tue, 4 Aug 2009 20:29:57 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S933871AbZHEA3y (ORCPT ); Tue, 4 Aug 2009 20:29:54 -0400 Received: from tundra.namei.org ([65.99.196.166]:49525 "EHLO tundra.namei.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933870AbZHEA3w (ORCPT ); Tue, 4 Aug 2009 20:29:52 -0400 Date: Wed, 5 Aug 2009 10:29:29 +1000 (EST) From: James Morris To: David Howells cc: torvalds@osdl.org, akpm@linux-foundation.org, serue@us.ibm.com, linux-kernel@vger.kernel.org, linux-security-module@vger.kernel.org Subject: Re: [PATCH 4/6] KEYS: Add garbage collection for dead, revoked and expired keys. [ver #2] In-Reply-To: <20090804203938.25094.21014.stgit@warthog.procyon.org.uk> Message-ID: References: <20090804203923.25094.78702.stgit@warthog.procyon.org.uk> <20090804203938.25094.21014.stgit@warthog.procyon.org.uk> User-Agent: Alpine 2.00 (LRH 1167 2008-08-23) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 4 Aug 2009, David Howells wrote: > +void key_schedule_gc(time_t gc_at) > +{ > + unsigned long expires; > + time_t now = current_kernel_time().tv_sec; > + > + kenter("%ld", gc_at - now); > + > + gc_at += key_gc_delay; > + > + if (now >= gc_at) { Should you be using time_after() and friends for these comparisons? -- James Morris