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
| ||
|
Date: Mon, 16 Jul 2007 10:37:09 -0700 From: Roland Dreier <rdreier@...co.com> To: Joachim Fenkes <fenkes@...ibm.com> Cc: "LinuxPPC-Dev" <linuxppc-dev@...abs.org>, LKML <linux-kernel@...r.kernel.org>, "OF-General" <general@...ts.openfabrics.org>, Roland Dreier <rolandd@...co.com>, "Hoang-Nam Nguyen" <hnguyen@...ibm.com>, Christoph Raisch <raisch@...ibm.com>, Stefan Roscher <stefan.roscher@...ibm.com> Subject: Re: [PATCH 10/10] IB/ehca: Support large page MRs > Add support for MR pages larger than 4K on eHCA2. This reduces firmware > memory consumption. If enabled via the mr_largepage module parameter, the MR > page size will be determined based on the MR length and the hardware > capabilities - if the MR is >= 16M, 16M pages are used, for example. Why the module parameter? Is there any reason a user would want to turn this off? Or conversely, why is it off by default? Also this patch seems to depend heavily on the multiple EQ patch, which I am holding off on now. So you may want to rebase to my current tree, which has all the ehca patches except the EQ one. > static ssize_t ehca_show_nr_eqs(struct device *dev, > struct device_attribute *attr, > char *buf) > { > return sprintf(buf, "%d\n", ehca_nr_eqs); > } > - > static DEVICE_ATTR(nr_eqs, S_IRUGO, ehca_show_nr_eqs, NULL); Although trivial, this chunk doesn't really belong in this patch -- just fix it up in the multiple EQ patch (which I haven't merged yet). - R. - 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/
Powered by blists - more mailing lists