From owner-freebsd-hackers@FreeBSD.ORG Mon May 2 16:24:59 2011 Return-Path: Delivered-To: freebsd-hackers@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 9AC35106566C for ; Mon, 2 May 2011 16:24:59 +0000 (UTC) (envelope-from yanegomi@gmail.com) Received: from mail-vw0-f54.google.com (mail-vw0-f54.google.com [209.85.212.54]) by mx1.freebsd.org (Postfix) with ESMTP id 4E3578FC18 for ; Mon, 2 May 2011 16:24:59 +0000 (UTC) Received: by vws18 with SMTP id 18so5764743vws.13 for ; Mon, 02 May 2011 09:24:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:in-reply-to:references:date :message-id:subject:from:to:content-type; bh=3jEQR6SsN/uwczRto1nOcDUdYCkhsnh+2EC1mESJy3Q=; b=JHXjIr0q/gUKNQ2VD2eO4JAhLHT1YzKPN6KSWfixN5kFGWfUoxbpKzOJR33GkbEW0f YwkoAu+ltq6vhLXAda57XGDzXA3vAQF698PA6eLbI5DNXTO93zqugrGDfv7EHS7+ZN/2 ksfGCCc0BQugut10tdX6HBS/NFcYeuBvgHuCI= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; b=bI3av7kD8sR1nRdebU22XWJalHbqfxWuMYgkTp0ARl2zKb/lloXqDt5x4NPh6XHy0m bsywWPbaK5Sycddfd5PbU+I7a+uRX6VDUgBhSIpaQBfXqs26jbVKmt+XcPiIS95Mo+Gn 3KCHdWG4gTjElz+0SF7PHH8Zue663MBZLmfWk= MIME-Version: 1.0 Received: by 10.220.203.13 with SMTP id fg13mr2419749vcb.11.1304353498593; Mon, 02 May 2011 09:24:58 -0700 (PDT) Received: by 10.220.199.130 with HTTP; Mon, 2 May 2011 09:24:58 -0700 (PDT) In-Reply-To: References: Date: Mon, 2 May 2011 09:24:58 -0700 Message-ID: From: Garrett Cooper To: freebsd-hackers@freebsd.org Content-Type: multipart/mixed; boundary=90e6ba53a7b4f1faaa04a24d763b Subject: Fwd: [PATCH] draft patch to make usr.bin/kdump WARNS?= 6 clean X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 02 May 2011 16:24:59 -0000 --90e6ba53a7b4f1faaa04a24d763b Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable ---------- Forwarded message ---------- From: Garrett Cooper Date: Mon, May 2, 2011 at 9:24 AM Subject: Re: [PATCH] draft patch to make usr.bin/kdump WARNS?=3D 6 clean To: Arnaud Lacombe On Mon, May 2, 2011 at 9:21 AM, Arnaud Lacombe wrote: > Hi, > > On Mon, May 2, 2011 at 12:10 PM, Garrett Cooper wrot= e: >> =A0 =A0I wanted to do something different this weekend, and I picked >> usr.bin/kdump as a likely 'victim' for converting from WARNS?=3D 0 to >> WARNS?=3D 6. I'm curious as to whether or not this is on the right >> track, but here's the reasoning I used: >> >> 1. Conditionally include diskmbr.h or diskpc98.h based on whether or >> not an architecture was non-pc98 or pc98 to avoid duplicate >> definitions, because the beforementioned headers are mutually >> exclusive. >> 2. Move the sockfamilyname declaration to kdump_subr.h as it's used in >> the generated ioctl.c file. >> 3. Fix a signed vs unsigned comparison with a simple cast because the >> size_t value will be sufficiently small that it can be converted to a >> signed comparison. >> 4. Fix a cast assignment type source//dest value alignment issue on >> ia64 assigning a struct sockaddr value to either struct sockaddr_in or >> struct sockaddr_in6 by using calloc and memcpy. >> 5. Fix structure alignment issues on arm by marking some structures as _= _packed. >> 6. Fix a shadowed declaration for flags by renaming a locally scoped >> variable to _flags; add appropriate type to field. >> 7. Remove unused argument to ktruser_malloc. >> 8. Add missing declarations for ktruser_malloc and ktruser_rtld. >> >> =A0 =A0I've run some basic tests and things seem sane (in particular >> ktrace'ing ktrace :)... ktrace'ing 'ssh ::1' and ktrace'ing 'ssh >> localhost', but I was wondering if there was anything I was missing or >> if someone else who ran arm or ia64 could test this patch out for me. >> =A0 =A0I've run make universe on amd64, i386, ia64, mips, and pc98, and >> things seem sane, but I can't play around with those machines to >> determine whether or not they're functional at runtime with the above >> changes. >> Thanks! >> -Garrett >> > I do not see any patch, either inline or attached. > > =A0- Arnaud > >> PS Oh yeah... no commit bit means that I can't commit this either, but >> I was curious if my approach was correct before getting to that step >> :). Yeah... I'm stupid for not attaching it. Need to get more sleep. -Garrett --90e6ba53a7b4f1faaa04a24d763b Content-Type: text/x-patch; charset=US-ASCII; name="usr-bin-kdump-WARNS-6-clean.patch" Content-Disposition: attachment; filename="usr-bin-kdump-WARNS-6-clean.patch" Content-Transfer-Encoding: base64 X-Attachment-Id: f_gn7md5980 SW5kZXg6IHVzci5iaW4va2R1bXAva2R1bXAuYwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSB1c3IuYmluL2tkdW1w L2tkdW1wLmMJKHJldmlzaW9uIDIyMTIxOSkKKysrIHVzci5iaW4va2R1bXAva2R1bXAuYwkod29y a2luZyBjb3B5KQpAQCAtOTksOCArOTksOSBAQAogdm9pZCBrdHJzb2NrYWRkcihzdHJ1Y3Qgc29j a2FkZHIgKik7CiB2b2lkIGt0cnN0YXQoc3RydWN0IHN0YXQgKik7CiB2b2lkIGt0cnN0cnVjdChj aGFyICosIHNpemVfdCk7Cit2b2lkIGt0cnVzZXJfbWFsbG9jKHVuc2lnbmVkIGNoYXIgKnApOwor dm9pZCBrdHJ1c2VyX3J0bGQoaW50IGxlbiwgdW5zaWduZWQgY2hhciAqcCk7CiB2b2lkIHVzYWdl KHZvaWQpOwotdm9pZCBzb2NrZmFtaWx5bmFtZShpbnQpOwogY29uc3QgY2hhciAqaW9jdGxuYW1l KHVfbG9uZyk7CiAKIGludCB0aW1lc3RhbXAsIGRlY2ltYWwsIGZhbmN5ID0gMSwgc3VwcHJlc3Nk YXRhLCB0YWlsLCB0aHJlYWRzLCBtYXhkYXRhLApAQCAtNTI4LDEzICs1MjksMTMgQEAKIAkJCQlp cCsrOwogCQkJCW5hcmctLTsKIAkJCX0gZWxzZSBpZiAoa3RyLT5rdHJfY29kZSA9PSBTWVNfb3Bl bikgewotCQkJCWludAlmbGFnczsKKwkJCQl1X2ludAlfZmxhZ3M7CiAJCQkJaW50CW1vZGU7CiAJ CQkJcHJpbnRfbnVtYmVyKGlwLG5hcmcsYyk7Ci0JCQkJZmxhZ3MgPSAqaXA7CisJCQkJX2ZsYWdz ID0gKmlwOwogCQkJCW1vZGUgPSAqKytpcDsKIAkJCQkodm9pZClwdXRjaGFyKCcsJyk7Ci0JCQkJ ZmxhZ3NhbmRtb2RlbmFtZSAoZmxhZ3MsIG1vZGUsIGRlY2ltYWwpOworCQkJCWZsYWdzYW5kbW9k ZW5hbWUgKF9mbGFncywgbW9kZSwgZGVjaW1hbCk7CiAJCQkJaXArKzsKIAkJCQluYXJnLT0yOwog CQkJfSBlbHNlIGlmIChrdHItPmt0cl9jb2RlID09IFNZU193YWl0NCkgewpAQCAtMTI1Niw3ICsx MjU3LDcgQEAKIH07CiAKIHZvaWQKLWt0cnVzZXJfbWFsbG9jKGludCBsZW4sIHVuc2lnbmVkIGNo YXIgKnApCitrdHJ1c2VyX21hbGxvYyh1bnNpZ25lZCBjaGFyICpwKQogewogCXN0cnVjdCB1dHJh Y2VfbWFsbG9jICp1dCA9IChzdHJ1Y3QgdXRyYWNlX21hbGxvYyAqKXA7CiAKQEAgLTEyODAsNyAr MTI4MSw3IEBACiAJfQogCiAJaWYgKGxlbiA9PSBzaXplb2Yoc3RydWN0IHV0cmFjZV9tYWxsb2Mp KSB7Ci0JCWt0cnVzZXJfbWFsbG9jKGxlbiwgcCk7CisJCWt0cnVzZXJfbWFsbG9jKHApOwogCQly ZXR1cm47CiAJfQogCkBAIC0xNDgwLDggKzE0ODEsOCBAQAogCWlmIChkYXRhbGVuID09IDApCiAJ CWdvdG8gaW52YWxpZDsKIAkvKiBzYW5pdHkgY2hlY2sgKi8KLQlmb3IgKGkgPSAwOyBpIDwgbmFt ZWxlbjsgKytpKQotCQlpZiAoIWlzYWxwaGEoKHVuc2lnbmVkIGNoYXIpbmFtZVtpXSkpCisJZm9y IChpID0gMDsgaSA8IChpbnQpbmFtZWxlbjsgKytpKQorCQlpZiAoIWlzYWxwaGEobmFtZVtpXSkp CiAJCQlnb3RvIGludmFsaWQ7CiAJaWYgKHN0cmNtcChuYW1lLCAic3RhdCIpID09IDApIHsKIAkJ aWYgKGRhdGFsZW4gIT0gc2l6ZW9mKHN0cnVjdCBzdGF0KSkKSW5kZXg6IHVzci5iaW4va2R1bXAv a2R1bXBfc3Vici5oCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIHVzci5iaW4va2R1bXAva2R1bXBfc3Vici5oCShy ZXZpc2lvbiAyMjEyMTkpCisrKyB1c3IuYmluL2tkdW1wL2tkdW1wX3N1YnIuaAkod29ya2luZyBj b3B5KQpAQCAtNDUsMyArNDUsNCBAQAogdm9pZCBtaW5oZXJpdG5hbWUgKGludCk7CiB2b2lkIHF1 b3RhY3RsbmFtZSAoaW50KTsKIHZvaWQgcHRyYWNlb3BuYW1lIChpbnQpOwordm9pZCBzb2NrZmFt aWx5bmFtZShpbnQpOwpJbmRleDogdXNyLmJpbi9rZHVtcC9ta2lvY3Rscwo9PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0t LSB1c3IuYmluL2tkdW1wL21raW9jdGxzCShyZXZpc2lvbiAyMjEyMTkpCisrKyB1c3IuYmluL2tk dW1wL21raW9jdGxzCSh3b3JraW5nIGNvcHkpCkBAIC0yMiwxMiArMjIsMjEgQEAKICMgWFhYIHNo b3VsZCB3ZSB1c2UgYW4gQU5TSSBjcHA/CiBpb2N0bF9pbmNsdWRlcz1gCiAJY2QgJDEKLQlmaW5k IC1IIC1zICogLW5hbWUgJyouaCcgfAorCWZpbmQgLUggLXMgKiAtbmFtZSAnKi5oJyAtYW5kIFwo IC1uYW1lICEgJ2Rpc2ttYnIuaCcgLW9yIC1uYW1lICEgJ2Rpc2twYzk4LmgnIFwpIHwKIAkJeGFy Z3MgZWdyZXAgLWwgXAogJ14jWyAJXSpkZWZpbmVbIAldK1tBLVphLXpfXVtBLVphLXowLTlfXSpb IAldK19JT1teYS16MC05X10nIHwKIAkJYXdrICd7cHJpbnRmKCIjaW5jbHVkZSA8JXM+XFxcXG4i LCAkMSl9JwogYAogCitjYXNlICJgdW5hbWUgLW1gIiBpbgorKnBjOTgqKQorCWlvY3RsX2luY2x1 ZGVzPSIkaW9jdGxfaW5jbHVkZXMjaW5jbHVkZSA8c3lzL2Rpc2twYzk4Lmg+XFxuIgorCTs7Cisq KQorCWlvY3RsX2luY2x1ZGVzPSIkaW9jdGxfaW5jbHVkZXMjaW5jbHVkZSA8c3lzL2Rpc2ttYnIu aD5cXG4iCisJOzsKK2VzYWMKKwogYXdrIC12IHg9IiRpb2N0bF9pbmNsdWRlcyIgJ0JFR0lOIHtw cmludCB4fScgfAogCWdjYyAtRSAtSSQxIC1kTSAtRENPTVBBVF80M1RUWSAtIHwKIAlhd2sgLXYg aW9jdGxfaW5jbHVkZXM9IiRpb2N0bF9pbmNsdWRlcyIgLXYgdXNlX3N3aXRjaD0iJHVzZV9zd2l0 Y2giICcKSW5kZXg6IHVzci5iaW4va2R1bXAvTWFrZWZpbGUKPT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gdXNyLmJp bi9rZHVtcC9NYWtlZmlsZQkocmV2aXNpb24gMjIxMjE5KQorKysgdXNyLmJpbi9rZHVtcC9NYWtl ZmlsZQkod29ya2luZyBjb3B5KQpAQCAtMTUsNyArMTUsNyBAQAogU1JDUys9CQlsaW51eF9zeXNj YWxscy5jCiAuZW5kaWYKIAotV0FSTlM/PQkJMAorV0FSTlM/PQkJNgogCiBDTEVBTkZJTEVTPQlp b2N0bC5jIGtkdW1wX3N1YnIuYyBsaW51eF9zeXNjYWxscy5jCiAK --90e6ba53a7b4f1faaa04a24d763b--