From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alan Jenkins Date: Fri, 05 Sep 2008 15:08:55 +0000 Subject: Re: [PATCH] threadsafe rules iteration Message-Id: <48C14B87.5060406@tuffmail.co.uk> List-Id: References: <48C113FC.3080203@tuffmail.co.uk> In-Reply-To: <48C113FC.3080203@tuffmail.co.uk> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: linux-hotplug@vger.kernel.org Alan Jenkins wrote: > Move ->current out of struct udev_rules and into a new struct udev_rules_iter. > > Signed-off-by: Alan Jenkins > > diff --git a/udev/udev_rules.c b/udev/udev_rules.c > index da7a62a..c21d762 100644 > --- a/udev/udev_rules.c > +++ b/udev/udev_rules.c > > @@ -1561,14 +1562,15 @@ int udev_rules_get_name(struct udev_rules *rules, struct udevice *udev) > > int udev_rules_get_run(struct udev_rules *rules, struct udevice *udev) > { > + struct udev_rules_iter *iter; > Sorry, don't apply that. It should be struct udev_rules_iter iter; I think we should be building with -Werror, but I'm not sure how to tell autoconf/automake. udev_rules.c: In function ‘udev_rules_get_run’: udev_rules.c:1571: warning: passing argument 1 of ‘udev_rules_iter_init’ from incompatible pointer type udev_rules.c:1573: warning: passing argument 1 of ‘udev_rules_iter_next’ from incompatible pointer type udev_rules.c:1624: warning: passing argument 1 of ‘udev_rules_iter_label’ from incompatible pointer type