Re: BLKPG changes pushed

[Jim Meyering]

Curtis Gedak wrote:

> Curtis Gedak wrote:
>> Jim Meyering wrote:
>>>     $ sudo ./parted -s $dev u B mkpart F13-root $s2 $e2
>>>     Warning: WARNING: the kernel failed to re-read the partition
>>> table on
>>>     /dev/sda (Device or resource busy).  As a result, it may not
>>> reflect all
>>>     of your changes until after reboot.
>>>
>>
>> I have been able to confirm the above error using the latest parted
>> from the git repository (April 24, 2010).  My testing used the
>> Ubuntu 10.04 Beta2 distribution, a GPT partition table, and an
>> existing partition already mounted.  When I try to create a new
>> partition I receive the warning message.
>>
>> Next I took a plain vanilla copy of parted-2.2 and applied only the
>> two patches from this mailing list item:
>> http://lists.alioth.debian.org/pipermail/parted-devel/2010-April/003552.html
>>
>>
>> My testing with plain vanilla parted-2.2 plus the two patches worked
>> 100% without any warning being displayed.
>>
>> This leads me to believe that there is some other change in the git
>> repository since parted-2.2 was released that is causing the warning
>> message to be displayed.
>
> I have looked through the patches since parted-2.2 was released.  The
> patch that causes the warning message to be displayed is:
> libparted: _disk_sync_part_table: always return 0 upon failure
> http://git.debian.org/?p=parted/parted.git;a=commit;h=81ed7fc413375a8b8ed5bd792e7385dacaf8a3e1

Thanks for investigating.
I've written a test to exercise this and will address it shortly.