On Mon, Feb 29, 2016 at 1:45 PM, Mike Christie <michaelc@xxxxxxxxxxx> wrote: > On 02/25/2016 12:15 PM, Lee Duncan wrote: >> From: Hannes Reinecke <hare@xxxxxxx> >> >> The iBFT table only specifies a prefix length, not a netmask. >> And the netmask is pretty much pointless for IPv6. >> So introduce a new attribute 'prefix-len' and display the >> netmask attribute only for IPv4 addresses. > > The code looks ok to me, but I think this ipv4 comment was left over > from the last posting since it conflicts with the comment below about > always displaying it. Maybe it can just be edited when the patch is > merged to avoid having to resend again. If so, > > Reviewed-by: Mike Christie <michaelc@xxxxxxxxxxx> > Cool. Let me roll it up and send the git pull to Linus. This does not seem be super-urgent so it can wait for the upcoming merge window? > >> >> Some older user-space code might rely on the netmask attribute >> being present, so we should always display it. >> >> Changes from v1: >> - Combined two patches into one >> >> Signed-off-by: Hannes Reinecke <hare@xxxxxxx> >> Signed-off-by: Lee Duncan <lduncan@xxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
