Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 107334 - nvclock flooding open fd's
Summary: nvclock flooding open fd's
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Philip Walls (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-09-26 14:56 UTC by Prakash Punnoor
Modified: 2007-04-08 16:36 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
ebuild with nvclock use flag (cpufreqd-2.0.0_pre1.ebuild,1.58 KB, text/plain)
2005-09-26 14:57 UTC, Prakash Punnoor
Details
cpufreqd nvclock fix (cpufreqd_nv.diff,878 bytes, patch)
2005-10-11 13:58 UTC, Prakash Punnoor
Details | Diff
nvclock fd fix (nvclock_fd.diff,436 bytes, patch)
2005-10-11 13:59 UTC, Prakash Punnoor
Details | Diff
cpufreqd nvcock fix (updated/conservative) (cpufreqd_nv2.diff,1.03 KB, patch)
2005-10-16 23:59 UTC, Prakash Punnoor
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Prakash Punnoor 2005-09-26 14:56:21 UTC
Hi, the new package also includes my nvclock plugin. I attached an ebuild which
builds it if the use flag is specified. You might want to add a few warning that
the plugin will oc an NVidia card w/o further checks and possible damage it, if
the wrong values are put into the config file.

Reproducible: Always
Steps to Reproduce:
1.
2.
3.
Comment 1 Prakash Punnoor 2005-09-26 14:57:07 UTC
Created attachment 69295 [details]
ebuild with nvclock use flag
Comment 2 Carsten Lohrke (RETIRED) gentoo-dev 2005-09-26 16:50:43 UTC
- Please attach always in plain text.
- The ebuild header is invalid.

The plugin should probably better be an ebuild, instead adding a yet another
superfluous use flag.
Comment 3 Prakash Punnoor 2005-09-26 23:38:37 UTC
I ticked auto when I attached the patch. I just modified the preivious ebuild,
thus the header. Of course I left it to the maintainer to clean it up, but just
wanted to show hot to compile the nvclock plugin. If the maintainer thinks it
should be a seperate ebuild, he may have fun doing so. I wonder how to compile
the plugin itself w/o major patching of the cpufreqd's build system.

This ebuild works for me and I'll keep it in my local overly.
Comment 4 Henrik Brix Andersen 2005-10-05 14:30:19 UTC
Please always submit proposed ebuild changes as a unified diff (diff -u) 
against the latest version available in portage.
Comment 5 Prakash Punnoor 2005-10-11 13:58:59 UTC
Created attachment 70413 [details, diff]
cpufreqd nvclock fix

In case you want to enable nvclock support, you might want to use these two
patches, as well, which prevent cpufreqd/nvclock from flooding open fd's.
Comment 6 Prakash Punnoor 2005-10-11 13:59:30 UTC
Created attachment 70414 [details, diff]
nvclock fd fix
Comment 7 Henrik Brix Andersen 2005-10-16 14:28:39 UTC
cpufreqd-2.0.0-pre1 added to portage including the above patches.

malverian, perhaps attachment #70414 [details, diff] should be added to the stand-alone nvclock
ebuilds as well?
Comment 8 Prakash Punnoor 2005-10-16 23:59:38 UTC
Created attachment 70834 [details, diff]
cpufreqd nvcock fix (updated/conservative)

After talking with author of nvclock, he suggest using this way to set mem/core
clock. This one disallows overclocking laptop and integrated graphics, but
according to author that isn't very safe anyway.
Comment 9 Henrik Brix Andersen 2005-10-17 01:13:57 UTC
Thanks, applied. In the future, please provide patches that apply with -p1, not
-p0 as this makes it much easier to use them from within ebuild. It also makes
it easier to see which package, the patch is intended for.
Comment 10 Prakash Punnoor 2005-10-21 01:10:42 UTC
Your useflag is named wrongly. It should be "nvidia" not "nvidea"...
Comment 11 Jakub Moc (RETIRED) gentoo-dev 2007-04-08 16:36:47 UTC
*nvclock-0.8_beta2 (14 Oct 2006)

  14 Oct 2006; <malverian@gentoo.org> +files/nvclock_acinclude_qtlibs.patch,
  nvclock-0.7-r1.ebuild, +nvclock-0.8_beta2.ebuild, -nvclock-0.8b.ebuild:
  Fix compile problem with USE=qt3 when Qt4 is installed (#114872).
  Compilation fix with USE=qt3 on AMD64. Version bump (#143023, #107334).