Re: [PATCH v2] nfs-utils: Support binding to source address.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 06/09/2011 10:18 AM, Steve Dickson wrote:
Hello,


On 06/08/2011 07:01 PM, greearb@xxxxxxxxxxxxxxx wrote:
From: Ben Greear<greearb@xxxxxxxxxxxxxxx>

This lets one specify the source IP address for
sockets, allowing users to leverage routing rules
on multi-homed systems.
Right out of the box I do think some support like
this would be good... but I not convinced on how needed
something like this is which means there is a lot of
change here for basically not... But lets continue
to talk about to see where we end up....

One thing you must do is break up this patch in to
a more readable patch series. There is no way I going
to commit a patch that is close to 1500 lines
that so invasive.

The vast majority of the patch is adding an extra
argument to various methods to get things to compile.

Do you want me to break those changes into individual
chunks, or should I just separate the parsing & documentation
chunks out?

I can dummy stuff out with passing NULL around to make
lots of smaller patches, but I'm not sure it helps
any readability.

Thanks,
Ben


--
Ben Greear <greearb@xxxxxxxxxxxxxxx>
Candela Technologies Inc  http://www.candelatech.com

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


[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux