linux-kernel - Re: [PATCH]ibft: Fix the display of a few fields in the NIC attribute structure in sysfs

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: <20090415143744.84d6c598.akpm@linux-foundation.org>
Date:	Wed, 15 Apr 2009 14:37:44 -0700
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	ketuzsezr@...nok.org
Cc:	James.Bottomley@...senpartnership.com, ashutosh.naik@...il.com,
	linux-kernel@...r.kernel.org, linux-scsi@...r.kernel.org,
	pjones@...hat.com, konradr@...ux.vnet.ibm.com, vishnu@...lsio.com
Subject: Re: [PATCH]ibft: Fix the display of a few fields in the NIC
 attribute  structure in sysfs

On Sun, 12 Apr 2009 09:06:59 -0400
Konrad Rzeszutek <ketuzsezr@...nok.org> wrote:

> > The second question is who picks this up?  The drivers/firmware
> > directory doesn't seem to have a dedicated tree, so I can take this
> > through the SCSI tree if required or Andrew can take it through -mm.
> 
> Which way works best for everybody? I was thinking SCSI, since the end result 
> of this module is to configure the storage used to bootstrap the box.

I can merge merge it later this week.

However...


: From: Ashutosh Naik <ashutosh.naik@...il.com>
: 
: Fix the display of a few fields in the iBFT NIC attribute structure in
: sysfs.
: 
: Signed-off-by: Ashutosh Naik <ashutosh.naik@...il.com>
: Cc: Vishnu V <vishnu@...lsio.com>
: Signed-off-by: Andrew Morton <akpm@...ux-foundation.org>
: ---
: 
:  drivers/firmware/iscsi_ibft.c |    4 ++--
:  1 file changed, 2 insertions(+), 2 deletions(-)
: 
: diff -puN drivers/firmware/iscsi_ibft.c~ibft-fix-the-display-of-a-few-fields-in-the-nic-attribute-structure-in-sysfs drivers/firmware/iscsi_ibft.c
: --- a/drivers/firmware/iscsi_ibft.c~ibft-fix-the-display-of-a-few-fields-in-the-nic-attribute-structure-in-sysfs
: +++ a/drivers/firmware/iscsi_ibft.c
: @@ -754,11 +754,11 @@ static int __init ibft_check_nic_for(str
:  		rc = 1;
:  		break;
:  	case ibft_eth_ip_addr:
: -		if (!memcmp(nic->dhcp, nulls, sizeof(nic->dhcp)))
: +		if (memcmp(nic->ip_addr, nulls, sizeof(nic->ip_addr)))
:  			rc = 1;
:  		break;
:  	case ibft_eth_subnet_mask:
: -		if (!memcmp(nic->dhcp, nulls, sizeof(nic->dhcp)))
: +		if (nic->subnet_mask_prefix)
:  			rc = 1;
:  		break;
:  	case ibft_eth_origin:

The changelog is bad.

What does "fix" mean?  What does the old output look like and how does
this patch change it?

This matters, because this is a userspace interface and who knows, there
might be applications out there which somehow depend on the old and
broken behaviour.

Of course, that might be totally silly and unrealistic but because we
weren't told what the patch actually does, how can I tell?

Please send a new and complete description of this change, thanks.

--
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/