Author: juhliarik Date: Tue Jan 12 13:02:19 2010 New Revision: 60330 URL: http://svn.opensuse.org/viewcvs/yast?rev=60330&view=rev Log: deleted offset of crashkernel param for kdump Modified: trunk/kdump/VERSION trunk/kdump/package/yast2-kdump.changes trunk/kdump/src/Kdump.ycp trunk/kdump/src/kdump_finish.ycp Modified: trunk/kdump/VERSION URL: http://svn.opensuse.org/viewcvs/yast/trunk/kdump/VERSION?rev=60330&r1=60329&r2=60330&view=diff ============================================================================== --- trunk/kdump/VERSION (original) +++ trunk/kdump/VERSION Tue Jan 12 13:02:19 2010 @@ -1 +1 @@ -2.18.1 +2.19.0 Modified: trunk/kdump/package/yast2-kdump.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/kdump/package/yast2-kdump.changes?rev=60330&r1=60329&r2=60330&view=diff ============================================================================== --- trunk/kdump/package/yast2-kdump.changes (original) +++ trunk/kdump/package/yast2-kdump.changes Tue Jan 12 13:02:19 2010 @@ -1,4 +1,10 @@ ------------------------------------------------------------------- +Tue Jan 12 10:05:15 CET 2010 - juhliarik@suse.cz + +- deleted offset "@16M" for crashkernel value (bnc#567572) +- 2.19.0 + +------------------------------------------------------------------- Thu Mar 5 13:44:29 CET 2009 - juhliarik@suse.cz - added fix for validation of autoyast profil (bnc#480466) Modified: trunk/kdump/src/Kdump.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/kdump/src/Kdump.ycp?rev=60330&r1=60329&r2=60330&view=diff ============================================================================== --- trunk/kdump/src/Kdump.ycp (original) +++ trunk/kdump/src/Kdump.ycp Tue Jan 12 13:02:19 2010 @@ -362,6 +362,23 @@ string result = ""; string kernel_boot_options = ""; integer min_dif_size = 1000; + if (Mode::update()) + { + result = Bootloader::getDefaultSection (); + integer section_position = -1; + foreach(map section, BootCommon::sections, + { + section_position = section_position +1; + string name = tostring(section["name"]:nil); + + if ((name == result) && (section["xen_append"]:nil != nil)) + { + section_pos = section_position; + kernel_version = "xen"; + } + }); + return result; + } // reading bootloader settings boolean old_progress = Progress::set (false); @@ -537,11 +554,10 @@ string crash_value = ""; crash_value = alocated_memory+"M"; - if ((Arch::i386()) ||(Arch::x86_64())) { - crash_value = crash_value + "@16M"; - y2milestone("Common offset 16M"); - } else if ((Arch::ia64()) || Arch::ppc64()) { - y2milestone("ia64 or ppc64 platform without offset"); + + // bnc#563905 problem with offset in crashkernel + if ((Arch::i386()) ||(Arch::x86_64()) || (Arch::ia64()) || Arch::ppc64()) { + y2milestone("i386, x86_64, ia64 and ppc64 platforms are without offset"); } else { if (Mode::normal()) Popup::Error(_("Unsupported architecture, \"crashkernel\" was not added")); @@ -571,9 +587,9 @@ if (crash != "") { crash_value = getAlocatedMemory(crash)+"M"; - - if ((Arch::i386()) ||(Arch::x86_64()) || Arch::ppc64()) - crash_value = crash_value + "@16M"; + // bnc#563905 problem with offset in crashkernel + //if ((Arch::i386()) ||(Arch::x86_64()) || Arch::ppc64()) + // crash_value = crash_value + "@16M"; } y2milestone("Converting crashkernel value from: (%1) to :(%2)",crash, crash_value); @@ -873,6 +889,17 @@ if (Mode::autoinst()) crash_value = crashkernel_param_value; + + if (Mode::update()) + { + if (search(crash_value, "@") != nil) + { + list<string> tmp_crash_value = splitstring(crash_value, "@"); + crash_value = tmp_crash_value[0]:""; + y2milestone("Delete offset crashkernel value: %1", crash_value); + } + } + // write crashkernel option to boot section if (section_pos == -1) { @@ -980,6 +1007,20 @@ } /** + * Update crashkernel argument during update of OS + * @return true on success + */ + +global boolean Update() +{ + y2milestone("Update kdump settings"); + ReadKdumpKernelParam (); + WriteKdumpBootParameter (); + return true; +} + + +/** * Write all kdump settings * @return true on success */ Modified: trunk/kdump/src/kdump_finish.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/kdump/src/kdump_finish.ycp?rev=60330&r1=60329&r2=60330&view=diff ============================================================================== --- trunk/kdump/src/kdump_finish.ycp (original) +++ trunk/kdump/src/kdump_finish.ycp Tue Jan 12 13:02:19 2010 @@ -50,7 +50,10 @@ // propose settings for kdump if autoyast doesn't include settings for yast2-kdump if ((!Kdump::import_called) && (Mode::autoinst())) Kdump::Propose(); - Kdump::Write(); + if (Mode::update()) + Kdump::Update(); + else + Kdump::Write(); } else { -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org