Bug 191745
Summary: | Review Request: sysprof-kmod - kernel module for the sysprof profiler | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Gianluca Sforna <giallu> |
Component: | Package Review | Assignee: | Parag AN(पराग) <panemade> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | j, panemade, sandmann |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2006-10-08 08:53:09 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 163779, 191743, 208687 |
Description
Gianluca Sforna
2006-05-15 15:37:24 UTC
I updated the spec to remove some diffs from the standard template: Spec URL: http://giallu.interfree.it/fedora/sysprof-kmod.spec SRPM URL: http://giallu.interfree.it/fedora/sysprof-kmod-1.0.2-2.2.6.16_1.2111_FC5.src.rpm Some comments:- rpmlint sysprof-kmod-1.0.2-2.2.6.16_1.2111_FC5.src.rpm gives output as W: sysprof-kmod summary-not-capitalized sysprof kernel module W: sysprof-kmod strange-permission kmodtool 0666 E: sysprof-kmod configure-without-libdir-spec I am leaving for a couple of weeks, so I can not look into this until I come back. thanks for your patience... Here I am again. I updated to 1.0.3 and silenced the first warning. Permissions on kmodtool seems to be the same as other kmod packages, so I think I will leave it alone. The error is a false positive, since I completely skip configure (see comment in the spec) for the kmod build. Spec URL: http://giallu.interfree.it/fedora/sysprof-kmod.spec SRPM URL: http://giallu.interfree.it/fedora/sysprof-kmod-1.0.3-1.2.6.16_1.2133_FC5.src.rpm According to http://fedoraproject.org/wiki/Packaging/KernelModules you will need to provide certain information to the Extras steering committee, in particular: A publishable explanation from the author(s) why the module is not merged with the mainline kernel yet and when it's planned to get merged. Could you please provide this information so that the committee can consider this module? (In reply to comment #5) > According to http://fedoraproject.org/wiki/Packaging/KernelModules you will need > to provide certain information to the Extras steering committee, in particular: > > A publishable explanation from the author(s) why the module is not merged with > the mainline kernel yet and when it's planned to get merged. > > Could you please provide this information so that the committee can consider > this module? OK, I asked the author to provide that info. My opinion is that this particular package does not really fall into the picture covered by the guidelines: this is not a driver, it's a (small) module which is exposing CPU sampling data for the GUI to show. Of course, it would be easier if I had only to package the GUI... (In reply to comment #6) > My opinion is that this particular package does not really fall into the picture > covered by the guidelines: this is not a driver, it's a (small) module which is > exposing CPU sampling data for the GUI to show. The main reason for this extras-work is that we want to tell everyone: "Get your shit into the kernel soon, that the proper place for it. kmod are only a interim solution" (sorry, sounds a bit harsh, but I think these words describe the situation well). In the begining we even considered rules like "kmods are only allowed for 18 months in Extras -- that should be enough time to get stuff upstream (e.g. into the kernel)" This is the response I got from the author which is, BTW, a Red Hat employee (though sysprof is not a Red Hat project): <snippet from original email> The reason I didn't reply is that the issue with the sysprof kernel module is a little tricky. Basically, the kernel developers I have talked to are saying that sysprof overlaps with the oprofile module, so I haven't attempted getting it upstream. One possibility is that I port sysprof to use the oprofile module, but there are some issues with that as well. <end snippet> Moreover, I was told it already exists a proof of concept of the latter solution (using the oprofile module), so chances are the kmod could go away in a reasonable timeframe. kmod was allowed by FESCo (review pending) Will Do Full Review today I think kernels above 2.6.16 do not require any smp kernels so either SPEC need to have some kversion checking or remove smp build lines. Here is Review for your package which i built for 2.6.18 which do not have smp support so i remove those lines form SPEC Package Built successfully for i386 FC6 development Rpmlint on SRPM W: sysprof-kmod strange-permission kmodtool 0666 A file that you listed to include in your package has strange permissions. Usually, a file should have 0644 permissions. => This is well known rpmlint Warning on kmod packages. Forget it. E: sysprof-kmod configure-without-libdir-spec A configure script is run without specifying the libdir. configure options must be augmented with something like --libdir=%{_libdir}. I am not able to found any ./configure call in SPEC. Why this Error is coming Gianluca, any idea? Rpmlint on RPM W: kmod-sysprof summary-not-capitalized sysprof kernel module(s) Summary doesn't begin with a capital letter. => This is well known rpmlint Warning on kmod packages. Forget it. W: kmod-sysprof unstripped-binary-or-object /lib/modules/2.6.18-1.2699.fc6/extra/sysprof/sysprof-module.ko => This is well known rpmlint Warning on kmod packages. Forget it. W: kmod-sysprof no-documentation The package contains no documentation (README, doc, etc). You have to include documentation files. =>Ignore this Otherwise package looks ok. I am not able to find sysprof-kmod-common package. When i tried to install sysprof-kmod, i got following output. error: Failed dependencies: sysprof-kmod-common = 1.0.3 is needed by kmod-sysprof-1.0.3-1.2.6.18_1.2699.fc6.i686 In this case i suggest you to modify kmodtool to remove dependency of sysprof-kmod-common for sysprof-kmod and also ask same on Fedora-extras mailing list. (In reply to comment #11) > I think kernels above 2.6.16 do not require any smp kernels so either SPEC need > to have some kversion checking or remove smp build lines. Though they are not required, we are still building them at least until 2.6.17-1.2187, which I am happily running at home. I hope there will be an announcement when they become history... > E: sysprof-kmod configure-without-libdir-spec > A configure script is run without specifying the libdir. configure > options must be augmented with something like --libdir=%{_libdir}. > > I am not able to found any ./configure call in SPEC. Why this Error is coming > Gianluca, any idea? it seems rpmlint is not ignoring the comment line which says: # ./configure breaks in mock builds (missing BRs), so we create here and spit that warning... I think I can easily fix it (or maybe just submit a bug report against rpmlint :P ) (In reply to comment #12) > I am not able to find sysprof-kmod-common package. When i tried to install > sysprof-kmod, i got following output. > sysprof-kmod-common is provided by sysprof-common Parag: Seems you are reviewing this package, so I am changing the blocker to FE- REVIEW. If thats not the case you can change it back to FE-NEW and reassign to nobody any update on libdir rpmlint warning? Once you update the new package i will approve this package. (In reply to comment #15) > any update on libdir rpmlint warning? > Once you update the new package i will approve this package. As I said, in comment #13, the warning is a false positive, so it does not seem to be a showstopper, is it? Anyway, if you like a cleaner rpmlint output, I think you can approve the package with the provision I import it with the warning fixed ( which I will happily do... ). OK. I did review in comment #11 and with rpmlint warning which is false one (see comment #16), I am going to approve this package. APPROVED. Don't forget to close this NEXTRELASE when you have imported and built it. (In reply to comment #17) > APPROVED. thx for review. Get's also an APPROVED from me as (kmod packages need a second approval from someone that's faimilar with kernel module packaging -- see http://www.fedoraproject.org/wiki/Packaging/KernelModules ) Small note: The header that make sure that modules for all shipped kernels get builds needs some adjustments for both devel (xen and smp stuff) and FC5 (no i586 smp). Please do that after importing. If you need some hints take a look at the em8300-kmod package (or ask on IRC for help) Thanks Thorsten, I used that as a template and the build was (mostly) flawless. Closing |