From owner-cvs-all Fri Jan 11 14:41: 3 2002 Delivered-To: cvs-all@freebsd.org Received: from flood.ping.uio.no (flood.ping.uio.no [129.240.78.31]) by hub.freebsd.org (Postfix) with ESMTP id 7D18837B41C; Fri, 11 Jan 2002 14:40:48 -0800 (PST) Received: by flood.ping.uio.no (Postfix, from userid 2602) id D842114C53; Fri, 11 Jan 2002 23:40:46 +0100 (CET) X-URL: http://www.ofug.org/~des/ X-Disclaimer: The views expressed in this message do not necessarily coincide with those of any organisation or company with which I am or have been affiliated. To: Alfred Perlstein Cc: Marcel Moolenaar , cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/compat/linux linux_ioctl.c References: <200201100536.g0A5aao01807@freefall.freebsd.org> <20020110191209.A318@dhcp01.pn.xcllnt.net> <20020110203918.A440@dhcp01.pn.xcllnt.net> <20020111112851.A2103@kayak.xcllnt.net> <20020111122534.K7984@elvis.mu.org> From: Dag-Erling Smorgrav Date: 11 Jan 2002 23:40:46 +0100 In-Reply-To: <20020111122534.K7984@elvis.mu.org> Message-ID: Lines: 39 User-Agent: Gnus/5.0808 (Gnus v5.8.8) Emacs/21.1 MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG Alfred Perlstein writes: > * Dag-Erling Smorgrav [020111 12:20] wrote: > > Since when does "talking it over first" mean "commit a patch without > > discussing it with the person who wrote the code it changes"? > > Especially when similar patches have been suggested, discussed, and > > rejected before (see for instance the audit trail for PR 26546). > You don't give a reason for your: > "Partial fix committed, more work remains to be done." See the log entry for revision 1.68 of linux_ioctl.c, and succeeding revisions. > This should have included _why_ you didn't incorperate the > diffs instead of just stating that you didn't. I *did* incorporate the diffs in the PR, with modifications, but they did not fully solve the problem - they were just a hack to get the aironet config utility working. I busted my ass for two weeks, working literally night and day, to get interface ioctls to work under Linux emulation - not just the three or four ioctls the patch in the PR fixes, but as many as I could actually test with the hardware and software I have, to the point where a Linux ifconfig(8) binary can now correctly list all interfaces, their Ethernet and IP addresses, and all the interface flags for which I could find a meaningful counterpart in Linux. The sparse interface numbering is a cosmetic nit, which I did not consider important enough to warrant the pain of fixing it properly. It apparently trips up at least two Linux applications, but nobody bothered to submit a PR about it, so I've never had the chance to actually analyze the problem. > It would make both the PR originator as well as fellow developers > understand why and how you are doing things. Both ambrisko (who submitted the PR) and marcel were involved in the development of the patches I *did* commit. DES -- Dag-Erling Smorgrav - des@ofug.org To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message