linux-kernel - Re: [PATCH] infiniband: hw: amso1100: Fixed compile warning.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <AANLkTimLbkqmkijkLk+kALOUK6TVB0AJA3Ca+X-wcjpQ@mail.gmail.com>
Date:	Mon, 24 Jan 2011 20:42:36 +0100
From:	Bart Van Assche <bvanassche@....org>
To:	Ralf Thielow <ralf.thielow@...glemail.com>
Cc:	tom@...ngridcomputing.com, swise@...ngridcomputing.com,
	linux-rdma@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] infiniband: hw: amso1100: Fixed compile warning.

On Sun, Jan 23, 2011 at 5:30 PM, Ralf Thielow
<ralf.thielow@...glemail.com> wrote:
>
> Fixed compile warning by use "unsigned long" instead of "u64"
> on assign NULL to "c2_vq_req->reply_msg".
>
> Signed-off-by: Ralf Thielow <ralf.thielow@...glemail.com>
> ---
>  drivers/infiniband/hw/amso1100/c2_vq.c |    6 +++---
>  1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/infiniband/hw/amso1100/c2_vq.c b/drivers/infiniband/hw/amso1100/c2_vq.c
> index 9ce7819..807b730 100644
> --- a/drivers/infiniband/hw/amso1100/c2_vq.c
> +++ b/drivers/infiniband/hw/amso1100/c2_vq.c
> @@ -107,7 +107,7 @@ struct c2_vq_req *vq_req_alloc(struct c2_dev *c2dev)
>        r = kmalloc(sizeof(struct c2_vq_req), GFP_KERNEL);
>        if (r) {
>                init_waitqueue_head(&r->wait_object);
> -               r->reply_msg = (u64) NULL;
> +               r->reply_msg = (unsigned long) NULL;
>                r->event = 0;
>                r->cm_id = NULL;
>                r->qp = NULL;
> @@ -123,7 +123,7 @@ struct c2_vq_req *vq_req_alloc(struct c2_dev *c2dev)
>  */
>  void vq_req_free(struct c2_dev *c2dev, struct c2_vq_req *r)
>  {
> -       r->reply_msg = (u64) NULL;
> +       r->reply_msg = (unsigned long) NULL;
>        if (atomic_dec_and_test(&r->refcnt)) {
>                kfree(r);
>        }
> @@ -151,7 +151,7 @@ void vq_req_get(struct c2_dev *c2dev, struct c2_vq_req *r)
>  void vq_req_put(struct c2_dev *c2dev, struct c2_vq_req *r)
>  {
>        if (atomic_dec_and_test(&r->refcnt)) {
> -               if (r->reply_msg != (u64) NULL)
> +               if (r->reply_msg != (unsigned long) NULL)
>                        vq_repbuf_free(c2dev,
>                                       (void *) (unsigned long) r->reply_msg);
>                kfree(r);

Hello Ralf,

Have you considered using "0" instead of "(u64) NULL" or "(unsigned
long) NULL" ?

Bart.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/