netdev - Re: [PATCH 02/15] i40iw: add main, hdr, status

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: <1450297873.24782.7.camel@perches.com>
Date:	Wed, 16 Dec 2015 12:31:13 -0800
From:	Joe Perches <joe@...ches.com>
To:	Faisal Latif <faisal.latif@...el.com>, dledford@...hat.com
Cc:	linux-rdma@...r.kernel.org, netdev@...r.kernel.org,
	jeffrey.t.kirsher@...el.com, e1000-rdma@...ts.sourceforge.net
Subject: Re: [PATCH 02/15] i40iw: add main, hdr, status

On Wed, 2015-12-16 at 13:58 -0600, Faisal Latif wrote:
> i40iw_main.c contains routines for i40e <=> i40iw interface and setup.
> i40iw.h is header file for main device data structures.
> i40iw_status.h is for return status codes.
[]
> diff --git a/drivers/infiniband/hw/i40iw/i40iw.h b/drivers/infiniband/hw/i40iw/i40iw.h
[]
> +#define i40iw_pr_err(fmt, args ...) pr_err("%s: error " fmt, __func__, ## args)
> +
> +#define i40iw_pr_info(fmt, args ...) pr_info("%s: " fmt, __func__, ## args)
> +
> +#define i40iw_pr_warn(fmt, args ...) pr_warn("%s: " fmt, __func__, ## args)

Using "error " in the output doesn't really add much
as there's already a KERN_ERR with the output.

Using __func__ hardly adds anything.

Using netdev_<level> is generally preferred

> +
> +struct i40iw_cqp_request {
> +	struct cqp_commands_info info;
> +	wait_queue_head_t waitq;
> +	struct list_head list;
> +	atomic_t refcount;
> +	void (*callback_fcn)(struct i40iw_cqp_request*, u32);
> +	void *param;
> +	struct i40iw_cqp_compl_info compl_info;
> +	u8 waiting:1;
> +	u8 request_done:1;
> +	u8 dynamic:1;
> +	u8 polling:1;

These would bitfields might be better as bool

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html